David C Somayajulu wrote: > Signed-off-by: David Somayajulu <david.somayajulu@xxxxxxxxxx> > > --- > drivers/scsi/qla4xxx/ql4_mbx.c | 190 ++++++++++------ > > diff --git a/drivers/scsi/qla4xxx/ql4_mbx.c b/drivers/scsi/qla4xxx/ql4_mbx.c > index 7f28657..0ef777a 100644 > --- a/drivers/scsi/qla4xxx/ql4_mbx.c > +++ b/drivers/scsi/qla4xxx/ql4_mbx.c > @@ -6,6 +6,10 @@ > */ > > #include "ql4_def.h" > +#include "ql4_version.h" > +#include "ql4_glbl.h" > +#include "ql4_dbg.h" > +#include "ql4_inline.h" Did you need to bring in all of these? You did not need version did you? > > - memset((void *)fw_ddb_entry->iSCSIAlias, 0, > - sizeof(fw_ddb_entry->iSCSIAlias)); > + memset((void *)fw_ddb_entry->iscsi_alias, 0, > + sizeof(fw_ddb_entry->iscsi_alias)); > > - memset((void *)fw_ddb_entry->iscsiName, 0, > - sizeof(fw_ddb_entry->iscsiName)); > + memset((void *)fw_ddb_entry->iscsi_name, 0, > + sizeof(fw_ddb_entry->iscsi_name)); > > - memset((void *)fw_ddb_entry->ipAddr, 0, sizeof(fw_ddb_entry->ipAddr)); > - memset((void *)fw_ddb_entry->targetAddr, 0, > - sizeof(fw_ddb_entry->targetAddr)); > + memset((void *)fw_ddb_entry->ip_addr, 0, sizeof(fw_ddb_entry->ip_addr)); > + memset((void *)fw_ddb_entry->tgt_addr, 0, > + sizeof(fw_ddb_entry->tgt_addr)); > I am not sure if it is a big deal or if it clutters the code, but I do not think you need the void * casts in there. - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html