[PATCH v7 05/23] j721e: dts: binman: Package tiboot3.bin, sysfw.itb, tispl.bin, u-boot.img

[Neha Malcom Francis]

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