[PATCH] spi: cadence-qspi: Move ref clock calculation to probe

Vignesh Raghavendra vigneshr at ti.com
Wed Feb 26 08:29:44 CET 2020


+Simon who converted driver to use clk_get* APIs

On 24/02/20 12:40 pm, Pratyush Yadav wrote:
> "assigned-clock-parents" and "assigned-clock-rates" DT properties take
> effect only after ofdata_to_platdata() when clk_set_defaults() is called
> in device_probe(). Therefore clk get rate() would return a wrong value
> in ofdata_to_platdata() when compared with probe. Hence it needs to be
> moved to probe.
> 
> Tested on u-boot-ti/next.
> 

Acked-by: Vignesh Raghavendra <vigneshr at ti.com>

Regards
Vignesh

> Signed-off-by: Pratyush Yadav <p.yadav at ti.com>
> ---
>  drivers/spi/cadence_qspi.c | 33 +++++++++++++++++----------------
>  1 file changed, 17 insertions(+), 16 deletions(-)
> 
> diff --git a/drivers/spi/cadence_qspi.c b/drivers/spi/cadence_qspi.c
> index 83b114ffe7..994a5948f1 100644
> --- a/drivers/spi/cadence_qspi.c
> +++ b/drivers/spi/cadence_qspi.c
> @@ -166,11 +166,28 @@ static int cadence_spi_probe(struct udevice *bus)
>  {
>  	struct cadence_spi_platdata *plat = bus->platdata;
>  	struct cadence_spi_priv *priv = dev_get_priv(bus);
> +	struct clk clk;
>  	int ret;
>  
>  	priv->regbase = plat->regbase;
>  	priv->ahbbase = plat->ahbbase;
>  
> +	if (plat->ref_clk_hz == 0) {
> +		ret = clk_get_by_index(bus, 0, &clk);
> +		if (ret) {
> +#ifdef CONFIG_CQSPI_REF_CLK
> +			plat->ref_clk_hz = CONFIG_CQSPI_REF_CLK;
> +#else
> +			return ret;
> +#endif
> +		} else {
> +			plat->ref_clk_hz = clk_get_rate(&clk);
> +			clk_free(&clk);
> +			if (IS_ERR_VALUE(plat->ref_clk_hz))
> +				return plat->ref_clk_hz;
> +		}
> +	}
> +
>  	ret = reset_get_bulk(bus, &priv->resets);
>  	if (ret)
>  		dev_warn(bus, "Can't get reset: %d\n", ret);
> @@ -268,8 +285,6 @@ static int cadence_spi_ofdata_to_platdata(struct udevice *bus)
>  {
>  	struct cadence_spi_platdata *plat = bus->platdata;
>  	ofnode subnode;
> -	struct clk clk;
> -	int ret;
>  
>  	plat->regbase = (void *)devfdt_get_addr_index(bus, 0);
>  	plat->ahbbase = (void *)devfdt_get_addr_size_index(bus, 1,
> @@ -305,20 +320,6 @@ static int cadence_spi_ofdata_to_platdata(struct udevice *bus)
>  	plat->tchsh_ns = ofnode_read_u32_default(subnode, "cdns,tchsh-ns", 20);
>  	plat->tslch_ns = ofnode_read_u32_default(subnode, "cdns,tslch-ns", 20);
>  
> -	ret = clk_get_by_index(bus, 0, &clk);
> -	if (ret) {
> -#ifdef CONFIG_CQSPI_REF_CLK
> -		plat->ref_clk_hz = CONFIG_CQSPI_REF_CLK;
> -#else
> -		return ret;
> -#endif
> -	} else {
> -		plat->ref_clk_hz = clk_get_rate(&clk);
> -		clk_free(&clk);
> -		if (IS_ERR_VALUE(plat->ref_clk_hz))
> -			return plat->ref_clk_hz;
> -	}
> -
>  	debug("%s: regbase=%p ahbbase=%p max-frequency=%d page-size=%d\n",
>  	      __func__, plat->regbase, plat->ahbbase, plat->max_hz,
>  	      plat->page_size);
> 




More information about the U-Boot mailing list