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

From: Tom Rini <trini@konsulko.com>
To: Patrick DELAUNAY <patrick.delaunay@foss.st.com>
Cc: "U-Boot Mailing List" <u-boot@lists.denx.de>,
	"Patrice CHOTARD" <patrice.chotard@foss.st.com>,
	"U-Boot STM32" <uboot-stm32@st-md-mailman.stormreply.com>,
	"Marek Vasut" <marex@denx.de>, "Pali Rohár" <pali@kernel.org>
Subject: Re: [PULL] Pull request for u-boot master / v2023.01 = u-boot-stm32-20230106
Date: Fri, 6 Jan 2023 11:52:50 -0500	[thread overview]
Message-ID: <20230106165250.GD3787616@bill-the-cat> (raw)
In-Reply-To: <348bc74e-fb7c-3911-680a-b6dff96a70b4@foss.st.com>

[-- 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 --]