[U-Boot] [PATCH v2] spi: cadence_qspi: support DM_CLK

Simon Goldschmidt simon.k.r.goldschmidt at gmail.com
Thu Oct 24 18:25:33 UTC 2019


Am 24.10.2019 um 20:23 schrieb Simon Goldschmidt:
> From: Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com>
> 
> Support loading clk speed via DM instead of requiring ad-hoc code.
> 
> Signed-off-by: Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com>
> Signed-off-by: Simon Goldschmidt <goldsimon at gmx.de>

That gmx adress somehow slipped in after cloning u-boot-spi. Can you 
remove it when applying or should I resend?

Regards,
Simon

> ---
> 
> Changes in v2:
> - check return value of clk_get_rate for error
> 
>   drivers/spi/cadence_qspi.c | 22 ++++++++++++++++++++--
>   1 file changed, 20 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/spi/cadence_qspi.c b/drivers/spi/cadence_qspi.c
> index e2e54cd277..60af99a16a 100644
> --- a/drivers/spi/cadence_qspi.c
> +++ b/drivers/spi/cadence_qspi.c
> @@ -5,6 +5,7 @@
>    */
>   
>   #include <common.h>
> +#include <clk.h>
>   #include <dm.h>
>   #include <fdtdec.h>
>   #include <malloc.h>
> @@ -22,12 +23,29 @@ static int cadence_spi_write_speed(struct udevice *bus, uint hz)
>   {
>   	struct cadence_spi_platdata *plat = bus->platdata;
>   	struct cadence_spi_priv *priv = dev_get_priv(bus);
> +	unsigned int ref_clk_hz;
> +	struct clk clk;
> +	int ret;
> +
> +	ret = clk_get_by_index(bus, 0, &clk);
> +	if (ret) {
> +#ifdef CONFIG_CQSPI_REF_CLK
> +		ref_clk_hz = CONFIG_CQSPI_REF_CLK;
> +#else
> +		return ret;
> +#endif
> +	} else {
> +		ref_clk_hz = clk_get_rate(&clk);
> +		clk_free(&clk);
> +		if (IS_ERR_VALUE(ref_clk_hz))
> +			return ref_clk_hz;
> +	}
>   
>   	cadence_qspi_apb_config_baudrate_div(priv->regbase,
> -					     CONFIG_CQSPI_REF_CLK, hz);
> +					     ref_clk_hz, hz);
>   
>   	/* Reconfigure delay timing if speed is changed. */
> -	cadence_qspi_apb_delay(priv->regbase, CONFIG_CQSPI_REF_CLK, hz,
> +	cadence_qspi_apb_delay(priv->regbase, ref_clk_hz, hz,
>   			       plat->tshsl_ns, plat->tsd2d_ns,
>   			       plat->tchsh_ns, plat->tslch_ns);
>   
> 



More information about the U-Boot mailing list