Hi,
Michael,
On 7/11/25 4:58 AM, Michael Nazzareno Trimarchi wrote:
Hi
Could be html for my mobile. Some comments inline.
Il ven 11 lug 2025, 11:44 Anshul Dalal <ansh...@ti.com
<mailto:ansh...@ti.com>> ha scritto:
On Thu Jul 10, 2025 at 10:41 PM IST, Judith Mendez wrote:
> This removes support from booting from UDA when in eMMC boot.
>
> When user selects eMMC boot [0], that is, boot from eMMC boot0 or
> boot1 HW partitions, we should not boot from eMMC UDA. The current
> logic confuses the end user and mixes two different boot modes.
>
This is correct, we should not be changing the eMMC boot mode based on
what U-Boot config is enabled.
The end user can always use MMCSD as the backup boot mode if they want
a fallback.
> The logic was added for AM62A/AM62P with commit: 3b7893145e36, but
> this is not a clean solution and fix will come later for falcon boot.
>
> Logic was also added for AM62X with commit: 664593464634, but this
> logic is incorrect according to TRM [0] under Primary Boot Mode
Selection
> for 'eMMC Boot'.
>
> Fixes: 3b7893145e36 ("mach-k3: add eMMC FS boot support for
am62[ap]")
> Fixes: 664593464634 ("arm: mach-k3: am625: Relax emmc boot
condition")
> Signed-off-by: Judith Mendez <j...@ti.com <mailto:j...@ti.com>>
Reviewed-by: Anshul Dalal <ansh...@ti.com <mailto:ansh...@ti.com>>
First of all this is not a fix. A fix when you have something that is
not working . Then the commit was having a reason the boot0 and boot1
can have limited size that does not allow to have all the TI ecosystem
on boot partition and then you must use the the rest of emmc. Booting
from this scenario is totally fine
It is not working as expected, so it is a fix IMO, but will not fight on
that.
AND no, eMMC boot is not expected to boot from UDA. If you want to boot
from eMMC UDA, please use MMCSD boot which is the proper boot mode to
boot from eMMC UDA in RAW or FS mode.
Michael
Thanks for the fix,
Anshul
