Re: [dunfell][PATCH v3] conf: machine: wic: Fix HS image boot partition
Manorit Chawdhry
On 09:59-20220609, Andrew Davis wrote:
From: Manorit Chawdhry <m-chawdhry@...>LGTM. Acked-by: Manorit Chawdhry <m-chawdhry@...>
|
|
AM4378 QSPI Boot
mrkanaly@...
Hello,
I am booting a AM4378 platform and am running in to some issues. I have modeled my board after am43xx_evm_defconfig and have achieved an executable SPL that I can debug with JTAG. U-Boot SPL 2020.01-g2781231a33 (Sep 01 2021 - 12:00:00 +0000)After programming to the NOR that is connected via QSPI, I am able to break at the QSPI memory mapped location (0x30000000) on boot and single step through instructions that I know resemble u-boot-spl (haven’t gotten symbol loading to work right with the memory map location). However I am running in to trouble as at some point in the boot a data abort is thrown and I haven’t figured out why/where yet. Before I spend too long trying to debug this I wanted to ask a few questions: * Is CONFIG_QSPI_BOOT required to boot an AM4378? Quoting the config, “This is not a must, some SoCs need this, some not.” My searches only see it coming up in regard to TI_SECURE_DEVICE, which I am not attempting to do anything with (i.e. not doing a secure boot). * am43xx_evm_qspiboot_defconfig Doesn’t use an SPL: * * Can it? * * Is an SPL necessary? My u-boot image currently exceeds available on chip memory and I thought an SPL would be required so that u-boot would have RAM available to it. Does the “eXecute in place” work around this on this platform such that you can skip using an SPL? DDR will get initialized by u-boot since it is executing from the NOR and didn’t have to be loaded in to memory, then it can init the DDR so it has the room to load Linux later on. * * When I build this config with CONFIG_SPL=y I get the error: arm-oe-linux-gnueabi-ld.bfd: invalid length for memory region .sramBut it does look as if the link script wasn’t processed correctly for this build configuration, as all the variables aren’t filled in: MEMORY { .sram : ORIGIN = 0x402F4000, LENGTH = IMAGE_MAX_SIZE }* Are these options and the extra options redundant or required? CONFIG_SYS_EXTRA_OPTIONS="CONS_INDEX=1,QSPI,QSPI_BOOT"; CONFIG_QSPI_BOOT=y; I wasn’t able to locate any CONFIG_QSPI in the code, only QSPI_BOOT. * I am currently building with CONFIG_OF_EMBED=y in order to get the above output from the UART. I understand this makes the DTB encoded in the executable ELF that I am debugging. Does this also mean the same for a binary that is flashed? Or does a flashed binary have to use OF_EMBED=n and the DTB appended to the end of the image (u-boot[-spl].bin, versus (u-boot[-spl]-no-dtb.bin). I appreciate any insight anyone might be able to offer. Thanks, Mike
|
|
[dunfell][PATCH v3] conf: machine: wic: Fix HS image boot partition
Andrew Davis
From: Manorit Chawdhry <m-chawdhry@...>
IMAGE_BOOT_FILES was being overwritten in the HS recipes leading to incomplete binaries in /boot and the naming of the binaries were not correct. Fixes /boot partition for HS devices. Signed-off-by: Manorit Chawdhry <m-chawdhry@...> [afd: Extended fix to all HS platforms] Signed-off-by: Andrew Davis <afd@...> --- Changes from v2: - Added UBOOT_SYMLINK to fix missing u-boot.img image conf/machine/am64xx-hs-evm.conf | 3 +-- conf/machine/am65xx-hs-evm.conf | 3 +-- conf/machine/include/k3.inc | 5 +++-- conf/machine/j7-hs-evm.conf | 3 +-- conf/machine/j7200-hs-evm.conf | 3 +-- 5 files changed, 7 insertions(+), 10 deletions(-) diff --git a/conf/machine/am64xx-hs-evm.conf b/conf/machine/am64xx-hs-evm.conf index 44e81463..ebbe2e62 100644 --- a/conf/machine/am64xx-hs-evm.conf +++ b/conf/machine/am64xx-hs-evm.conf @@ -15,8 +15,7 @@ UBOOT_DTBO_LOADADDRESS = "0x83080000" 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_SYMLINK = "u-boot.img" TI_SECURE_DEV_PKG = "${TI_SECURE_DEV_PKG_K3}" diff --git a/conf/machine/am65xx-hs-evm.conf b/conf/machine/am65xx-hs-evm.conf index e4eebb6a..de92cacb 100644 --- a/conf/machine/am65xx-hs-evm.conf +++ b/conf/machine/am65xx-hs-evm.conf @@ -15,8 +15,7 @@ UBOOT_DTBO_LOADADDRESS = "0x83080000" 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_SYMLINK = "u-boot.img" TI_SECURE_DEV_PKG = "${TI_SECURE_DEV_PKG_K3}" diff --git a/conf/machine/include/k3.inc b/conf/machine/include/k3.inc index e0970698..f85e6b98 100644 --- a/conf/machine/include/k3.inc +++ b/conf/machine/include/k3.inc @@ -22,6 +22,7 @@ UBOOT_ENTRYPOINT = "0x80008000" UBOOT_LOADADDRESS = "0x80008000" SPL_BINARY = "tispl.bin" +SPL_BINARYNAME = "tispl.bin" UBOOT_SUFFIX = "img" EXTRA_IMAGEDEPENDS += "virtual/bootloader" @@ -40,8 +41,8 @@ MACHINE_FEATURES = "kernel26 apm usbgadget usbhost vfat ext2 alsa ethernet pci" IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap" -IMAGE_BOOT_FILES_LEGACY = "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} tiboot3.bin sysfw.itb" -IMAGE_BOOT_FILES_MULTI_CERT = "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} tiboot3.bin" +IMAGE_BOOT_FILES_LEGACY = "${SPL_BINARYNAME} u-boot.${UBOOT_SUFFIX} tiboot3.bin sysfw.itb" +IMAGE_BOOT_FILES_MULTI_CERT = "${SPL_BINARYNAME} u-boot.${UBOOT_SUFFIX} tiboot3.bin" IMAGE_BOOT_FILES ?= "${IMAGE_BOOT_FILES_LEGACY}" diff --git a/conf/machine/j7-hs-evm.conf b/conf/machine/j7-hs-evm.conf index f2e3ccf5..ece0d85e 100644 --- a/conf/machine/j7-hs-evm.conf +++ b/conf/machine/j7-hs-evm.conf @@ -15,9 +15,8 @@ UBOOT_DTBO_LOADADDRESS = "0x83080000" 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_SYMLINK = "u-boot.img" BBMULTICONFIG += "k3r5-sr1-1" diff --git a/conf/machine/j7200-hs-evm.conf b/conf/machine/j7200-hs-evm.conf index a5bd637e..8821ffac 100644 --- a/conf/machine/j7200-hs-evm.conf +++ b/conf/machine/j7200-hs-evm.conf @@ -15,8 +15,7 @@ UBOOT_DTBO_LOADADDRESS = "0x83080000" 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_SYMLINK = "u-boot.img" TI_SECURE_DEV_PKG = "${TI_SECURE_DEV_PKG_K3}" -- 2.36.1
|
|
Re: [dunfell][PATCH v2] conf: machine: wic: Fix HS image boot partition
Andrew Davis
On 6/9/22 8:45 AM, Manorit Chawdhry wrote:
On 15:02-20220608, Andrew Davis wrote:From: Manorit Chawdhry <m-chawdhry@...>Hi, the builds are failing for this. Maybe try some clean builds? Had to completely remove all build and caches to get it to fail here, odd would have expected only removing the build dir to show this issue. This turned out a bit more involved than I was expecting, but think I found a way to pull in the _HS version and do the rename without effecting anything else.. sending v3. Thanks, Andrew Regards,
|
|
Re: [dunfell][PATCH v2] conf: machine: wic: Fix HS image boot partition
Manorit Chawdhry
On 15:02-20220608, Andrew Davis wrote:
From: Manorit Chawdhry <m-chawdhry@...>Hi, the builds are failing for this. Maybe try some clean builds? Regards, Manorit ---
|
|
[dunfell][PATCH v3 2/2] optee: Upgrade to upstream 3.17.0
Andrew Davis
Update to the latest upstream master which is 3.17.
We do this to add HUK and AM64x/AM62x support. With this we need to also switch OPTEEMACHINE for AM64x/AM62x. To build against the latest OP-TEE the support recipes need updated also. Move these here into meta-ti so that builds will still work when not using meta-arago-distro. The fix patches do not look to be needed anymore. While here switch to git checkout with https. Signed-off-by: Andrew Davis <afd@...> --- Changes from v2: - Switch OPTEEMACHINE - Update optee-examples conf/machine/include/am62xx.inc | 2 +- conf/machine/include/am64xx.inc | 2 +- .../optee/optee-client_%.bbappend | 2 ++ .../optee/optee-examples_%.bbappend | 2 ++ ...ow-setting-sysroot-for-libgcc-lookup.patch | 34 ------------------- ...0007-allow-setting-sysroot-for-clang.patch | 29 ---------------- recipes-security/optee/optee-os_%.bbappend | 14 +++----- recipes-security/optee/optee-test_%.bbappend | 5 +++ 8 files changed, 15 insertions(+), 75 deletions(-) create mode 100644 recipes-security/optee/optee-client_%.bbappend create mode 100644 recipes-security/optee/optee-examples_%.bbappend delete mode 100644 recipes-security/optee/optee-os/0006-allow-setting-sysroot-for-libgcc-lookup.patch delete mode 100644 recipes-security/optee/optee-os/0007-allow-setting-sysroot-for-clang.patch create mode 100644 recipes-security/optee/optee-test_%.bbappend diff --git a/conf/machine/include/am62xx.inc b/conf/machine/include/am62xx.inc index ff857b97..0364edd1 100644 --- a/conf/machine/include/am62xx.inc +++ b/conf/machine/include/am62xx.inc @@ -15,5 +15,5 @@ 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" -OPTEEMACHINE = "k3-am65x" +OPTEEMACHINE = "k3-am62x" OPTEEOUTPUTMACHINE = "k3" diff --git a/conf/machine/include/am64xx.inc b/conf/machine/include/am64xx.inc index 7646c70a..60ffacd5 100644 --- a/conf/machine/include/am64xx.inc +++ b/conf/machine/include/am64xx.inc @@ -20,7 +20,7 @@ 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" -OPTEEMACHINE = "k3-am65x" +OPTEEMACHINE = "k3-am64x" OPTEEOUTPUTMACHINE = "k3" # Set the list of device targets for ti-pdk class recipes diff --git a/recipes-security/optee/optee-client_%.bbappend b/recipes-security/optee/optee-client_%.bbappend new file mode 100644 index 00000000..5675570a --- /dev/null +++ b/recipes-security/optee/optee-client_%.bbappend @@ -0,0 +1,2 @@ +PV_ti-soc = "3.17.0+git${SRCPV}" +SRCREV_ti-soc = "a5c30b1277466a9bf85b62f45a6b00e79774e29c" diff --git a/recipes-security/optee/optee-examples_%.bbappend b/recipes-security/optee/optee-examples_%.bbappend new file mode 100644 index 00000000..19e32d9d --- /dev/null +++ b/recipes-security/optee/optee-examples_%.bbappend @@ -0,0 +1,2 @@ +PV_ti-soc = "3.17.0+git${SRCPV}" +SRCREV_ti-soc = "65fc74309e12189ad5b6ce3ffec37c8011088a5a" diff --git a/recipes-security/optee/optee-os/0006-allow-setting-sysroot-for-libgcc-lookup.patch b/recipes-security/optee/optee-os/0006-allow-setting-sysroot-for-libgcc-lookup.patch deleted file mode 100644 index 17005396..00000000 --- a/recipes-security/optee/optee-os/0006-allow-setting-sysroot-for-libgcc-lookup.patch +++ /dev/null @@ -1,34 +0,0 @@ -From 0bab935695ebcf0c533b49896ab18ff33d4a47d1 Mon Sep 17 00:00:00 2001 -From: Ross Burton <ross.burton@...> -Date: Tue, 26 May 2020 14:38:02 -0500 -Subject: [PATCH] allow setting sysroot for libgcc lookup - -Explicitly pass the new variable LIBGCC_LOCATE_CFLAGS variable when searching -for the compiler libraries as there's no easy way to reliably pass --sysroot -otherwise. - -Upstream-Status: Pending [https://github.com/OP-TEE/optee_os/issues/4188] -Signed-off-by: Ross Burton <ross.burton@...> ---- - mk/gcc.mk | 6 +++--- - 1 file changed, 3 insertions(+), 3 deletions(-) - -diff --git a/mk/gcc.mk b/mk/gcc.mk -index adc77a24..81bfa78a 100644 ---- a/mk/gcc.mk -+++ b/mk/gcc.mk -@@ -13,11 +13,11 @@ nostdinc$(sm) := -nostdinc -isystem $(shell $(CC$(sm)) \ - -print-file-name=include 2> /dev/null) - - # Get location of libgcc from gcc --libgcc$(sm) := $(shell $(CC$(sm)) $(CFLAGS$(arch-bits-$(sm))) \ -+libgcc$(sm) := $(shell $(CC$(sm)) $(LIBGCC_LOCATE_CFLAGS) $(CFLAGS$(arch-bits-$(sm))) \ - -print-libgcc-file-name 2> /dev/null) --libstdc++$(sm) := $(shell $(CXX$(sm)) $(CXXFLAGS$(arch-bits-$(sm))) $(comp-cxxflags$(sm)) \ -+libstdc++$(sm) := $(shell $(CXX$(sm)) $(LIBGCC_LOCATE_CFLAGS) $(CXXFLAGS$(arch-bits-$(sm))) $(comp-cxxflags$(sm)) \ - -print-file-name=libstdc++.a 2> /dev/null) --libgcc_eh$(sm) := $(shell $(CXX$(sm)) $(CXXFLAGS$(arch-bits-$(sm))) $(comp-cxxflags$(sm)) \ -+libgcc_eh$(sm) := $(shell $(CXX$(sm)) $(LIBGCC_LOCATE_CFLAGS) $(CXXFLAGS$(arch-bits-$(sm))) $(comp-cxxflags$(sm)) \ - -print-file-name=libgcc_eh.a 2> /dev/null) - - # Define these to something to discover accidental use diff --git a/recipes-security/optee/optee-os/0007-allow-setting-sysroot-for-clang.patch b/recipes-security/optee/optee-os/0007-allow-setting-sysroot-for-clang.patch deleted file mode 100644 index 5c0d0a56..00000000 --- a/recipes-security/optee/optee-os/0007-allow-setting-sysroot-for-clang.patch +++ /dev/null @@ -1,29 +0,0 @@ -From 3167f2c0dba4db59d61b60a8fe66f969d20aafa9 Mon Sep 17 00:00:00 2001 -From: Brett Warren <brett.warren@...> -Date: Wed, 23 Sep 2020 09:27:34 +0100 -Subject: [PATCH] optee: enable clang support - -When compiling with clang, the LIBGCC_LOCATE_CFLAG variable used -to provide a sysroot wasn't included, which results in not locating -compiler-rt. This is mitigated by including the variable as ammended. - -Upstream-Status: Pending -ChangeId: 8ba69a4b2eb8ebaa047cb266c9aa6c2c3da45701 -Signed-off-by: Brett Warren <brett.warren@...> ---- - mk/clang.mk | 2 +- - 1 file changed, 1 insertion(+), 1 deletion(-) - -diff --git a/mk/clang.mk b/mk/clang.mk -index 0f48c836..47465523 100644 ---- a/mk/clang.mk -+++ b/mk/clang.mk -@@ -27,7 +27,7 @@ comp-cflags-warns-clang := -Wno-language-extension-token \ - - # Note, use the compiler runtime library (libclang_rt.builtins.*.a) instead of - # libgcc for clang --libgcc$(sm) := $(shell $(CC$(sm)) $(CFLAGS$(arch-bits-$(sm))) \ -+libgcc$(sm) := $(shell $(CC$(sm)) $(LIBGCC_LOCATE_CFLAGS) $(CFLAGS$(arch-bits-$(sm))) \ - -rtlib=compiler-rt -print-libgcc-file-name 2> /dev/null) - - # Core ASLR relies on the executable being ready to run from its preferred load diff --git a/recipes-security/optee/optee-os_%.bbappend b/recipes-security/optee/optee-os_%.bbappend index 4e01ea74..00673152 100644 --- a/recipes-security/optee/optee-os_%.bbappend +++ b/recipes-security/optee/optee-os_%.bbappend @@ -1,14 +1,8 @@ -FILESEXTRAPATHS_prepend_ti-soc := "${THISDIR}/${PN}:" +PV_ti-soc = "3.17.0+git${SRCPV}" +SRCREV_ti-soc = "3ab148c8f4a049a05aa54614e5313bf7b5fb563f" +SRC_URI_ti-soc = "git://github.com/OP-TEE/optee_os.git;protocol=https" -PV_ti-soc = "3.12.0+git${SRCPV}" - -SRCREV_ti-soc = "3d47a131bca1d9ed511bfd516aa5e70269e12c1d" - -SRC_URI_ti-soc = " \ - git://github.com/OP-TEE/optee_os.git \ - file://0006-allow-setting-sysroot-for-libgcc-lookup.patch \ - file://0007-allow-setting-sysroot-for-clang.patch \ -" +DEPENDS_append_ti-soc = " python3-cryptography-native" EXTRA_OEMAKE_append_k3 = "${@ 'CFG_CONSOLE_UART='+ d.getVar('OPTEE_K3_USART') if d.getVar('OPTEE_K3_USART') else ''}" diff --git a/recipes-security/optee/optee-test_%.bbappend b/recipes-security/optee/optee-test_%.bbappend new file mode 100644 index 00000000..d81ae521 --- /dev/null +++ b/recipes-security/optee/optee-test_%.bbappend @@ -0,0 +1,5 @@ +PV_ti-soc = "3.17.0+git${SRCPV}" +SRCREV_ti-soc = "8a698baf9e8e010e4d8d52e6aded42dfc31e5b25" +SRC_URI_ti-soc = "git://github.com/OP-TEE/optee_test.git;protocol=https" + +DEPENDS_append_ti-soc = " python3-cryptography-native" -- 2.36.1
|
|
[dunfell][PATCH v3 1/2] trusted-firmware-a: Update to latest upstream master
Andrew Davis
For TF-A we work against upstream first, our fork has only been
used in the rare cases we need to ship a feature before we can get it upstream. This is not currently the case so lets switch to upstream repo until there is a need for our fork. Signed-off-by: Andrew Davis <afd@...> --- recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend b/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend index 68bfa899..207f3dac 100644 --- a/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend +++ b/recipes-bsp/trusted-firmware-a/trusted-firmware-a_%.bbappend @@ -1,8 +1,6 @@ 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 = "6541aa24f834daa80da701e0054dfe3a16cda0ce" +SRCREV_tfa_k3 = "453abc80b2eacbf2816c838e47e40f063227d934" COMPATIBLE_MACHINE_k3 = "k3" TFA_BUILD_TARGET_k3 = "all" TFA_INSTALL_TARGET_k3 = "bl31" -- 2.36.1
|
|
[dunfell][PATCH v2] conf: machine: wic: Fix HS image boot partition
Andrew Davis
From: Manorit Chawdhry <m-chawdhry@...>
IMAGE_BOOT_FILES was being overwritten in the HS recipes leading to incomplete binaries in /boot and the naming of the binaries were not correct. Fixes /boot partition for HS devices. Signed-off-by: Manorit Chawdhry <m-chawdhry@...> [afd: Extended fix to all HS platforms] Signed-off-by: Andrew Davis <afd@...> --- Changes from v1: - Use signed u-boot.img file for HS conf/machine/am64xx-hs-evm.conf | 2 -- conf/machine/am65xx-hs-evm.conf | 2 -- conf/machine/include/k3.inc | 5 +++-- conf/machine/j7-hs-evm.conf | 2 -- conf/machine/j7200-hs-evm.conf | 2 -- 5 files changed, 3 insertions(+), 10 deletions(-) diff --git a/conf/machine/am64xx-hs-evm.conf b/conf/machine/am64xx-hs-evm.conf index 44e81463..c63d382e 100644 --- a/conf/machine/am64xx-hs-evm.conf +++ b/conf/machine/am64xx-hs-evm.conf @@ -15,8 +15,6 @@ UBOOT_DTBO_LOADADDRESS = "0x83080000" UBOOT_DTBO_OFFSET = "0x00010000" SPL_BINARY = "tispl.bin_HS" -SPL_BINARYNAME = "tispl.bin" UBOOT_BINARY = "u-boot.img_HS" -IMAGE_BOOT_FILES = "${UBOOT_BINARY}" TI_SECURE_DEV_PKG = "${TI_SECURE_DEV_PKG_K3}" diff --git a/conf/machine/am65xx-hs-evm.conf b/conf/machine/am65xx-hs-evm.conf index e4eebb6a..a28b45dc 100644 --- a/conf/machine/am65xx-hs-evm.conf +++ b/conf/machine/am65xx-hs-evm.conf @@ -15,8 +15,6 @@ UBOOT_DTBO_LOADADDRESS = "0x83080000" UBOOT_DTBO_OFFSET = "0x00010000" SPL_BINARY = "tispl.bin_HS" -SPL_BINARYNAME = "tispl.bin" UBOOT_BINARY = "u-boot.img_HS" -IMAGE_BOOT_FILES = "${UBOOT_BINARY}" TI_SECURE_DEV_PKG = "${TI_SECURE_DEV_PKG_K3}" diff --git a/conf/machine/include/k3.inc b/conf/machine/include/k3.inc index e0970698..f85e6b98 100644 --- a/conf/machine/include/k3.inc +++ b/conf/machine/include/k3.inc @@ -22,6 +22,7 @@ UBOOT_ENTRYPOINT = "0x80008000" UBOOT_LOADADDRESS = "0x80008000" SPL_BINARY = "tispl.bin" +SPL_BINARYNAME = "tispl.bin" UBOOT_SUFFIX = "img" EXTRA_IMAGEDEPENDS += "virtual/bootloader" @@ -40,8 +41,8 @@ MACHINE_FEATURES = "kernel26 apm usbgadget usbhost vfat ext2 alsa ethernet pci" IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap" -IMAGE_BOOT_FILES_LEGACY = "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} tiboot3.bin sysfw.itb" -IMAGE_BOOT_FILES_MULTI_CERT = "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} tiboot3.bin" +IMAGE_BOOT_FILES_LEGACY = "${SPL_BINARYNAME} u-boot.${UBOOT_SUFFIX} tiboot3.bin sysfw.itb" +IMAGE_BOOT_FILES_MULTI_CERT = "${SPL_BINARYNAME} u-boot.${UBOOT_SUFFIX} tiboot3.bin" IMAGE_BOOT_FILES ?= "${IMAGE_BOOT_FILES_LEGACY}" diff --git a/conf/machine/j7-hs-evm.conf b/conf/machine/j7-hs-evm.conf index f2e3ccf5..c7766def 100644 --- a/conf/machine/j7-hs-evm.conf +++ b/conf/machine/j7-hs-evm.conf @@ -15,9 +15,7 @@ UBOOT_DTBO_LOADADDRESS = "0x83080000" UBOOT_DTBO_OFFSET = "0x00010000" SPL_BINARY = "tispl.bin_HS" -SPL_BINARYNAME = "tispl.bin" UBOOT_BINARY = "u-boot.img_HS" -IMAGE_BOOT_FILES = "${UBOOT_BINARY}" BBMULTICONFIG += "k3r5-sr1-1" diff --git a/conf/machine/j7200-hs-evm.conf b/conf/machine/j7200-hs-evm.conf index a5bd637e..93c0c0ce 100644 --- a/conf/machine/j7200-hs-evm.conf +++ b/conf/machine/j7200-hs-evm.conf @@ -15,8 +15,6 @@ UBOOT_DTBO_LOADADDRESS = "0x83080000" UBOOT_DTBO_OFFSET = "0x00010000" SPL_BINARY = "tispl.bin_HS" -SPL_BINARYNAME = "tispl.bin" UBOOT_BINARY = "u-boot.img_HS" -IMAGE_BOOT_FILES = "${UBOOT_BINARY}" TI_SECURE_DEV_PKG = "${TI_SECURE_DEV_PKG_K3}" -- 2.36.1
|
|
Re: [dunfell][PATCH v2 1/2] trusted-firmware-a: Update to latest upstream master
Andrew Davis
On 6/8/22 2:43 PM, Denys Dmytriyenko wrote:
On Wed, Jun 08, 2022 at 02:21:40PM -0500, Ryan Eatmon wrote:What is the pressing need to update optee in dunfell to the very latest? It pulls in new features for our plats like HUK and AM64x/AM62x support. Andrew On 6/8/2022 14:20, Denys Dmytriyenko wrote:What is the difference between v1 and v2 of these patches?
|
|
Re: [dunfell][PATCH v2 1/2] trusted-firmware-a: Update to latest upstream master
Denys Dmytriyenko
On Wed, Jun 08, 2022 at 02:21:40PM -0500, Ryan Eatmon wrote:
What is the pressing need to update optee in dunfell to the very latest? On 6/8/2022 14:20, Denys Dmytriyenko wrote:What is the difference between v1 and v2 of these patches?
|
|
Re: [dunfell PATCH v2 1/5] conf: machine: Add j721s2 hs platform
Denys Dmytriyenko
On Wed, Jun 08, 2022 at 01:11:53PM -0500, Nishanth Menon via lists.yoctoproject.org wrote:
On 16:05-20220607, Jayesh Choudhary via lists.yoctoproject.org wrote:These are addresses to be used inside FIT image for HS devices and they goMachine conf for J721S2 HS platform.Is there any way we can get rid of addresses from recipes? into machine configs for each platform. Are there sane defaults in U-boot? If so, we can drop these from generated FIT image .its file a let U-boot handle it? +
|
|
Re: [dunfell PATCH v2 4/5] ti-graphics: Add j721s2-hs-evm as a compatible machine
Denys Dmytriyenko
On Wed, Jun 08, 2022 at 01:09:21PM -0500, Nishanth Menon via lists.yoctoproject.org wrote:
On 16:05-20220607, Jayesh Choudhary via lists.yoctoproject.org wrote:Platform and DDK cleanup went into kirkstone/master, but not dunfell.Add HS platform to the list of compatible machines.^^ DDK km build does'nt again change based on board.. so why are we TARGET_PRODUCT_am62xx = "am62_linux"BVNC does'nt change based on evm - so why are we do it based on board?
|
|
Re: [dunfell][PATCH v2 1/2] trusted-firmware-a: Update to latest upstream master
Ryan Eatmon
He sent this one as a mistake. He is going to send a V3 that moves the optee-[client|test|examples] out of meta-arago and into meta-ti so that all of the optee stuff is in one place.
toggle quoted messageShow quoted text
On 6/8/2022 14:20, Denys Dmytriyenko wrote:
What is the difference between v1 and v2 of these patches? --
Ryan Eatmon reatmon@... ----------------------------------------- Texas Instruments, Inc. - LCPD - MGTS
|
|
Re: [dunfell][PATCH v2 1/2] trusted-firmware-a: Update to latest upstream master
Denys Dmytriyenko
What is the difference between v1 and v2 of these patches?
toggle quoted messageShow quoted text
On Wed, Jun 08, 2022 at 12:14:17PM -0500, Andrew Davis via lists.yoctoproject.org wrote:
For TF-A we work against upstream first, our fork has only been
|
|
Re: [dunfell/master PATCH] linux-ti-mainline: Bump kernel version to v5.18
Denys Dmytriyenko
On Wed, Jun 08, 2022 at 12:19:58PM -0500, Nishanth Menon wrote:
This adds up numerous fixes for various k3 platforms, adds platform suchReviewed-by: Denys Dmytriyenko <denys@...> ---
|
|
Re: [dunfell PATCH v2 5/5] cnm-wave-fw: add compatible for j721s2-hs-evm
Andrew Davis
On 6/8/22 1:07 PM, Nishanth Menon wrote:
On 16:05-20220607, Jayesh Choudhary via lists.yoctoproject.org wrote:Add the HS platform to the compatible list in the recipe.Just curious.. We didn't have a SoC name for j721s2, it used the same name "j7" for all J7 like SoCs, so we had to add specifics on a per-board basis. Don't worry, I fixed it in master branch[0], but it would have been too much churn for a stable like dunfell, so we just have to live with adding for each specific board instead of per SoC for a bit longer. [0] https://git.ti.com/cgit/arago-project/meta-ti/commit/?id=531fd1af2333fe831c70a88e7f592a11a58bf94d Andrew ALTERNATIVE_LINK_NAME[wave521c_codec_fw.bin] = "${base_libdir}/firmware/${TARGET_WAVE521C}"
|
|
Re: [dunfell PATCH v2 1/5] conf: machine: Add j721s2 hs platform
On 16:05-20220607, Jayesh Choudhary via lists.yoctoproject.org wrote:
Machine conf for J721S2 HS platform.Is there any way we can get rid of addresses from recipes? + -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
|
|
Re: [dunfell PATCH v2 4/5] ti-graphics: Add j721s2-hs-evm as a compatible machine
On 16:05-20220607, Jayesh Choudhary via lists.yoctoproject.org wrote:
Add HS platform to the list of compatible machines.^^ DDK km build does'nt again change based on board.. so why are we doint this based on board? TARGET_PRODUCT_am62xx = "am62_linux"BVNC does'nt change based on evm - so why are we do it based on board? TARGET_BVNC_am62xx = "33.15.11.3"^^ Same question here as well.. am62xx seems to have done it right. TARGET_PRODUCT_am62xx = "am62_linux" -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
|
|
Re: [dunfell PATCH v2 5/5] cnm-wave-fw: add compatible for j721s2-hs-evm
On 16:05-20220607, Jayesh Choudhary via lists.yoctoproject.org wrote:
Add the HS platform to the compatible list in the recipe.Just curious.. Is the codec binary changing per board? if not, why not just plug it based on SoC?
-- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
|
|
[dunfell/master PATCH] linux-ti-mainline: Bump kernel version to v5.18
This adds up numerous fixes for various k3 platforms, adds platform such
as AM62x, J721S2. While at it, bump the ti-upstream-tools repo as this sync up gives us: - Switch over to 4k as default, configs needed for containers for k3 platforms among other config updates - Enable fixes for various pinmuxes and overlays for system test images. Signed-off-by: Nishanth Menon <nm@...> --- recipes-bsp/u-boot/u-boot-ti-mainline_git.bb | 6 +++--- recipes-kernel/linux/linux-ti-mainline_git.bb | 8 ++++---- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/recipes-kernel/linux/linux-ti-mainline_git.bb b/recipes-kernel/linux/linux-ti-mainline_git.bb index 4a5034fd048c..48c8bd4f440a 100644 --- a/recipes-kernel/linux/linux-ti-mainline_git.bb +++ b/recipes-kernel/linux/linux-ti-mainline_git.bb @@ -20,9 +20,9 @@ S = "${WORKDIR}/git" BRANCH = "master" TOOLS_BRANCH = "master" -# 5.15 Mainline version -SRCREV = "8bb7eca972ad531c9b149c0a51ab43a417385813" -PV = "5.15+git${SRCPV}" +# 5.18 Mainline version +SRCREV = "4b0986a3613c92f4ec1bdc7f60ec66fea135991f" +PV = "5.18+git${SRCPV}" # Append to the MACHINE_KERNEL_PR so that a new SRCREV will cause a rebuild MACHINE_KERNEL_PR_append = "b" @@ -36,7 +36,7 @@ SRC_URI += " \ file://defconfig \ " -SRCREV_ti-upstream-tools = "0f60697843bba6f8d721b14da92b1652563ccb95" +SRCREV_ti-upstream-tools = "2536a656f0fb620bae9e3a0793a23caee0e11023" SRCREV_FORMAT = "linux" KERNEL_DEVICETREE = "" -- 2.31.1
|
|