Re: [PATCH 3/5] scsi: bnx2fc: clean up header definitions

2017-05-05 Thread Johannes Thumshirn

On 04/10/2017 07:12 PM, Sebastian Andrzej Siewior wrote:
> - All symbols which are only used within one .c file are marked static
>   and removed from the bnx2fc.h file if possible.
>
> - the declarion of bnx2fc_percpu is moved into the header file
>
> This patch was only compile-tested due to -ENODEV.
>
> Cc: qlogic-storage-upstr...@qlogic.com
> Cc: Christoph Hellwig 
> Signed-off-by: Sebastian Andrzej Siewior 
> ---
Reviewed-by: Johannes Thumshirn 



Re: [PATCH 3/5] scsi: bnx2fc: clean up header definitions

2017-05-05 Thread Christoph Hellwig
Looks fine,

Reviewed-by: Christoph Hellwig