Hi Stephen,
Thanks for your comments.
The patch will modify according to your comments and will be sent in
the next kernel revision 5.19.rc1
On Thu, 26 May 2022 at 22:24, Stephen Boyd <sboyd@xxxxxxxxxx> wrote:
>
> Quoting Tomer Maimon (2022-05-22 08:50:34)
> > diff --git a/Documentation/devicetree/bindings/clock/nuvoton,npcm845-clk.yaml b/Documentation/devicetree/bindings/clock/nuvoton,npcm845-clk.yaml
> > new file mode 100644
> > index 000000000000..f305c7c7eaf0
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/clock/nuvoton,npcm845-clk.yaml
> > @@ -0,0 +1,68 @@
> [...]
> > +
> > +required:
> > + - compatible
> > + - reg
> > + - "#clock-cells"
>
> Are clocks not required because sometimes the reference clk isn't
> connected?
require, will be fixed
>
> > +
> > +additionalProperties: false
> > +
> > +examples:
> > + # Clock Control Module node:
> > + - |
> > +
> > + ahb {
>
> drop ahb node please.
>
> > + #address-cells = <2>;
> > + #size-cells = <2>;
> > +
> > + clk: clock-controller@f0801000 {
>
> Drop label 'clk' as well please.
>
> > + compatible = "nuvoton,npcm845-clk";
> > + reg = <0x0 0xf0801000 0x0 0x1000>;
> > + #clock-cells = <1>;
> > + };
> > + };
> > +
> > +...
Best regards,
Tomer
