2020-11-24 20:31:24

by Colin King

[permalink] [raw]
Subject: [PATCH][next] scsi: pm8001: remove space in a debug message

From: Colin Ian King <[email protected]>

There are two words that need separating with a space in a
pm8001_dbg message. Fix it.

Signed-off-by: Colin Ian King <[email protected]>
---
drivers/scsi/pm8001/pm8001_hwi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/scsi/pm8001/pm8001_hwi.c b/drivers/scsi/pm8001/pm8001_hwi.c
index 08d6cc9b50db..c8d4d87c5473 100644
--- a/drivers/scsi/pm8001/pm8001_hwi.c
+++ b/drivers/scsi/pm8001/pm8001_hwi.c
@@ -1031,7 +1031,7 @@ pm8001_chip_soft_rst(struct pm8001_hba_info *pm8001_ha)
regVal = pm8001_cr32(pm8001_ha, 2, GSM_WRITE_DATA_PARITY_CHECK);
pm8001_cw32(pm8001_ha, 2, GSM_WRITE_DATA_PARITY_CHECK, regVal3);
pm8001_dbg(pm8001_ha, INIT,
- "GSM 0x700048 - Write Data Parity Check Enableis set to = 0x%x\n",
+ "GSM 0x700048 - Write Data Parity Check Enable is set to = 0x%x\n",
pm8001_cr32(pm8001_ha, 2, GSM_WRITE_DATA_PARITY_CHECK));

/* step 13: bring the IOP and AAP1 out of reset */
--
2.29.2


2020-11-25 00:05:34

by Ewan Milne

[permalink] [raw]
Subject: Re: [PATCH][next] scsi: pm8001: remove space in a debug message

On Tue, 2020-11-24 at 09:38 +0000, Colin King wrote:
> From: Colin Ian King <[email protected]>
>
> There are two words that need separating with a space in a
> pm8001_dbg message. Fix it.
>
> Signed-off-by: Colin Ian King <[email protected]>
> ---
> drivers/scsi/pm8001/pm8001_hwi.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/scsi/pm8001/pm8001_hwi.c
> b/drivers/scsi/pm8001/pm8001_hwi.c
> index 08d6cc9b50db..c8d4d87c5473 100644
> --- a/drivers/scsi/pm8001/pm8001_hwi.c
> +++ b/drivers/scsi/pm8001/pm8001_hwi.c
> @@ -1031,7 +1031,7 @@ pm8001_chip_soft_rst(struct pm8001_hba_info
> *pm8001_ha)
> regVal = pm8001_cr32(pm8001_ha, 2,
> GSM_WRITE_DATA_PARITY_CHECK);
> pm8001_cw32(pm8001_ha, 2, GSM_WRITE_DATA_PARITY_CHECK,
> regVal3);
> pm8001_dbg(pm8001_ha, INIT,
> - "GSM 0x700048 - Write Data Parity Check Enableis set
> to = 0x%x\n",
> + "GSM 0x700048 - Write Data Parity Check Enable is
> set to = 0x%x\n",
> pm8001_cr32(pm8001_ha, 2,
> GSM_WRITE_DATA_PARITY_CHECK));
>
> /* step 13: bring the IOP and AAP1 out of reset */

Reviewed-by: Ewan D. Milne <[email protected]>


2020-11-26 14:05:38

by Jinpu Wang

[permalink] [raw]
Subject: Re: [PATCH][next] scsi: pm8001: remove space in a debug message

On Tue, Nov 24, 2020 at 10:38 AM Colin King <[email protected]> wrote:
>
> From: Colin Ian King <[email protected]>
>
> There are two words that need separating with a space in a
> pm8001_dbg message. Fix it.
>
> Signed-off-by: Colin Ian King <[email protected]>
Acked-by: Jack Wang <[email protected]>
Thanks
> ---
> drivers/scsi/pm8001/pm8001_hwi.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/scsi/pm8001/pm8001_hwi.c b/drivers/scsi/pm8001/pm8001_hwi.c
> index 08d6cc9b50db..c8d4d87c5473 100644
> --- a/drivers/scsi/pm8001/pm8001_hwi.c
> +++ b/drivers/scsi/pm8001/pm8001_hwi.c
> @@ -1031,7 +1031,7 @@ pm8001_chip_soft_rst(struct pm8001_hba_info *pm8001_ha)
> regVal = pm8001_cr32(pm8001_ha, 2, GSM_WRITE_DATA_PARITY_CHECK);
> pm8001_cw32(pm8001_ha, 2, GSM_WRITE_DATA_PARITY_CHECK, regVal3);
> pm8001_dbg(pm8001_ha, INIT,
> - "GSM 0x700048 - Write Data Parity Check Enableis set to = 0x%x\n",
> + "GSM 0x700048 - Write Data Parity Check Enable is set to = 0x%x\n",
> pm8001_cr32(pm8001_ha, 2, GSM_WRITE_DATA_PARITY_CHECK));
>
> /* step 13: bring the IOP and AAP1 out of reset */
> --
> 2.29.2
>

2020-12-01 04:40:20

by Martin K. Petersen

[permalink] [raw]
Subject: Re: [PATCH][next] scsi: pm8001: remove space in a debug message


Colin,

> There are two words that need separating with a space in a
> pm8001_dbg message. Fix it.

Applied to 5.11/scsi-staging, thanks!

--
Martin K. Petersen Oracle Linux Engineering

2020-12-08 04:58:17

by Martin K. Petersen

[permalink] [raw]
Subject: Re: [PATCH][next] scsi: pm8001: remove space in a debug message

On Tue, 24 Nov 2020 09:38:28 +0000, Colin King wrote:

> There are two words that need separating with a space in a
> pm8001_dbg message. Fix it.

Applied to 5.11/scsi-queue, thanks!

[1/1] scsi: pm8001: remove space in a debug message
https://git.kernel.org/mkp/scsi/c/c6131854e28a

--
Martin K. Petersen Oracle Linux Engineering