* [PATCH][next] mtd: lpddr: Remove space before newline
@ 2025-07-31 9:52 Colin Ian King
2025-07-31 15:58 ` Dan Carpenter
2025-09-01 12:29 ` Miquel Raynal
0 siblings, 2 replies; 3+ messages in thread
From: Colin Ian King @ 2025-07-31 9:52 UTC (permalink / raw)
To: Miquel Raynal, Richard Weinberger, Vignesh Raghavendra, linux-mtd
Cc: kernel-janitors, linux-kernel
There is an extraneous space before a newline in a handful of printk
messages. Remove the spaces.
Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
---
drivers/mtd/lpddr/lpddr_cmds.c | 10 +++++-----
drivers/mtd/lpddr/qinfo_probe.c | 4 ++--
2 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/mtd/lpddr/lpddr_cmds.c b/drivers/mtd/lpddr/lpddr_cmds.c
index 14e36ae71958..290fd0119e98 100644
--- a/drivers/mtd/lpddr/lpddr_cmds.c
+++ b/drivers/mtd/lpddr/lpddr_cmds.c
@@ -142,7 +142,7 @@ static int wait_for_ready(struct map_info *map, struct flchip *chip,
if (dsr & DSR_READY_STATUS)
break;
if (!timeo) {
- printk(KERN_ERR "%s: Flash timeout error state %d \n",
+ printk(KERN_ERR "%s: Flash timeout error state %d\n",
map->name, chip_state);
ret = -ETIME;
break;
@@ -186,7 +186,7 @@ static int wait_for_ready(struct map_info *map, struct flchip *chip,
if (dsr & DSR_ERR) {
/* Clear DSR*/
map_write(map, CMD(~(DSR_ERR)), map->pfow_base + PFOW_DSR);
- printk(KERN_WARNING"%s: Bad status on wait: 0x%x \n",
+ printk(KERN_WARNING"%s: Bad status on wait: 0x%x\n",
map->name, dsr);
print_drs_error(dsr);
ret = -EIO;
@@ -321,7 +321,7 @@ static int chip_ready(struct map_info *map, struct flchip *chip, int mode)
/* Resume and pretend we weren't here. */
put_chip(map, chip);
printk(KERN_ERR "%s: suspend operation failed."
- "State may be wrong \n", map->name);
+ "State may be wrong\n", map->name);
return -EIO;
}
chip->erase_suspended = 1;
@@ -468,7 +468,7 @@ static int do_write_buffer(struct map_info *map, struct flchip *chip,
chip->state = FL_WRITING;
ret = wait_for_ready(map, chip, (1<<lpddr->qinfo->ProgBufferTime));
if (ret) {
- printk(KERN_WARNING"%s Buffer program error: %d at %lx; \n",
+ printk(KERN_WARNING"%s Buffer program error: %d at %lx\n",
map->name, ret, adr);
goto out;
}
@@ -736,7 +736,7 @@ static int do_xxlock(struct mtd_info *mtd, loff_t adr, uint32_t len, int thunk)
ret = wait_for_ready(map, chip, 1);
if (ret) {
- printk(KERN_ERR "%s: block unlock error status %d \n",
+ printk(KERN_ERR "%s: block unlock error status %d\n",
map->name, ret);
goto out;
}
diff --git a/drivers/mtd/lpddr/qinfo_probe.c b/drivers/mtd/lpddr/qinfo_probe.c
index 137ae5f0a19b..42281e460c62 100644
--- a/drivers/mtd/lpddr/qinfo_probe.c
+++ b/drivers/mtd/lpddr/qinfo_probe.c
@@ -55,7 +55,7 @@ static long lpddr_get_qinforec_pos(struct map_info *map, char *id_str)
return minor | (major << bankwidth);
}
}
- printk(KERN_ERR"%s qinfo id string is wrong! \n", map->name);
+ printk(KERN_ERR"%s qinfo id string is wrong!\n", map->name);
BUG();
return -1;
}
@@ -112,7 +112,7 @@ static int lpddr_pfow_present(struct map_info *map, struct lpddr_private *lpddr)
return 1; /* "PFOW" is found */
out:
- printk(KERN_WARNING"%s: PFOW string at 0x%lx is not found \n",
+ printk(KERN_WARNING"%s: PFOW string at 0x%lx is not found\n",
map->name, map->pfow_base);
return 0;
}
--
2.50.0
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH][next] mtd: lpddr: Remove space before newline
2025-07-31 9:52 [PATCH][next] mtd: lpddr: Remove space before newline Colin Ian King
@ 2025-07-31 15:58 ` Dan Carpenter
2025-09-01 12:29 ` Miquel Raynal
1 sibling, 0 replies; 3+ messages in thread
From: Dan Carpenter @ 2025-07-31 15:58 UTC (permalink / raw)
To: Colin Ian King
Cc: Miquel Raynal, Richard Weinberger, Vignesh Raghavendra,
linux-mtd, kernel-janitors, linux-kernel
On Thu, Jul 31, 2025 at 10:52:47AM +0100, Colin Ian King wrote:
> @@ -321,7 +321,7 @@ static int chip_ready(struct map_info *map, struct flchip *chip, int mode)
> /* Resume and pretend we weren't here. */
> put_chip(map, chip);
> printk(KERN_ERR "%s: suspend operation failed."
> - "State may be wrong \n", map->name);
> + "State may be wrong\n", map->name);
Not related to your patch, but ideally there would be a space after
the period.
regards,
dan carpenter
> return -EIO;
> }
> chip->erase_suspended = 1;
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH][next] mtd: lpddr: Remove space before newline
2025-07-31 9:52 [PATCH][next] mtd: lpddr: Remove space before newline Colin Ian King
2025-07-31 15:58 ` Dan Carpenter
@ 2025-09-01 12:29 ` Miquel Raynal
1 sibling, 0 replies; 3+ messages in thread
From: Miquel Raynal @ 2025-09-01 12:29 UTC (permalink / raw)
To: Richard Weinberger, Vignesh Raghavendra, linux-mtd, Colin Ian King
Cc: kernel-janitors, linux-kernel
On Thu, 31 Jul 2025 10:52:47 +0100, Colin Ian King wrote:
> There is an extraneous space before a newline in a handful of printk
> messages. Remove the spaces.
>
>
Applied to mtd/next, thanks!
[1/1] mtd: lpddr: Remove space before newline
commit: 4b7ea37ebb6ea59724f69e3888e7b276014e17e8
Patche(s) should be available on mtd/linux.git and will be
part of the next PR (provided that no robot complains by then).
Kind regards,
Miquèl
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-09-01 12:29 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-07-31 9:52 [PATCH][next] mtd: lpddr: Remove space before newline Colin Ian King
2025-07-31 15:58 ` Dan Carpenter
2025-09-01 12:29 ` Miquel Raynal
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®