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...