On 2023/5/4 15:04, Krzysztof Kozlowski wrote: > On 04/05/2023 08:53, Changhuang Liang wrote: >>>> }; >>>> }; >>>> >>>> Add a "regmap" property which is phandle. And it can keep the present child-node >>>> structure. This is more consistent with our soc design. >>> >>> Adding property from child to parent does not make any sense. Didn't you >>> already receive comment on this? >>> >>> Best regards, >>> Krzysztof >>> >> >> Krzysztof, >> >> I am confused about what to do next. How to add this power-controller's >> node in device tree? >> > > You just move power-domain-cells up. > > Best regards, > Krzysztof > Like this? aon_syscon: syscon@17010000 { compatible = "starfive,jh7110-aon-syscon", "syscon", "starfive,jh7110-aon-pmu"; reg = <0x0 0x17010000 0x0 0x1000>; #power-domain-cells = <1>; }; If right? I will tell the syscon patch's owner delete the "simple-mfd" in aon_syscon node. Best regards, Krzysztof