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

[Francesco Dolcini]

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.

Francesco