[PULL] Pull request for u-boot master / v2023.01 = u-boot-stm32-20230106

[PULL] Pull request for u-boot master / v2023.01 = u-boot-stm32-20230106

[Patrick DELAUNAY]

Hi Tom,

Please pull the STM32 related fixes for u-boot/master, v2023.01: 
u-boot-stm32-20230106


- stm32mp: Fix board_get_usable_ram_top(): workaround to avoid issue 
after the

   commit 777aaaa706b ("common/memsize.c: Fix get_effective_memsize() to 
check

   for overflow") because the effective DDR effective size is reduce by 4KiB
   and sometime the board hang on boot


CI status: 
https://source.denx.de/u-boot/custodians/u-boot-stm/-/pipelines/14623


This last patch for v2023.01 avoids a hang on 1GB DDR STM32MP15 board 
detected

by Marek and caused by Pali patch.


For my point a view , this patch is an acceptable workaround for master 
branch and v2023.01 delivery

but the initial patch should be revisited or reverted (in master or in 
next ?).


For details see comments in patch

"arm: stm32mp: Fix board_get_usable_ram_top() again"

http://patchwork.ozlabs.org/project/uboot/patch/20230105012222.238075-1-marex@denx.de/

https://lore.kernel.org/u-boot/20230105012222.238075-1-marex@denx.de/


Thanks,
Patrick

=====


The following changes since commit 8d6cbf5e6bc4e10e38b954763fa025caed517cc2:

   Merge tag 'efi-2023-01-rc5-3' of 
https://source.denx.de/u-boot/custodians/u-boot-efi (2023-01-04 08:50:24 
-0500)

are available in the Git repository at:

   https://gitlab.denx.de/u-boot/custodians/u-boot-stm.git 
tags/u-boot-stm32-20230106

for you to fetch changes up to a2e0b041d662e4c80502cd5c9a8326d026f9deb1:

   arm: stm32mp: Fix board_get_usable_ram_top() again (2023-01-06 
11:02:17 +0100)

----------------------------------------------------------------
- stm32mp: Fix board_get_usable_ram_top(): workaround to avoid issue 
after the
   commit 777aaaa706b ("common/memsize.c: Fix get_effective_memsize() to 
check
   for overflow") because the effective DDR effective size is reduce by 4KiB
   and sometime the board hang on boot

----------------------------------------------------------------
Marek Vasut (1):
       arm: stm32mp: Fix board_get_usable_ram_top() again

  arch/arm/mach-stm32mp/dram_init.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

Re: [PULL] Pull request for u-boot master / v2023.01 = u-boot-stm32-20230106

[Tom Rini]

[-- Attachment #1: Type: text/plain, Size: 1969 bytes --]

On Fri, Jan 06, 2023 at 11:48:54AM +0100, Patrick DELAUNAY wrote:

> Hi Tom,
> 
> Please pull the STM32 related fixes for u-boot/master, v2023.01:
> u-boot-stm32-20230106
> 
> 
> - stm32mp: Fix board_get_usable_ram_top(): workaround to avoid issue after
> the
> 
>   commit 777aaaa706b ("common/memsize.c: Fix get_effective_memsize() to
> check
> 
>   for overflow") because the effective DDR effective size is reduce by 4KiB
>   and sometime the board hang on boot
> 
> 
> CI status:
> https://source.denx.de/u-boot/custodians/u-boot-stm/-/pipelines/14623
> 
> 
> This last patch for v2023.01 avoids a hang on 1GB DDR STM32MP15 board
> detected
> 
> by Marek and caused by Pali patch.
> 
> 
> For my point a view , this patch is an acceptable workaround for master
> branch and v2023.01 delivery
> 
> but the initial patch should be revisited or reverted (in master or in next
> ?).
> 
> 
> For details see comments in patch
> 
> "arm: stm32mp: Fix board_get_usable_ram_top() again"
> 
> http://patchwork.ozlabs.org/project/uboot/patch/20230105012222.238075-1-marex@denx.de/
> 
> https://lore.kernel.org/u-boot/20230105012222.238075-1-marex@denx.de/

Yes, I want to see how this progresses, thanks for taking a work-around
for now.  But for now I have also posted a revert, to help spur
discussion.

> 
> Thanks,
> Patrick
> 
> =====
> 
> 
> The following changes since commit 8d6cbf5e6bc4e10e38b954763fa025caed517cc2:
> 
>   Merge tag 'efi-2023-01-rc5-3' of
> https://source.denx.de/u-boot/custodians/u-boot-efi (2023-01-04 08:50:24
> -0500)
> 
> are available in the Git repository at:
> 
>   https://gitlab.denx.de/u-boot/custodians/u-boot-stm.git
> tags/u-boot-stm32-20230106
> 
> for you to fetch changes up to a2e0b041d662e4c80502cd5c9a8326d026f9deb1:
> 
>   arm: stm32mp: Fix board_get_usable_ram_top() again (2023-01-06 11:02:17
> +0100)
> 

Applied to u-boot/master, thanks!

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]