Message ID | 20171228120903.17591-3-Eugeniy.Paltsev@synopsys.com |
---|---|
State | Accepted |
Commit | 58c125b9e2b232ce73ed7b24ba7b1ca5ff41c5bd |
Delegated to: | Jagannadha Sutradharudu Teki |
Headers | show |
Series | DW SPI: Get clock value from Device Tree | expand |
Hi Jagan, Could you please pull these patches to your tree? It would be really nice to see this patch in the nearest release. Thanks. On Thu, 2017-12-28 at 15:09 +0300, Eugeniy Paltsev wrote: > Add option to set spi controller clock frequency via device tree > using standard clock bindings. > > Define dw_spi_get_clk function as 'weak' as some targets > (like SOCFPGA_GEN5 and SOCFPGA_ARRIA10) don't use standard clock API > and implement dw_spi_get_clk their own way in their clock manager. > > Get rid of clock_manager.h include as we don't use > cm_get_spi_controller_clk_hz function anymore. (we use redefined > dw_spi_get_clk in SOCFPGA clock managers instead) > > Reviewed-by: Marek Vasut <marex@denx.de> > Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> > --- > drivers/spi/designware_spi.c | 45 ++++++++++++++++++++++++++++++++++++++++++-- > 1 file changed, 43 insertions(+), 2 deletions(-) > > diff --git a/drivers/spi/designware_spi.c b/drivers/spi/designware_spi.c > index 5aa507b..c501aee 100644 > --- a/drivers/spi/designware_spi.c > +++ b/drivers/spi/designware_spi.c > @@ -11,6 +11,7 @@ > */ > > #include <common.h> > +#include <clk.h> > #include <dm.h> > #include <errno.h> > #include <malloc.h> > @@ -18,7 +19,6 @@ > #include <fdtdec.h> > #include <linux/compat.h> > #include <asm/io.h> > -#include <asm/arch/clock_manager.h> > > DECLARE_GLOBAL_DATA_PTR; > > @@ -94,6 +94,8 @@ struct dw_spi_priv { > void __iomem *regs; > unsigned int freq; /* Default frequency */ > unsigned int mode; > + struct clk clk; > + unsigned long bus_clk_rate; > > int bits_per_word; > u8 cs; /* chip select pin */ > @@ -176,14 +178,53 @@ static void spi_hw_init(struct dw_spi_priv *priv) > debug("%s: fifo_len=%d\n", __func__, priv->fifo_len); > } > > +/* > + * We define dw_spi_get_clk function as 'weak' as some targets > + * (like SOCFPGA_GEN5 and SOCFPGA_ARRIA10) don't use standard clock API > + * and implement dw_spi_get_clk their own way in their clock manager. > + */ > +__weak int dw_spi_get_clk(struct udevice *bus, ulong *rate) > +{ > + struct dw_spi_priv *priv = dev_get_priv(bus); > + int ret; > + > + ret = clk_get_by_index(bus, 0, &priv->clk); > + if (ret) > + return ret; > + > + ret = clk_enable(&priv->clk); > + if (ret && ret != -ENOSYS && ret != -ENOTSUPP) > + return ret; > + > + *rate = clk_get_rate(&priv->clk); > + if (!*rate) > + goto err_rate; > + > + debug("%s: get spi controller clk via device tree: %lu Hz\n", > + __func__, *rate); > + > + return 0; > + > +err_rate: > + clk_disable(&priv->clk); > + clk_free(&priv->clk); > + > + return -EINVAL; > +} > + > static int dw_spi_probe(struct udevice *bus) > { > struct dw_spi_platdata *plat = dev_get_platdata(bus); > struct dw_spi_priv *priv = dev_get_priv(bus); > + int ret; > > priv->regs = plat->regs; > priv->freq = plat->frequency; > > + ret = dw_spi_get_clk(bus, &priv->bus_clk_rate); > + if (ret) > + return ret; > + > /* Currently only bits_per_word == 8 supported */ > priv->bits_per_word = 8; > > @@ -369,7 +410,7 @@ static int dw_spi_set_speed(struct udevice *bus, uint speed) > spi_enable_chip(priv, 0); > > /* clk_div doesn't support odd number */ > - clk_div = cm_get_spi_controller_clk_hz() / speed; > + clk_div = priv->bus_clk_rate / speed; > clk_div = (clk_div + 1) & 0xfffe; > dw_writel(priv, DW_SPI_BAUDR, clk_div); > -- Eugeniy Paltsev
Hi Jagan, On Fri, 2018-01-19 at 14:17 +0100, Eugeniy Paltsev wrote: > Hi Jagan, > > Could you please pull these patches to your tree? > It would be really nice to see this patch in the nearest release. > > Thanks. > > On Thu, 2017-12-28 at 15:09 +0300, Eugeniy Paltsev wrote: > > Add option to set spi controller clock frequency via device tree > > using standard clock bindings. > > > > Define dw_spi_get_clk function as 'weak' as some targets > > (like SOCFPGA_GEN5 and SOCFPGA_ARRIA10) don't use standard clock API > > and implement dw_spi_get_clk their own way in their clock manager. > > > > Get rid of clock_manager.h include as we don't use > > cm_get_spi_controller_clk_hz function anymore. (we use redefined > > dw_spi_get_clk in SOCFPGA clock managers instead) > > > > Reviewed-by: Marek Vasut <marex@denx.de> > > Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> > > --- Any chance for this one to be applied sometime soon? Note this is a prerequisite for our further work and it's been floating and discussed on the mailing list for a couple of months now, that said we'd really like to get this as a part of U-Boot 2018.03. -Alexey
On Thu, Jan 25, 2018 at 6:15 PM, Alexey Brodkin <Alexey.Brodkin@synopsys.com> wrote: > Hi Jagan, > > On Fri, 2018-01-19 at 14:17 +0100, Eugeniy Paltsev wrote: >> Hi Jagan, >> >> Could you please pull these patches to your tree? >> It would be really nice to see this patch in the nearest release. >> >> Thanks. >> >> On Thu, 2017-12-28 at 15:09 +0300, Eugeniy Paltsev wrote: >> > Add option to set spi controller clock frequency via device tree >> > using standard clock bindings. >> > >> > Define dw_spi_get_clk function as 'weak' as some targets >> > (like SOCFPGA_GEN5 and SOCFPGA_ARRIA10) don't use standard clock API >> > and implement dw_spi_get_clk their own way in their clock manager. >> > >> > Get rid of clock_manager.h include as we don't use >> > cm_get_spi_controller_clk_hz function anymore. (we use redefined >> > dw_spi_get_clk in SOCFPGA clock managers instead) >> > >> > Reviewed-by: Marek Vasut <marex@denx.de> >> > Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> >> > --- > > Any chance for this one to be applied sometime soon? > Note this is a prerequisite for our further work and it's been floating > and discussed on the mailing list for a couple of months now, > that said we'd really like to get this as a part of U-Boot 2018.03. Since delegate has marked to 'marek' it wasn't showed with me, yes will take this.
On Thu, Jan 25, 2018 at 06:37:20PM +0530, Jagan Teki wrote: > On Thu, Jan 25, 2018 at 6:15 PM, Alexey Brodkin > <Alexey.Brodkin@synopsys.com> wrote: > > Hi Jagan, > > > > On Fri, 2018-01-19 at 14:17 +0100, Eugeniy Paltsev wrote: > >> Hi Jagan, > >> > >> Could you please pull these patches to your tree? > >> It would be really nice to see this patch in the nearest release. > >> > >> Thanks. > >> > >> On Thu, 2017-12-28 at 15:09 +0300, Eugeniy Paltsev wrote: > >> > Add option to set spi controller clock frequency via device tree > >> > using standard clock bindings. > >> > > >> > Define dw_spi_get_clk function as 'weak' as some targets > >> > (like SOCFPGA_GEN5 and SOCFPGA_ARRIA10) don't use standard clock API > >> > and implement dw_spi_get_clk their own way in their clock manager. > >> > > >> > Get rid of clock_manager.h include as we don't use > >> > cm_get_spi_controller_clk_hz function anymore. (we use redefined > >> > dw_spi_get_clk in SOCFPGA clock managers instead) > >> > > >> > Reviewed-by: Marek Vasut <marex@denx.de> > >> > Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> > >> > --- > > > > Any chance for this one to be applied sometime soon? > > Note this is a prerequisite for our further work and it's been floating > > and discussed on the mailing list for a couple of months now, > > that said we'd really like to get this as a part of U-Boot 2018.03. > > Since delegate has marked to 'marek' it wasn't showed with me, yes > will take this. As always, oops, my bad. I think I saw Marek commenting at one point and kept delegating to him. And I missed that it now has his Reviewed-by, so yes, please pick this up in patchwork, thanks!
On Thu, Jan 25, 2018 at 10:01 PM, Tom Rini <trini@konsulko.com> wrote: > On Thu, Jan 25, 2018 at 06:37:20PM +0530, Jagan Teki wrote: >> On Thu, Jan 25, 2018 at 6:15 PM, Alexey Brodkin >> <Alexey.Brodkin@synopsys.com> wrote: >> > Hi Jagan, >> > >> > On Fri, 2018-01-19 at 14:17 +0100, Eugeniy Paltsev wrote: >> >> Hi Jagan, >> >> >> >> Could you please pull these patches to your tree? >> >> It would be really nice to see this patch in the nearest release. >> >> >> >> Thanks. >> >> >> >> On Thu, 2017-12-28 at 15:09 +0300, Eugeniy Paltsev wrote: >> >> > Add option to set spi controller clock frequency via device tree >> >> > using standard clock bindings. >> >> > >> >> > Define dw_spi_get_clk function as 'weak' as some targets >> >> > (like SOCFPGA_GEN5 and SOCFPGA_ARRIA10) don't use standard clock API >> >> > and implement dw_spi_get_clk their own way in their clock manager. >> >> > >> >> > Get rid of clock_manager.h include as we don't use >> >> > cm_get_spi_controller_clk_hz function anymore. (we use redefined >> >> > dw_spi_get_clk in SOCFPGA clock managers instead) >> >> > >> >> > Reviewed-by: Marek Vasut <marex@denx.de> >> >> > Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> >> >> > --- >> > >> > Any chance for this one to be applied sometime soon? >> > Note this is a prerequisite for our further work and it's been floating >> > and discussed on the mailing list for a couple of months now, >> > that said we'd really like to get this as a part of U-Boot 2018.03. >> >> Since delegate has marked to 'marek' it wasn't showed with me, yes >> will take this. > > As always, oops, my bad. I think I saw Marek commenting at one point > and kept delegating to him. And I missed that it now has his > Reviewed-by, so yes, please pick this up in patchwork, thanks! Applied to u-boot-spi/master, thanks!
diff --git a/drivers/spi/designware_spi.c b/drivers/spi/designware_spi.c index 5aa507b..c501aee 100644 --- a/drivers/spi/designware_spi.c +++ b/drivers/spi/designware_spi.c @@ -11,6 +11,7 @@ */ #include <common.h> +#include <clk.h> #include <dm.h> #include <errno.h> #include <malloc.h> @@ -18,7 +19,6 @@ #include <fdtdec.h> #include <linux/compat.h> #include <asm/io.h> -#include <asm/arch/clock_manager.h> DECLARE_GLOBAL_DATA_PTR; @@ -94,6 +94,8 @@ struct dw_spi_priv { void __iomem *regs; unsigned int freq; /* Default frequency */ unsigned int mode; + struct clk clk; + unsigned long bus_clk_rate; int bits_per_word; u8 cs; /* chip select pin */ @@ -176,14 +178,53 @@ static void spi_hw_init(struct dw_spi_priv *priv) debug("%s: fifo_len=%d\n", __func__, priv->fifo_len); } +/* + * We define dw_spi_get_clk function as 'weak' as some targets + * (like SOCFPGA_GEN5 and SOCFPGA_ARRIA10) don't use standard clock API + * and implement dw_spi_get_clk their own way in their clock manager. + */ +__weak int dw_spi_get_clk(struct udevice *bus, ulong *rate) +{ + struct dw_spi_priv *priv = dev_get_priv(bus); + int ret; + + ret = clk_get_by_index(bus, 0, &priv->clk); + if (ret) + return ret; + + ret = clk_enable(&priv->clk); + if (ret && ret != -ENOSYS && ret != -ENOTSUPP) + return ret; + + *rate = clk_get_rate(&priv->clk); + if (!*rate) + goto err_rate; + + debug("%s: get spi controller clk via device tree: %lu Hz\n", + __func__, *rate); + + return 0; + +err_rate: + clk_disable(&priv->clk); + clk_free(&priv->clk); + + return -EINVAL; +} + static int dw_spi_probe(struct udevice *bus) { struct dw_spi_platdata *plat = dev_get_platdata(bus); struct dw_spi_priv *priv = dev_get_priv(bus); + int ret; priv->regs = plat->regs; priv->freq = plat->frequency; + ret = dw_spi_get_clk(bus, &priv->bus_clk_rate); + if (ret) + return ret; + /* Currently only bits_per_word == 8 supported */ priv->bits_per_word = 8; @@ -369,7 +410,7 @@ static int dw_spi_set_speed(struct udevice *bus, uint speed) spi_enable_chip(priv, 0); /* clk_div doesn't support odd number */ - clk_div = cm_get_spi_controller_clk_hz() / speed; + clk_div = priv->bus_clk_rate / speed; clk_div = (clk_div + 1) & 0xfffe; dw_writel(priv, DW_SPI_BAUDR, clk_div);