Re: [PATCH v2 4/6] qla2xxx: Fix sparse warnings triggered by the PCI state checking code

Roman Bolshakov
2020-02-25T18:09:39Z

On Wed, Jan 22, 2020 at 08:23:43PM -0800, Bart Van Assche wrote:
> This patch fixes the following sparse warnings:
>
> drivers/scsi/qla2xxx/qla_mbx.c:120:21: warning: restricted pci_channel_state_t degrades to integer
> drivers/scsi/qla2xxx/qla_mbx.c:120:37: warning: restricted pci_channel_state_t degrades to integer
>
> From include/linux/pci.h:
>
> enum pci_channel_state {
> /* I/O channel is in normal state */
> pci_channel_io_normal = (__force pci_channel_state_t) 1,
>
> /* I/O to channel is blocked */
> pci_channel_io_frozen = (__force pci_channel_state_t) 2,
>
> /* PCI card is dead */
> pci_channel_io_perm_failure = (__force pci_channel_state_t) 3,
> };
>
> Cc: Quinn Tran <qutran@marvell.com>
> Cc: Martin Wilck <mwilck@suse.com>
> Cc: Daniel Wagner <dwagner@suse.de>
> Cc: Roman Bolshakov <r.bolshakov@yadro.com>
> Signed-off-by: Bart Van Assche <bvanassche@acm.org>
> ---
> drivers/scsi/qla2xxx/qla_mbx.c | 5 ++---
> drivers/scsi/qla2xxx/qla_mr.c | 5 ++---
> 2 files changed, 4 insertions(+), 6 deletions(-)
>

Hi Bart,

Reviewed-by: Roman Bolshakov <r.bolshakov@yadro.com>

Best regards,
Roman