From: Padmarao Begari <[email protected]>

The BOOTENV_DEV_UFS macro is incomplete - it only initializes UFS
and scans SCSI devices but does not actually boot anything. This
causes the boot sequence to stop after scsi scan without loading
a kernel.

Update BOOTENV_DEV_UFS to set the devnum environment variable to
the UFS instance number, initialize the device using $devnum, and
delegate the boot process to the shared scsi_boot script. Remove
the explicit scsi scan, as it is already handled by scsi_boot.

This matches the pattern used by other boot device macros and
ensures that UFS devices can successfully boot using the distro
boot framework.

Signed-off-by: Padmarao Begari <[email protected]>
Signed-off-by: Michal Simek <[email protected]>
---

 include/configs/amd_versal2.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/include/configs/amd_versal2.h b/include/configs/amd_versal2.h
index 404af2cd4c65..38d0bfc315b7 100644
--- a/include/configs/amd_versal2.h
+++ b/include/configs/amd_versal2.h
@@ -108,7 +108,8 @@
 #define BOOT_TARGET_DEVICES_UFS(func)  func(UFS, ufs, 0)
 
 #define BOOTENV_DEV_UFS(devtypeu, devtypel, instance) \
-       "bootcmd_" #devtypel "=" #devtypel " init " #instance "; scsi scan;\0"
+       "bootcmd_" #devtypel "=devnum=" #instance "; " \
+       #devtypel " init $devnum; run scsi_boot\0"
 
 #define BOOTENV_DEV_NAME_UFS(devtypeu, devtypel, instance) \
        "ufs "
-- 
2.43.0

base-commit: f431a25f6a088a4a296d13c2f00290e71a552d24
branch: debian-sent3

Reply via email to