Re: [PATCH] mpt2sas: Fix hex/decimal typo in _base_sas_log_info()

From: Chris Dunlop
Date: Wed Mar 14 2012 - 14:08:25 EST


On Wed, Mar 14, 2012 at 02:56:10PM +1100, Chris Dunlop wrote:
> The log_info constant is supposed to be IOP_LOGINFO_CODE_TASK_TERMINATED
> (see drivers/message/fusion/lsi/mpi_log_sas.h) rather than the
> non-sensical 0x1CA86D0.

Oops, I see Sathisha Nanjappa has already submitted this fix:

http://article.gmane.org/gmane.linux.scsi/73446


> Signed-off-by: Chris Dunlop <chris@xxxxxxxxxxxx>
> ---
> drivers/scsi/mpt2sas/mpt2sas_base.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/scsi/mpt2sas/mpt2sas_base.c b/drivers/scsi/mpt2sas/mpt2sas_base.c
> index a78036f..224ed29 100644
> --- a/drivers/scsi/mpt2sas/mpt2sas_base.c
> +++ b/drivers/scsi/mpt2sas/mpt2sas_base.c
> @@ -657,7 +657,7 @@ _base_sas_log_info(struct MPT2SAS_ADAPTER *ioc , u32 log_info)
> return;
>
> /* eat the loginfos associated with task aborts */
> - if (ioc->ignore_loginfos && (log_info == 30050000 || log_info ==
> + if (ioc->ignore_loginfos && (log_info == 0x30050000 || log_info ==
> 0x31140000 || log_info == 0x31130000))
> return;
>
> --
> 1.7.0.4
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/