Re: [PATCH v8 01/11] dt-bindings: clock: meson: add A1 PLL clock controller bindings

From: Dmitry Rokosov
Date: Fri Dec 02 2022 - 06:18:14 EST


>
> Node names should be generic.
> https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation

Thank you for the link, will change it in the next version.

>
> > + compatible = "amlogic,a1-pll-clkc";
>
> Use 4 spaces for example indentation.

Good point, I've fixed it in the my patches above. But as you mentioned
in the all changes, I have to squash them....

--
Thank you,
Dmitry