diff mbox series

[4/4] ARM: omap2+: Downgrade u-boot version warnings to debug statements

Message ID 20230911040828.39386-4-tony@atomide.com
State New
Headers show
Series [1/4] ARM: dts: ti: omap: Fix bandgap thermal cells addressing for omap3/4 | expand

Commit Message

Tony Lindgren Sept. 11, 2023, 4:08 a.m. UTC
We should be able to see real issues with dmesg -l err,warn. The u-boot
revision warning should be a debug statement rather than a warning.

Cc: Ivaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>
Cc: Carl Philipp Klemm <philipp@uvos.xyz>
Cc: Merlijn Wajer <merlijn@wizzup.org>
Cc: Pavel Machek <pavel@ucw.cz>
Cc: Sebastian Reichel <sre@kernel.org>
Signed-off-by: Tony Lindgren <tony@atomide.com>
---
 arch/arm/mach-omap2/pm44xx.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Sebastian Reichel Sept. 12, 2023, 3:27 p.m. UTC | #1
Hi,

On Mon, Sep 11, 2023 at 07:08:27AM +0300, Tony Lindgren wrote:
> We should be able to see real issues with dmesg -l err,warn. The u-boot
> revision warning should be a debug statement rather than a warning.
> 
> Cc: Ivaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>
> Cc: Carl Philipp Klemm <philipp@uvos.xyz>
> Cc: Merlijn Wajer <merlijn@wizzup.org>
> Cc: Pavel Machek <pavel@ucw.cz>
> Cc: Sebastian Reichel <sre@kernel.org>
> Signed-off-by: Tony Lindgren <tony@atomide.com>
> ---

Reviewed-by: Sebastian Reichel <sebastian.reichel@collabora.com>

-- Sebastian

>  arch/arm/mach-omap2/pm44xx.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/pm44xx.c b/arch/arm/mach-omap2/pm44xx.c
> --- a/arch/arm/mach-omap2/pm44xx.c
> +++ b/arch/arm/mach-omap2/pm44xx.c
> @@ -99,7 +99,7 @@ static int omap4_pm_suspend(void)
>  		 * possible causes.
>  		 * http://www.spinics.net/lists/arm-kernel/msg218641.html
>  		 */
> -		pr_warn("A possible cause could be an old bootloader - try u-boot >= v2012.07\n");
> +		pr_debug("A possible cause could be an old bootloader - try u-boot >= v2012.07\n");
>  	} else {
>  		pr_info("Successfully put all powerdomains to target state\n");
>  	}
> @@ -257,7 +257,7 @@ int __init omap4_pm_init(void)
>  	 * http://www.spinics.net/lists/arm-kernel/msg218641.html
>  	 */
>  	if (cpu_is_omap44xx())
> -		pr_warn("OMAP4 PM: u-boot >= v2012.07 is required for full PM support\n");
> +		pr_debug("OMAP4 PM: u-boot >= v2012.07 is required for full PM support\n");
>  
>  	ret = pwrdm_for_each(pwrdms_setup, NULL);
>  	if (ret) {
> -- 
> 2.42.0
diff mbox series

Patch

diff --git a/arch/arm/mach-omap2/pm44xx.c b/arch/arm/mach-omap2/pm44xx.c
--- a/arch/arm/mach-omap2/pm44xx.c
+++ b/arch/arm/mach-omap2/pm44xx.c
@@ -99,7 +99,7 @@  static int omap4_pm_suspend(void)
 		 * possible causes.
 		 * http://www.spinics.net/lists/arm-kernel/msg218641.html
 		 */
-		pr_warn("A possible cause could be an old bootloader - try u-boot >= v2012.07\n");
+		pr_debug("A possible cause could be an old bootloader - try u-boot >= v2012.07\n");
 	} else {
 		pr_info("Successfully put all powerdomains to target state\n");
 	}
@@ -257,7 +257,7 @@  int __init omap4_pm_init(void)
 	 * http://www.spinics.net/lists/arm-kernel/msg218641.html
 	 */
 	if (cpu_is_omap44xx())
-		pr_warn("OMAP4 PM: u-boot >= v2012.07 is required for full PM support\n");
+		pr_debug("OMAP4 PM: u-boot >= v2012.07 is required for full PM support\n");
 
 	ret = pwrdm_for_each(pwrdms_setup, NULL);
 	if (ret) {