Thread (131 messages) 131 messages, 6 authors, 2018-02-09

Re: [PATCH v6 40/41] ARM: da8xx-dt: switch to device tree clocks

From: Bartosz Golaszewski <hidden>
Date: 2018-02-05 15:33:42
Also in: linux-arm-kernel, linux-clk, lkml

2018-02-05 15:04 GMT+01:00 Bartosz Golaszewski [off-list ref]:
2018-01-24 4:26 GMT+01:00 David Lechner [off-list ref]:
quoted
On 01/20/2018 11:14 AM, David Lechner wrote:
quoted
This removes all of the clock init code from da8xx-dt.c. This includes
all of the OF_DEV_AUXDATA that was just used for looking up clocks.

Signed-off-by: David Lechner <david@lechnology.com>
---

v6 changes:
- removed misleading statement from commit message

  arch/arm/mach-davinci/da8xx-dt.c | 61
+---------------------------------------
  1 file changed, 1 insertion(+), 60 deletions(-)
diff --git a/arch/arm/mach-davinci/da8xx-dt.c
b/arch/arm/mach-davinci/da8xx-dt.c
quoted
  static void __init da850_init_machine(void)
  {
-       /* All existing boards use 100MHz SATA refclkpn */
-       static const unsigned long sata_refclkpn = 100 * 1000 * 1000;
-
-       int ret;
-
-       ret = da8xx_register_usb20_phy_clk(false);
-       if (ret)
-               pr_warn("%s: registering USB 2.0 PHY clock failed: %d",
-                       __func__, ret);
-       ret = da8xx_register_usb11_phy_clk(false);
-       if (ret)
-               pr_warn("%s: registering USB 1.1 PHY clock failed: %d",
-                       __func__, ret);
-
-       ret = da850_register_sata_refclk(sata_refclkpn);
-       if (ret)
-               pr_warn("%s: registering SATA REFCLK failed: %d",
-                       __func__, ret);
-
-       of_platform_default_populate(NULL, da850_auxdata_lookup, NULL);
+       of_platform_default_populate(NULL, NULL, NULL);

of_platform_default_populate() can actually be removed completely.
of_platform_default_populate_init() is called implicitly during
arch_initcall_sync
quoted
        davinci_pm_init();
        pdata_quirks_init();
  }
Hi David,

this patch breaks nand on da850-lcdk (and probably on da850-evm) since
the nand driver can no longer clk_get() the nand clock. We would need
to keep the dev_auxdata at least for aemif.

Thanks,
Bartosz
Actually scratch that last e-mail, I missed the fact that the nand
node is only there in the board dts files. I've just sent you a patch
that enables nand on da850-lcdk. Please include it in your series.
I'll do the same for evm shortly.

Thanks,
Bartosz
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help