[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <3a66d740-6764-de25-097f-c910fe5eb5de@suse.de>
Date: Wed, 14 Sep 2016 14:46:54 +0200
From: Hannes Reinecke <hare@...e.de>
To: John Garry <john.garry@...wei.com>, jejb@...ux.vnet.ibm.com,
martin.petersen@...cle.com
Cc: linuxarm@...wei.com, zhangfei.gao@...aro.org, xuwei5@...ilicon.com,
john.garry2@...l.dcu.ie, linux-scsi@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 06/15] hisi_sas: disable dlvry queues once at reset for v2
hw
On 09/06/2016 05:36 PM, John Garry wrote:
> The Delivery queue enable register should only be written
> to once at reset for v2 hw.
>
> Signed-off-by: John Garry <john.garry@...wei.com>
> ---
> drivers/scsi/hisi_sas/hisi_sas_v2_hw.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/scsi/hisi_sas/hisi_sas_v2_hw.c b/drivers/scsi/hisi_sas/hisi_sas_v2_hw.c
> index 25e7c58..aca2b80 100644
> --- a/drivers/scsi/hisi_sas/hisi_sas_v2_hw.c
> +++ b/drivers/scsi/hisi_sas/hisi_sas_v2_hw.c
> @@ -685,9 +685,7 @@ static int reset_hw_v2_hw(struct hisi_hba *hisi_hba)
> else
> reset_val = 0x7ffff;
>
> - /* Disable all of the DQ */
> - for (i = 0; i < HISI_SAS_MAX_QUEUES; i++)
> - hisi_sas_write32(hisi_hba, DLVRY_QUEUE_ENABLE, 0);
> + hisi_sas_write32(hisi_hba, DLVRY_QUEUE_ENABLE, 0);
>
> /* Disable all of the PHYs */
> for (i = 0; i < hisi_hba->n_phy; i++) {
>
Reviewed-by: Hannes Reinecke <hare@...e.com>
Cheers,
Hannes
--
Dr. Hannes Reinecke zSeries & Storage
hare@...e.de +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg)
Powered by blists - more mailing lists