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