[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110315123539.GD14548@game.jcrosoft.org>
Date:	Tue, 15 Mar 2011 13:35:39 +0100
From:	Jean-Christophe PLAGNIOL-VILLARD <plagnioj@...osoft.com>
To:	Jamie Iles <jamie@...ieiles.com>
Cc:	netdev@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
	nicolas.ferre@...el.com
Subject: Re: [PATCHv2 1/9] at91: provide macb clks with "pclk" and "hclk"
 name
On 10:14 Tue 15 Mar     , Jamie Iles wrote:
> The macb driver expects clocks with the names "pclk" and "hclk".  We
> currently provide "macb_clk" but to fit in line with other
> architectures (namely AVR32), provide "pclk" and a fake "hclk".
> 
> Cc: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@...osoft.com>
> Signed-off-by: Jamie Iles <jamie@...ieiles.com>
> ---
>  arch/arm/mach-at91/at572d940hf.c |    8 +++++++-
>  arch/arm/mach-at91/at91cap9.c    |    8 +++++++-
>  arch/arm/mach-at91/at91sam9260.c |    8 +++++++-
>  arch/arm/mach-at91/at91sam9263.c |    8 +++++++-
>  arch/arm/mach-at91/at91sam9g45.c |    8 +++++++-
>  5 files changed, 35 insertions(+), 5 deletions(-)
> 
> diff --git a/arch/arm/mach-at91/at572d940hf.c b/arch/arm/mach-at91/at572d940hf.c
> index a6b9c68..9b3a37e 100644
> --- a/arch/arm/mach-at91/at572d940hf.c
> +++ b/arch/arm/mach-at91/at572d940hf.c
> @@ -71,10 +71,15 @@ static struct clk pioC_clk = {
>  	.type		= CLK_TYPE_PERIPHERAL,
>  };
>  static struct clk macb_clk = {
> -	.name		= "macb_clk",
> +	.name		= "pclk",
>  	.pmc_mask	= 1 << AT572D940HF_ID_EMAC,
>  	.type		= CLK_TYPE_PERIPHERAL,
>  };
> +static struct clk macb_hclk = {
> +	.name		= "hclk",
> +	.pmc_mask	= 0,
> +	.type		= CLK_TYPE_PERIPHERAL,
> +};
for the fake clock you must specify the parent as macb_clk
take a look on the tcb1_clk for the 9g45
Best Regards,
J.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists
 
