Re: [PATCH] bfa: turn bfa_mem_{kva,dma}_setup into inline functions

2016-11-21 Thread Martin K. Petersen
> "Arnd" == Arnd Bergmann writes: Arnd> These two macros cause lots of warnings with gcc-7: Arnd> drivers/scsi/bfa/bfa_svc.c: In function 'bfa_fcxp_meminfo': Arnd> drivers/scsi/bfa/bfa_svc.c:521:103: error: '*' in boolean context, Arnd> suggest '&&' instead

RE: [PATCH] bfa: turn bfa_mem_{kva,dma}_setup into inline functions

2016-11-20 Thread Gurumurthy, Anil
Martin K. Petersen <martin.peter...@oracle.com>; Anil Gurumurthy <anil.gurumur...@qlogic.com>; Sudarsana Kalluru <sudarsana.kall...@qlogic.com>; linux-scsi@vger.kernel.org; linux-ker...@vger.kernel.org Subject: Re: [PATCH] bfa: turn bfa_mem_{kva,dma}_setup into inline functions On W

Re: [PATCH] bfa: turn bfa_mem_{kva,dma}_setup into inline functions

2016-11-18 Thread Johannes Thumshirn
On Wed, Nov 16, 2016 at 04:14:27PM +0100, Arnd Bergmann wrote: > These two macros cause lots of warnings with gcc-7: > > drivers/scsi/bfa/bfa_svc.c: In function 'bfa_fcxp_meminfo': > drivers/scsi/bfa/bfa_svc.c:521:103: error: '*' in boolean context, suggest > '&&' instead

[PATCH] bfa: turn bfa_mem_{kva,dma}_setup into inline functions

2016-11-16 Thread Arnd Bergmann
These two macros cause lots of warnings with gcc-7: drivers/scsi/bfa/bfa_svc.c: In function 'bfa_fcxp_meminfo': drivers/scsi/bfa/bfa_svc.c:521:103: error: '*' in boolean context, suggest '&&' instead [-Werror=int-in-bool-context] Using inline functions makes them much more readable and avoids