| Message ID | 20250625171626.2863841-3-anshuld@ti.com |
|---|---|
| State | Under Review |
| Delegated to: | Ryan Eatmon |
| Headers | show |
| Series | Add support for AM62x SIP | expand |
On 6/25/25 12:16 PM, Anshul Dalal via lists.yoctoproject.org wrote: > Due to 512MiB DDR limitation of am62x SiP, the memory map has been > modified with OP-TEE moved to bottom of DDR after atf to 0x80080000, > this requires modifications to ATF's BL32_BASE. PRELOADED_BL33_BASE has > also been updated to 0x81880000 to reflect the updated SPL_TEXT_BASE > from u-boot. > > Signed-off-by: Anshul Dalal <anshuld@ti.com> > --- > .../recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc b/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc > index cb956a00..805ded8e 100644 > --- a/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc > +++ b/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc > @@ -19,4 +19,5 @@ TFA_SPD:am62lxx = "" > > EXTRA_OEMAKE += "${@ 'K3_USART=' + d.getVar('TFA_K3_USART') if d.getVar('TFA_K3_USART') else ''}" > EXTRA_OEMAKE += "${@ 'K3_PM_SYSTEM_SUSPEND=' + d.getVar('TFA_K3_SYSTEM_SUSPEND') if d.getVar('TFA_K3_SYSTEM_SUSPEND') else ''}" > +EXTRA_OEMAKE:append:am62xxsip-evm:bsp-ti-6_12 = " PRELOADED_BL33_BASE=0x81880000 BL32_BASE=0x80080000" I wonder if we want to do something like we did with TFA_K3_USART and TFA_K3_SYSTEM_SUSPEND here so that we can set this is the machine .conf file, vs out here in the recipe. Might be something we can do later, as the same needs done for CFG_TEE_CORE_LOG_LEVEL in OP-TEE. Andrew > EXTRA_OEMAKE:append:ti-falcon = " PRELOADED_BL33_BASE=0x82000000 K3_HW_CONFIG_BASE=0x88000000" > > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#18672): https://lists.yoctoproject.org/g/meta-ti/message/18672 > Mute This Topic: https://lists.yoctoproject.org/mt/113829067/3619733 > Group Owner: meta-ti+owner@lists.yoctoproject.org > Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [afd@ti.com] > -=-=-=-=-=-=-=-=-=-=-=- >
On 6/25/2025 12:45 PM, Andrew Davis wrote: > On 6/25/25 12:16 PM, Anshul Dalal via lists.yoctoproject.org wrote: >> Due to 512MiB DDR limitation of am62x SiP, the memory map has been >> modified with OP-TEE moved to bottom of DDR after atf to 0x80080000, >> this requires modifications to ATF's BL32_BASE. PRELOADED_BL33_BASE has >> also been updated to 0x81880000 to reflect the updated SPL_TEXT_BASE >> from u-boot. >> >> Signed-off-by: Anshul Dalal <anshuld@ti.com> >> --- >> .../recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git >> a/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc >> b/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc >> index cb956a00..805ded8e 100644 >> --- >> a/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc >> +++ >> b/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc >> @@ -19,4 +19,5 @@ TFA_SPD:am62lxx = "" >> EXTRA_OEMAKE += "${@ 'K3_USART=' + d.getVar('TFA_K3_USART') if >> d.getVar('TFA_K3_USART') else ''}" >> EXTRA_OEMAKE += "${@ 'K3_PM_SYSTEM_SUSPEND=' + >> d.getVar('TFA_K3_SYSTEM_SUSPEND') if d.getVar('TFA_K3_SYSTEM_SUSPEND') >> else ''}" >> +EXTRA_OEMAKE:append:am62xxsip-evm:bsp-ti-6_12 = " >> PRELOADED_BL33_BASE=0x81880000 BL32_BASE=0x80080000" > > I wonder if we want to do something like we did with TFA_K3_USART and > TFA_K3_SYSTEM_SUSPEND here > so that we can set this is the machine .conf file, vs out here in the > recipe. > > Might be something we can do later, as the same needs done for > CFG_TEE_CORE_LOG_LEVEL in OP-TEE. I like that plan for a follow up patch. > Andrew > >> EXTRA_OEMAKE:append:ti-falcon = " PRELOADED_BL33_BASE=0x82000000 >> K3_HW_CONFIG_BASE=0x88000000" >> >> >> >> -=-=-=-=-=-=-=-=-=-=-=- >> Links: You receive all messages sent to this group. >> View/Reply Online (#18672): >> https://lists.yoctoproject.org/g/meta-ti/message/18672 >> Mute This Topic: https://lists.yoctoproject.org/mt/113829067/3619733 >> Group Owner: meta-ti+owner@lists.yoctoproject.org >> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [afd@ti.com] >> -=-=-=-=-=-=-=-=-=-=-=- >>
diff --git a/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc b/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc index cb956a00..805ded8e 100644 --- a/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc +++ b/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc @@ -19,4 +19,5 @@ TFA_SPD:am62lxx = "" EXTRA_OEMAKE += "${@ 'K3_USART=' + d.getVar('TFA_K3_USART') if d.getVar('TFA_K3_USART') else ''}" EXTRA_OEMAKE += "${@ 'K3_PM_SYSTEM_SUSPEND=' + d.getVar('TFA_K3_SYSTEM_SUSPEND') if d.getVar('TFA_K3_SYSTEM_SUSPEND') else ''}" +EXTRA_OEMAKE:append:am62xxsip-evm:bsp-ti-6_12 = " PRELOADED_BL33_BASE=0x81880000 BL32_BASE=0x80080000" EXTRA_OEMAKE:append:ti-falcon = " PRELOADED_BL33_BASE=0x82000000 K3_HW_CONFIG_BASE=0x88000000"
Due to 512MiB DDR limitation of am62x SiP, the memory map has been modified with OP-TEE moved to bottom of DDR after atf to 0x80080000, this requires modifications to ATF's BL32_BASE. PRELOADED_BL33_BASE has also been updated to 0x81880000 to reflect the updated SPL_TEXT_BASE from u-boot. Signed-off-by: Anshul Dalal <anshuld@ti.com> --- .../recipes-bsp/trusted-firmware-a/trusted-firmware-a-ti.inc | 1 + 1 file changed, 1 insertion(+)