diff mbox series

qla2xxx: Add new FC-NVMe enable BIT to enable FC-NVMe feature

Message ID 20190130175044.3012-1-hmadhani@marvell.com (mailing list archive)
State Mainlined
Commit 171e4909eac7912c8d9ffaec9a2872b3271dd6e9
Headers show
Series qla2xxx: Add new FC-NVMe enable BIT to enable FC-NVMe feature | expand

Commit Message

Himanshu Madhani Jan. 30, 2019, 5:50 p.m. UTC
From: Giridhar Malavali <gmalavali@marvell.com>

This patch adds new BIT detection to enable FC-NVMe feature in
the driver.

Signed-off-by: Giridhar Malavali <gmalalvali@marvell.com>
Signed-off-by: Himanshu Madhani <hmadhani@marvell.com>
---
Hi Martin,

This patch adds additional bit to enable FC-NVMe in the driver. 

Please apply this patch to 5.1/scsi-queue at your earliest convenience.

Thanks,
Himanshu
---
 drivers/scsi/qla2xxx/qla_def.h | 3 +++
 drivers/scsi/qla2xxx/qla_mbx.c | 4 +++-
 2 files changed, 6 insertions(+), 1 deletion(-)

Comments

Ewan Milne Feb. 1, 2019, 1:32 p.m. UTC | #1
On Wed, 2019-01-30 at 09:50 -0800, Himanshu Madhani wrote:
> From: Giridhar Malavali <gmalavali@marvell.com>
> 
> This patch adds new BIT detection to enable FC-NVMe feature in
> the driver.
> 
> Signed-off-by: Giridhar Malavali <gmalalvali@marvell.com>
> Signed-off-by: Himanshu Madhani <hmadhani@marvell.com>
> ---
> Hi Martin,
> 
> This patch adds additional bit to enable FC-NVMe in the driver. 
> 
> Please apply this patch to 5.1/scsi-queue at your earliest convenience.
> 
> Thanks,
> Himanshu
> ---
>  drivers/scsi/qla2xxx/qla_def.h | 3 +++
>  drivers/scsi/qla2xxx/qla_mbx.c | 4 +++-
>  2 files changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/scsi/qla2xxx/qla_def.h b/drivers/scsi/qla2xxx/qla_def.h
> index 05b5c6fa011d..199713f29cbd 100644
> --- a/drivers/scsi/qla2xxx/qla_def.h
> +++ b/drivers/scsi/qla2xxx/qla_def.h
> @@ -3958,6 +3958,9 @@ struct qla_hw_data {
>  	uint16_t	fw_subminor_version;
>  	uint16_t	fw_attributes;
>  	uint16_t	fw_attributes_h;
> +#define FW_ATTR_H_NVME		BIT_10
> +#define FW_ATTR_H_NVME_UPDATED  BIT_14
> +
>  	uint16_t	fw_attributes_ext[2];
>  	uint32_t	fw_memory_size;
>  	uint32_t	fw_transfer_size;
> diff --git a/drivers/scsi/qla2xxx/qla_mbx.c b/drivers/scsi/qla2xxx/qla_mbx.c
> index 3181235c3a32..f4adf6baee69 100644
> --- a/drivers/scsi/qla2xxx/qla_mbx.c
> +++ b/drivers/scsi/qla2xxx/qla_mbx.c
> @@ -1109,7 +1109,9 @@ qla2x00_get_fw_version(scsi_qla_host_t *vha)
>  		 * FW supports nvme and driver load parameter requested nvme.
>  		 * BIT 26 of fw_attributes indicates NVMe support.
>  		 */
> -		if ((ha->fw_attributes_h & 0x400) && ql2xnvmeenable) {
> +		if ((ha->fw_attributes_h &
> +		    (FW_ATTR_H_NVME | FW_ATTR_H_NVME_UPDATED)) &&
> +			ql2xnvmeenable) {
>  			vha->flags.nvme_enabled = 1;
>  			ql_log(ql_log_info, vha, 0xd302,
>  			    "%s: FC-NVMe is Enabled (0x%x)\n",

Reviewed-by: Ewan D. Milne <emilne@redhat.com>
Martin K. Petersen Feb. 1, 2019, 11:15 p.m. UTC | #2
Himanshu,

> This patch adds new BIT detection to enable FC-NVMe feature in
> the driver.
>
> Signed-off-by: Giridhar Malavali <gmalalvali@marvell.com>
                                         ^

Fixed Giridhar's address and applied to 5.1/scsi-queue. Thanks!
diff mbox series

Patch

diff --git a/drivers/scsi/qla2xxx/qla_def.h b/drivers/scsi/qla2xxx/qla_def.h
index 05b5c6fa011d..199713f29cbd 100644
--- a/drivers/scsi/qla2xxx/qla_def.h
+++ b/drivers/scsi/qla2xxx/qla_def.h
@@ -3958,6 +3958,9 @@  struct qla_hw_data {
 	uint16_t	fw_subminor_version;
 	uint16_t	fw_attributes;
 	uint16_t	fw_attributes_h;
+#define FW_ATTR_H_NVME		BIT_10
+#define FW_ATTR_H_NVME_UPDATED  BIT_14
+
 	uint16_t	fw_attributes_ext[2];
 	uint32_t	fw_memory_size;
 	uint32_t	fw_transfer_size;
diff --git a/drivers/scsi/qla2xxx/qla_mbx.c b/drivers/scsi/qla2xxx/qla_mbx.c
index 3181235c3a32..f4adf6baee69 100644
--- a/drivers/scsi/qla2xxx/qla_mbx.c
+++ b/drivers/scsi/qla2xxx/qla_mbx.c
@@ -1109,7 +1109,9 @@  qla2x00_get_fw_version(scsi_qla_host_t *vha)
 		 * FW supports nvme and driver load parameter requested nvme.
 		 * BIT 26 of fw_attributes indicates NVMe support.
 		 */
-		if ((ha->fw_attributes_h & 0x400) && ql2xnvmeenable) {
+		if ((ha->fw_attributes_h &
+		    (FW_ATTR_H_NVME | FW_ATTR_H_NVME_UPDATED)) &&
+			ql2xnvmeenable) {
 			vha->flags.nvme_enabled = 1;
 			ql_log(ql_log_info, vha, 0xd302,
 			    "%s: FC-NVMe is Enabled (0x%x)\n",