Re: [PATCH 2/2] power: reset: add missing space in error message
From: Krzysztof Kozlowski
Date: Mon Sep 07 2026 - 03:25:49 EST
On 07/09/2026 08:39, Hemanth Selam wrote:
> The two string literals are concatenated, so the message prints "has to
> contain atleast one entry". Add the missing space.
>
> checkpatch.pl misses this one: neither literal is misspelled on its
> own, the fault only appears once they are joined.
>
> Only the message text changes, no code changes.
>
> Assisted-by: Cursor:claude-opus-5
> Signed-off-by: Hemanth Selam <hemanth.selam@xxxxxxxxx>
> ---
> drivers/power/reset/keystone-reset.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/power/reset/keystone-reset.c b/drivers/power/reset/keystone-reset.c
> index 3c44cd6cee0a..dbf2e4274dc6 100644
> --- a/drivers/power/reset/keystone-reset.c
> +++ b/drivers/power/reset/keystone-reset.c
> @@ -112,7 +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"
> + dev_err(dev, "ti,wdt-list property has to contain at "
> "least one entry\n");
Instead, string should not be wrapped.
You need to improve your interactions/prompts with your AI slop tool, so
it will follow Linux kernel coding style.
Best regards,
Krzysztof