On 02:57-20220513, M, Aparna wrote:
> 
> On 12/05/22 21:05, Ryan Eatmon wrote:
> >
> >
> > On 5/12/2022 10:28, Aparna M wrote:
> >> Add ti-sci-firmware binaries required for uboot build to the
> >> prebuilt-images directory of SDK.
> >>
> >> Signed-off-by: Aparna M <[email protected]>
> >> ---
> >>   meta-arago-distro/recipes-core/images/tisdk-core-bundle.bb | 3 +++
> >>   1 file changed, 3 insertions(+)
> >>
> >> This patch is dependent on
> >> https://lists.yoctoproject.org/g/meta-ti/topic/dunfell_patch_ti_sci_fw/91058279
> >>  
> >>
> 
> As per suggestions on previous patch, this is the new patch dependency:
> https://lists.yoctoproject.org/g/meta-ti/topic/dunfell_patch/91076133
> 
> >>
> >> diff --git 
> >> a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.bb 
> >> b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.bb
> >> index 9ebcbb0e..d5a11f36 100644
> >> --- a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.bb
> >> +++ b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.bb
> >> @@ -21,6 +21,9 @@ DEPLOY_IMAGES_NAME_append_j7-evm = " sysfw.itb"
> >>   DEPLOY_IMAGES_NAME_append_j7-hs-evm = " sysfw.itb"
> >>   DEPLOY_IMAGES_NAME_append_am65xx-evm = " 
> >> ti-sci-firmware-am65x-gp.bin sysfw-am65x-evm.itb 
> >> sysfw-am65x_sr2-evm.itb"
> >>   DEPLOY_IMAGES_NAME_append_j7-evm = " ti-fs-firmware-j721e-gp.bin"
> >> +DEPLOY_IMAGES_NAME_append_am64xx-evm = " ti-sci-firmware-am64x-gp.bin"
> >> +DEPLOY_IMAGES_NAME_append_am64xx-hs-evm = " 
> >> ti-sci-firmware-am64x-hs-cert.bin ti-sci-firmware-am64x-hs-enc.bin 
> >> fitImage-its.its"
> >> +DEPLOY_IMAGES_NAME_append_am62xx-evm = " ti-fs-firmware-am62x-gp.bin 
> >> ipc_echo_testb_mcu1_0_release_strip.xer5f"
> >>     ARAGO_TISDK_IMAGE ?= "tisdk-core-bundle"
> >>   export IMAGE_BASENAME = "${ARAGO_TISDK_IMAGE}"
> >
> >
> > Same as the other patch on meta-ti.  I think the 
> > ipc_echo_testb_mcu1_0_release_strip.xer5f file needs to not be in this 
> > file.  The other changes are fine, but remove that one.
> 
> The ipc_echo_testb_mcu1_0_release_strip.xer5f file is required in the 
> SDK for toplevel makefile uboot build for am62xx-evm. So it needs to be 
> packaged in the prebuilt-images folder of the SDK.


I dont understand why are we forcing this dependency? u-boot is built
from source, the firmwares are in linux-firmware repositories. why
are'nt the top level makefiles using the linux-firmware repo to pick
up the correct binaries?

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 
849D 1736 249D


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#13865): 
https://lists.yoctoproject.org/g/meta-arago/message/13865
Mute This Topic: https://lists.yoctoproject.org/mt/91060571/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to