Re: [DO NOT MERGE v8 13/36] dt-bindings: clock: sh7750-cpg: Add renesas,sh7750-cpg header.
Geert Uytterhoeven <[email protected]>
| Newsgroups | gmane.linux.ide,gmane.linux.ports.sh.devel,gmane.linux.drivers.devicetree,gmane.linux.kernel,gmane.linux.kernel.renesas-soc,gmane.linux.kernel.clk,gmane.comp.video.dri.devel,gmane.linux.kernel.pci,gmane.linux.serial |
|---|---|
| Message-ID | <CAMuHMdWdR=SfNBFdABXRqdng8b=68mP_BnnXfpQcUvjCmvOmVA@mail.gmail.com> |
Hi Sato-san, Thanks for the update! On Wed, May 29, 2024 at 10:01 AM Yoshinori Sato <[email protected]> wrote: > SH7750 CPG Clock output define. (from my comments on v6 and v7) Please improve the patch description. > Signed-off-by: Yoshinori Sato <[email protected]> > --- /dev/null > +++ b/Documentation/devicetree/bindings/clock/renesas,sh7750-cpg.yaml > @@ -0,0 +1,107 @@ > +examples: > + - | > + #include <dt-bindings/clock/sh7750-cpg.h> > + cpg: clock-controller@ffc00000 { > + compatible = "renesas,sh7751r-cpg"; > + reg = <0xffc00000 20>, <0xfe0a0000 16>; > + reg-names = "FRQCR", "CLKSTP00"; > + clocks = <&extal>; > + clock-names = "extal"; > + renesas,mode = <0>; Nit: please move "renesas,mode" last. > + #clock-cells = <1>; > + #power-domain-cells = <0>; > + }; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected] In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds