On Fri 11 Jan 09:01 PST 2019, Georgi Djakov wrote:
> Hi Marc,
>
> On 1/11/19 16:42, Marc Gonzalez wrote:
> > Add MSM8998 Resource Power Manager Clock Controller DT node.
> >
> > Signed-off-by: Marc Gonzalez <marc.w.gonzalez@xxxxxxx>
> > ---
> > arch/arm64/boot/dts/qcom/msm8998.dtsi | 6 ++++++
> > 1 file changed, 6 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/msm8998.dtsi b/arch/arm64/boot/dts/qcom/msm8998.dtsi
> > index 8d41b69ec2da..de37415be0a8 100644
> > --- a/arch/arm64/boot/dts/qcom/msm8998.dtsi
> > +++ b/arch/arm64/boot/dts/qcom/msm8998.dtsi
> > @@ -3,6 +3,7 @@
> >
> > #include <dt-bindings/interrupt-controller/arm-gic.h>
> > #include <dt-bindings/clock/qcom,gcc-msm8998.h>
> > +#include <dt-bindings/clock/qcom,rpmcc.h>
> > #include <dt-bindings/gpio/gpio.h>
> >
> > / {
> > @@ -266,6 +267,11 @@
> > rpm_requests: rpm-requests {
> > compatible = "qcom,rpm-msm8998";
> > qcom,glink-channels = "rpm_requests";
> > +
> > + rpmcc: qcom,rpmcc {
>
> Maybe use "clock-controller" as node name instead of "qcom,rpmcc".
>
I agree with Georgi, name this "clock-controller".
With that, you have my
Reviewed-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>
Regards,
Bjorn
