On Thu, Mar 30, 2023 at 08:12:10PM -0400, Denys Dmytriyenko wrote: > On Thu, Mar 30, 2023 at 04:24:38PM -0500, Ryan Eatmon via > lists.yoctoproject.org wrote: > > We are seeing some testing issues where the new code that signs all of > > the files at all times is causing issues. So rollback the logic and > > only sign for platforms that support it. > > This is unfortunate that am65x GP has a bug/feature and fails to boot with > signed images. Hopefully this can be resolved soon, otherwise it will block > combining GP and HS-SE/HS-FS builds into one platform with multiconfig. > Especially this one, ironically: > https://patchwork.yoctoproject.org/project/ti/patch/[email protected]/ > > As we discussed offline, I'm fine with this as a temporary workaround for now.
And forgot to ack, d'oh :facepalm: > > Signed-off-by: Ryan Eatmon <[email protected]> Acked-by: Denys Dmytriyenko <[email protected]> > > --- > > .../trusted-firmware-a_%.bbappend | 40 ++++++++++++++++++- > > 1 file changed, 38 insertions(+), 2 deletions(-) > > > > diff --git > > a/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend > > b/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend > > index 9b8dd142..c65ecd9c 100644 > > --- > > a/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend > > +++ > > b/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend > > @@ -12,12 +12,48 @@ inherit ti-secdev > > EXTRA_OEMAKE:append:k3 = "${@ ' K3_USART=' + d.getVar('TFA_K3_USART') if > > d.getVar('TFA_K3_USART') else ''}" > > EXTRA_OEMAKE:append:k3 = "${@ ' K3_PM_SYSTEM_SUSPEND=' + > > d.getVar('TFA_K3_SYSTEM_SUSPEND') if d.getVar('TFA_K3_SYSTEM_SUSPEND') else > > ''}" > > > > -# Signing procedure for K3 devices > > -do_compile:append:k3() { > > +# Signing procedure for K3 HS devices > > +tfa_sign_k3hs() { > > mv ${BUILD_DIR}/bl31.bin ${BUILD_DIR}/bl31.bin.unsigned > > ${TI_SECURE_DEV_PKG}/scripts/secure-binary-image.sh > > ${BUILD_DIR}/bl31.bin.unsigned ${BUILD_DIR}/bl31.bin > > } > > > > +do_compile:append:am65xx-hs-evm() { > > + tfa_sign_k3hs > > +} > > + > > +do_compile:append:am64xx-evm() { > > + tfa_sign_k3hs > > +} > > + > > +do_compile:append:am62xx-evm() { > > + tfa_sign_k3hs > > +} > > + > > +do_compile:append:am62xx-lp-evm() { > > + tfa_sign_k3hs > > +} > > + > > +do_compile:append:am62axx-evm() { > > + tfa_sign_k3hs > > +} > > + > > +do_compile:append:j721e-hs-evm() { > > + tfa_sign_k3hs > > +} > > + > > +do_compile:append:j7200-hs-evm() { > > + tfa_sign_k3hs > > +} > > + > > +do_compile:append:j721s2-hs-evm() { > > + tfa_sign_k3hs > > +} > > + > > +do_compile:append:j784s4-hs-evm() { > > + tfa_sign_k3hs > > +} > > + > > do_install:append:k3() { > > if [ -f ${BUILD_DIR}/bl31.bin.unsigned ]; then > > echo "Install bl31.bin.unsigned" > > -- > > 2.17.1
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#16272): https://lists.yoctoproject.org/g/meta-ti/message/16272 Mute This Topic: https://lists.yoctoproject.org/mt/97960832/21656 Group Owner: [email protected] Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/leave/6695321/21656/1393940836/xyzzy [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
