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

[Marcel Ziswiler]

On Fri, 2023-08-04 at 10:15 +0200, Francesco Dolcini wrote:
> On Fri, Aug 04, 2023 at 08:12:36AM +0000, Marcel Ziswiler wrote:
> > 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.
> 
> Will it work? in k3-binman.dtsi the schema is referenced with such
> relative path: `schema = "../common/schema.yaml";` there.
> 
> There is an assumption of this being used from board/ti/$boardname if I
> understand it correctly.

Yes, but we can still override only that with labels rather than duplicating the whole thing. I am cleaning it
up that minute...