On Sat, 2016-03-12 at 13:30:37 +0530, Shubhrajyoti Datta wrote:
> Add clock description for spi-xilinx
>
> Signed-off-by: Shubhrajyoti Datta <shubhraj@xxxxxxxxxx>
> ---
> v3
> Add a clock name
> v4
> Correct the name
> .../devicetree/bindings/spi/spi-xilinx.txt | 5 +++++
> 1 files changed, 5 insertions(+), 0 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/spi/spi-xilinx.txt b/Documentation/devicetree/bindings/spi/spi-xilinx.txt
> index c7b7856..07ec09a 100644
> --- a/Documentation/devicetree/bindings/spi/spi-xilinx.txt
> +++ b/Documentation/devicetree/bindings/spi/spi-xilinx.txt
> @@ -7,6 +7,9 @@ Required properties:
> - interrupts : Property with a value describing the interrupt
> number.
> - interrupt-parent : Must be core interrupt controller
> +- clocks: Input clock specifier. Refer to common clock bindings.
> +- clock-names: tuple listing input clock names.
> + Required elements: "s_axi_aclk"
I thought there were too clocks? If the IP has two clocks, I think both
should be added and supported here.
>
> Optional properties:
> - xlnx,num-ss-bits : Number of chip selects used.
> @@ -14,6 +17,8 @@ Optional properties:
> Example:
> axi_quad_spi@41e00000 {
> compatible = "xlnx,xps-spi-2.00.a";
> + clocks = <&clkc 15>;
> + clock-names = "spi_fclk";
Doesn't match the binding documented above.
Sören
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
