[patch 1/2] drivers/message/fusion/mptsas.c: fix warning

[patch 1/2] drivers/message/fusion/mptsas.c: fix warning

[akpm]

From: Andrew Morton <akpm@linux-foundation.org>

drivers/message/fusion/mptsas.c: In function 'mptsas_event_process':
drivers/message/fusion/mptsas.c:5010: warning: unused variable 'log_info'

Cc: "Moore, Eric Dean" <Eric.Moore@lsil.com>
Cc: James Bottomley <James.Bottomley@HansenPartnership.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---

 drivers/message/fusion/mptsas.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff -puN drivers/message/fusion/mptsas.c~drivers-message-fusion-mptsasc-fix-warning drivers/message/fusion/mptsas.c
--- a/drivers/message/fusion/mptsas.c~drivers-message-fusion-mptsasc-fix-warning
+++ a/drivers/message/fusion/mptsas.c
@@ -5009,7 +5009,6 @@ mptsas_event_process(MPT_ADAPTER *ioc, E
 			(ioc_stat & MPI_IOCSTATUS_FLAG_LOG_INFO_AVAILABLE)) {
 			VirtTarget *vtarget = NULL;
 			u8		id, channel;
-			u32	 log_info = le32_to_cpu(reply->IOCLogInfo);
 
 			id = sas_event_data->TargetID;
 			channel = sas_event_data->Bus;
@@ -5020,7 +5019,8 @@ mptsas_event_process(MPT_ADAPTER *ioc, E
 				    "LogInfo (0x%x) available for "
 				   "INTERNAL_DEVICE_RESET"
 				   "fw_id %d fw_channel %d\n", ioc->name,
-				   log_info, id, channel));
+				   le32_to_cpu(reply->IOCLogInfo), id,
+				   channel));
 				if (vtarget->raidVolume) {
 					devtprintk(ioc, printk(MYIOC_s_DEBUG_FMT
 					"Skipping Raid Volume for inDMD\n",
_

RE: [patch 1/2] drivers/message/fusion/mptsas.c: fix warning

[Desai, Kashyap]

Without debug enable log_info will not be used.
Please consider this patch as an ACKed.

Thanks, Kashyap

> -----Original Message-----
> From: linux-scsi-owner@vger.kernel.org [mailto:linux-scsi-
> owner@vger.kernel.org] On Behalf Of akpm@linux-foundation.org
> Sent: Wednesday, January 19, 2011 3:32 AM
> To: James.Bottomley@HansenPartnership.com
> Cc: linux-scsi@vger.kernel.org; akpm@linux-foundation.org; Moore, Eric
> Subject: [patch 1/2] drivers/message/fusion/mptsas.c: fix warning
> 
> From: Andrew Morton <akpm@linux-foundation.org>
> 
> drivers/message/fusion/mptsas.c: In function 'mptsas_event_process':
> drivers/message/fusion/mptsas.c:5010: warning: unused variable
> 'log_info'
> 
> Cc: "Moore, Eric Dean" <Eric.Moore@lsil.com>
> Cc: James Bottomley <James.Bottomley@HansenPartnership.com>
> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> ---
> 
>  drivers/message/fusion/mptsas.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff -puN drivers/message/fusion/mptsas.c~drivers-message-fusion-
> mptsasc-fix-warning drivers/message/fusion/mptsas.c
> --- a/drivers/message/fusion/mptsas.c~drivers-message-fusion-mptsasc-
> fix-warning
> +++ a/drivers/message/fusion/mptsas.c
> @@ -5009,7 +5009,6 @@ mptsas_event_process(MPT_ADAPTER *ioc, E
>  			(ioc_stat & MPI_IOCSTATUS_FLAG_LOG_INFO_AVAILABLE)) {
>  			VirtTarget *vtarget = NULL;
>  			u8		id, channel;
> -			u32	 log_info = le32_to_cpu(reply->IOCLogInfo);
> 
>  			id = sas_event_data->TargetID;
>  			channel = sas_event_data->Bus;
> @@ -5020,7 +5019,8 @@ mptsas_event_process(MPT_ADAPTER *ioc, E
>  				    "LogInfo (0x%x) available for "
>  				   "INTERNAL_DEVICE_RESET"
>  				   "fw_id %d fw_channel %d\n", ioc->name,
> -				   log_info, id, channel));
> +				   le32_to_cpu(reply->IOCLogInfo), id,
> +				   channel));
>  				if (vtarget->raidVolume) {
>  					devtprintk(ioc, printk(MYIOC_s_DEBUG_FMT
>  					"Skipping Raid Volume for inDMD\n",
> _
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi"
> in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html