* [PATCH v2] power: reset: keystone: fix missing space in error message
@ 2026-09-07 9:12 Hemanth Selam
2026-09-09 20:49 ` Sebastian Reichel
0 siblings, 1 reply; 2+ messages in thread
From: Hemanth Selam @ 2026-09-07 9:12 UTC (permalink / raw)
To: Santosh Shilimkar, Sebastian Reichel
Cc: linux-kernel, linux-pm, Krzysztof Kozlowski
The two string literals are concatenated, so the message prints
"has to contain atleast one entry".
v1 added the missing space but kept the string split across two lines.
Krzysztof Kozlowski pointed out the string should not be wrapped at all,
which also matches coding-style.rst: never break user-visible strings,
because that breaks the ability to grep for them. Join it into one
line instead; the result exceeds 80 columns, which is expected for a
user-visible string.
Assisted-by: Cursor:claude-opus-5
Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>
---
drivers/power/reset/keystone-reset.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/power/reset/keystone-reset.c b/drivers/power/reset/keystone-reset.c
index 3c44cd6cee0a..491ba490b8ea 100644
--- a/drivers/power/reset/keystone-reset.c
+++ b/drivers/power/reset/keystone-reset.c
@@ -112,8 +112,7 @@ static int rsctrl_probe(struct platform_device *pdev)
for (i = 0; i < WDT_MUX_NUMBER; i++) {
ret = of_property_read_u32_index(np, "ti,wdt-list", i, &val);
if (ret == -EOVERFLOW && !i) {
- dev_err(dev, "ti,wdt-list property has to contain at"
- "least one entry\n");
+ dev_err(dev, "ti,wdt-list property has to contain at least one entry\n");
return -EINVAL;
} else if (ret) {
break;
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH v2] power: reset: keystone: fix missing space in error message
2026-09-07 9:12 [PATCH v2] power: reset: keystone: fix missing space in error message Hemanth Selam
@ 2026-09-09 20:49 ` Sebastian Reichel
0 siblings, 0 replies; 2+ messages in thread
From: Sebastian Reichel @ 2026-09-09 20:49 UTC (permalink / raw)
To: Santosh Shilimkar, Sebastian Reichel, Hemanth Selam
Cc: linux-kernel, linux-pm, Krzysztof Kozlowski
On Mon, 07 Sep 2026 14:42:12 +0530, Hemanth Selam wrote:
> The two string literals are concatenated, so the message prints
> "has to contain atleast one entry".
>
> v1 added the missing space but kept the string split across two lines.
> Krzysztof Kozlowski pointed out the string should not be wrapped at all,
> which also matches coding-style.rst: never break user-visible strings,
> because that breaks the ability to grep for them. Join it into one
> line instead; the result exceeds 80 columns, which is expected for a
> user-visible string.
>
> [...]
Applied, thanks!
[1/1] power: reset: keystone: fix missing space in error message
commit: 49ee04619f820ac0e5ca618dfc0657229eadeddc
Best regards,
--
Sebastian Reichel <sebastian.reichel@collabora.com>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-09-09 20:49 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-09-07 9:12 [PATCH v2] power: reset: keystone: fix missing space in error message Hemanth Selam
2026-09-09 20:49 ` Sebastian Reichel
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®