* [PATCH 1/1] ARM: mx51/53: correct clocks that take ipg_perclk as parent
@ 2011-11-04 9:34 Richard Zhao
2011-11-07 7:52 ` Sascha Hauer
0 siblings, 1 reply; 4+ messages in thread
From: Richard Zhao @ 2011-11-04 9:34 UTC (permalink / raw)
To: linux-arm-kernel
gpt_clk, pwm1_clk and pwm2_clk parent clock are all ipg_perclk.
Signed-off-by: Richard Zhao <richard.zhao@linaro.org>
---
arch/arm/mach-mx5/clock-mx51-mx53.c | 6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/mach-mx5/clock-mx51-mx53.c b/arch/arm/mach-mx5/clock-mx51-mx53.c
index 2aacf41..a3ca4ce 100644
--- a/arch/arm/mach-mx5/clock-mx51-mx53.c
+++ b/arch/arm/mach-mx5/clock-mx51-mx53.c
@@ -1278,12 +1278,12 @@ DEFINE_CLOCK(uart5_clk, 4, MXC_CCM_CCGR7, MXC_CCM_CCGRx_CG7_OFFSET,
DEFINE_CLOCK(gpt_ipg_clk, 0, MXC_CCM_CCGR2, MXC_CCM_CCGRx_CG10_OFFSET,
NULL, NULL, &ipg_clk, NULL);
DEFINE_CLOCK(gpt_clk, 0, MXC_CCM_CCGR2, MXC_CCM_CCGRx_CG9_OFFSET,
- NULL, NULL, &ipg_clk, &gpt_ipg_clk);
+ NULL, NULL, &ipg_perclk, &gpt_ipg_clk);
DEFINE_CLOCK(pwm1_clk, 0, MXC_CCM_CCGR2, MXC_CCM_CCGRx_CG6_OFFSET,
- NULL, NULL, &ipg_clk, NULL);
+ NULL, NULL, &ipg_perclk, NULL);
DEFINE_CLOCK(pwm2_clk, 0, MXC_CCM_CCGR2, MXC_CCM_CCGRx_CG8_OFFSET,
- NULL, NULL, &ipg_clk, NULL);
+ NULL, NULL, &ipg_perclk, NULL);
/* I2C */
DEFINE_CLOCK(i2c1_clk, 0, MXC_CCM_CCGR1, MXC_CCM_CCGRx_CG9_OFFSET,
--
1.7.5.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH 1/1] ARM: mx51/53: correct clocks that take ipg_perclk as parent
2011-11-04 9:34 [PATCH 1/1] ARM: mx51/53: correct clocks that take ipg_perclk as parent Richard Zhao
@ 2011-11-07 7:52 ` Sascha Hauer
2011-11-08 0:41 ` Richard Zhao
0 siblings, 1 reply; 4+ messages in thread
From: Sascha Hauer @ 2011-11-07 7:52 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Nov 04, 2011 at 05:34:23PM +0800, Richard Zhao wrote:
> gpt_clk, pwm1_clk and pwm2_clk parent clock are all ipg_perclk.
>
> Signed-off-by: Richard Zhao <richard.zhao@linaro.org>
> ---
> arch/arm/mach-mx5/clock-mx51-mx53.c | 6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/mach-mx5/clock-mx51-mx53.c b/arch/arm/mach-mx5/clock-mx51-mx53.c
> index 2aacf41..a3ca4ce 100644
> --- a/arch/arm/mach-mx5/clock-mx51-mx53.c
> +++ b/arch/arm/mach-mx5/clock-mx51-mx53.c
> @@ -1278,12 +1278,12 @@ DEFINE_CLOCK(uart5_clk, 4, MXC_CCM_CCGR7, MXC_CCM_CCGRx_CG7_OFFSET,
> DEFINE_CLOCK(gpt_ipg_clk, 0, MXC_CCM_CCGR2, MXC_CCM_CCGRx_CG10_OFFSET,
> NULL, NULL, &ipg_clk, NULL);
> DEFINE_CLOCK(gpt_clk, 0, MXC_CCM_CCGR2, MXC_CCM_CCGRx_CG9_OFFSET,
> - NULL, NULL, &ipg_clk, &gpt_ipg_clk);
> + NULL, NULL, &ipg_perclk, &gpt_ipg_clk);
No. The gpt has several parents and the one we are using is the ipg
clock. Try a sleep 10 with your patch applied.
Sascha
>
> DEFINE_CLOCK(pwm1_clk, 0, MXC_CCM_CCGR2, MXC_CCM_CCGRx_CG6_OFFSET,
> - NULL, NULL, &ipg_clk, NULL);
> + NULL, NULL, &ipg_perclk, NULL);
> DEFINE_CLOCK(pwm2_clk, 0, MXC_CCM_CCGR2, MXC_CCM_CCGRx_CG8_OFFSET,
> - NULL, NULL, &ipg_clk, NULL);
> + NULL, NULL, &ipg_perclk, NULL);
>
> /* I2C */
> DEFINE_CLOCK(i2c1_clk, 0, MXC_CCM_CCGR1, MXC_CCM_CCGRx_CG9_OFFSET,
> --
> 1.7.5.4
>
>
>
--
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] 4+ messages in thread
* [PATCH 1/1] ARM: mx51/53: correct clocks that take ipg_perclk as parent
2011-11-07 7:52 ` Sascha Hauer
@ 2011-11-08 0:41 ` Richard Zhao
2011-11-08 13:08 ` Sascha Hauer
0 siblings, 1 reply; 4+ messages in thread
From: Richard Zhao @ 2011-11-08 0:41 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Nov 07, 2011 at 08:52:54AM +0100, Sascha Hauer wrote:
> On Fri, Nov 04, 2011 at 05:34:23PM +0800, Richard Zhao wrote:
> > gpt_clk, pwm1_clk and pwm2_clk parent clock are all ipg_perclk.
> >
> > Signed-off-by: Richard Zhao <richard.zhao@linaro.org>
> > ---
> > arch/arm/mach-mx5/clock-mx51-mx53.c | 6 +++---
> > 1 files changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/arch/arm/mach-mx5/clock-mx51-mx53.c b/arch/arm/mach-mx5/clock-mx51-mx53.c
> > index 2aacf41..a3ca4ce 100644
> > --- a/arch/arm/mach-mx5/clock-mx51-mx53.c
> > +++ b/arch/arm/mach-mx5/clock-mx51-mx53.c
> > @@ -1278,12 +1278,12 @@ DEFINE_CLOCK(uart5_clk, 4, MXC_CCM_CCGR7, MXC_CCM_CCGRx_CG7_OFFSET,
> > DEFINE_CLOCK(gpt_ipg_clk, 0, MXC_CCM_CCGR2, MXC_CCM_CCGRx_CG10_OFFSET,
> > NULL, NULL, &ipg_clk, NULL);
> > DEFINE_CLOCK(gpt_clk, 0, MXC_CCM_CCGR2, MXC_CCM_CCGRx_CG9_OFFSET,
> > - NULL, NULL, &ipg_clk, &gpt_ipg_clk);
> > + NULL, NULL, &ipg_perclk, &gpt_ipg_clk);
>
> No. The gpt has several parents and the one we are using is the ipg
> clock. Try a sleep 10 with your patch applied.
You are right. Sounds like gpt_clk can not be defined here. Is it better to pass
three clocks (ipg, ipg_per, ckil) to gpt driver and let gpt decide what to use?
Anyway, it's beyond this patch. I'll keep this line as it was.
Richard
>
> Sascha
>
> >
> > DEFINE_CLOCK(pwm1_clk, 0, MXC_CCM_CCGR2, MXC_CCM_CCGRx_CG6_OFFSET,
> > - NULL, NULL, &ipg_clk, NULL);
> > + NULL, NULL, &ipg_perclk, NULL);
> > DEFINE_CLOCK(pwm2_clk, 0, MXC_CCM_CCGR2, MXC_CCM_CCGRx_CG8_OFFSET,
> > - NULL, NULL, &ipg_clk, NULL);
> > + NULL, NULL, &ipg_perclk, NULL);
> >
> > /* I2C */
> > DEFINE_CLOCK(i2c1_clk, 0, MXC_CCM_CCGR1, MXC_CCM_CCGRx_CG9_OFFSET,
> > --
> > 1.7.5.4
> >
> >
> >
>
> --
> 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 |
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 1/1] ARM: mx51/53: correct clocks that take ipg_perclk as parent
2011-11-08 0:41 ` Richard Zhao
@ 2011-11-08 13:08 ` Sascha Hauer
0 siblings, 0 replies; 4+ messages in thread
From: Sascha Hauer @ 2011-11-08 13:08 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Nov 08, 2011 at 08:41:31AM +0800, Richard Zhao wrote:
> On Mon, Nov 07, 2011 at 08:52:54AM +0100, Sascha Hauer wrote:
> > On Fri, Nov 04, 2011 at 05:34:23PM +0800, Richard Zhao wrote:
> > > gpt_clk, pwm1_clk and pwm2_clk parent clock are all ipg_perclk.
> > >
> > > Signed-off-by: Richard Zhao <richard.zhao@linaro.org>
> > > ---
> > > arch/arm/mach-mx5/clock-mx51-mx53.c | 6 +++---
> > > 1 files changed, 3 insertions(+), 3 deletions(-)
> > >
> > > diff --git a/arch/arm/mach-mx5/clock-mx51-mx53.c b/arch/arm/mach-mx5/clock-mx51-mx53.c
> > > index 2aacf41..a3ca4ce 100644
> > > --- a/arch/arm/mach-mx5/clock-mx51-mx53.c
> > > +++ b/arch/arm/mach-mx5/clock-mx51-mx53.c
> > > @@ -1278,12 +1278,12 @@ DEFINE_CLOCK(uart5_clk, 4, MXC_CCM_CCGR7, MXC_CCM_CCGRx_CG7_OFFSET,
> > > DEFINE_CLOCK(gpt_ipg_clk, 0, MXC_CCM_CCGR2, MXC_CCM_CCGRx_CG10_OFFSET,
> > > NULL, NULL, &ipg_clk, NULL);
> > > DEFINE_CLOCK(gpt_clk, 0, MXC_CCM_CCGR2, MXC_CCM_CCGRx_CG9_OFFSET,
> > > - NULL, NULL, &ipg_clk, &gpt_ipg_clk);
> > > + NULL, NULL, &ipg_perclk, &gpt_ipg_clk);
> >
> > No. The gpt has several parents and the one we are using is the ipg
> > clock. Try a sleep 10 with your patch applied.
> You are right. Sounds like gpt_clk can not be defined here. Is it better to pass
> three clocks (ipg, ipg_per, ckil) to gpt driver and let gpt decide what to use?
> Anyway, it's beyond this patch. I'll keep this line as it was.
Yes, this would be the correct solution.
Sascha
--
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] 4+ messages in thread
end of thread, other threads:[~2011-11-08 13:08 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-11-04 9:34 [PATCH 1/1] ARM: mx51/53: correct clocks that take ipg_perclk as parent Richard Zhao
2011-11-07 7:52 ` Sascha Hauer
2011-11-08 0:41 ` Richard Zhao
2011-11-08 13:08 ` Sascha Hauer
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).