[U-Boot] [PATCH 06/19] spi: Fix zynq SPI binding

Jagan Teki jteki at openedev.com
Thu Aug 13 20:36:12 CEST 2015


On 22 July 2015 at 21:08, Michal Simek <michal.simek at xilinx.com> wrote:
> Zynq is using Cadence IP where binding is documented in the Linux kernel
> and there is no reason to use different binding.
> Synchronize it.
>
> Signed-off-by: Michal Simek <michal.simek at xilinx.com>

Reviewed-by: Jagan Teki <jteki at openedev.com>

Updated patchwork, just take along with the series.

> ---
>
>  arch/arm/dts/zynq-7000.dtsi               |  4 ++--
>  doc/device-tree-bindings/spi/spi-zynq.txt | 35 +++++++++++++++++--------------
>  drivers/spi/zynq_spi.c                    |  2 +-
>  3 files changed, 22 insertions(+), 19 deletions(-)
>
> diff --git a/arch/arm/dts/zynq-7000.dtsi b/arch/arm/dts/zynq-7000.dtsi
> index 7679cf242f91..d212b60ca6b2 100644
> --- a/arch/arm/dts/zynq-7000.dtsi
> +++ b/arch/arm/dts/zynq-7000.dtsi
> @@ -166,7 +166,7 @@
>                 };
>
>                 spi0: spi at e0006000 {
> -                       compatible = "xlnx,zynq-spi";
> +                       compatible = "xlnx,zynq-spi-r1p6";
>                         reg = <0xe0006000 0x1000>;
>                         status = "disabled";
>                         interrupt-parent = <&intc>;
> @@ -179,7 +179,7 @@
>                 };
>
>                 spi1: spi at e0007000 {
> -                       compatible = "xlnx,zynq-spi";
> +                       compatible = "xlnx,zynq-spi-r1p6";
>                         reg = <0xe0007000 0x1000>;
>                         status = "disabled";
>                         interrupt-parent = <&intc>;
> diff --git a/doc/device-tree-bindings/spi/spi-zynq.txt b/doc/device-tree-bindings/spi/spi-zynq.txt
> index f397a36d6801..cb2945789d05 100644
> --- a/doc/device-tree-bindings/spi/spi-zynq.txt
> +++ b/doc/device-tree-bindings/spi/spi-zynq.txt
> @@ -1,29 +1,32 @@
> -Zynq SPI controller Device Tree Bindings
> -----------------------------------------
> +Cadence SPI controller Device Tree Bindings
> +-------------------------------------------
>
>  Required properties:
> -- compatible           : Should be "xlnx,spi-zynq".
> +- compatible           : Should be "cdns,spi-r1p6" or "xlnx,zynq-spi-r1p6".
>  - reg                  : Physical base address and size of SPI registers map.
> -- status               : Status will be disabled in dtsi and enabled in required dts.
> -- interrupt-parent     : Must be core interrupt controller.
>  - interrupts           : Property with a value describing the interrupt
>                           number.
> -- clocks               : Clock phandles (see clock bindings for details).
> +- interrupt-parent     : Must be core interrupt controller
>  - clock-names          : List of input clock names - "ref_clk", "pclk"
>                           (See clock bindings for details).
> +- clocks               : Clock phandles (see clock bindings for details).
>  - spi-max-frequency    : Maximum SPI clocking speed of device in Hz
>
> +Optional properties:
> +- num-cs               : Number of chip selects used.
> +                         If a decoder is used, this will be the number of
> +                         chip selects after the decoder.
> +- is-decoded-cs                : Flag to indicate whether decoder is used or not.
> +
>  Example:
>
> -       spi at e0006000 {
> -               compatible = "xlnx,zynq-spi";
> -               reg = <0xe0006000 0x1000>;
> -               status = "disabled";
> -               interrupt-parent = <&intc>;
> -               interrupts = <0 26 4>;
> -               clocks = <&clkc 25>, <&clkc 34>;
> +       spi at e0007000 {
> +               compatible = "xlnx,zynq-spi-r1p6";
>                 clock-names = "ref_clk", "pclk";
> -               spi-max-frequency = <166666700>;
> -               #address-cells = <1>;
> -               #size-cells = <0>;
> +               clocks = <&clkc 26>, <&clkc 35>;
> +               interrupt-parent = <&intc>;
> +               interrupts = <0 49 4>;
> +               num-cs = <4>;
> +               is-decoded-cs = <0>;
> +               reg = <0xe0007000 0x1000>;
>         } ;
> diff --git a/drivers/spi/zynq_spi.c b/drivers/spi/zynq_spi.c
> index c5c3e1044fda..0208afc4a636 100644
> --- a/drivers/spi/zynq_spi.c
> +++ b/drivers/spi/zynq_spi.c
> @@ -309,7 +309,7 @@ static const struct dm_spi_ops zynq_spi_ops = {
>  };
>
>  static const struct udevice_id zynq_spi_ids[] = {
> -       { .compatible = "xlnx,zynq-spi" },
> +       { .compatible = "xlnx,zynq-spi-r1p6" },
>         { }
>  };
>
> --
> 2.3.5

thanks!
-- 
Jagan | openedev.


More information about the U-Boot mailing list