RE: [PATCH 2/4] Fix for crash when bfa_itnim is NULL

2015-02-02 Thread Anil Gurumurthy
Hi Julian,
   You are right. It makes sense to move it to the next patch. I will re-send 
the patch series.

Thanks,
Anil

-Original Message-
From: Julian Calaby [mailto:julian.cal...@gmail.com] 
Sent: 30 January 2015 06:45
To: Anil Gurumurthy
Cc: Jej B; Christoph Hellwig; linux-scsi
Subject: Re: [PATCH 2/4] Fix for crash when bfa_itnim is NULL

Hi Anil,

On Thu, Jan 29, 2015 at 7:55 PM,  anil.gurumur...@qlogic.com wrote:
 From: Anil Gurumurthy anil.gurumur...@qlogic.com

 Signed-off-by: Sudarsana Kalluru sudarsana.kall...@qlogic.com
 Signed-off-by: Anil Gurumurthy anil.gurumur...@qlogic.com
 ---
  drivers/scsi/bfa/bfa_fcs_lport.c |2 +-
  drivers/scsi/bfa/bfad_im.c   |   26 ++
  2 files changed, 27 insertions(+), 1 deletions(-)

 diff --git a/drivers/scsi/bfa/bfa_fcs_lport.c 
 b/drivers/scsi/bfa/bfa_fcs_lport.c
 index d823792..4631630 100644
 --- a/drivers/scsi/bfa/bfa_fcs_lport.c
 +++ b/drivers/scsi/bfa/bfa_fcs_lport.c
 @@ -2654,7 +2654,7 @@ bfa_fcs_fdmi_get_hbaattr(struct bfa_fcs_lport_fdmi_s 
 *fdmi,

 strncpy(hba_attr-node_sym_name.symname,
 port-port_cfg.node_sym_name.symname, BFA_SYMNAME_MAXLEN);
 -   strcpy(hba_attr-vendor_info, BROCADE);
 +   strcpy(hba_attr-vendor_info, QLogic);

Shouldn't this be in the next patch?

Thanks,

-- 
Julian Calaby

Email: julian.cal...@gmail.com
Profile: http://www.google.com/profiles/julian.calaby/
attachment: winmail.dat

Re: [PATCH 2/4] Fix for crash when bfa_itnim is NULL

2015-01-29 Thread Julian Calaby
Hi Anil,

On Thu, Jan 29, 2015 at 7:55 PM,  anil.gurumur...@qlogic.com wrote:
 From: Anil Gurumurthy anil.gurumur...@qlogic.com

 Signed-off-by: Sudarsana Kalluru sudarsana.kall...@qlogic.com
 Signed-off-by: Anil Gurumurthy anil.gurumur...@qlogic.com
 ---
  drivers/scsi/bfa/bfa_fcs_lport.c |2 +-
  drivers/scsi/bfa/bfad_im.c   |   26 ++
  2 files changed, 27 insertions(+), 1 deletions(-)

 diff --git a/drivers/scsi/bfa/bfa_fcs_lport.c 
 b/drivers/scsi/bfa/bfa_fcs_lport.c
 index d823792..4631630 100644
 --- a/drivers/scsi/bfa/bfa_fcs_lport.c
 +++ b/drivers/scsi/bfa/bfa_fcs_lport.c
 @@ -2654,7 +2654,7 @@ bfa_fcs_fdmi_get_hbaattr(struct bfa_fcs_lport_fdmi_s 
 *fdmi,

 strncpy(hba_attr-node_sym_name.symname,
 port-port_cfg.node_sym_name.symname, BFA_SYMNAME_MAXLEN);
 -   strcpy(hba_attr-vendor_info, BROCADE);
 +   strcpy(hba_attr-vendor_info, QLogic);

Shouldn't this be in the next patch?

Thanks,

-- 
Julian Calaby

Email: julian.cal...@gmail.com
Profile: http://www.google.com/profiles/julian.calaby/
--
To unsubscribe from this list: send the line unsubscribe linux-scsi in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html