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>

Reviewed-by: Anshul Dalal <ansh...@ti.com>

Thanks for the fix,
Anshul

Reply via email to