* [PATCH] ARM: imx: add imx5x usb controller clock lookups
@ 2012-08-23 10:38 Michael Grzeschik
2012-08-23 11:01 ` [PATCH v2] " Michael Grzeschik
2012-08-23 14:18 ` [PATCH] " Shawn Guo
0 siblings, 2 replies; 7+ messages in thread
From: Michael Grzeschik @ 2012-08-23 10:38 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Michael Grzeschik <m.grzeschik@pengutronix.de>
---
arch/arm/mach-imx/clk-imx51-imx53.c | 36 +++++++++++++++++++++++------------
1 file changed, 24 insertions(+), 12 deletions(-)
diff --git a/arch/arm/mach-imx/clk-imx51-imx53.c b/arch/arm/mach-imx/clk-imx51-imx53.c
index 4bdcaa9..6ce49b1 100644
--- a/arch/arm/mach-imx/clk-imx51-imx53.c
+++ b/arch/arm/mach-imx/clk-imx51-imx53.c
@@ -258,18 +258,6 @@ static void __init mx5_clocks_common_init(unsigned long rate_ckil,
clk_register_clkdev(clk[pwm2_ipg_gate], "pwm", "mxc_pwm.1");
clk_register_clkdev(clk[i2c1_gate], NULL, "imx-i2c.0");
clk_register_clkdev(clk[i2c2_gate], NULL, "imx-i2c.1");
- clk_register_clkdev(clk[usboh3_per_gate], "per", "mxc-ehci.0");
- clk_register_clkdev(clk[usboh3_gate], "ipg", "mxc-ehci.0");
- clk_register_clkdev(clk[usboh3_gate], "ahb", "mxc-ehci.0");
- clk_register_clkdev(clk[usboh3_per_gate], "per", "mxc-ehci.1");
- clk_register_clkdev(clk[usboh3_gate], "ipg", "mxc-ehci.1");
- clk_register_clkdev(clk[usboh3_gate], "ahb", "mxc-ehci.1");
- clk_register_clkdev(clk[usboh3_per_gate], "per", "mxc-ehci.2");
- clk_register_clkdev(clk[usboh3_gate], "ipg", "mxc-ehci.2");
- clk_register_clkdev(clk[usboh3_gate], "ahb", "mxc-ehci.2");
- clk_register_clkdev(clk[usboh3_per_gate], "per", "fsl-usb2-udc");
- clk_register_clkdev(clk[usboh3_gate], "ipg", "fsl-usb2-udc");
- clk_register_clkdev(clk[usboh3_gate], "ahb", "fsl-usb2-udc");
clk_register_clkdev(clk[nfc_gate], NULL, "mxc_nand");
clk_register_clkdev(clk[ssi1_ipg_gate], NULL, "imx-ssi.0");
clk_register_clkdev(clk[ssi2_ipg_gate], NULL, "imx-ssi.1");
@@ -351,6 +339,18 @@ int __init mx51_clocks_init(unsigned long rate_ckil, unsigned long rate_osc,
clk_register_clkdev(clk[ipu_di0_gate], "di0", "imx51-ipu");
clk_register_clkdev(clk[ipu_di1_gate], "di1", "imx51-ipu");
clk_register_clkdev(clk[ipu_gate], "hsp", "imx51-ipu");
+ clk_register_clkdev(clk[usboh3_per_gate], "per", "73f80600.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ipg", "73f80600.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ahb", "73f80600.usb");
+ clk_register_clkdev(clk[usboh3_per_gate], "per", "73f80400.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ipg", "73f80400.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ahb", "73f80400.usb");
+ clk_register_clkdev(clk[usboh3_per_gate], "per", "73f80200.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ipg", "73f80200.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ahb", "73f80200.usb");
+ clk_register_clkdev(clk[usboh3_per_gate], "per", "73f80000.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ipg", "73f80000.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ahb", "73f80000.usb");
clk_register_clkdev(clk[usb_phy_gate], "phy", "mxc-ehci.0");
clk_register_clkdev(clk[esdhc1_ipg_gate], "ipg", "sdhci-esdhc-imx51.0");
clk_register_clkdev(clk[dummy], "ahb", "sdhci-esdhc-imx51.0");
@@ -439,6 +439,18 @@ int __init mx53_clocks_init(unsigned long rate_ckil, unsigned long rate_osc,
clk_register_clkdev(clk[ipu_di0_gate], "di0", "imx53-ipu");
clk_register_clkdev(clk[ipu_di1_gate], "di1", "imx53-ipu");
clk_register_clkdev(clk[ipu_gate], "hsp", "imx53-ipu");
+ clk_register_clkdev(clk[usboh3_per_gate], "per", "53f80600.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ipg", "53f80600.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ahb", "53f80600.usb");
+ clk_register_clkdev(clk[usboh3_per_gate], "per", "53f80400.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ipg", "53f80400.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ahb", "53f80400.usb");
+ clk_register_clkdev(clk[usboh3_per_gate], "per", "53f80200.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ipg", "53f80200.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ahb", "53f80200.usb");
+ clk_register_clkdev(clk[usboh3_per_gate], "per", "53f80000.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ipg", "53f80000.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ahb", "53f80000.usb");
clk_register_clkdev(clk[usb_phy1_gate], "usb_phy1", "mxc-ehci.0");
clk_register_clkdev(clk[esdhc1_ipg_gate], "ipg", "sdhci-esdhc-imx53.0");
clk_register_clkdev(clk[dummy], "ahb", "sdhci-esdhc-imx53.0");
--
1.7.10.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH v2] ARM: imx: add imx5x usb controller clock lookups
2012-08-23 10:38 [PATCH] ARM: imx: add imx5x usb controller clock lookups Michael Grzeschik
@ 2012-08-23 11:01 ` Michael Grzeschik
2012-08-23 11:13 ` Fabio Estevam
2012-08-23 14:18 ` [PATCH] " Shawn Guo
1 sibling, 1 reply; 7+ messages in thread
From: Michael Grzeschik @ 2012-08-23 11:01 UTC (permalink / raw)
To: linux-arm-kernel
Signed-off-by: Michael Grzeschik <m.grzeschik@pengutronix.de>
---
arch/arm/mach-imx/clk-imx51-imx53.c | 24 ++++++++++++++++++++++++
1 file changed, 24 insertions(+)
diff --git a/arch/arm/mach-imx/clk-imx51-imx53.c b/arch/arm/mach-imx/clk-imx51-imx53.c
index 4bdcaa9..7eba6ab 100644
--- a/arch/arm/mach-imx/clk-imx51-imx53.c
+++ b/arch/arm/mach-imx/clk-imx51-imx53.c
@@ -351,6 +351,18 @@ int __init mx51_clocks_init(unsigned long rate_ckil, unsigned long rate_osc,
clk_register_clkdev(clk[ipu_di0_gate], "di0", "imx51-ipu");
clk_register_clkdev(clk[ipu_di1_gate], "di1", "imx51-ipu");
clk_register_clkdev(clk[ipu_gate], "hsp", "imx51-ipu");
+ clk_register_clkdev(clk[usboh3_per_gate], "per", "73f80600.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ipg", "73f80600.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ahb", "73f80600.usb");
+ clk_register_clkdev(clk[usboh3_per_gate], "per", "73f80400.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ipg", "73f80400.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ahb", "73f80400.usb");
+ clk_register_clkdev(clk[usboh3_per_gate], "per", "73f80200.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ipg", "73f80200.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ahb", "73f80200.usb");
+ clk_register_clkdev(clk[usboh3_per_gate], "per", "73f80000.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ipg", "73f80000.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ahb", "73f80000.usb");
clk_register_clkdev(clk[usb_phy_gate], "phy", "mxc-ehci.0");
clk_register_clkdev(clk[esdhc1_ipg_gate], "ipg", "sdhci-esdhc-imx51.0");
clk_register_clkdev(clk[dummy], "ahb", "sdhci-esdhc-imx51.0");
@@ -439,6 +451,18 @@ int __init mx53_clocks_init(unsigned long rate_ckil, unsigned long rate_osc,
clk_register_clkdev(clk[ipu_di0_gate], "di0", "imx53-ipu");
clk_register_clkdev(clk[ipu_di1_gate], "di1", "imx53-ipu");
clk_register_clkdev(clk[ipu_gate], "hsp", "imx53-ipu");
+ clk_register_clkdev(clk[usboh3_per_gate], "per", "53f80600.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ipg", "53f80600.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ahb", "53f80600.usb");
+ clk_register_clkdev(clk[usboh3_per_gate], "per", "53f80400.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ipg", "53f80400.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ahb", "53f80400.usb");
+ clk_register_clkdev(clk[usboh3_per_gate], "per", "53f80200.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ipg", "53f80200.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ahb", "53f80200.usb");
+ clk_register_clkdev(clk[usboh3_per_gate], "per", "53f80000.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ipg", "53f80000.usb");
+ clk_register_clkdev(clk[usboh3_gate], "ahb", "53f80000.usb");
clk_register_clkdev(clk[usb_phy1_gate], "usb_phy1", "mxc-ehci.0");
clk_register_clkdev(clk[esdhc1_ipg_gate], "ipg", "sdhci-esdhc-imx53.0");
clk_register_clkdev(clk[dummy], "ahb", "sdhci-esdhc-imx53.0");
--
1.7.10.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH v2] ARM: imx: add imx5x usb controller clock lookups
2012-08-23 11:01 ` [PATCH v2] " Michael Grzeschik
@ 2012-08-23 11:13 ` Fabio Estevam
2012-08-23 11:36 ` Michael Grzeschik
0 siblings, 1 reply; 7+ messages in thread
From: Fabio Estevam @ 2012-08-23 11:13 UTC (permalink / raw)
To: linux-arm-kernel
Hi Michael,
On Thu, Aug 23, 2012 at 8:01 AM, Michael Grzeschik
<m.grzeschik@pengutronix.de> wrote:
> Signed-off-by: Michael Grzeschik <m.grzeschik@pengutronix.de>
> ---
> arch/arm/mach-imx/clk-imx51-imx53.c | 24 ++++++++++++++++++++++++
Just curious: does the current Chip Idea USB driver work 'as is' on mx5?
Are you also going to submit the <soc>.dtsi and <board>.dts patches?
I would like to try it on a mx53qsb.
Regards,
Fabio Estevam
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2] ARM: imx: add imx5x usb controller clock lookups
2012-08-23 11:13 ` Fabio Estevam
@ 2012-08-23 11:36 ` Michael Grzeschik
2012-08-23 11:54 ` Fabio Estevam
0 siblings, 1 reply; 7+ messages in thread
From: Michael Grzeschik @ 2012-08-23 11:36 UTC (permalink / raw)
To: linux-arm-kernel
Hi Fabio,
On Thu, Aug 23, 2012 at 08:13:38AM -0300, Fabio Estevam wrote:
> On Thu, Aug 23, 2012 at 8:01 AM, Michael Grzeschik
> <m.grzeschik@pengutronix.de> wrote:
> > Signed-off-by: Michael Grzeschik <m.grzeschik@pengutronix.de>
> > ---
> > arch/arm/mach-imx/clk-imx51-imx53.c | 24 ++++++++++++++++++++++++
>
> Just curious: does the current Chip Idea USB driver work 'as is' on mx5?
Not "as is" but we are working on it to get all patches for that
mainline. Currently there is the static CI13XXX_REQUIRE_TRANSCEIVER flag
in the ci13xxx_imx_data, which makes the socs with no real xceiver
depend on the nop-xceiver. But that one lags clk_prepare_{enable, disable}.
But we already prepared that in some, "hopefully ready for mainline", patches.
> Are you also going to submit the <soc>.dtsi and <board>.dts patches?
Yes, the dtsi patch is alread on the list:
http://www.spinics.net/lists/arm-kernel/msg190737.html
> I would like to try it on a mx53qsb.
we got it already running with that board and the above mentioned
dependencies.
Cheers,
Michael
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2] ARM: imx: add imx5x usb controller clock lookups
2012-08-23 11:36 ` Michael Grzeschik
@ 2012-08-23 11:54 ` Fabio Estevam
0 siblings, 0 replies; 7+ messages in thread
From: Fabio Estevam @ 2012-08-23 11:54 UTC (permalink / raw)
To: linux-arm-kernel
Hi Michael,
On Thu, Aug 23, 2012 at 8:36 AM, Michael Grzeschik <mgr@pengutronix.de> wrote:
> we got it already running with that board and the above mentioned
> dependencies.
This is very good news. I will wait for the patches to reach the
mainling lists and then will have a try.
Thanks,
Fabio Estevam
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: imx: add imx5x usb controller clock lookups
2012-08-23 10:38 [PATCH] ARM: imx: add imx5x usb controller clock lookups Michael Grzeschik
2012-08-23 11:01 ` [PATCH v2] " Michael Grzeschik
@ 2012-08-23 14:18 ` Shawn Guo
2012-08-23 14:41 ` Marc Kleine-Budde
1 sibling, 1 reply; 7+ messages in thread
From: Shawn Guo @ 2012-08-23 14:18 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Aug 23, 2012 at 12:38:48PM +0200, Michael Grzeschik wrote:
> Signed-off-by: Michael Grzeschik <m.grzeschik@pengutronix.de>
> ---
> arch/arm/mach-imx/clk-imx51-imx53.c | 36 +++++++++++++++++++++++------------
> 1 file changed, 24 insertions(+), 12 deletions(-)
>
> diff --git a/arch/arm/mach-imx/clk-imx51-imx53.c b/arch/arm/mach-imx/clk-imx51-imx53.c
> index 4bdcaa9..6ce49b1 100644
> --- a/arch/arm/mach-imx/clk-imx51-imx53.c
> +++ b/arch/arm/mach-imx/clk-imx51-imx53.c
> @@ -258,18 +258,6 @@ static void __init mx5_clocks_common_init(unsigned long rate_ckil,
> clk_register_clkdev(clk[pwm2_ipg_gate], "pwm", "mxc_pwm.1");
> clk_register_clkdev(clk[i2c1_gate], NULL, "imx-i2c.0");
> clk_register_clkdev(clk[i2c2_gate], NULL, "imx-i2c.1");
> - clk_register_clkdev(clk[usboh3_per_gate], "per", "mxc-ehci.0");
> - clk_register_clkdev(clk[usboh3_gate], "ipg", "mxc-ehci.0");
> - clk_register_clkdev(clk[usboh3_gate], "ahb", "mxc-ehci.0");
> - clk_register_clkdev(clk[usboh3_per_gate], "per", "mxc-ehci.1");
> - clk_register_clkdev(clk[usboh3_gate], "ipg", "mxc-ehci.1");
> - clk_register_clkdev(clk[usboh3_gate], "ahb", "mxc-ehci.1");
> - clk_register_clkdev(clk[usboh3_per_gate], "per", "mxc-ehci.2");
> - clk_register_clkdev(clk[usboh3_gate], "ipg", "mxc-ehci.2");
> - clk_register_clkdev(clk[usboh3_gate], "ahb", "mxc-ehci.2");
> - clk_register_clkdev(clk[usboh3_per_gate], "per", "fsl-usb2-udc");
> - clk_register_clkdev(clk[usboh3_gate], "ipg", "fsl-usb2-udc");
> - clk_register_clkdev(clk[usboh3_gate], "ahb", "fsl-usb2-udc");
Removing those will break some in-tree users, right?
> clk_register_clkdev(clk[nfc_gate], NULL, "mxc_nand");
> clk_register_clkdev(clk[ssi1_ipg_gate], NULL, "imx-ssi.0");
> clk_register_clkdev(clk[ssi2_ipg_gate], NULL, "imx-ssi.1");
> @@ -351,6 +339,18 @@ int __init mx51_clocks_init(unsigned long rate_ckil, unsigned long rate_osc,
> clk_register_clkdev(clk[ipu_di0_gate], "di0", "imx51-ipu");
> clk_register_clkdev(clk[ipu_di1_gate], "di1", "imx51-ipu");
> clk_register_clkdev(clk[ipu_gate], "hsp", "imx51-ipu");
> + clk_register_clkdev(clk[usboh3_per_gate], "per", "73f80600.usb");
> + clk_register_clkdev(clk[usboh3_gate], "ipg", "73f80600.usb");
> + clk_register_clkdev(clk[usboh3_gate], "ahb", "73f80600.usb");
> + clk_register_clkdev(clk[usboh3_per_gate], "per", "73f80400.usb");
> + clk_register_clkdev(clk[usboh3_gate], "ipg", "73f80400.usb");
> + clk_register_clkdev(clk[usboh3_gate], "ahb", "73f80400.usb");
> + clk_register_clkdev(clk[usboh3_per_gate], "per", "73f80200.usb");
> + clk_register_clkdev(clk[usboh3_gate], "ipg", "73f80200.usb");
> + clk_register_clkdev(clk[usboh3_gate], "ahb", "73f80200.usb");
> + clk_register_clkdev(clk[usboh3_per_gate], "per", "73f80000.usb");
> + clk_register_clkdev(clk[usboh3_gate], "ipg", "73f80000.usb");
> + clk_register_clkdev(clk[usboh3_gate], "ahb", "73f80000.usb");
I haven't seen new driver requesting these clocks, so would postpone
the changes until the driver actually requests them.
Regards,
Shawn
> clk_register_clkdev(clk[usb_phy_gate], "phy", "mxc-ehci.0");
> clk_register_clkdev(clk[esdhc1_ipg_gate], "ipg", "sdhci-esdhc-imx51.0");
> clk_register_clkdev(clk[dummy], "ahb", "sdhci-esdhc-imx51.0");
> @@ -439,6 +439,18 @@ int __init mx53_clocks_init(unsigned long rate_ckil, unsigned long rate_osc,
> clk_register_clkdev(clk[ipu_di0_gate], "di0", "imx53-ipu");
> clk_register_clkdev(clk[ipu_di1_gate], "di1", "imx53-ipu");
> clk_register_clkdev(clk[ipu_gate], "hsp", "imx53-ipu");
> + clk_register_clkdev(clk[usboh3_per_gate], "per", "53f80600.usb");
> + clk_register_clkdev(clk[usboh3_gate], "ipg", "53f80600.usb");
> + clk_register_clkdev(clk[usboh3_gate], "ahb", "53f80600.usb");
> + clk_register_clkdev(clk[usboh3_per_gate], "per", "53f80400.usb");
> + clk_register_clkdev(clk[usboh3_gate], "ipg", "53f80400.usb");
> + clk_register_clkdev(clk[usboh3_gate], "ahb", "53f80400.usb");
> + clk_register_clkdev(clk[usboh3_per_gate], "per", "53f80200.usb");
> + clk_register_clkdev(clk[usboh3_gate], "ipg", "53f80200.usb");
> + clk_register_clkdev(clk[usboh3_gate], "ahb", "53f80200.usb");
> + clk_register_clkdev(clk[usboh3_per_gate], "per", "53f80000.usb");
> + clk_register_clkdev(clk[usboh3_gate], "ipg", "53f80000.usb");
> + clk_register_clkdev(clk[usboh3_gate], "ahb", "53f80000.usb");
> clk_register_clkdev(clk[usb_phy1_gate], "usb_phy1", "mxc-ehci.0");
> clk_register_clkdev(clk[esdhc1_ipg_gate], "ipg", "sdhci-esdhc-imx53.0");
> clk_register_clkdev(clk[dummy], "ahb", "sdhci-esdhc-imx53.0");
> --
> 1.7.10.4
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: imx: add imx5x usb controller clock lookups
2012-08-23 14:18 ` [PATCH] " Shawn Guo
@ 2012-08-23 14:41 ` Marc Kleine-Budde
0 siblings, 0 replies; 7+ messages in thread
From: Marc Kleine-Budde @ 2012-08-23 14:41 UTC (permalink / raw)
To: linux-arm-kernel
On 08/23/2012 04:18 PM, Shawn Guo wrote:
> On Thu, Aug 23, 2012 at 12:38:48PM +0200, Michael Grzeschik wrote:
>> Signed-off-by: Michael Grzeschik <m.grzeschik@pengutronix.de>
>> ---
>> arch/arm/mach-imx/clk-imx51-imx53.c | 36 +++++++++++++++++++++++------------
>> 1 file changed, 24 insertions(+), 12 deletions(-)
>>
>> diff --git a/arch/arm/mach-imx/clk-imx51-imx53.c b/arch/arm/mach-imx/clk-imx51-imx53.c
>> index 4bdcaa9..6ce49b1 100644
>> --- a/arch/arm/mach-imx/clk-imx51-imx53.c
>> +++ b/arch/arm/mach-imx/clk-imx51-imx53.c
>> @@ -258,18 +258,6 @@ static void __init mx5_clocks_common_init(unsigned long rate_ckil,
>> clk_register_clkdev(clk[pwm2_ipg_gate], "pwm", "mxc_pwm.1");
>> clk_register_clkdev(clk[i2c1_gate], NULL, "imx-i2c.0");
>> clk_register_clkdev(clk[i2c2_gate], NULL, "imx-i2c.1");
>> - clk_register_clkdev(clk[usboh3_per_gate], "per", "mxc-ehci.0");
>> - clk_register_clkdev(clk[usboh3_gate], "ipg", "mxc-ehci.0");
>> - clk_register_clkdev(clk[usboh3_gate], "ahb", "mxc-ehci.0");
>> - clk_register_clkdev(clk[usboh3_per_gate], "per", "mxc-ehci.1");
>> - clk_register_clkdev(clk[usboh3_gate], "ipg", "mxc-ehci.1");
>> - clk_register_clkdev(clk[usboh3_gate], "ahb", "mxc-ehci.1");
>> - clk_register_clkdev(clk[usboh3_per_gate], "per", "mxc-ehci.2");
>> - clk_register_clkdev(clk[usboh3_gate], "ipg", "mxc-ehci.2");
>> - clk_register_clkdev(clk[usboh3_gate], "ahb", "mxc-ehci.2");
>> - clk_register_clkdev(clk[usboh3_per_gate], "per", "fsl-usb2-udc");
>> - clk_register_clkdev(clk[usboh3_gate], "ipg", "fsl-usb2-udc");
>> - clk_register_clkdev(clk[usboh3_gate], "ahb", "fsl-usb2-udc");
>
> Removing those will break some in-tree users, right?
Yes, there was a v2 fixing this.
>> clk_register_clkdev(clk[nfc_gate], NULL, "mxc_nand");
>> clk_register_clkdev(clk[ssi1_ipg_gate], NULL, "imx-ssi.0");
>> clk_register_clkdev(clk[ssi2_ipg_gate], NULL, "imx-ssi.1");
>> @@ -351,6 +339,18 @@ int __init mx51_clocks_init(unsigned long rate_ckil, unsigned long rate_osc,
>> clk_register_clkdev(clk[ipu_di0_gate], "di0", "imx51-ipu");
>> clk_register_clkdev(clk[ipu_di1_gate], "di1", "imx51-ipu");
>> clk_register_clkdev(clk[ipu_gate], "hsp", "imx51-ipu");
>> + clk_register_clkdev(clk[usboh3_per_gate], "per", "73f80600.usb");
>> + clk_register_clkdev(clk[usboh3_gate], "ipg", "73f80600.usb");
>> + clk_register_clkdev(clk[usboh3_gate], "ahb", "73f80600.usb");
>> + clk_register_clkdev(clk[usboh3_per_gate], "per", "73f80400.usb");
>> + clk_register_clkdev(clk[usboh3_gate], "ipg", "73f80400.usb");
>> + clk_register_clkdev(clk[usboh3_gate], "ahb", "73f80400.usb");
>> + clk_register_clkdev(clk[usboh3_per_gate], "per", "73f80200.usb");
>> + clk_register_clkdev(clk[usboh3_gate], "ipg", "73f80200.usb");
>> + clk_register_clkdev(clk[usboh3_gate], "ahb", "73f80200.usb");
>> + clk_register_clkdev(clk[usboh3_per_gate], "per", "73f80000.usb");
>> + clk_register_clkdev(clk[usboh3_gate], "ipg", "73f80000.usb");
>> + clk_register_clkdev(clk[usboh3_gate], "ahb", "73f80000.usb");
>
> I haven't seen new driver requesting these clocks, so would postpone
> the changes until the driver actually requests them.
I'm going to post a driver ci13xxx_imx driver update in a few minutes.
Marc
--
Pengutronix e.K. | Marc Kleine-Budde |
Industrial Linux Solutions | Phone: +49-231-2826-924 |
Vertretung West/Dortmund | Fax: +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 262 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120823/3c70cfa8/attachment-0001.sig>
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-08-23 14:41 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-08-23 10:38 [PATCH] ARM: imx: add imx5x usb controller clock lookups Michael Grzeschik
2012-08-23 11:01 ` [PATCH v2] " Michael Grzeschik
2012-08-23 11:13 ` Fabio Estevam
2012-08-23 11:36 ` Michael Grzeschik
2012-08-23 11:54 ` Fabio Estevam
2012-08-23 14:18 ` [PATCH] " Shawn Guo
2012-08-23 14:41 ` Marc Kleine-Budde
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).