> -----Original Message-----
> From: [email protected] [mailto:linux-omap-
> [email protected]] On Behalf Of Anand Gadiyar
> Sent: Monday, January 10, 2011 7:07 PM
> To: Santosh Shilimkar
> Cc: [email protected]; [email protected]; linux-arm-
> [email protected]; Paul Walmsley
> Subject: Re: [PATCH] omap3: clocks: Fix build error 'CK_3430ES2'
> undeclared here
>
> On 1/10/2011 3:46 AM, Santosh Shilimkar wrote:
> > At latest mainline commit 0c21e3aaf6a,  omap2plus build is broken.
> This
> > patch is trivial fix for the missed usb clock node for
> CK_3430ES2PLUS
> > flag update.
> >
> >  CHK     include/generated/compile.h
> >   CC      arch/arm/mach-omap2/clock3xxx_data.o
> > arch/arm/mach-omap2/clock3xxx_data.c:3289: error: 'CK_3430ES2'
> undeclared
> > here (not in a function)
> > make[1]: *** [arch/arm/mach-omap2/clock3xxx_data.o] Error 1
> > make: *** [arch/arm/mach-omap2] Error 2
> >
> > Signed-off-by: Santosh Shilimkar <[email protected]>
> > Cc: Paul Walmsley <[email protected]>
> > ---
> >  arch/arm/mach-omap2/clock3xxx_data.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/arch/arm/mach-omap2/clock3xxx_data.c b/arch/arm/mach-
> omap2/clock3xxx_data.c
> > index d3ab1c9..c54cc33 100644
> > --- a/arch/arm/mach-omap2/clock3xxx_data.c
> > +++ b/arch/arm/mach-omap2/clock3xxx_data.c
> > @@ -3286,7 +3286,7 @@ static struct omap_clk omap3xxx_clks[] = {
> >     CLK(NULL,       "cpefuse_fck",  &cpefuse_fck,   CK_3430ES2PLUS
> | CK_AM35XX | CK_36XX),
> >     CLK(NULL,       "ts_fck",       &ts_fck,        CK_3430ES2PLUS |
CK_AM35XX
> | CK_36XX),
> >     CLK(NULL,       "usbtll_fck",   &usbtll_fck,    CK_3430ES2PLUS
> | CK_AM35XX | CK_36XX),
> > -   CLK("ehci-omap.0",      "usbtll_fck",   &usbtll_fck,
>       CK_3430ES2 | CK_AM35XX),
> > +   CLK("ehci-omap.0",      "usbtll_fck",   &usbtll_fck,
>       CK_3430ES2PLUS | CK_AM35XX),
>
> Was about to post something similar.
>
> Can you update this line to add CK_36XX as well. Without this, we
> won't be able to use
> the USBHOST on 36xx/37xx platforms.
>
> With this, feel free to add my
> Acked-by: Anand Gadiyar <[email protected]>
>
ok. I can add CK_36XX and repost it with your ack

> - Anand
>
> >     CLK("omap-mcbsp.1",     "prcm_fck",     &core_96m_fck,
>       CK_3XXX),
> >     CLK("omap-mcbsp.5",     "prcm_fck",     &core_96m_fck,
>       CK_3XXX),
> >     CLK(NULL,       "core_96m_fck", &core_96m_fck,  CK_3XXX),
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-
> omap" in
> the body of a message to [email protected]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to