[PATCH] spi: cadence-qspi: Move ref clock calculation to probe
"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.
Signed-off-by: Pratyush Yadav p.yadav@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);
+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@ti.com
Regards Vignesh
Signed-off-by: Pratyush Yadav p.yadav@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);
Vignesh Raghavendra vigneshr@ti.com schrieb am Mi., 26. Feb. 2020, 08:29:
+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@ti.com
Fine by me. I actually moved it there after someone requested me to :-) I first had it in the set_rate function...
Acked-by: Simon Goldschmidt simon.k.r.goldschmidt@gmail.com
Regards Vignesh
Signed-off-by: Pratyush Yadav p.yadav@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);
On Wed, Feb 26, 2020 at 1:02 PM Simon Goldschmidt simon.k.r.goldschmidt@gmail.com wrote:
Vignesh Raghavendra vigneshr@ti.com schrieb am Mi., 26. Feb. 2020, 08:29:
+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@ti.com
Fine by me. I actually moved it there after someone requested me to :-) I first had it in the set_rate function...
Acked-by: Simon Goldschmidt simon.k.r.goldschmidt@gmail.com
So, does this fixing something or will it work even for wrong value? I'm thinking of picking it up for release, if associated boards got tested.
Jagan.
On Mon, Feb 24, 2020 at 12:40 PM Pratyush Yadav p.yadav@ti.com 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.
Signed-off-by: Pratyush Yadav p.yadav@ti.com
Applied to u-boot-spi/master
participants (4)
-
Jagan Teki -
Pratyush Yadav -
Simon Goldschmidt -
Vignesh Raghavendra