diff options
| author | Bart Van Assche <bart.vanassche@sandisk.com> | 2015-07-09 07:23:46 -0700 |
|---|---|---|
| committer | James Bottomley <JBottomley@Odin.com> | 2015-08-26 10:35:35 -0700 |
| commit | df3f4cd0defd5f832a806ca1f0dd6638a2df17a5 (patch) | |
| tree | 71247576da136aa11c258ffaf9724bc43d86e94b /drivers/scsi/qla2xxx/qla_nx2.h | |
| parent | 52c82823cc686d4faf9e9bc4feec4bf48c36fded (diff) | |
qla2xxx: Replace two macros with an inline function
Replace the QLA82XX_ADDR_IN_RANGE() and QLA8044_ADDR_IN_RANGE() macros
with the inline function addr_in_range(). This avoids that the compiler
reports the following warning when building with W=1: comparison of
unsigned expression >= 0 is always true.
Signed-off-by: Bart Van Assche <bart.vanassche@sandisk.com>
Acked-by: Himanshu Madhani <himanshu.madhani@qlogic.com>
Signed-off-by: James Bottomley <JBottomley@Odin.com>
Diffstat (limited to 'drivers/scsi/qla2xxx/qla_nx2.h')
| -rw-r--r-- | drivers/scsi/qla2xxx/qla_nx2.h | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/scsi/qla2xxx/qla_nx2.h b/drivers/scsi/qla2xxx/qla_nx2.h index ada36057d7cd..02fe3c4cdf55 100644 --- a/drivers/scsi/qla2xxx/qla_nx2.h +++ b/drivers/scsi/qla2xxx/qla_nx2.h @@ -58,8 +58,10 @@ #define QLA8044_PCI_QDR_NET_MAX ((unsigned long)0x043fffff) /* PCI Windowing for DDR regions. */ -#define QLA8044_ADDR_IN_RANGE(addr, low, high) \ - (((addr) <= (high)) && ((addr) >= (low))) +static inline bool addr_in_range(u64 addr, u64 low, u64 high) +{ + return addr <= high && addr >= low; +} /* Indirectly Mapped Registers */ #define QLA8044_FLASH_SPI_STATUS 0x2808E010 |