[PATCH] pinctrl: Demote subsystem banner message

Linus Walleij posted 1 patch 4 months ago
drivers/pinctrl/core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] pinctrl: Demote subsystem banner message
Posted by Linus Walleij 4 months ago
There is no reason to print any "hello world" from pin control
unless (maybe) if we are debugging.

Drop the banner.

Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
 drivers/pinctrl/core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pinctrl/core.c b/drivers/pinctrl/core.c
index c5dbf4e9db84424ae9ceb90ed3681341654d1871..157510157d47aad96b459abf68cc0e4eed957f2d 100644
--- a/drivers/pinctrl/core.c
+++ b/drivers/pinctrl/core.c
@@ -2416,7 +2416,7 @@ EXPORT_SYMBOL_GPL(devm_pinctrl_unregister);
 
 static int __init pinctrl_init(void)
 {
-	pr_info("initialized pinctrl subsystem\n");
+	pr_debug("initialized pinctrl subsystem\n");
 	pinctrl_init_debugfs();
 	return 0;
 }

---
base-commit: c746c3b5169831d7fb032a1051d8b45592ae8d78
change-id: 20251007-pinctrl-drop-advert-d1150d425942

Best regards,
-- 
Linus Walleij <linus.walleij@linaro.org>
Re: [PATCH] pinctrl: Demote subsystem banner message
Posted by Linus Walleij 3 months, 3 weeks ago
On Tue, Oct 7, 2025 at 3:18 PM Linus Walleij <linus.walleij@linaro.org> wrote:

> There is no reason to print any "hello world" from pin control
> unless (maybe) if we are debugging.
>
> Drop the banner.
>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

Patch applied!

Yours,
Linus Walleij
Re: [PATCH] pinctrl: Demote subsystem banner message
Posted by Bartosz Golaszewski 4 months ago
On Tue, 7 Oct 2025 15:18:50 +0200, Linus Walleij
<linus.walleij@linaro.org> said:
> There is no reason to print any "hello world" from pin control
> unless (maybe) if we are debugging.
>
> Drop the banner.
>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
>  drivers/pinctrl/core.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/pinctrl/core.c b/drivers/pinctrl/core.c
> index c5dbf4e9db84424ae9ceb90ed3681341654d1871..157510157d47aad96b459abf68cc0e4eed957f2d 100644
> --- a/drivers/pinctrl/core.c
> +++ b/drivers/pinctrl/core.c
> @@ -2416,7 +2416,7 @@ EXPORT_SYMBOL_GPL(devm_pinctrl_unregister);
>
>  static int __init pinctrl_init(void)
>  {
> -	pr_info("initialized pinctrl subsystem\n");
> +	pr_debug("initialized pinctrl subsystem\n");
>  	pinctrl_init_debugfs();
>  	return 0;
>  }
>
> ---
> base-commit: c746c3b5169831d7fb032a1051d8b45592ae8d78
> change-id: 20251007-pinctrl-drop-advert-d1150d425942
>
> Best regards,
> --
> Linus Walleij <linus.walleij@linaro.org>
>
>
>

Good idea!

Reviewed-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>