Re: [PATCH 3/3] arm64: dts: add support for C3 power domain controller

[Xianwei Zhao <xianwei.zhao@xxxxxxxxxxx>]

Hi Krzysztof,
   Thanks for your reply.

On 2023/7/3 21:12, Krzysztof Kozlowski wrote:
> [ EXTERNAL EMAIL ]
>
> On 03/07/2023 11:31, =Xianwei Zhao wrote:
> > From: Xianwei Zhao <xianwei.zhao@xxxxxxxxxxx>
> >
> > Enable power domain controller for Amlogic C3 SoC
> >
> > Signed-off-by: Xianwei Zhao <xianwei.zhao@xxxxxxxxxxx>
> > ---
> >   arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi | 10 ++++++++++
> >   1 file changed, 10 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi b/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
> > index 60ad4f3eef9d..826c51b1aff6 100644
> > --- a/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
> > +++ b/arch/arm64/boot/dts/amlogic/amlogic-c3.dtsi
> > @@ -47,6 +47,16 @@ xtal: xtal-clk {
> >                #clock-cells = <0>;
> >        };
> >
> > +     sm: secure-monitor {
> > +             compatible = "amlogic,meson-gxbb-sm";
> > +
> > +             pwrc: power-controller {
> > +                     compatible = "amlogic,c3-pwrc";
> > +                     #power-domain-cells = <1>;
> > +                     status = "okay";
>
> Why do you need it? okay is by default.Will move "status" in next version.
>
> Best regards,
> Krzysztof