[PATCH v5 3/3] board: toradex: add verdin am62 support

[Marcel Ziswiler]

Hi Neha

On Fri, 2023-08-04 at 13:35 +0530, Neha Malcom Francis wrote:
> Hi Francesco
> 
> On 04/08/23 13:24, Francesco Dolcini wrote:
> > On Fri, Aug 04, 2023 at 12:04:59PM +0530, Neha Malcom Francis wrote:
> > > On 03/08/23 19:30, Marcel Ziswiler wrote:
> > > > +       combined-sysfw-cfg {
> > > > +               filename = "combined-sysfw-cfg.bin";
> > > > +               ti-board-config {
> > > > +                       board-cfg {
> > > > +                               config = "board-cfg.yaml";
> > > > +                               schema = "../../ti/common/schema.yaml";
> > > > +                       };
> > > > +                       sec-cfg {
> > > > +                               config = "sec-cfg.yaml";
> > > > +                               schema = "../../ti/common/schema.yaml";
> > > > +                       };
> > > > +                       pm-cfg {
> > > > +                               config = "pm-cfg.yaml";
> > > > +                               schema = "../../ti/common/schema.yaml";
> > > > +                       };
> > > > +                       rm-cfg {
> > > > +                               config = "rm-cfg.yaml";
> > > > +                               schema = "../../ti/common/schema.yaml";
> > > > +                       };
> > > > +               };
> > > > +       };
> > > > +};
> > > > +
> > > 
> > > ^ If you are already including k3-binman.dtsi, why are you redefining these?
> > 
> > We want/need out own `config`. Any other solution apart of this one?
> > 
> 
> Ah I see, how about using the cfg phandle to replace it with your own? See 
> example in arch/arm/dts/k3-j7200-binman.dtsi

Yes, I missed that. Let me do it that way and post a v6.

Thanks!

> > Francesco

Cheers

Marcel