* [meta-ti][dunfell][PATCH] conf: machine: am64xx-hs-evm: Include required boot binaries in wic image @ 2022-03-02 0:08 Aparna M 2022-03-02 0:17 ` Denys Dmytriyenko 0 siblings, 1 reply; 4+ messages in thread From: Aparna M @ 2022-03-02 0:08 UTC (permalink / raw) To: meta-ti, denis, praneeth, reatmon; +Cc: devarsht, nikhil.nd, nsekhar, Aparna M The wic image generated for HS platforms do not contain all required boot files. This happens because the IMAGE_BOOT_FILES variable that contains boot binaries to be included in the wic image is not set correctly. Populate this variable to include all boot binaries required. 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. Use UBOOT_IMAGE variable to include the binary as u-boot.img in the wic image. 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..960f63cf 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 = "tiboot3.bin ${SPL_BINARYNAME} ${UBOOT_IMAGE}" TI_SECURE_DEV_PKG = "${TI_SECURE_DEV_PKG_K3}" -- 2.17.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [meta-ti][dunfell][PATCH] conf: machine: am64xx-hs-evm: Include required boot binaries in wic image 2022-03-02 0:08 [meta-ti][dunfell][PATCH] conf: machine: am64xx-hs-evm: Include required boot binaries in wic image Aparna M @ 2022-03-02 0:17 ` Denys Dmytriyenko 2022-03-03 12:05 ` Aparna M 0 siblings, 1 reply; 4+ messages in thread From: Denys Dmytriyenko @ 2022-03-02 0:17 UTC (permalink / raw) To: Aparna M; +Cc: meta-ti, praneeth, reatmon, devarsht, nikhil.nd, nsekhar, nm Thanks, Aparna. Was this tested on the actual HS platform? Can you describe the boot flow briefly? Where does fitImage with the kernel and DTBs go? I don't see it in wic. Also, what about other K3 HS platforms? Should they be updated similarly? Thanks. On Wed, Mar 02, 2022 at 05:38:57AM +0530, Aparna M wrote: > The wic image generated for HS platforms do not contain all required > boot files. This happens because the IMAGE_BOOT_FILES variable that > contains boot binaries to be included in the wic image is not set > correctly. Populate this variable to include all boot binaries required. > > 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. Use UBOOT_IMAGE > variable to include the binary as u-boot.img in the wic image. > > 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..960f63cf 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 = "tiboot3.bin ${SPL_BINARYNAME} ${UBOOT_IMAGE}" > > TI_SECURE_DEV_PKG = "${TI_SECURE_DEV_PKG_K3}" > -- > 2.17.1 > -- Regards, Denys Dmytriyenko <denis@denix.org> PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964 Fingerprint: 25FC E4A5 8A72 2F69 1186 6D76 4209 0272 9A92 C964 ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [meta-ti][dunfell][PATCH] conf: machine: am64xx-hs-evm: Include required boot binaries in wic image 2022-03-02 0:17 ` Denys Dmytriyenko @ 2022-03-03 12:05 ` Aparna M 2022-03-03 23:20 ` Denys Dmytriyenko 0 siblings, 1 reply; 4+ messages in thread From: Aparna M @ 2022-03-03 12:05 UTC (permalink / raw) To: Denys Dmytriyenko Cc: meta-ti, praneeth, reatmon, devarsht, nikhil.nd, nsekhar, nm On 19:17-20220301, Denys Dmytriyenko wrote: Hi Denys, Yes, the wic image was tested on the HS platform and the boot works. This should give you an idea about the boot flow : https://software-dl.ti.com/processor-sdk-linux/esd/AM65X/08_01_00_01/exports/docs/linux/Foundational_Components/U-Boot/UG-General-Info.html#boot-flow Boot images are x509 cert signed and the fitImage and DTBs are present in /boot in the rootfs partition of the wic image. Also, yes, I believe the other K3 HS platforms need similar fixes. > Thanks, Aparna. > > Was this tested on the actual HS platform? > Can you describe the boot flow briefly? > Where does fitImage with the kernel and DTBs go? I don't see it in wic. > Also, what about other K3 HS platforms? Should they be updated similarly? > Thanks. > > > On Wed, Mar 02, 2022 at 05:38:57AM +0530, Aparna M wrote: > > The wic image generated for HS platforms do not contain all required > > boot files. This happens because the IMAGE_BOOT_FILES variable that > > contains boot binaries to be included in the wic image is not set > > correctly. Populate this variable to include all boot binaries required. > > > > 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. Use UBOOT_IMAGE > > variable to include the binary as u-boot.img in the wic image. > > > > 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..960f63cf 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 = "tiboot3.bin ${SPL_BINARYNAME} ${UBOOT_IMAGE}" > > > > TI_SECURE_DEV_PKG = "${TI_SECURE_DEV_PKG_K3}" > > -- > > 2.17.1 > > > > -- > Regards, > Denys Dmytriyenko <denis@denix.org> > PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964 > Fingerprint: 25FC E4A5 8A72 2F69 1186 6D76 4209 0272 9A92 C964 ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [meta-ti][dunfell][PATCH] conf: machine: am64xx-hs-evm: Include required boot binaries in wic image 2022-03-03 12:05 ` Aparna M @ 2022-03-03 23:20 ` Denys Dmytriyenko 0 siblings, 0 replies; 4+ messages in thread From: Denys Dmytriyenko @ 2022-03-03 23:20 UTC (permalink / raw) To: Aparna M; +Cc: meta-ti, praneeth, reatmon, devarsht, nikhil.nd, nsekhar, nm On Thu, Mar 03, 2022 at 05:35:04PM +0530, Aparna M wrote: > On 19:17-20220301, Denys Dmytriyenko wrote: > Hi Denys, > > Yes, the wic image was tested on the HS platform and the boot works. > This should give you an idea about the boot flow : > https://software-dl.ti.com/processor-sdk-linux/esd/AM65X/08_01_00_01/exports/docs/linux/Foundational_Components/U-Boot/UG-General-Info.html#boot-flow Yeah, I'm well aware of the K3 GP boot flow. I was asking about the same for K3 HS, as few things aren't that clear. E.g. in legacy HS devices OPTEE was part of FIT image loaded by U-boot. Now OPTEE loads much earlier. I was trying to clarify few things like that and how certs and validation are handled... > Boot images are x509 cert signed and the fitImage and DTBs are present > in /boot in the rootfs partition of the wic image. Also, yes, I Ah, right, see that now. > believe the other K3 HS platforms need similar fixes. Do you plan to submit similar patches for those platforms? Thanks. > > Thanks, Aparna. > > > > Was this tested on the actual HS platform? > > Can you describe the boot flow briefly? > > Where does fitImage with the kernel and DTBs go? I don't see it in wic. > > Also, what about other K3 HS platforms? Should they be updated similarly? > > Thanks. > > > > > > On Wed, Mar 02, 2022 at 05:38:57AM +0530, Aparna M wrote: > > > The wic image generated for HS platforms do not contain all required > > > boot files. This happens because the IMAGE_BOOT_FILES variable that > > > contains boot binaries to be included in the wic image is not set > > > correctly. Populate this variable to include all boot binaries required. > > > > > > 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. Use UBOOT_IMAGE > > > variable to include the binary as u-boot.img in the wic image. > > > > > > 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..960f63cf 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 = "tiboot3.bin ${SPL_BINARYNAME} ${UBOOT_IMAGE}" > > > > > > TI_SECURE_DEV_PKG = "${TI_SECURE_DEV_PKG_K3}" > > > -- > > > 2.17.1 -- Regards, Denys Dmytriyenko <denis@denix.org> PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964 Fingerprint: 25FC E4A5 8A72 2F69 1186 6D76 4209 0272 9A92 C964 ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-03-03 23:20 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-03-02 0:08 [meta-ti][dunfell][PATCH] conf: machine: am64xx-hs-evm: Include required boot binaries in wic image Aparna M 2022-03-02 0:17 ` Denys Dmytriyenko 2022-03-03 12:05 ` Aparna M 2022-03-03 23:20 ` Denys Dmytriyenko
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.