* [meta-ti][dunfell][PATCH] conf: machine: am64xx-hs-evm: Fix naming of u-boot.img @ 2022-02-21 8:13 Aparna M 2022-02-22 0:41 ` Denys Dmytriyenko 0 siblings, 1 reply; 5+ messages in thread From: Aparna M @ 2022-02-21 8:13 UTC (permalink / raw) To: meta-ti, praneeth, reatmon; +Cc: devarsht, nikhil.nd, nsekhar, Aparna M All the HS platforms generate the u-boot binary as u-boot.img_HS but the bootloader expects the binary to be named as u-boot.img Due to this mismatch, script cannot find the required image to be added in the WIC image. Fix this by overwriting the UBOOT_IMAGE variable via conf file. Signed-off-by: Nikhil Devshatwar <nikhil.nd@ti.com> Signed-off-by: Aparna M <a-m1@ti.com> --- conf/machine/am64xx-hs-evm.conf | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/conf/machine/am64xx-hs-evm.conf b/conf/machine/am64xx-hs-evm.conf index 44e81463..781b8318 100644 --- a/conf/machine/am64xx-hs-evm.conf +++ b/conf/machine/am64xx-hs-evm.conf @@ -17,6 +17,7 @@ UBOOT_DTBO_OFFSET = "0x00010000" SPL_BINARY = "tispl.bin_HS" SPL_BINARYNAME = "tispl.bin" UBOOT_BINARY = "u-boot.img_HS" -IMAGE_BOOT_FILES = "${UBOOT_BINARY}" +UBOOT_IMAGE = "u-boot.img" +IMAGE_BOOT_FILES = "${UBOOT_IMAGE}" TI_SECURE_DEV_PKG = "${TI_SECURE_DEV_PKG_K3}" -- 2.17.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [meta-ti][dunfell][PATCH] conf: machine: am64xx-hs-evm: Fix naming of u-boot.img 2022-02-21 8:13 [meta-ti][dunfell][PATCH] conf: machine: am64xx-hs-evm: Fix naming of u-boot.img Aparna M @ 2022-02-22 0:41 ` Denys Dmytriyenko 2022-02-25 10:08 ` Aparna M 2022-02-25 20:51 ` Aparna M 0 siblings, 2 replies; 5+ messages in thread From: Denys Dmytriyenko @ 2022-02-22 0:41 UTC (permalink / raw) To: a-m1; +Cc: meta-ti, praneeth, reatmon, devarsht, nikhil.nd, nsekhar On Mon, Feb 21, 2022 at 01:43:31PM +0530, Aparna M via lists.yoctoproject.org wrote: > All the HS platforms generate the u-boot binary as u-boot.img_HS > but the bootloader expects the binary to be named as u-boot.img > > Due to this mismatch, script cannot find the required image to > be added in the WIC image. Fix this by overwriting the > UBOOT_IMAGE variable via conf file. Can you please clarify which script cannot find the image? Thanks! Also, how does it work for am65xx HS, j721e HS, j7200 HS? Why only am64xx HS? > Signed-off-by: Nikhil Devshatwar <nikhil.nd@ti.com> > > Signed-off-by: Aparna M <a-m1@ti.com> > --- > conf/machine/am64xx-hs-evm.conf | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/conf/machine/am64xx-hs-evm.conf b/conf/machine/am64xx-hs-evm.conf > index 44e81463..781b8318 100644 > --- a/conf/machine/am64xx-hs-evm.conf > +++ b/conf/machine/am64xx-hs-evm.conf > @@ -17,6 +17,7 @@ UBOOT_DTBO_OFFSET = "0x00010000" > SPL_BINARY = "tispl.bin_HS" > SPL_BINARYNAME = "tispl.bin" > UBOOT_BINARY = "u-boot.img_HS" > -IMAGE_BOOT_FILES = "${UBOOT_BINARY}" > +UBOOT_IMAGE = "u-boot.img" > +IMAGE_BOOT_FILES = "${UBOOT_IMAGE}" > > TI_SECURE_DEV_PKG = "${TI_SECURE_DEV_PKG_K3}" > -- > 2.17.1 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [meta-ti][dunfell][PATCH] conf: machine: am64xx-hs-evm: Fix naming of u-boot.img 2022-02-22 0:41 ` Denys Dmytriyenko @ 2022-02-25 10:08 ` Aparna M 2022-02-25 21:17 ` Denys Dmytriyenko 2022-02-25 20:51 ` Aparna M 1 sibling, 1 reply; 5+ messages in thread From: Aparna M @ 2022-02-25 10:08 UTC (permalink / raw) To: Denys Dmytriyenko Cc: meta-ti, praneeth, reatmon, nikhil.nd, devarsht, nsekhar On 19:41-20220221, Denys Dmytriyenko wrote: > On Mon, Feb 21, 2022 at 01:43:31PM +0530, Aparna M via lists.yoctoproject.org wrote: > > All the HS platforms generate the u-boot binary as u-boot.img_HS > > but the bootloader expects the binary to be named as u-boot.img > > > > Due to this mismatch, script cannot find the required image to > > be added in the WIC image. Fix this by overwriting the > > UBOOT_IMAGE variable via conf file. > > Can you please clarify which script cannot find the image? Thanks! The wic file generation scripts that pick up files for the boot partition from the IMAGE_BOOT_FILES variable is unable to find u-boot.img. As a result, the generated wic image does not package the uboot binary and fails. > > Also, how does it work for am65xx HS, j721e HS, j7200 HS? Why only > am64xx HS? The previous releases for these three platforms did not include wic images. I believe this issue might be present in these platforms as well. > > > Signed-off-by: Nikhil Devshatwar <nikhil.nd@ti.com> > > > > Signed-off-by: Aparna M <a-m1@ti.com> > > --- > > conf/machine/am64xx-hs-evm.conf | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/conf/machine/am64xx-hs-evm.conf b/conf/machine/am64xx-hs-evm.conf > > index 44e81463..781b8318 100644 > > --- a/conf/machine/am64xx-hs-evm.conf > > +++ b/conf/machine/am64xx-hs-evm.conf > > @@ -17,6 +17,7 @@ UBOOT_DTBO_OFFSET = "0x00010000" > > SPL_BINARY = "tispl.bin_HS" > > SPL_BINARYNAME = "tispl.bin" > > UBOOT_BINARY = "u-boot.img_HS" > > -IMAGE_BOOT_FILES = "${UBOOT_BINARY}" > > +UBOOT_IMAGE = "u-boot.img" > > +IMAGE_BOOT_FILES = "${UBOOT_IMAGE}" > > > > TI_SECURE_DEV_PKG = "${TI_SECURE_DEV_PKG_K3}" > > -- > > 2.17.1 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [meta-ti][dunfell][PATCH] conf: machine: am64xx-hs-evm: Fix naming of u-boot.img 2022-02-25 10:08 ` Aparna M @ 2022-02-25 21:17 ` Denys Dmytriyenko 0 siblings, 0 replies; 5+ messages in thread From: Denys Dmytriyenko @ 2022-02-25 21:17 UTC (permalink / raw) To: a-m1; +Cc: meta-ti, praneeth, reatmon, nikhil.nd, devarsht, nsekhar On Fri, Feb 25, 2022 at 03:38:30PM +0530, Aparna M via lists.yoctoproject.org wrote: > On 19:41-20220221, Denys Dmytriyenko wrote: > > On Mon, Feb 21, 2022 at 01:43:31PM +0530, Aparna M via lists.yoctoproject.org wrote: > > > All the HS platforms generate the u-boot binary as u-boot.img_HS > > > but the bootloader expects the binary to be named as u-boot.img > > > > > > Due to this mismatch, script cannot find the required image to > > > be added in the WIC image. Fix this by overwriting the > > > UBOOT_IMAGE variable via conf file. > > > > Can you please clarify which script cannot find the image? Thanks! > > The wic file generation scripts that pick up files for the boot > partition from the IMAGE_BOOT_FILES variable is unable to find > u-boot.img. As a result, the generated wic image does not package the > uboot binary and fails. I don't see the failure. wic image picks up everything listed in IMAGE_BOOT_FILES variable and it is set right there to contain UBOOT_BINARY, which is set to u-boot.img_HS HS platforms boot differently and they use FIT image along with other artifacts. I don't believe wic image was ever used for HS platforms... So, I believe you have some other problem and this is not the right fix for it. > > Also, how does it work for am65xx HS, j721e HS, j7200 HS? Why only > > am64xx HS? > > The previous releases for these three platforms did not include wic > images. I believe this issue might be present in these platforms as > well. > > > > > > Signed-off-by: Nikhil Devshatwar <nikhil.nd@ti.com> > > > > > > Signed-off-by: Aparna M <a-m1@ti.com> > > > --- > > > conf/machine/am64xx-hs-evm.conf | 3 ++- > > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > > > diff --git a/conf/machine/am64xx-hs-evm.conf b/conf/machine/am64xx-hs-evm.conf > > > index 44e81463..781b8318 100644 > > > --- a/conf/machine/am64xx-hs-evm.conf > > > +++ b/conf/machine/am64xx-hs-evm.conf > > > @@ -17,6 +17,7 @@ UBOOT_DTBO_OFFSET = "0x00010000" > > > SPL_BINARY = "tispl.bin_HS" > > > SPL_BINARYNAME = "tispl.bin" > > > UBOOT_BINARY = "u-boot.img_HS" > > > -IMAGE_BOOT_FILES = "${UBOOT_BINARY}" > > > +UBOOT_IMAGE = "u-boot.img" > > > +IMAGE_BOOT_FILES = "${UBOOT_IMAGE}" > > > > > > TI_SECURE_DEV_PKG = "${TI_SECURE_DEV_PKG_K3}" > > > -- > > > 2.17.1 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [meta-ti][dunfell][PATCH] conf: machine: am64xx-hs-evm: Fix naming of u-boot.img 2022-02-22 0:41 ` Denys Dmytriyenko 2022-02-25 10:08 ` Aparna M @ 2022-02-25 20:51 ` Aparna M 1 sibling, 0 replies; 5+ messages in thread From: Aparna M @ 2022-02-25 20:51 UTC (permalink / raw) To: Denys Dmytriyenko; +Cc: meta-ti, reatmon, praneeth, nikhil.nd, devarsht Hi, I looked into wic images generated for am64xx HS, am65xx HS, j7 HS and j7200 HS - the issue seems to be similar. The boot partition contains u-boot.img_HS file but other boot binaries are missing. Please wait before further review. Will get back with updates. Thanks, Aparna ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-02-25 21:17 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-02-21 8:13 [meta-ti][dunfell][PATCH] conf: machine: am64xx-hs-evm: Fix naming of u-boot.img Aparna M 2022-02-22 0:41 ` Denys Dmytriyenko 2022-02-25 10:08 ` Aparna M 2022-02-25 21:17 ` Denys Dmytriyenko 2022-02-25 20:51 ` Aparna M
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.