diff mbox

[3/4] clk: ux500: Provide an alias for the SMSC911x Ethernet chip

Message ID 1355937587-31730-3-git-send-email-lee.jones@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Lee Jones Dec. 19, 2012, 5:19 p.m. UTC
In the case of some of the ux500 platforms, an Ethernet chip is
placed on an extended bus which is traditionally used as a NAND
flash chip placeholder. The p3_pclk0 clock is used to control it,
so we are required to provide and easy way to access it from the
SMSC911x driver. We do this using an alias provided by this patch.

Cc: Mike Turquette <mturquette@linaro.org>
Cc: Ulf Hansson <ulf.hansson@linaro.org>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/clk/ux500/u8500_clk.c |    1 +
 1 file changed, 1 insertion(+)

Comments

Lee Jones Jan. 9, 2013, 8:56 a.m. UTC | #1
On Wed, 19 Dec 2012, Lee Jones wrote:

> In the case of some of the ux500 platforms, an Ethernet chip is
> placed on an extended bus which is traditionally used as a NAND
> flash chip placeholder. The p3_pclk0 clock is used to control it,
> so we are required to provide and easy way to access it from the
> SMSC911x driver. We do this using an alias provided by this patch.
> 
> Cc: Mike Turquette <mturquette@linaro.org>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
> ---
>  drivers/clk/ux500/u8500_clk.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/clk/ux500/u8500_clk.c b/drivers/clk/ux500/u8500_clk.c
> index a601802..9d9add1 100644
> --- a/drivers/clk/ux500/u8500_clk.c
> +++ b/drivers/clk/ux500/u8500_clk.c
> @@ -325,6 +325,7 @@ void u8500_clk_init(void)
>  	clk = clk_reg_prcc_pclk("p3_pclk0", "per3clk", U8500_CLKRST3_BASE,
>  				BIT(0), 0);
>  	clk_register_clkdev(clk, "fsmc", NULL);
> +	clk_register_clkdev(clk, NULL, "smsc911x");
>  
>  	clk = clk_reg_prcc_pclk("p3_pclk1", "per3clk", U8500_CLKRST3_BASE,
>  				BIT(1), 0);
> -- 
> 1.7.9.5
> 

I still need a maintiner Ack for this before I can push it.

Mike?
Ulf Hansson Jan. 9, 2013, 4:26 p.m. UTC | #2
On 9 January 2013 09:56, Lee Jones <lee.jones@linaro.org> wrote:
> On Wed, 19 Dec 2012, Lee Jones wrote:
>
>> In the case of some of the ux500 platforms, an Ethernet chip is
>> placed on an extended bus which is traditionally used as a NAND
>> flash chip placeholder. The p3_pclk0 clock is used to control it,
>> so we are required to provide and easy way to access it from the
>> SMSC911x driver. We do this using an alias provided by this patch.
>>
>> Cc: Mike Turquette <mturquette@linaro.org>
>> Cc: Ulf Hansson <ulf.hansson@linaro.org>
>> Signed-off-by: Lee Jones <lee.jones@linaro.org>
>> ---
>>  drivers/clk/ux500/u8500_clk.c |    1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/clk/ux500/u8500_clk.c b/drivers/clk/ux500/u8500_clk.c
>> index a601802..9d9add1 100644
>> --- a/drivers/clk/ux500/u8500_clk.c
>> +++ b/drivers/clk/ux500/u8500_clk.c
>> @@ -325,6 +325,7 @@ void u8500_clk_init(void)
>>       clk = clk_reg_prcc_pclk("p3_pclk0", "per3clk", U8500_CLKRST3_BASE,
>>                               BIT(0), 0);
>>       clk_register_clkdev(clk, "fsmc", NULL);
>> +     clk_register_clkdev(clk, NULL, "smsc911x");
>>
>>       clk = clk_reg_prcc_pclk("p3_pclk1", "per3clk", U8500_CLKRST3_BASE,
>>                               BIT(1), 0);
>> --
>> 1.7.9.5
>>
>
> I still need a maintiner Ack for this before I can push it.
>
> Mike?
>
> --
> Lee Jones
> Linaro ST-Ericsson Landing Team Lead
> Linaro.org ? Open source software for ARM SoCs
> Follow Linaro: Facebook | Twitter | Blog

Acked-by: Ulf Hansson <ulf.hansson@linaro.org>
Mike Turquette Jan. 15, 2013, 11:53 p.m. UTC | #3
Quoting Lee Jones (2013-01-09 00:56:36)
> On Wed, 19 Dec 2012, Lee Jones wrote:
> 
> > In the case of some of the ux500 platforms, an Ethernet chip is
> > placed on an extended bus which is traditionally used as a NAND
> > flash chip placeholder. The p3_pclk0 clock is used to control it,
> > so we are required to provide and easy way to access it from the
> > SMSC911x driver. We do this using an alias provided by this patch.
> > 
> > Cc: Mike Turquette <mturquette@linaro.org>
> > Cc: Ulf Hansson <ulf.hansson@linaro.org>
> > Signed-off-by: Lee Jones <lee.jones@linaro.org>
> > ---
> >  drivers/clk/ux500/u8500_clk.c |    1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/clk/ux500/u8500_clk.c b/drivers/clk/ux500/u8500_clk.c
> > index a601802..9d9add1 100644
> > --- a/drivers/clk/ux500/u8500_clk.c
> > +++ b/drivers/clk/ux500/u8500_clk.c
> > @@ -325,6 +325,7 @@ void u8500_clk_init(void)
> >       clk = clk_reg_prcc_pclk("p3_pclk0", "per3clk", U8500_CLKRST3_BASE,
> >                               BIT(0), 0);
> >       clk_register_clkdev(clk, "fsmc", NULL);
> > +     clk_register_clkdev(clk, NULL, "smsc911x");
> >  
> >       clk = clk_reg_prcc_pclk("p3_pclk1", "per3clk", U8500_CLKRST3_BASE,
> >                               BIT(1), 0);
> > -- 
> > 1.7.9.5
> > 
> 
> I still need a maintiner Ack for this before I can push it.
> 
> Mike?
> 

Acked-by: Mike Turquette <mturquette@linaro.org>

> -- 
> Lee Jones
> Linaro ST-Ericsson Landing Team Lead
> Linaro.org ? Open source software for ARM SoCs
> Follow Linaro: Facebook | Twitter | Blog
diff mbox

Patch

diff --git a/drivers/clk/ux500/u8500_clk.c b/drivers/clk/ux500/u8500_clk.c
index a601802..9d9add1 100644
--- a/drivers/clk/ux500/u8500_clk.c
+++ b/drivers/clk/ux500/u8500_clk.c
@@ -325,6 +325,7 @@  void u8500_clk_init(void)
 	clk = clk_reg_prcc_pclk("p3_pclk0", "per3clk", U8500_CLKRST3_BASE,
 				BIT(0), 0);
 	clk_register_clkdev(clk, "fsmc", NULL);
+	clk_register_clkdev(clk, NULL, "smsc911x");
 
 	clk = clk_reg_prcc_pclk("p3_pclk1", "per3clk", U8500_CLKRST3_BASE,
 				BIT(1), 0);