> -----Original Message----- > From: linux-omap-owner@xxxxxxxxxxxxxxx > [mailto:linux-omap-owner@xxxxxxxxxxxxxxx] On Behalf Of Tony Lindgren > Sent: Wednesday, August 11, 2010 2:50 PM > To: Igor Grinberg > Cc: Stanley.Miao; linux-omap@xxxxxxxxxxxxxxx > Subject: Re: [PATCH] OMAP2: Fix a cpu type check problem. > > * Igor Grinberg <grinberg@xxxxxxxxxxxxxx> [100810 17:25]: > > On 08/10/10 15:36, Stanley.Miao wrote: > > > cpu_is_omap3517() and cpu_is_omap3505() are the subgroups > of cpu_is_omap34xx(), > > > so we should check cpu_is_omap3517() and > cpu_is_omap3505() first, then check > > > cpu_is_omap34xx(). > > > > > > Signed-off-by: Stanley.Miao <stanley.miao@xxxxxxxxxxxxx> > > > > > > > Tested-by: Igor Grinberg <grinberg@xxxxxxxxxxxxxx> > > > > I've just ran into this yesterday evening. > > Having a patch for this on the next day made me :) > > Tested on AM3517. > > Can you please describe what breaks so we can merge this as a fix? > cpu_is_34xx() will be true for all OMAP3 based devices including AM3517. So, if we want to perform operations specific to AM3517, the check for AM3517 should be done first else you match for 34xx would return true; and you wouldn't go far enough to check for am3517. ~sanjeev > Regards, > > Tony > > > > --- > > > arch/arm/mach-omap2/clock3xxx_data.c | 14 +++++++------- > > > 1 files changed, 7 insertions(+), 7 deletions(-) > > > > > > diff --git a/arch/arm/mach-omap2/clock3xxx_data.c > b/arch/arm/mach-omap2/clock3xxx_data.c > > > index 138646d..dfdce2d 100644 > > > --- a/arch/arm/mach-omap2/clock3xxx_data.c > > > +++ b/arch/arm/mach-omap2/clock3xxx_data.c > > > @@ -3417,7 +3417,13 @@ int __init omap3xxx_clk_init(void) > > > struct omap_clk *c; > > > u32 cpu_clkflg = CK_3XXX; > > > > > > - if (cpu_is_omap34xx()) { > > > + if (cpu_is_omap3517()) { > > > + cpu_mask = RATE_IN_3XXX | RATE_IN_3430ES2PLUS; > > > + cpu_clkflg |= CK_3517; > > > + } else if (cpu_is_omap3505()) { > > > + cpu_mask = RATE_IN_3XXX | RATE_IN_3430ES2PLUS; > > > + cpu_clkflg |= CK_3505; > > > + } else if (cpu_is_omap34xx()) { > > > cpu_mask = RATE_IN_3XXX; > > > cpu_clkflg |= CK_343X; > > > > > > @@ -3432,12 +3438,6 @@ int __init omap3xxx_clk_init(void) > > > cpu_mask |= RATE_IN_3430ES2PLUS; > > > cpu_clkflg |= CK_3430ES2; > > > } > > > - } else if (cpu_is_omap3517()) { > > > - cpu_mask = RATE_IN_3XXX | RATE_IN_3430ES2PLUS; > > > - cpu_clkflg |= CK_3517; > > > - } else if (cpu_is_omap3505()) { > > > - cpu_mask = RATE_IN_3XXX | RATE_IN_3430ES2PLUS; > > > - cpu_clkflg |= CK_3505; > > > } > > > > > > if (omap3_has_192mhz_clk()) > > > > > > > -- > > Regards, > > Igor. > > > -- > To unsubscribe from this list: send the line "unsubscribe > linux-omap" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html