On Mon, Jun 09, 2025 at 01:05:36PM +0530, Anshul Dalal wrote:
> On Sat Jun 7, 2025 at 12:39 AM IST, Tom Rini wrote:
> > On Tue, Jun 03, 2025 at 07:54:47PM +0530, Anshul Dalal wrote:
> >
> >> The SPL_FS_LOAD_ARGS_NAME config is used for the arguments to the kernel
> >> (dtb in our case) in falcon boot.
> >> 
> >> Setting it in board specific Kconfig allows us to reuse the same config
> >> fragment 'am62x_r5_falcon.config' for all 3 platforms for enabling
> >> falcon boot.
> >> 
> >> Signed-off-by: Anshul Dalal <ansh...@ti.com>
> >> ---
> >>  board/ti/am62ax/Kconfig | 3 +++
> >>  board/ti/am62px/Kconfig | 3 +++
> >>  board/ti/am62x/Kconfig  | 3 +++
> >>  3 files changed, 9 insertions(+)
> >
> > Why can't whatever is using the fragment also set this? Is it because
> > you're only using this as part of say:
> > make am62ax_evm_r5_defconfig am62x_falcon.config
> > ?
> 
> Correct, having three different config fragments with only changes to the
> dtb path doesn't seem very useful imo.

Then these belong back in the main defconfig itself I think.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to