Hi Nishanth,
On 15/07/23 12:00, Neha Malcom Francis wrote:
Hi Nishanth
On 15/07/23 11:44, Nishanth Menon wrote:
On 19:20-20230714, Neha Malcom Francis wrote:
Since you are going to respin anyways, I will complain about this:
+#ifdef CONFIG_TARGET_J721E_A72_EVM
+
+#define SPL_NODTB "spl/u-boot-spl-nodtb.bin"
+#define SPL_J721E_EVM_DTB "spl/dts/k3-j721e-common-proc-board.dtb"
+#define SPL_J721E_SK_DTB "spl/dts/k3-j721e-sk.dtb"
+
+#define UBOOT_NODTB "u-boot-nodtb.bin"
+#define J721E_EVM_DTB "arch/arm/dts/k3-j721e-common-proc-board.dtb"
+#define J721E_SK_DTB "arch/arm/dts/k3-j721e-sk.dtb"
+
I was trying to add beaglebone-ai64 support and discovered that this
messes the reuse of j721e-binman.dtsi for ai64. I am forced to introduce
j721e-sk.dtb - so why not just use SPL_NODTB SPL_J721E_BOARD_DTB,
UBOOT_NODTB J721E_EVM_BOARD_DTB (generic names) and drop the specific sk
stuff? And define the macros in the board.dtsi prior to including the
files?
I can do that, yes. Thanks for catching this.
I'd like to just bring up that this would mean that the same A72 build and
binaries will no longer work for both EVM and SK, rather will have to modify the
CONFIG_DEFAULT_DEVICE_TREE for each. I am okay with this, but just putting it
out there.
--
Thanking You
Neha Malcom Francis
