Re: [PATCH AUTOSEL 4.9 17/18] scsi: be2iscsi: Fix some missing space in some messages

From: Marion & Christophe JAILLET
Date: Fri Jul 16 2021 - 15:22:50 EST

Le 14/07/2021 à 21:48, Sasha Levin a écrit :
From: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>

[ Upstream commit c7fa2c855e892721bafafdf6393342c000e0ef77 ]

Fix a few style issues reported by

- Avoid duplicated word in comment.

- Add missing space in messages.

- Unneeded continuation line character.

- Unneeded extra spaces.

- Unneeded log message after memory allocation failure.

Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>
Signed-off-by: Martin K. Petersen <martin.petersen@xxxxxxxxxx>
Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>


I always appreciate to have some patches backported, but in this particular case, I wonder what is the rational to backport up to 4.9 some checkpatch warning about log message?

Keeping code aligned to ease other future backport?

I thought that the rule for backport was that it needed to fix a real issue (and sometimes a real 'potential' issue)