Re: [patch] [SCSI] qla4xxx: silence a Sparse warning

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



-----Original Message-----
From: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
Date: Mon, 20 Feb 2012 23:30:12 -0800
To: Ravi Anand <ravi.anand@xxxxxxxxxx>
Cc: Vikas Chaudhary <vikas.chaudhary@xxxxxxxxxx>, Dept-Eng iSCSI Driver
<Dept-iSCSIDriver@xxxxxxxxxx>, "James E.J. Bottomley"
<JBottomley@xxxxxxxxxxxxx>, scsi <linux-scsi@xxxxxxxxxxxxxxx>,
"kernel-janitors@xxxxxxxxxxxxxxx" <kernel-janitors@xxxxxxxxxxxxxxx>
Subject: [patch] [SCSI] qla4xxx: silence a Sparse warning

>Sparse complains that the definition of this funciton doesn't match the
>implementation.  The "uint32_t aen_code" should be an enum.
>
>drivers/scsi/qla4xxx/ql4_os.c:2966:5: error: symbol
>       'qla4xxx_post_aen_work' redeclared with different type
>       (originally declared at drivers/scsi/qla4xxx/ql4_glbl.h:184)
>       - incompatible argument 2 (different signedness)
>
>Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
>
>diff --git a/drivers/scsi/qla4xxx/ql4_glbl.h
>b/drivers/scsi/qla4xxx/ql4_glbl.h
>index 954ba91..71bbc07 100644
>--- a/drivers/scsi/qla4xxx/ql4_glbl.h
>+++ b/drivers/scsi/qla4xxx/ql4_glbl.h
>@@ -181,7 +181,8 @@ int qla4xxx_flash_ddb_change(struct scsi_qla_host
>*ha, uint32_t fw_ddb_index,
> int qla4xxx_ddb_change(struct scsi_qla_host *ha, uint32_t fw_ddb_index,
>                      struct ddb_entry *ddb_entry, uint32_t state);
> void qla4xxx_build_ddb_list(struct scsi_qla_host *ha, int is_reset);
>-int qla4xxx_post_aen_work(struct scsi_qla_host *ha, uint32_t aen_code,
>+int qla4xxx_post_aen_work(struct scsi_qla_host *ha,
>+                        enum iscsi_host_event_code aen_code,
>                         uint32_t data_size, uint8_t *data);
> int qla4xxx_ping_iocb(struct scsi_qla_host *ha, uint32_t options,
>                     uint32_t payload_size, uint32_t pid, uint8_t *ipaddr);

Acked-by: Vikas Chaudhary <vikas.chaudhary@xxxxxxxxxx>


This message and any attached documents contain information from QLogic Corporation or its wholly-owned subsidiaries that may be confidential. If you are not the intended recipient, you may not read, copy, distribute, or use this information. If you have received this transmission in error, please notify the sender immediately by reply e-mail and then delete this message.

--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux