|
[dunfell][PATCH v3 1/2] trusted-firmware-a: upgrade to v2.6+
Ryan Eatmon
From: Praneeth Bajjuri <praneeth@...>
upgrade arm-trusted-firmare to master which is currently v2.6+ This upgrade picks up am62x low power mode feature. Signed-off-by: Praneeth Bajjuri <praneeth@...> Signed-off-by: Ryan Eatmon <reatmon@...> --- recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend b/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend index b84da1f3..80b1d664 100644 --- a/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend +++ b/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend @@ -1,8 +1,8 @@ -PV_k3 = "2.5" +PV_k3 = "2.6" LIC_FILES_CHKSUM_k3 = "file://license.rst;md5=1dd070c98a281d18d9eefd938729b031" BRANCH_k3 = "ti-atf" SRC_URI_k3 = "git://git.ti.com/atf/arm-trusted-firmware.git;branch=${BRANCH};name=tfa" -SRCREV_tfa_k3 = "0693f356eb2a25adf8758d98058120bed082cc73" +SRCREV_tfa_k3 = "6541aa24f834daa80da701e0054dfe3a16cda0ce" COMPATIBLE_MACHINE_k3 = "k3" TFA_BUILD_TARGET_k3 = "all" TFA_INSTALL_TARGET_k3 = "bl31" -- 2.17.1
|
|
|
|
Re: [dunfell PATCH v2 2/2] conf/machine: am62xx: Update build flag for low power mode
Denys Dmytriyenko
On Wed, May 04, 2022 at 06:49:30PM -0500, Praneeth Bajjuri wrote:
From: Nishanth Menon <nm@...>NAK Either append or +=, not both! do_compile_append_am65xx-hs-evm() {-- Regards, Denys Dmytriyenko <denis@...> PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964 Fingerprint: 25FC E4A5 8A72 2F69 1186 6D76 4209 0272 9A92 C964
|
|
|
|
Re: [dunfell][PATCH] recipes-bsp: tifs-lpm-stub: Introduce new recipe
praneeth
On 5/4/2022 4:49 PM, Ryan Eatmon wrote:
On 5/4/2022 16:43, Denys Dmytriyenko wrote:Yes, this signed firmware is applicable for GP only.On Wed, May 04, 2022 at 04:38:44PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:Not sure how to answer that. I was just told to pick up this change for the release and I cleaned it up to match a more consistent style.This firmware stub is required by the TI foundational securityIs it GP only FW? We dont have a HS variant yet. Signed-off-by: Nikhil Devshatwar <nikhil.nd@...>
|
|
|
|
[dunfell PATCH v2 0/2] am62xx: Upgrade ATF to get LP support
praneeth
This series upgrades ATF to current master which is 2.6+.
This gets much needed AM62 Low power mode feature. Changes in v2: * Fixed the build flag patch as provided by Nishanth Menon * updated RFC patch sent earlier to move from ti-atf-next to ti-atf Nishanth Menon (1): conf/machine: am62xx: Update build flag for low power mode Praneeth Bajjuri (1): trusted-firmware-a: upgrade to v2.6+ conf/machine/include/am62xx.inc | 2 ++ recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend | 5 +++-- 2 files changed, 5 insertions(+), 2 deletions(-) -- 2.17.1
|
|
|
|
[dunfell PATCH v2 1/2] trusted-firmware-a: upgrade to v2.6+
praneeth
upgrade arm-trusted-firmare to master which is currently v2.6+
This upgrade picks up am62x low power mode feature. Signed-off-by: Praneeth Bajjuri <praneeth@...> --- recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend b/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend index b84da1f3..80b1d664 100644 --- a/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend +++ b/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend @@ -1,8 +1,8 @@ -PV_k3 = "2.5" +PV_k3 = "2.6" LIC_FILES_CHKSUM_k3 = "file://license.rst;md5=1dd070c98a281d18d9eefd938729b031" BRANCH_k3 = "ti-atf" SRC_URI_k3 = "git://git.ti.com/atf/arm-trusted-firmware.git;branch=${BRANCH};name=tfa" -SRCREV_tfa_k3 = "0693f356eb2a25adf8758d98058120bed082cc73" +SRCREV_tfa_k3 = "6541aa24f834daa80da701e0054dfe3a16cda0ce" COMPATIBLE_MACHINE_k3 = "k3" TFA_BUILD_TARGET_k3 = "all" TFA_INSTALL_TARGET_k3 = "bl31" -- 2.17.1
|
|
|
|
[dunfell PATCH v2 2/2] conf/machine: am62xx: Update build flag for low power mode
praneeth
From: Nishanth Menon <nm@...>
Update build flag for arm-trusted-firmware to include low power mode support K3_PM_SYSTEM_SUSPEND is added in arm-trusted-firmware now which enables low power mode support for am62xx. Signed-off-by: Nishanth Menon <nm@...> Signed-off-by: Praneeth Bajjuri <praneeth@...> --- conf/machine/include/am62xx.inc | 2 ++ recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend | 1 + 2 files changed, 3 insertions(+) diff --git a/conf/machine/include/am62xx.inc b/conf/machine/include/am62xx.inc index e8823dd2..ff857b97 100644 --- a/conf/machine/include/am62xx.inc +++ b/conf/machine/include/am62xx.inc @@ -6,6 +6,8 @@ MACHINE_FEATURES += "screen touchscreen" SERIAL_CONSOLES = "115200;ttyS2" SERIAL_CONSOLES_CHECK = "${SERIAL_CONSOLES}" +TFA_K3_SYSTEM_SUSPEND = "1" + # AM62 supports multi-certificate images, use the same IMAGE_BOOT_FILES = "${IMAGE_BOOT_FILES_MULTI_CERT}" diff --git a/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend b/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend index 80b1d664..5264fa97 100644 --- a/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend +++ b/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend @@ -9,6 +9,7 @@ TFA_INSTALL_TARGET_k3 = "bl31" TFA_SPD_k3 = "opteed" 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 ''}" do_compile_append_am65xx-hs-evm() { export TI_SECURE_DEV_PKG=${TI_SECURE_DEV_PKG} -- 2.17.1
|
|
|
|
Re: [dunfell PATCH] conf/machine: am62xx: Update build flag for low power mode
praneeth
On 5/3/2022 3:16 PM, Denys Dmytriyenko wrote:
On Tue, May 03, 2022 at 02:25:09PM -0500, Praneeth Bajjuri wrote:This patch is wrong, Will submit a new one.Update build flag for arm-trusted-firmware to include low power modeHow is this getting passed to ATF? OPTEEMACHINE = "k3-am65x"
|
|
|
|
Re: [dunfell][PATCH] recipes-bsp: tifs-lpm-stub: Introduce new recipe
Denys Dmytriyenko
On Wed, May 04, 2022 at 04:49:23PM -0500, Ryan Eatmon wrote:
Adding Nikhil to direct recipients. The reason I'm asking - the change is specific to am62xx-evm and not a more generic am62xx family (GP, HS, custom(er) boards). Plus the FW name is "ti-fs-stub-firmware-am62x-gp-signed.bin" with GP in the name, hence the question. Signed-off-by: Nikhil Devshatwar <nikhil.nd@...>
|
|
|
|
Re: [dunfell][PATCH] recipes-bsp: tifs-lpm-stub: Introduce new recipe
Ryan Eatmon
On 5/4/2022 16:43, Denys Dmytriyenko wrote:
On Wed, May 04, 2022 at 04:38:44PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:Not sure how to answer that. I was just told to pick up this change for the release and I cleaned it up to match a more consistent style.This firmware stub is required by the TI foundational securityIs it GP only FW? --Signed-off-by: Nikhil Devshatwar <nikhil.nd@...> Ryan Eatmon reatmon@... ----------------------------------------- Texas Instruments, Inc. - LCPD - MGTS
|
|
|
|
Re: [dunfell][PATCH] recipes-bsp: tifs-lpm-stub: Introduce new recipe
Denys Dmytriyenko
On Wed, May 04, 2022 at 04:38:44PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
This firmware stub is required by the TI foundational securityIs it GP only FW? Signed-off-by: Nikhil Devshatwar <nikhil.nd@...>
|
|
|
|
[dunfell][PATCH] recipes-bsp: tifs-lpm-stub: Introduce new recipe
Ryan Eatmon
This firmware stub is required by the TI foundational security
software while transitioning to low power mode. Add a recipe for this compatible with am62xx-evm Signed-off-by: Nikhil Devshatwar <nikhil.nd@...> Signed-off-by: Ryan Eatmon <reatmon@...> --- recipes-bsp/ti-linux-fw/ti-linux-fw.inc | 1 + .../tifs-lpm-stub/tifs-lpm-stub_git.bb | 24 +++++++++++++++++++ recipes-kernel/linux/kernel-rdepends.inc | 4 ++++ 3 files changed, 29 insertions(+) create mode 100644 recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb diff --git a/recipes-bsp/ti-linux-fw/ti-linux-fw.inc b/recipes-bsp/ti-linux-fw/ti-linux-fw.inc index 3ebdae30..47e76347 100644 --- a/recipes-bsp/ti-linux-fw/ti-linux-fw.inc +++ b/recipes-bsp/ti-linux-fw/ti-linux-fw.inc @@ -18,6 +18,7 @@ GOODIX_FW_VERSION = "1.0.0.0" CADENCE_MHDP_FW_VERSION = "1.2.17" IMG_DEC_FW_VERSION = "1.0" CNM_WAVE521_FW_VERSION = "1.0.00" +TIFS_LPM_STUB_FW_VERSION = "08.03.02" TI_LINUX_FW_SRCREV ?= "917625c2721eef8677df9e47d919005b0d4f93aa" SRCREV = "${TI_LINUX_FW_SRCREV}" diff --git a/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb b/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb new file mode 100644 index 00000000..642a8d3a --- /dev/null +++ b/recipes-bsp/tifs-lpm-stub/tifs-lpm-stub_git.bb @@ -0,0 +1,24 @@ +SUMMARY = "TI Foundational security Low Power Management Stub" + +require recipes-bsp/ti-linux-fw/ti-linux-fw.inc + +PV = "${TIFS_LPM_STUB_FW_VERSION}" +PR = "${INC_PR}.0" + +CLEANBROKEN = "1" + +COMPATIBLE_MACHINE = "am62xx-evm" + +PACKAGE_ARCH = "${MACHINE_ARCH}" + +S = "${WORKDIR}/git" + +TARGET = "ti-fs-stub-firmware-am62x-gp-signed.bin" + +do_install() { + install -d ${D}${base_libdir}/firmware/ti-sysfw/ + install -m 0644 ${S}/ti-sysfw/${TARGET} ${D}${base_libdir}/firmware/ti-sysfw/${TARGET} +} + +FILES_${PN} = "${base_libdir}/firmware" + diff --git a/recipes-kernel/linux/kernel-rdepends.inc b/recipes-kernel/linux/kernel-rdepends.inc index 522d9674..17aee4c2 100644 --- a/recipes-kernel/linux/kernel-rdepends.inc +++ b/recipes-kernel/linux/kernel-rdepends.inc @@ -45,3 +45,7 @@ RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_j7 = " vxd-dec-fw" # Add run-time dependency for Chips&Media Wave521 firmware to the rootfs RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_j721s2-evm = " cnm-wave-fw" + +# Add run-time dependency for TIFS Low Power Module stub +RDEPENDS_${KERNEL_PACKAGE_NAME}-base_append_am62xx-evm = " tifs-lpm-stub" + -- 2.17.1
|
|
|
|
[master][PATCH] optee-os, trusted-firmware-a: add leading whitespace when using append override
Denys Dmytriyenko
From: Denys Dmytriyenko <denys@...>
As append override does not add any whitespaces, it could lead to potenti= al issues when two items in space-separated list get concatenated. It is alw= ays recommended to add a leading whitespace in such cases, like EXTRA_OEMAKE. Signed-off-by: Denys Dmytriyenko <denys@...> --- .../trusted-firmware-a/trusted-firmware-a_%.bbappend | 2 +- meta-ti-bsp/recipes-security/optee/optee-os_%.bbappend | 2 +- 2 files changed, 2 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-firmwar= e-a_%.bbappend index 35cb9c95..5a2b4510 100644 --- a/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bba= ppend +++ b/meta-ti-bsp/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bba= ppend @@ -8,7 +8,7 @@ TFA_BUILD_TARGET:k3 =3D "all" TFA_INSTALL_TARGET:k3 =3D "bl31" TFA_SPD:k3 =3D "opteed" =20 -EXTRA_OEMAKE:append:k3 =3D "${@ 'K3_USART=3D' + d.getVar('TFA_K3_USART')= if d.getVar('TFA_K3_USART') else ''}" +EXTRA_OEMAKE:append:k3 =3D "${@ ' K3_USART=3D' + d.getVar('TFA_K3_USART'= ) if d.getVar('TFA_K3_USART') else ''}" =20 do_compile:append:am65xx-hs-evm() { export TI_SECURE_DEV_PKG=3D${TI_SECURE_DEV_PKG} diff --git a/meta-ti-bsp/recipes-security/optee/optee-os_%.bbappend b/met= a-ti-bsp/recipes-security/optee/optee-os_%.bbappend index 5af57e16..6097580f 100644 --- a/meta-ti-bsp/recipes-security/optee/optee-os_%.bbappend +++ b/meta-ti-bsp/recipes-security/optee/optee-os_%.bbappend @@ -1,4 +1,4 @@ -EXTRA_OEMAKE:append:k3 =3D "${@ 'CFG_CONSOLE_UART=3D'+ d.getVar('OPTEE_K= 3_USART') if d.getVar('OPTEE_K3_USART') else ''}" +EXTRA_OEMAKE:append:k3 =3D "${@ ' CFG_CONSOLE_UART=3D'+ d.getVar('OPTEE_= K3_USART') if d.getVar('OPTEE_K3_USART') else ''}" =20 do_compile:prepend:ti-soc() { export TI_SECURE_DEV_PKG=3D${TI_SECURE_DEV_PKG} --=20 2.25.1
|
|
|
|
Re: [PATCH v2] ffa-debugfs-mod: Do not build for TI SOCs
Denys Dmytriyenko
On Tue, May 03, 2022 at 09:09:49AM -0700, Khem Raj wrote:
The module can not be compiled without patching kernel 5.10Besides couple minor typos in the comment, looks good to me. Signed-off-by: Khem Raj <raj.khem@...>Reviewed-by: Denys Dmytriyenko <denys@...> ---
|
|
|
|
Re: [dunfell PATCH] conf/machine: am62xx: Update build flag for low power mode
Denys Dmytriyenko
On Tue, May 03, 2022 at 02:25:09PM -0500, Praneeth Bajjuri wrote:
Update build flag for arm-trusted-firmware to include low power modeHow is this getting passed to ATF? OPTEEMACHINE = "k3-am65x"
|
|
|
|
Re: [dunfell PATCH] conf/machine: am62xx: Update build flag for low power mode
praneeth
On 5/3/2022 2:25 PM, praneeth via lists.yoctoproject.org wrote:
Update build flag for arm-trusted-firmware to include low power modeI would like a dunfell-next build with this patch and https://lists.yoctoproject.org/g/meta-ti/message/14680 together. Would like to check nightly build and boot test across all K3 platforms. Will resubmit both patches together after merging ti-atf-next to ti-atf once i review the test results Thanks Praneeth ---
|
|
|
|
[dunfell PATCH] conf/machine: am62xx: Update build flag for low power mode
praneeth
Update build flag for arm-trusted-firmware to include low power mode
support K3_PM_SYSTEM_SUSPEND is added in arm-trusted-firmware now which enables low power mode support for am62xx. Signed-off-by: Praneeth Bajjuri <praneeth@...> --- conf/machine/include/am62xx.inc | 1 + 1 file changed, 1 insertion(+) diff --git a/conf/machine/include/am62xx.inc b/conf/machine/include/am62xx.inc index e8823dd2..59cb7053 100644 --- a/conf/machine/include/am62xx.inc +++ b/conf/machine/include/am62xx.inc @@ -13,5 +13,6 @@ do_image_wic[mcdepends] = "mc::k3r5:ti-sci-fw:do_deploy" do_image_tar[mcdepends] = "mc::k3r5:ti-sci-fw:do_deploy" TFA_BOARD = "lite" +K3_PM_SYSTEM_SUSPEND = "1" OPTEEMACHINE = "k3-am65x" OPTEEOUTPUTMACHINE = "k3" -- 2.17.1
|
|
|
|
[PATCH v2] ffa-debugfs-mod: Do not build for TI SOCs
The module can not be compiled without patching kernel 5.10
This helps meta-ti to live in multi-BSP envs Signed-off-by: Khem Raj <raj.khem@...> --- .../linux/ffa-debugfs/ffa-debugfs-mod_%.bbappend | 5 +++++ 1 file changed, 5 insertions(+) create mode 100644 meta-ti-bsp/recipes-kernel/linux/ffa-debugfs/ffa-debugfs-mod_%.bbappend diff --git a/meta-ti-bsp/recipes-kernel/linux/ffa-debugfs/ffa-debugfs-mod_%.bbappend b/meta-ti-bsp/recipes-kernel/linux/ffa-debugfs/ffa-debugfs-mod_%.bbappend new file mode 100644 index 00000000..67fcf6ec --- /dev/null +++ b/meta-ti-bsp/recipes-kernel/linux/ffa-debugfs/ffa-debugfs-mod_%.bbappend @@ -0,0 +1,5 @@ +# Added my meta-arm, this needs ceetain backports in 5.10 +# Disable until kernel upgrades to newer than 5.10 where it +# is already upstreamed +# Revisit when kernel is upgraded to 5.15 or newer e.g. +COMPATIBLE_HOST:ti-soc = "null" -- 2.36.0
|
|
|
|
Re: [master][PATCH] uio-module-drv-test: Fix build with clang
Denys Dmytriyenko
On Tue, May 03, 2022 at 11:00:48AM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
Point to an updated SRCREV that fixes a compile error with the clangReviewed-by: Denys Dmytriyenko <denys@...> ---
|
|
|
|
[master][PATCH] uio-module-drv-test: Fix build with clang
Ryan Eatmon
Point to an updated SRCREV that fixes a compile error with the clang
compiler. Signed-off-by: Khem Raj <raj.khem@...> Signed-off-by: Ryan Eatmon <reatmon@...> --- meta-ti-bsp/recipes-bsp/uio-module-drv/uio-module-drv.inc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/meta-ti-bsp/recipes-bsp/uio-module-drv/uio-module-drv.inc b/meta-ti-bsp/recipes-bsp/uio-module-drv/uio-module-drv.inc index 2c22e0cf..5b9ac50e 100644 --- a/meta-ti-bsp/recipes-bsp/uio-module-drv/uio-module-drv.inc +++ b/meta-ti-bsp/recipes-bsp/uio-module-drv/uio-module-drv.inc @@ -2,10 +2,10 @@ LICENSE = "BSD-3-Clause" LIC_FILES_CHKSUM = "file://LICENSE.txt;md5=2b7c96eb2aed7b0f6f922214dbde9016" BRANCH = "master" -# This corresponds to version 02.02.01.00 -UIO_MODULE_DRV_SRCREV = "bda9260f2220840a60ec0fbc8011cbadf3cf1b91" +# This corresponds to version 02.03.01.00 +UIO_MODULE_DRV_SRCREV = "78c535afe8f8cc0b715500a63eab0bc3606c0579" -PV = "2.2.1.0+git${SRCPV}" +PV = "2.3.1.0+git${SRCPV}" UIO_MODULE_DRV_GIT_URI = "git://git.ti.com/keystone-linux/uio-module-drv.git" UIO_MODULE_DRV_GIT_PROTOCOL = "git" -- 2.17.1
|
|
|
|
Re: [PATCH] ffa-debugfs-mod: Do not build for TI SOCs
Denys Dmytriyenko
On Tue, May 03, 2022 at 08:09:27AM -0700, Khem Raj wrote:
The module can not be compiled without patching kernel 5.10This doesn't seem to be needed ^ +# Added my meta-arm, this needs ceetain backports in 5.10
|
|
|