Ping for review feedback on this patch, it'd be good to have the fix
included in the same release as the merge was included.
Joel
On 2020-04-25 20:54, Joel Johnson wrote:
Update quoted string alignment to address checkpatch.pl warning
originally introduced in
commit f60a66ef5d7d ("cmd: mvebu: bubt: show image boot device").
Signed-off-by: Joel Johnson <mrj...@lixil.net>
---
I am sort of inferring that this is the desired fix, preferring line
length infractions over split quoted strings. As such, I've combined
the
static part of the string for grepping and left the dynamic parts on
the
next line.
v2 change:
- syntax fix
v3 change:
- actually add syntax fix...
---
cmd/mvebu/bubt.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/cmd/mvebu/bubt.c b/cmd/mvebu/bubt.c
index dc0ce01c35..ef53153c46 100644
--- a/cmd/mvebu/bubt.c
+++ b/cmd/mvebu/bubt.c
@@ -730,8 +730,8 @@ static int bubt_check_boot_mode(const struct
bubt_dev *dst)
for (int i = 0; i < ARRAY_SIZE(a38x_boot_modes); i++) {
if (a38x_boot_modes[i].id == hdr->blockid) {
- printf("Error: A38x image meant to be "
- "booted from \"%s\", not \"%s\"!\n",
+ printf("Error: A38x image meant to be booted from
"
+ "\"%s\", not \"%s\"!\n",
a38x_boot_modes[i].name, dst->name);
return -ENOEXEC;
}