[PATCH] clk: SPEAr600: Fix ethernet clock name for DT based probing

[PATCH] clk: SPEAr600: Fix ethernet clock name for DT based probing

[Stefan Roese]

Signed-off-by: Stefan Roese <sr@denx.de>
Cc: Viresh Kumar <viresh.kumar@st.com>
Cc: "Mike Turquette" <mturquette@ti.com>
---
 drivers/clk/spear/spear6xx_clock.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clk/spear/spear6xx_clock.c b/drivers/clk/spear/spear6xx_clock.c
index f9a20b3..bf33c1d 100644
--- a/drivers/clk/spear/spear6xx_clock.c
+++ b/drivers/clk/spear/spear6xx_clock.c
@@ -298,7 +298,7 @@ void __init spear6xx_clk_init(void)
 
 	clk = clk_register_gate(NULL, "gmac_clk", "ahb_clk", 0, PERIP1_CLK_ENB,
 			GMAC_CLK_ENB, 0, &_lock);
-	clk_register_clkdev(clk, NULL, "gmac");
+	clk_register_clkdev(clk, NULL, "e0800000.ethernet");
 
 	clk = clk_register_gate(NULL, "i2c_clk", "ahb_clk", 0, PERIP1_CLK_ENB,
 			I2C_CLK_ENB, 0, &_lock);
-- 
1.7.10.1

[PATCH] clk: SPEAr600: Fix ethernet clock name for DT based probing

[Viresh Kumar]

On 5/11/2012 2:11 PM, Stefan Roese wrote:
> diff --git a/drivers/clk/spear/spear6xx_clock.c b/drivers/clk/spear/spear6xx_clock.c
> index f9a20b3..bf33c1d 100644
> --- a/drivers/clk/spear/spear6xx_clock.c
> +++ b/drivers/clk/spear/spear6xx_clock.c
> @@ -298,7 +298,7 @@ void __init spear6xx_clk_init(void)
>  
>  	clk = clk_register_gate(NULL, "gmac_clk", "ahb_clk", 0, PERIP1_CLK_ENB,
>  			GMAC_CLK_ENB, 0, &_lock);
> -	clk_register_clkdev(clk, NULL, "gmac");
> +	clk_register_clkdev(clk, NULL, "e0800000.ethernet");
>  
>  	clk = clk_register_gate(NULL, "i2c_clk", "ahb_clk", 0, PERIP1_CLK_ENB,
>  			I2C_CLK_ENB, 0, &_lock);

Acked-by: Viresh Kumar <viresh.kumar@st.com>

-- 
viresh

[PATCH] clk: SPEAr600: Fix ethernet clock name for DT based probing

[Turquette, Mike]

On Sun, May 13, 2012 at 8:34 PM, Viresh Kumar <viresh.kumar@st.com> wrote:
> On 5/11/2012 2:11 PM, Stefan Roese wrote:
>> diff --git a/drivers/clk/spear/spear6xx_clock.c b/drivers/clk/spear/spear6xx_clock.c
>> index f9a20b3..bf33c1d 100644
>> --- a/drivers/clk/spear/spear6xx_clock.c
>> +++ b/drivers/clk/spear/spear6xx_clock.c
>> @@ -298,7 +298,7 @@ void __init spear6xx_clk_init(void)
>>
>> ? ? ? clk = clk_register_gate(NULL, "gmac_clk", "ahb_clk", 0, PERIP1_CLK_ENB,
>> ? ? ? ? ? ? ? ? ? ? ? GMAC_CLK_ENB, 0, &_lock);
>> - ? ? clk_register_clkdev(clk, NULL, "gmac");
>> + ? ? clk_register_clkdev(clk, NULL, "e0800000.ethernet");
>>
>> ? ? ? clk = clk_register_gate(NULL, "i2c_clk", "ahb_clk", 0, PERIP1_CLK_ENB,
>> ? ? ? ? ? ? ? ? ? ? ? I2C_CLK_ENB, 0, &_lock);
>
> Acked-by: Viresh Kumar <viresh.kumar@st.com>

Do you want me to take this?  Spear didn't originally go through me
but I can take it if that is the intent.

Regards,
Mike

>
> --
> viresh

[PATCH] clk: SPEAr600: Fix ethernet clock name for DT based probing

[viresh kumar]

On May 16, 2012 10:32 AM, "Turquette, Mike" <mturquette@ti.com> wrote:
>
> On Sun, May 13, 2012 at 8:34 PM, Viresh Kumar <viresh.kumar@st.com> wrote:
> > On 5/11/2012 2:11 PM, Stefan Roese wrote:
> >> diff --git a/drivers/clk/spear/spear6xx_clock.c
b/drivers/clk/spear/spear6xx_clock.c
> >> index f9a20b3..bf33c1d 100644
> >> --- a/drivers/clk/spear/spear6xx_clock.c
> >> +++ b/drivers/clk/spear/spear6xx_clock.c
> >> @@ -298,7 +298,7 @@ void __init spear6xx_clk_init(void)
> >>
> >>       clk = clk_register_gate(NULL, "gmac_clk", "ahb_clk", 0,
PERIP1_CLK_ENB,
> >>                       GMAC_CLK_ENB, 0, &_lock);
> >> -     clk_register_clkdev(clk, NULL, "gmac");
> >> +     clk_register_clkdev(clk, NULL, "e0800000.ethernet");
> >>
> >>       clk = clk_register_gate(NULL, "i2c_clk", "ahb_clk", 0,
PERIP1_CLK_ENB,
> >>                       I2C_CLK_ENB, 0, &_lock);
> >
> > Acked-by: Viresh Kumar <viresh.kumar@st.com>
>
> Do you want me to take this?  Spear didn't originally go through me
> but I can take it if that is the intent.
>

Sure. Go ahead.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120516/178decb8/attachment-0001.html>

[PATCH] clk: SPEAr600: Fix ethernet clock name for DT based probing

[Turquette, Mike]

On Tue, May 15, 2012 at 10:56 PM, viresh kumar <viresh.linux@gmail.com> wrote:
>
> On May 16, 2012 10:32 AM, "Turquette, Mike" <mturquette@ti.com> wrote:
>>
>> On Sun, May 13, 2012 at 8:34 PM, Viresh Kumar <viresh.kumar@st.com> wrote:
>> > On 5/11/2012 2:11 PM, Stefan Roese wrote:
>> >> diff --git a/drivers/clk/spear/spear6xx_clock.c
>> >> b/drivers/clk/spear/spear6xx_clock.c
>> >> index f9a20b3..bf33c1d 100644
>> >> --- a/drivers/clk/spear/spear6xx_clock.c
>> >> +++ b/drivers/clk/spear/spear6xx_clock.c
>> >> @@ -298,7 +298,7 @@ void __init spear6xx_clk_init(void)
>> >>
>> >> ? ? ? clk = clk_register_gate(NULL, "gmac_clk", "ahb_clk", 0,
>> >> PERIP1_CLK_ENB,
>> >> ? ? ? ? ? ? ? ? ? ? ? GMAC_CLK_ENB, 0, &_lock);
>> >> - ? ? clk_register_clkdev(clk, NULL, "gmac");
>> >> + ? ? clk_register_clkdev(clk, NULL, "e0800000.ethernet");
>> >>
>> >> ? ? ? clk = clk_register_gate(NULL, "i2c_clk", "ahb_clk", 0,
>> >> PERIP1_CLK_ENB,
>> >> ? ? ? ? ? ? ? ? ? ? ? I2C_CLK_ENB, 0, &_lock);
>> >
>> > Acked-by: Viresh Kumar <viresh.kumar@st.com>
>>
>> Do you want me to take this? ?Spear didn't originally go through me
>> but I can take it if that is the intent.
>>
>
> Sure. Go ahead.

Oops.  I'm working too late.  Wasn't thinking.

I don't have spear in my clk-next branch (since clk-next is stable and
the spear clock stuff went straight to arm-soc).  Instead of having me
make a new branch based on arm-soc's latest can you just send this
Arnd/Olof?

Thanks,
Mike

[PATCH] clk: SPEAr600: Fix ethernet clock name for DT based probing

[Stefan Roese]

On Wednesday 16 May 2012 07:56:49 viresh kumar wrote:
> > > Acked-by: Viresh Kumar <viresh.kumar@st.com>
> > 
> > Do you want me to take this?  Spear didn't originally go through me
> > but I can take it if that is the intent.
> 
> Sure. Go ahead.

Ping.

Thanks,
Stefan

[PATCH] clk: SPEAr600: Fix ethernet clock name for DT based probing

[Stefan Roese]

On Wednesday 06 June 2012 08:46:07 Stefan Roese wrote:
> On Wednesday 16 May 2012 07:56:49 viresh kumar wrote:
> > > > Acked-by: Viresh Kumar <viresh.kumar@st.com>
> > > 
> > > Do you want me to take this?  Spear didn't originally go through me
> > > but I can take it if that is the intent.
> > 
> > Sure. Go ahead.
> 
> Ping.

Ping again (this is a real show-stopper for SPEAr600 in v3.5!).

Thanks,
Stefan

[PATCH] clk: SPEAr600: Fix ethernet clock name for DT based probing

[Mike Turquette]

On 20120618-11:09, Stefan Roese wrote:
> On Wednesday 06 June 2012 08:46:07 Stefan Roese wrote:
> > On Wednesday 16 May 2012 07:56:49 viresh kumar wrote:
> > > > > Acked-by: Viresh Kumar <viresh.kumar@st.com>
> > > > 
> > > > Do you want me to take this?  Spear didn't originally go through me
> > > > but I can take it if that is the intent.
> > > 
> > > Sure. Go ahead.
> > 
> > Ping.
> 
> Ping again (this is a real show-stopper for SPEAr600 in v3.5!).
> 

Hi Stefan,

I've taken this into my clk-next branch already.  Sorry if I did not
communicate that well enough.  I will request for clk-next to be
included into linux-next later this week.  Hopefully this patch will
start showing up in linux-next in a matter of days.

Let me know if there is anything else I can do.

Thanks,
Mike

> Thanks,
> Stefan

[PATCH] clk: SPEAr600: Fix ethernet clock name for DT based probing

[Stefan Roese]

Hi Mike,

On Monday 18 June 2012 19:20:42 Mike Turquette wrote:
> On 20120618-11:09, Stefan Roese wrote:
> > On Wednesday 06 June 2012 08:46:07 Stefan Roese wrote:
> > > On Wednesday 16 May 2012 07:56:49 viresh kumar wrote:
> > > > > > Acked-by: Viresh Kumar <viresh.kumar@st.com>
> > > > > 
> > > > > Do you want me to take this?  Spear didn't originally go through me
> > > > > but I can take it if that is the intent.
> > > > 
> > > > Sure. Go ahead.
> > > 
> > > Ping.
> > 
> > Ping again (this is a real show-stopper for SPEAr600 in v3.5!).
> 
> Hi Stefan,
> 
> I've taken this into my clk-next branch already.  Sorry if I did not
> communicate that well enough.  I will request for clk-next to be
> included into linux-next later this week.  Hopefully this patch will
> start showing up in linux-next in a matter of days.

Great. Thanks,
Stefan

[PATCH] clk: SPEAr600: Fix ethernet clock name for DT based probing

[Arnd Bergmann]

On Monday 18 June 2012, Mike Turquette wrote:
> > Ping again (this is a real show-stopper for SPEAr600 in v3.5!).
> > 
> I've taken this into my clk-next branch already.  Sorry if I did not
> communicate that well enough.  I will request for clk-next to be
> included into linux-next later this week.  Hopefully this patch will
> start showing up in linux-next in a matter of days.
> 
> Let me know if there is anything else I can do.

It sounds to me that Stefan needs this in 3.5, while the clk-next
patches are destined for 3.6. Is that correct? If so, do you also
have a clk-fixes branch that is targetted at 3.5 regressions?

	Arnd

[PATCH] clk: SPEAr600: Fix ethernet clock name for DT based probing

[Stefan Roese]

On Tuesday 19 June 2012 13:18:42 Arnd Bergmann wrote:
> On Monday 18 June 2012, Mike Turquette wrote:
> > > Ping again (this is a real show-stopper for SPEAr600 in v3.5!).
> > 
> > I've taken this into my clk-next branch already.  Sorry if I did not
> > communicate that well enough.  I will request for clk-next to be
> > included into linux-next later this week.  Hopefully this patch will
> > start showing up in linux-next in a matter of days.
> > 
> > Let me know if there is anything else I can do.
> 
> It sounds to me that Stefan needs this in 3.5,

Yes. Its definitely a fix for 3.5.

> while the clk-next
> patches are destined for 3.6. Is that correct? If so, do you also
> have a clk-fixes branch that is targetted at 3.5 regressions?

Thanks,
Stefan