Re: [PATCH v1] dt-bindings: opp: Allow multi-worded node names

From: Viresh Kumar
Date: Fri Oct 22 2021 - 03:45:56 EST


On 22-10-21, 10:39, Dmitry Osipenko wrote:
> What we currently have for Tegra is a tegra-opps.dtsi and tegra.dtsi
> which includes the OPP's dtsi.
>
> the tegra-opps.dtsi has this structure:
>
> table: devname-opp-table {
> opp: ...
> };
>
> and tegra.dtsi:
>
> #include "tegra-opps.dtsi"
>
> device@0000 {
> operating-points-v2 = <&table>;
> };
>
> It just occurred to me that there is no need to move all tables to
> tegra.dtsi, but change structure of tegra-opps.dtsi to:
>
> device@0000 {
> operating-points-v2 = <&table>;
>
> table: opp-table {
> opp: ...
> };
> };

I thought you would have already thought about that and I was surprised when you
saw the tables are big enough to be moved. I was wondering what does it really
mean :)

> Then there no need to change current naming scheme. Let me try to
> implement it and see how it goes.

That's good then.

--
viresh