Re: [PATCH v3 3/4] dt-bindings: clk: at91: add bindings for SAM9X60's slow clock controller

Alexandre Belloni
2019-05-10T21:33:19Z

On 10/05/2019 11:23:35+0000, Claudiu.Beznea@microchip.com wrote:
> From: Claudiu Beznea <claudiu.beznea@microchip.com>
>
> Add bindings for SAM9X60's slow clock controller.
>
> Signed-off-by: Claudiu Beznea <claudiu.beznea@microchip.com>
Reviewed-by: Alexandre Belloni <alexandre.belloni@bootlin.com>

> ---
>
> Hi Rob,
>
> I didn't added your Reviewed-by tag to this version since I changed
> the driver with regards to clock-cells DT binding (and I though you
> may want to comment on this).
>
> Thank you,
> Claudiu Beznea
>
> Documentation/devicetree/bindings/clock/at91-clock.txt | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/clock/at91-clock.txt b/Documentation/devicetree/bindings/clock/at91-clock.txt
> index b520280e33ff..13f45db3b66d 100644
> --- a/Documentation/devicetree/bindings/clock/at91-clock.txt
> +++ b/Documentation/devicetree/bindings/clock/at91-clock.txt
> @@ -9,10 +9,11 @@ Slow Clock controller:
> Required properties:
> - compatible : shall be one of the following:
> "atmel,at91sam9x5-sckc",
> - "atmel,sama5d3-sckc" or
> - "atmel,sama5d4-sckc":
> + "atmel,sama5d3-sckc",
> + "atmel,sama5d4-sckc" or
> + "microchip,sam9x60-sckc":
> at91 SCKC (Slow Clock Controller)
> -- #clock-cells : shall be 0.
> +- #clock-cells : shall be 1 for "microchip,sam9x60-sckc" otherwise shall be 0.
> - clocks : shall be the input parent clock phandle for the clock.
>
> Optional properties:
> --
> 2.7.4
>

--
Alexandre Belloni, Bootlin
Embedded Linux and Kernel engineering