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

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

Francesco