RE: [PATCH] OMAP2: Fix a cpu type check problem.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----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


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux