linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] scsi: qla2xxx: use MBX_TOV_SECONDS for mailbox command timeout values
@ 2020-08-05 20:05 Enzo Matsumiya
  2020-08-12 19:49 ` Himanshu Madhani
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Enzo Matsumiya @ 2020-08-05 20:05 UTC (permalink / raw)
  To: linux-scsi
  Cc: Nilesh Javali, GR-QLogic-Storage-Upstream, James E.J. Bottomley,
	Martin K. Petersen

Improves readability of qla_mbx.c

Signed-off-by: Enzo Matsumiya <ematsumiya@suse.de>
---
 drivers/scsi/qla2xxx/qla_mbx.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/drivers/scsi/qla2xxx/qla_mbx.c b/drivers/scsi/qla2xxx/qla_mbx.c
index df31ee0d59b2..b99eed55eb4a 100644
--- a/drivers/scsi/qla2xxx/qla_mbx.c
+++ b/drivers/scsi/qla2xxx/qla_mbx.c
@@ -5190,7 +5190,7 @@ qla2x00_read_ram_word(scsi_qla_host_t *vha, uint32_t risc_addr, uint32_t *data)
 	mcp->mb[8] = MSW(risc_addr);
 	mcp->out_mb = MBX_8|MBX_1|MBX_0;
 	mcp->in_mb = MBX_3|MBX_2|MBX_0;
-	mcp->tov = 30;
+	mcp->tov = MBX_TOV_SECONDS;
 	mcp->flags = 0;
 	rval = qla2x00_mailbox_command(vha, mcp);
 	if (rval != QLA_SUCCESS) {
@@ -5378,7 +5378,7 @@ qla2x00_write_ram_word(scsi_qla_host_t *vha, uint32_t risc_addr, uint32_t data)
 	mcp->mb[8] = MSW(risc_addr);
 	mcp->out_mb = MBX_8|MBX_3|MBX_2|MBX_1|MBX_0;
 	mcp->in_mb = MBX_1|MBX_0;
-	mcp->tov = 30;
+	mcp->tov = MBX_TOV_SECONDS;
 	mcp->flags = 0;
 	rval = qla2x00_mailbox_command(vha, mcp);
 	if (rval != QLA_SUCCESS) {
@@ -5650,7 +5650,7 @@ qla24xx_set_fcp_prio(scsi_qla_host_t *vha, uint16_t loop_id, uint16_t priority,
 	mcp->mb[9] = vha->vp_idx;
 	mcp->out_mb = MBX_9|MBX_4|MBX_3|MBX_2|MBX_1|MBX_0;
 	mcp->in_mb = MBX_4|MBX_3|MBX_1|MBX_0;
-	mcp->tov = 30;
+	mcp->tov = MBX_TOV_SECONDS;
 	mcp->flags = 0;
 	rval = qla2x00_mailbox_command(vha, mcp);
 	if (mb != NULL) {
@@ -5737,7 +5737,7 @@ qla82xx_mbx_intr_enable(scsi_qla_host_t *vha)
 
 	mcp->out_mb = MBX_1|MBX_0;
 	mcp->in_mb = MBX_0;
-	mcp->tov = 30;
+	mcp->tov = MBX_TOV_SECONDS;
 	mcp->flags = 0;
 
 	rval = qla2x00_mailbox_command(vha, mcp);
@@ -5772,7 +5772,7 @@ qla82xx_mbx_intr_disable(scsi_qla_host_t *vha)
 
 	mcp->out_mb = MBX_1|MBX_0;
 	mcp->in_mb = MBX_0;
-	mcp->tov = 30;
+	mcp->tov = MBX_TOV_SECONDS;
 	mcp->flags = 0;
 
 	rval = qla2x00_mailbox_command(vha, mcp);
@@ -5964,7 +5964,7 @@ qla81xx_set_led_config(scsi_qla_host_t *vha, uint16_t *led_cfg)
 	if (IS_QLA8031(ha))
 		mcp->out_mb |= MBX_6|MBX_5|MBX_4|MBX_3;
 	mcp->in_mb = MBX_0;
-	mcp->tov = 30;
+	mcp->tov = MBX_TOV_SECONDS;
 	mcp->flags = 0;
 
 	rval = qla2x00_mailbox_command(vha, mcp);
@@ -6000,7 +6000,7 @@ qla81xx_get_led_config(scsi_qla_host_t *vha, uint16_t *led_cfg)
 	mcp->in_mb = MBX_2|MBX_1|MBX_0;
 	if (IS_QLA8031(ha))
 		mcp->in_mb |= MBX_6|MBX_5|MBX_4|MBX_3;
-	mcp->tov = 30;
+	mcp->tov = MBX_TOV_SECONDS;
 	mcp->flags = 0;
 
 	rval = qla2x00_mailbox_command(vha, mcp);
-- 
2.28.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread
* Re: [PATCH] scsi: qla2xxx: use MBX_TOV_SECONDS for mailbox command timeout values
  2020-08-05 20:05 [PATCH] scsi: qla2xxx: use MBX_TOV_SECONDS for mailbox command timeout values Enzo Matsumiya
@ 2020-08-12 19:49 ` Himanshu Madhani
  2020-08-13  7:00 ` Roman Bolshakov
  2020-08-18  3:12 ` Martin K. Petersen
  2 siblings, 0 replies; 4+ messages in thread
From: Himanshu Madhani @ 2020-08-12 19:49 UTC (permalink / raw)
  To: Enzo Matsumiya
  Cc: linux-scsi, Nilesh Javali, GR-QLogic-Storage-Upstream,
	James E.J. Bottomley, Martin K. Petersen



> On Aug 5, 2020, at 3:05 PM, Enzo Matsumiya <ematsumiya@suse.de> wrote:
> 
> Improves readability of qla_mbx.c
> 
> Signed-off-by: Enzo Matsumiya <ematsumiya@suse.de>
> ---
> drivers/scsi/qla2xxx/qla_mbx.c | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/drivers/scsi/qla2xxx/qla_mbx.c b/drivers/scsi/qla2xxx/qla_mbx.c
> index df31ee0d59b2..b99eed55eb4a 100644
> --- a/drivers/scsi/qla2xxx/qla_mbx.c
> +++ b/drivers/scsi/qla2xxx/qla_mbx.c
> @@ -5190,7 +5190,7 @@ qla2x00_read_ram_word(scsi_qla_host_t *vha, uint32_t risc_addr, uint32_t *data)
> 	mcp->mb[8] = MSW(risc_addr);
> 	mcp->out_mb = MBX_8|MBX_1|MBX_0;
> 	mcp->in_mb = MBX_3|MBX_2|MBX_0;
> -	mcp->tov = 30;
> +	mcp->tov = MBX_TOV_SECONDS;
> 	mcp->flags = 0;
> 	rval = qla2x00_mailbox_command(vha, mcp);
> 	if (rval != QLA_SUCCESS) {
> @@ -5378,7 +5378,7 @@ qla2x00_write_ram_word(scsi_qla_host_t *vha, uint32_t risc_addr, uint32_t data)
> 	mcp->mb[8] = MSW(risc_addr);
> 	mcp->out_mb = MBX_8|MBX_3|MBX_2|MBX_1|MBX_0;
> 	mcp->in_mb = MBX_1|MBX_0;
> -	mcp->tov = 30;
> +	mcp->tov = MBX_TOV_SECONDS;
> 	mcp->flags = 0;
> 	rval = qla2x00_mailbox_command(vha, mcp);
> 	if (rval != QLA_SUCCESS) {
> @@ -5650,7 +5650,7 @@ qla24xx_set_fcp_prio(scsi_qla_host_t *vha, uint16_t loop_id, uint16_t priority,
> 	mcp->mb[9] = vha->vp_idx;
> 	mcp->out_mb = MBX_9|MBX_4|MBX_3|MBX_2|MBX_1|MBX_0;
> 	mcp->in_mb = MBX_4|MBX_3|MBX_1|MBX_0;
> -	mcp->tov = 30;
> +	mcp->tov = MBX_TOV_SECONDS;
> 	mcp->flags = 0;
> 	rval = qla2x00_mailbox_command(vha, mcp);
> 	if (mb != NULL) {
> @@ -5737,7 +5737,7 @@ qla82xx_mbx_intr_enable(scsi_qla_host_t *vha)
> 
> 	mcp->out_mb = MBX_1|MBX_0;
> 	mcp->in_mb = MBX_0;
> -	mcp->tov = 30;
> +	mcp->tov = MBX_TOV_SECONDS;
> 	mcp->flags = 0;
> 
> 	rval = qla2x00_mailbox_command(vha, mcp);
> @@ -5772,7 +5772,7 @@ qla82xx_mbx_intr_disable(scsi_qla_host_t *vha)
> 
> 	mcp->out_mb = MBX_1|MBX_0;
> 	mcp->in_mb = MBX_0;
> -	mcp->tov = 30;
> +	mcp->tov = MBX_TOV_SECONDS;
> 	mcp->flags = 0;
> 
> 	rval = qla2x00_mailbox_command(vha, mcp);
> @@ -5964,7 +5964,7 @@ qla81xx_set_led_config(scsi_qla_host_t *vha, uint16_t *led_cfg)
> 	if (IS_QLA8031(ha))
> 		mcp->out_mb |= MBX_6|MBX_5|MBX_4|MBX_3;
> 	mcp->in_mb = MBX_0;
> -	mcp->tov = 30;
> +	mcp->tov = MBX_TOV_SECONDS;
> 	mcp->flags = 0;
> 
> 	rval = qla2x00_mailbox_command(vha, mcp);
> @@ -6000,7 +6000,7 @@ qla81xx_get_led_config(scsi_qla_host_t *vha, uint16_t *led_cfg)
> 	mcp->in_mb = MBX_2|MBX_1|MBX_0;
> 	if (IS_QLA8031(ha))
> 		mcp->in_mb |= MBX_6|MBX_5|MBX_4|MBX_3;
> -	mcp->tov = 30;
> +	mcp->tov = MBX_TOV_SECONDS;
> 	mcp->flags = 0;
> 
> 	rval = qla2x00_mailbox_command(vha, mcp);
> -- 
> 2.28.0
> 

Reviewed-by: Himanshu Madhani <himanshu.madhani@oracle.com>

--
Himanshu Madhani	 Oracle Linux Engineering


^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [PATCH] scsi: qla2xxx: use MBX_TOV_SECONDS for mailbox command timeout values
  2020-08-05 20:05 [PATCH] scsi: qla2xxx: use MBX_TOV_SECONDS for mailbox command timeout values Enzo Matsumiya
  2020-08-12 19:49 ` Himanshu Madhani
@ 2020-08-13  7:00 ` Roman Bolshakov
  2020-08-18  3:12 ` Martin K. Petersen
  2 siblings, 0 replies; 4+ messages in thread
From: Roman Bolshakov @ 2020-08-13  7:00 UTC (permalink / raw)
  To: Enzo Matsumiya
  Cc: linux-scsi, Nilesh Javali, GR-QLogic-Storage-Upstream,
	James E.J. Bottomley, Martin K. Petersen

On Wed, Aug 05, 2020 at 05:05:46PM -0300, Enzo Matsumiya wrote:
> Improves readability of qla_mbx.c
> 
> Signed-off-by: Enzo Matsumiya <ematsumiya@suse.de>
> ---
>  drivers/scsi/qla2xxx/qla_mbx.c | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 

Reviewed-by: Roman Bolshakov <r.bolshakov@yadro.com>

Thanks,
Roman

^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [PATCH] scsi: qla2xxx: use MBX_TOV_SECONDS for mailbox command timeout values
  2020-08-05 20:05 [PATCH] scsi: qla2xxx: use MBX_TOV_SECONDS for mailbox command timeout values Enzo Matsumiya
  2020-08-12 19:49 ` Himanshu Madhani
  2020-08-13  7:00 ` Roman Bolshakov
@ 2020-08-18  3:12 ` Martin K. Petersen
  2 siblings, 0 replies; 4+ messages in thread
From: Martin K. Petersen @ 2020-08-18  3:12 UTC (permalink / raw)
  To: Enzo Matsumiya, linux-scsi
  Cc: Martin K . Petersen, GR-QLogic-Storage-Upstream,
	James E.J. Bottomley, Nilesh Javali

On Wed, 5 Aug 2020 17:05:46 -0300, Enzo Matsumiya wrote:

> Improves readability of qla_mbx.c

Applied to 5.9/scsi-fixes, thanks!

[1/1] scsi: qla2xxx: Use MBX_TOV_SECONDS for mailbox command timeout values
      https://git.kernel.org/mkp/scsi/c/c314a014b180

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-08-18  3:12 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-05 20:05 [PATCH] scsi: qla2xxx: use MBX_TOV_SECONDS for mailbox command timeout values Enzo Matsumiya
2020-08-12 19:49 ` Himanshu Madhani
2020-08-13  7:00 ` Roman Bolshakov
2020-08-18  3:12 ` Martin K. Petersen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).