[U-Boot] [PATCH v5 1/5] spi: cadence_spi: Sync DT bindings with Linux

[Jagan Teki]

On Sat, Jan 6, 2018 at 11:47 PM, Jason Rush <jarush at gmail.com> wrote:
> Adopt the Linux DT bindings. This also fixes an issue
> with the indaddrtrig register on the Cadence QSPI
> device being programmed with the wrong value for the
> socfpga arch.
>
> Tested on TI K2G platform:
> Tested-by: Vignesh R <vigneshr at ti.com>
>
> Tested on a socfpga-cyclonev board:
> Tested-by: Simon Goldschmidt <sgoldschmidt at de.pepperl-fuchs.com>
>
> Signed-off-by: Jason Rush <jarush at gmail.com>
> Cc: Marek Vasut <marex at denx.de>
> ---
> Changes for v5:
>    - Rebased
> Changes for v4:
>    - Rebased
> Changes for v3:
>    - None
>
>  drivers/spi/cadence_qspi.c     | 20 ++++++++++++--------
>  drivers/spi/cadence_qspi.h     |  6 +++++-
>  drivers/spi/cadence_qspi_apb.c | 15 ++++-----------
>  3 files changed, 21 insertions(+), 20 deletions(-)
>
> diff --git a/drivers/spi/cadence_qspi.c b/drivers/spi/cadence_qspi.c
> index 9a6e41f330..991a7160bd 100644
> --- a/drivers/spi/cadence_qspi.c
> +++ b/drivers/spi/cadence_qspi.c
> @@ -212,7 +212,7 @@ static int cadence_spi_xfer(struct udevice *dev, unsigned int bitlen,
>
>         /* Set Chip select */
>         cadence_qspi_apb_chipselect(base, spi_chip_select(dev),
> -                                   CONFIG_CQSPI_DECODER);
> +                                   plat->is_decoded_cs);
>
>         if ((flags & SPI_XFER_END) || (flags == 0)) {
>                 if (priv->cmd_len == 0) {
> @@ -296,7 +296,11 @@ static int cadence_spi_ofdata_to_platdata(struct udevice *bus)
>
>         plat->regbase = (void *)data[0];
>         plat->ahbbase = (void *)data[2];
> -       plat->sram_size = fdtdec_get_int(blob, node, "sram-size", 128);
> +       plat->is_decoded_cs = fdtdec_get_bool(blob, node, "cdns,is-decoded-cs");
> +       plat->fifo_depth = fdtdec_get_uint(blob, node, "cdns,fifo-depth", 128);
> +       plat->fifo_width = fdtdec_get_uint(blob, node, "cdns,fifo-width", 4);
> +       plat->trigger_address = fdtdec_get_uint(blob, node,
> +               "cdns,trigger-address", 0);

fix these checkpatch warnings.

>
>         /* All other paramters are embedded in the child node */
>         subnode = fdt_first_subnode(blob, node);
> @@ -310,12 +314,12 @@ static int cadence_spi_ofdata_to_platdata(struct udevice *bus)
>                                        500000);
>
>         /* Read other parameters from DT */
> -       plat->page_size = fdtdec_get_int(blob, subnode, "page-size", 256);
> -       plat->block_size = fdtdec_get_int(blob, subnode, "block-size", 16);
> -       plat->tshsl_ns = fdtdec_get_int(blob, subnode, "tshsl-ns", 200);
> -       plat->tsd2d_ns = fdtdec_get_int(blob, subnode, "tsd2d-ns", 255);
> -       plat->tchsh_ns = fdtdec_get_int(blob, subnode, "tchsh-ns", 20);
> -       plat->tslch_ns = fdtdec_get_int(blob, subnode, "tslch-ns", 20);
> +       plat->page_size = fdtdec_get_uint(blob, subnode, "page-size", 256);
> +       plat->block_size = fdtdec_get_uint(blob, subnode, "block-size", 16);
> +       plat->tshsl_ns = fdtdec_get_uint(blob, subnode, "cdns,tshsl-ns", 200);
> +       plat->tsd2d_ns = fdtdec_get_uint(blob, subnode, "cdns,tsd2d-ns", 255);
> +       plat->tchsh_ns = fdtdec_get_uint(blob, subnode, "cdns,tchsh-ns", 20);
> +       plat->tslch_ns = fdtdec_get_uint(blob, subnode, "cdns,tslch-ns", 20);
>
>         debug("%s: regbase=%p ahbbase=%p max-frequency=%d page-size=%d\n",
>               __func__, plat->regbase, plat->ahbbase, plat->max_hz,
> diff --git a/drivers/spi/cadence_qspi.h b/drivers/spi/cadence_qspi.h
> index d1927a4003..83154210bd 100644
> --- a/drivers/spi/cadence_qspi.h
> +++ b/drivers/spi/cadence_qspi.h
> @@ -18,14 +18,18 @@ struct cadence_spi_platdata {
>         unsigned int    max_hz;
>         void            *regbase;
>         void            *ahbbase;
> +       bool            is_decoded_cs;
> +       u32             fifo_depth;
> +       u32             fifo_width;
> +       u32             trigger_address;
>
> +       // Flash parameters

fix with proper single line comment /* ... */

except these,

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