[1/3] add comment about content of u-boot OVERLAYS var

Message ID 20230904055133.1851073-2-felix.moessbauer@siemens.com
State Accepted, archived
Headers show
Series Extend u-boot-script for DT overlays | expand

Commit Message

MOESSBAUER, Felix Sept. 4, 2023, 5:51 a.m. UTC
The update-u-boot-script implementation uses a runtime loop to iterate
over the content each individual overlay entry. This is required because
the entry might contain an u-boot variable, containing multiple overlays
itself.

To avoid incorrect code cleanups like in 680ad26, we add a comment about
this special handling.

Signed-off-by: Felix Moessbauer <felix.moessbauer@siemens.com>
---
 meta/recipes-bsp/u-boot-script/files/update-u-boot-script | 1 +
 1 file changed, 1 insertion(+)

Patch

diff --git a/meta/recipes-bsp/u-boot-script/files/update-u-boot-script b/meta/recipes-bsp/u-boot-script/files/update-u-boot-script
index e9ace15e..1d9c6d45 100755
--- a/meta/recipes-bsp/u-boot-script/files/update-u-boot-script
+++ b/meta/recipes-bsp/u-boot-script/files/update-u-boot-script
@@ -63,6 +63,7 @@  if [ -n "${OVERLAYS}" ]; then
 		if ! echo $OVERLAY | grep -q "^/"; then
 			OVERLAY_PATH=/usr/lib/linux-image-${KERNEL_VERSION}/
 		fi
+		# The ${OVERLAY} might contain an u-boot variable with multiple entries.
 		echo "for entry in ${OVERLAY}; do" >> ${BOOT_CMD}
 		echo "echo Loading ${OVERLAY_PATH}\${entry}..." >> ${BOOT_CMD}
 		echo "load \${devtype} \${devnum}:${ROOT_PARTITION}" \