Sriramakrishnan <s...@ti.com> writes:

> The emac driver uses generic name for the module and phy
> clocks. Updated the omap3xxx_clks table to match the names
> used by the Davinci emac driver.
>
> Signed-off-by: Sriramakrishnan <s...@ti.com>
> ---
>  arch/arm/mach-omap2/clock3xxx_data.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/clock3xxx_data.c 
> b/arch/arm/mach-omap2/clock3xxx_data.c
> index d5153b6..989da2e 100644
> --- a/arch/arm/mach-omap2/clock3xxx_data.c
> +++ b/arch/arm/mach-omap2/clock3xxx_data.c
> @@ -3472,8 +3472,8 @@ static struct omap_clk omap3xxx_clks[] = {
>       CLK(NULL,       "ipss_ick",     &ipss_ick,      CK_AM35XX),
>       CLK(NULL,       "rmii_ck",      &rmii_ck,       CK_AM35XX),
>       CLK(NULL,       "pclk_ck",      &pclk_ck,       CK_AM35XX),
> -     CLK("davinci_emac",     "ick",          &emac_ick,      CK_AM35XX),
> -     CLK("davinci_emac",     "fck",          &emac_fck,      CK_AM35XX),
> +     CLK("davinci_emac",     "emac_clk",     &emac_ick,      CK_AM35XX),
> +     CLK("davinci_emac",     "phy_clk",      &emac_fck,      CK_AM35XX),

As I mentioned in the review of the equivalent davinci changes,
I'm not crazy abou the _clk suffixes on the con_id.  I think
these should be called "main" (or "emac") and "phy".

Kevin
_______________________________________________
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to