[PATCH][next] scsi: mpt3sas: Remove trailing space after \n newline

Colin Ian King posted 1 patch 1 year, 3 months ago
There is a newer version of this series
drivers/scsi/mpt3sas/mpt3sas_base.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
[PATCH][next] scsi: mpt3sas: Remove trailing space after \n newline
Posted by Colin Ian King 1 year, 3 months ago
There is a extraneous space after a newline in an ioc_info message.
Remove it.

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
---
 drivers/scsi/mpt3sas/mpt3sas_base.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/scsi/mpt3sas/mpt3sas_base.c b/drivers/scsi/mpt3sas/mpt3sas_base.c
index 9a24f7776d64..ebe4cbbc16e4 100644
--- a/drivers/scsi/mpt3sas/mpt3sas_base.c
+++ b/drivers/scsi/mpt3sas/mpt3sas_base.c
@@ -8899,8 +8899,8 @@ _base_check_ioc_facts_changes(struct MPT3SAS_ADAPTER *ioc)
 		if (!device_remove_in_progress) {
 			ioc_info(ioc,
 			    "Unable to allocate the memory for "
-			    "device_remove_in_progress of sz: %d\n "
-			    , pd_handles_sz);
+			    "device_remove_in_progress of sz: %d\n",
+			    pd_handles_sz);
 			return -ENOMEM;
 		}
 		memset(device_remove_in_progress +
-- 
2.39.2
Re: [PATCH][next] scsi: mpt3sas: Remove trailing space after \n newline
Posted by Christophe JAILLET 1 year, 3 months ago
Le 02/09/2024 à 16:36, Colin Ian King a écrit :
> There is a extraneous space after a newline in an ioc_info message.
> Remove it.
> 
> Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
> ---
>   drivers/scsi/mpt3sas/mpt3sas_base.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/scsi/mpt3sas/mpt3sas_base.c b/drivers/scsi/mpt3sas/mpt3sas_base.c
> index 9a24f7776d64..ebe4cbbc16e4 100644
> --- a/drivers/scsi/mpt3sas/mpt3sas_base.c
> +++ b/drivers/scsi/mpt3sas/mpt3sas_base.c
> @@ -8899,8 +8899,8 @@ _base_check_ioc_facts_changes(struct MPT3SAS_ADAPTER *ioc)
>   		if (!device_remove_in_progress) {
>   			ioc_info(ioc,
>   			    "Unable to allocate the memory for "
> -			    "device_remove_in_progress of sz: %d\n "
> -			    , pd_handles_sz);
> +			    "device_remove_in_progress of sz: %d\n",

Hi,

I think that the 2 parts of the string should be put on the same line.

Another call just a few lines above is already written like that.

CJ

> +			    pd_handles_sz);
>   			return -ENOMEM;
>   		}
>   		memset(device_remove_in_progress +