mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH master] Revert "ARM: beaglebone: init MPU speed to 800Mhz"
@ 2021-12-03 18:51 Ahmad Fatoum
  2021-12-03 18:53 ` Ahmad Fatoum
  2021-12-07  8:18 ` Sascha Hauer
  0 siblings, 2 replies; 3+ messages in thread
From: Ahmad Fatoum @ 2021-12-03 18:51 UTC (permalink / raw)
  To: barebox; +Cc: Ahmad Fatoum

Commit 130c7d6715e9 ("ARM: beaglebone: init MPU speed to 800Mhz") causes
warm reboot from within Linux v5.14.0 to no longer succeed.
barebox MLO is entered, but hangs. Reset from within barebox is
unaffected.
This can be reproduced using DistroKit 949c5ef2b9a6 ("v7a: barebox:
version bump v2021.08.0 → v2021.11.0") on a Beaglebone Black.
Until that's figured out, revert the commit; functional reboot is more
important than the quicker boot up.

This reverts commit 130c7d6715e932a1e0b3e026fea97f700ab33ea9.

Fixes: 130c7d6715e9 ("ARM: beaglebone: init MPU speed to 800Mhz")
Reported-by: arwie # IRC
Cc: Marc Reilly <marc@cpdesign.com.au>
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
---
 arch/arm/boards/beaglebone/lowlevel.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boards/beaglebone/lowlevel.c b/arch/arm/boards/beaglebone/lowlevel.c
index 31211448f5e8..91d143e4150b 100644
--- a/arch/arm/boards/beaglebone/lowlevel.c
+++ b/arch/arm/boards/beaglebone/lowlevel.c
@@ -126,7 +126,7 @@ static noinline int beaglebone_sram_init(void)
 
 	/* Setup the PLLs and the clocks for the peripherals */
 	if (is_beaglebone_black()) {
-		am33xx_pll_init(MPUPLL_M_800, DDRPLL_M_400);
+		am33xx_pll_init(MPUPLL_M_500, DDRPLL_M_400);
 		am335x_sdram_init(0x18B, &ddr3_cmd_ctrl, &ddr3_regs,
 				&ddr3_data);
 	} else {
-- 
2.30.2


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH master] Revert "ARM: beaglebone: init MPU speed to 800Mhz"
  2021-12-03 18:51 [PATCH master] Revert "ARM: beaglebone: init MPU speed to 800Mhz" Ahmad Fatoum
@ 2021-12-03 18:53 ` Ahmad Fatoum
  2021-12-07  8:18 ` Sascha Hauer
  1 sibling, 0 replies; 3+ messages in thread
From: Ahmad Fatoum @ 2021-12-03 18:53 UTC (permalink / raw)
  To: barebox, Marc Reilly

Hello Marc,

On 03.12.21 19:51, Ahmad Fatoum wrote:
> Commit 130c7d6715e9 ("ARM: beaglebone: init MPU speed to 800Mhz") causes
> warm reboot from within Linux v5.14.0 to no longer succeed.
> barebox MLO is entered, but hangs. Reset from within barebox is
> unaffected.
> This can be reproduced using DistroKit 949c5ef2b9a6 ("v7a: barebox:
> version bump v2021.08.0 → v2021.11.0") on a Beaglebone Black.
> Until that's figured out, revert the commit; functional reboot is more
> important than the quicker boot up.
> 
> This reverts commit 130c7d6715e932a1e0b3e026fea97f700ab33ea9.

I send a revert, so barebox v2021.12.0 can ship with the regression fixed.
Would be great if you could take a look to see what needs to be done
differently to reinstate the clock frequency increase.

Cheers,
Ahmad

> 
> Fixes: 130c7d6715e9 ("ARM: beaglebone: init MPU speed to 800Mhz")
> Reported-by: arwie # IRC
> Cc: Marc Reilly <marc@cpdesign.com.au>
> Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
> ---
>  arch/arm/boards/beaglebone/lowlevel.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boards/beaglebone/lowlevel.c b/arch/arm/boards/beaglebone/lowlevel.c
> index 31211448f5e8..91d143e4150b 100644
> --- a/arch/arm/boards/beaglebone/lowlevel.c
> +++ b/arch/arm/boards/beaglebone/lowlevel.c
> @@ -126,7 +126,7 @@ static noinline int beaglebone_sram_init(void)
>  
>  	/* Setup the PLLs and the clocks for the peripherals */
>  	if (is_beaglebone_black()) {
> -		am33xx_pll_init(MPUPLL_M_800, DDRPLL_M_400);
> +		am33xx_pll_init(MPUPLL_M_500, DDRPLL_M_400);
>  		am335x_sdram_init(0x18B, &ddr3_cmd_ctrl, &ddr3_regs,
>  				&ddr3_data);
>  	} else {
> 


-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH master] Revert "ARM: beaglebone: init MPU speed to 800Mhz"
  2021-12-03 18:51 [PATCH master] Revert "ARM: beaglebone: init MPU speed to 800Mhz" Ahmad Fatoum
  2021-12-03 18:53 ` Ahmad Fatoum
@ 2021-12-07  8:18 ` Sascha Hauer
  1 sibling, 0 replies; 3+ messages in thread
From: Sascha Hauer @ 2021-12-07  8:18 UTC (permalink / raw)
  To: Ahmad Fatoum; +Cc: barebox

On Fri, Dec 03, 2021 at 07:51:54PM +0100, Ahmad Fatoum wrote:
> Commit 130c7d6715e9 ("ARM: beaglebone: init MPU speed to 800Mhz") causes
> warm reboot from within Linux v5.14.0 to no longer succeed.
> barebox MLO is entered, but hangs. Reset from within barebox is
> unaffected.
> This can be reproduced using DistroKit 949c5ef2b9a6 ("v7a: barebox:
> version bump v2021.08.0 → v2021.11.0") on a Beaglebone Black.
> Until that's figured out, revert the commit; functional reboot is more
> important than the quicker boot up.
> 
> This reverts commit 130c7d6715e932a1e0b3e026fea97f700ab33ea9.
> 
> Fixes: 130c7d6715e9 ("ARM: beaglebone: init MPU speed to 800Mhz")
> Reported-by: arwie # IRC
> Cc: Marc Reilly <marc@cpdesign.com.au>
> Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
> ---
>  arch/arm/boards/beaglebone/lowlevel.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied, thanks

Sascha

> 
> diff --git a/arch/arm/boards/beaglebone/lowlevel.c b/arch/arm/boards/beaglebone/lowlevel.c
> index 31211448f5e8..91d143e4150b 100644
> --- a/arch/arm/boards/beaglebone/lowlevel.c
> +++ b/arch/arm/boards/beaglebone/lowlevel.c
> @@ -126,7 +126,7 @@ static noinline int beaglebone_sram_init(void)
>  
>  	/* Setup the PLLs and the clocks for the peripherals */
>  	if (is_beaglebone_black()) {
> -		am33xx_pll_init(MPUPLL_M_800, DDRPLL_M_400);
> +		am33xx_pll_init(MPUPLL_M_500, DDRPLL_M_400);
>  		am335x_sdram_init(0x18B, &ddr3_cmd_ctrl, &ddr3_regs,
>  				&ddr3_data);
>  	} else {
> -- 
> 2.30.2
> 
> 
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-12-07  8:55 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-03 18:51 [PATCH master] Revert "ARM: beaglebone: init MPU speed to 800Mhz" Ahmad Fatoum
2021-12-03 18:53 ` Ahmad Fatoum
2021-12-07  8:18 ` Sascha Hauer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox