* [PATCH] ARM: OMAP: id: Add missing break statement in omap3xxx_check_revision @ 2012-01-23 7:56 Vaibhav Hiremath 2012-01-27 0:25 ` Tony Lindgren 0 siblings, 1 reply; 5+ messages in thread From: Vaibhav Hiremath @ 2012-01-23 7:56 UTC (permalink / raw) To: linux-arm-kernel Add missing break statement in the function omap3xxx_check_revision. The commit id 4390f5b2cb1f56 [ARM: OMAP: TI814X: Add cpu type macros and detection support], removed the 'break' statement from the function omap3xxx_check_revision(), resulting into wrong omap/cpu_revision initialization for AM335x devices. Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com> Cc: Tony Lindgren <tony@atomide.com> --- arch/arm/mach-omap2/id.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/arch/arm/mach-omap2/id.c b/arch/arm/mach-omap2/id.c index 325e12e..32cf203 100644 --- a/arch/arm/mach-omap2/id.c +++ b/arch/arm/mach-omap2/id.c @@ -395,6 +395,7 @@ void __init omap3xxx_check_revision(void) case 0xb944: omap_revision = AM335X_REV_ES1_0; cpu_rev = "1.0"; + break; case 0xb8f2: switch (rev) { case 0: -- 1.7.0.4 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] ARM: OMAP: id: Add missing break statement in omap3xxx_check_revision 2012-01-23 7:56 [PATCH] ARM: OMAP: id: Add missing break statement in omap3xxx_check_revision Vaibhav Hiremath @ 2012-01-27 0:25 ` Tony Lindgren 2012-01-27 5:32 ` Hiremath, Vaibhav 0 siblings, 1 reply; 5+ messages in thread From: Tony Lindgren @ 2012-01-27 0:25 UTC (permalink / raw) To: linux-arm-kernel * Vaibhav Hiremath <hvaibhav@ti.com> [120122 23:26]: > Add missing break statement in the function omap3xxx_check_revision. > > The commit id 4390f5b2cb1f56 [ARM: OMAP: TI814X: Add cpu type macros > and detection support], removed the 'break' statement from the function > omap3xxx_check_revision(), resulting into wrong omap/cpu_revision > initialization for AM335x devices. Looks like this is already fixed, can you please verify with v3.3-rc1? Regards, Tony > Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com> > Cc: Tony Lindgren <tony@atomide.com> > --- > arch/arm/mach-omap2/id.c | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/mach-omap2/id.c b/arch/arm/mach-omap2/id.c > index 325e12e..32cf203 100644 > --- a/arch/arm/mach-omap2/id.c > +++ b/arch/arm/mach-omap2/id.c > @@ -395,6 +395,7 @@ void __init omap3xxx_check_revision(void) > case 0xb944: > omap_revision = AM335X_REV_ES1_0; > cpu_rev = "1.0"; > + break; > case 0xb8f2: > switch (rev) { > case 0: > -- > 1.7.0.4 > ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] ARM: OMAP: id: Add missing break statement in omap3xxx_check_revision 2012-01-27 0:25 ` Tony Lindgren @ 2012-01-27 5:32 ` Hiremath, Vaibhav 2012-01-30 8:30 ` Hiremath, Vaibhav 0 siblings, 1 reply; 5+ messages in thread From: Hiremath, Vaibhav @ 2012-01-27 5:32 UTC (permalink / raw) To: linux-arm-kernel On Fri, Jan 27, 2012 at 05:55:43, Tony Lindgren wrote: > * Vaibhav Hiremath <hvaibhav@ti.com> [120122 23:26]: > > Add missing break statement in the function omap3xxx_check_revision. > > > > The commit id 4390f5b2cb1f56 [ARM: OMAP: TI814X: Add cpu type macros > > and detection support], removed the 'break' statement from the function > > omap3xxx_check_revision(), resulting into wrong omap/cpu_revision > > initialization for AM335x devices. > > Looks like this is already fixed, can you please verify with > v3.3-rc1? > Tony, I just looked at your linux-omap/master branch and 'break' statement is missing. So this patch is still required. I can still see, 394 } 395 break; 396 case 0xb944: 397 omap_revision = AM335X_REV_ES1_0; 398 cpu_rev = "1.0"; 399 case 0xb8f2: 400 switch (rev) { Thanks, Vaibhav > Regards, > > Tony > > > Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com> > > Cc: Tony Lindgren <tony@atomide.com> > > --- > > arch/arm/mach-omap2/id.c | 1 + > > 1 files changed, 1 insertions(+), 0 deletions(-) > > > > diff --git a/arch/arm/mach-omap2/id.c b/arch/arm/mach-omap2/id.c > > index 325e12e..32cf203 100644 > > --- a/arch/arm/mach-omap2/id.c > > +++ b/arch/arm/mach-omap2/id.c > > @@ -395,6 +395,7 @@ void __init omap3xxx_check_revision(void) > > case 0xb944: > > omap_revision = AM335X_REV_ES1_0; > > cpu_rev = "1.0"; > > + break; > > case 0xb8f2: > > switch (rev) { > > case 0: > > -- > > 1.7.0.4 > > > ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] ARM: OMAP: id: Add missing break statement in omap3xxx_check_revision 2012-01-27 5:32 ` Hiremath, Vaibhav @ 2012-01-30 8:30 ` Hiremath, Vaibhav 2012-02-29 22:42 ` Tony Lindgren 0 siblings, 1 reply; 5+ messages in thread From: Hiremath, Vaibhav @ 2012-01-30 8:30 UTC (permalink / raw) To: linux-arm-kernel On Fri, Jan 27, 2012 at 11:02:01, Hiremath, Vaibhav wrote: > On Fri, Jan 27, 2012 at 05:55:43, Tony Lindgren wrote: > > * Vaibhav Hiremath <hvaibhav@ti.com> [120122 23:26]: > > > Add missing break statement in the function omap3xxx_check_revision. > > > > > > The commit id 4390f5b2cb1f56 [ARM: OMAP: TI814X: Add cpu type macros > > > and detection support], removed the 'break' statement from the function > > > omap3xxx_check_revision(), resulting into wrong omap/cpu_revision > > > initialization for AM335x devices. > > > > Looks like this is already fixed, can you please verify with > > v3.3-rc1? > > > Tony, > > I just looked at your linux-omap/master branch and 'break' statement is missing. So this patch is still required. > > I can still see, > > 394 } > 395 break; > 396 case 0xb944: > 397 omap_revision = AM335X_REV_ES1_0; > 398 cpu_rev = "1.0"; > 399 case 0xb8f2: > 400 switch (rev) { > > Tony, Can this be merged? I can still see "break" statement missing in the latest linux-omap/master branch. Thanks, Vaibhav > Thanks, > Vaibhav > > > > Regards, > > > > Tony > > > > > Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com> > > > Cc: Tony Lindgren <tony@atomide.com> > > > --- > > > arch/arm/mach-omap2/id.c | 1 + > > > 1 files changed, 1 insertions(+), 0 deletions(-) > > > > > > diff --git a/arch/arm/mach-omap2/id.c b/arch/arm/mach-omap2/id.c > > > index 325e12e..32cf203 100644 > > > --- a/arch/arm/mach-omap2/id.c > > > +++ b/arch/arm/mach-omap2/id.c > > > @@ -395,6 +395,7 @@ void __init omap3xxx_check_revision(void) > > > case 0xb944: > > > omap_revision = AM335X_REV_ES1_0; > > > cpu_rev = "1.0"; > > > + break; > > > case 0xb8f2: > > > switch (rev) { > > > case 0: > > > -- > > > 1.7.0.4 > > > > > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-omap" in > the body of a message to majordomo at vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] ARM: OMAP: id: Add missing break statement in omap3xxx_check_revision 2012-01-30 8:30 ` Hiremath, Vaibhav @ 2012-02-29 22:42 ` Tony Lindgren 0 siblings, 0 replies; 5+ messages in thread From: Tony Lindgren @ 2012-02-29 22:42 UTC (permalink / raw) To: linux-arm-kernel * Hiremath, Vaibhav <hvaibhav@ti.com> [120129 23:59]: > On Fri, Jan 27, 2012 at 11:02:01, Hiremath, Vaibhav wrote: > > On Fri, Jan 27, 2012 at 05:55:43, Tony Lindgren wrote: > > > * Vaibhav Hiremath <hvaibhav@ti.com> [120122 23:26]: > > > > Add missing break statement in the function omap3xxx_check_revision. > > > > > > > > The commit id 4390f5b2cb1f56 [ARM: OMAP: TI814X: Add cpu type macros > > > > and detection support], removed the 'break' statement from the function > > > > omap3xxx_check_revision(), resulting into wrong omap/cpu_revision > > > > initialization for AM335x devices. > > > > > > Looks like this is already fixed, can you please verify with > > > v3.3-rc1? > > > > > Tony, > > > > I just looked at your linux-omap/master branch and 'break' statement is missing. So this patch is still required. > > > > I can still see, > > > > 394 } > > 395 break; > > 396 case 0xb944: > > 397 omap_revision = AM335X_REV_ES1_0; > > 398 cpu_rev = "1.0"; > > 399 case 0xb8f2: > > 400 switch (rev) { > > > > > Tony, > > Can this be merged? I can still see "break" statement missing in the latest > linux-omap/master branch. OK thanks applying into fixes for the -rc cycle. Tony ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-02-29 22:42 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-01-23 7:56 [PATCH] ARM: OMAP: id: Add missing break statement in omap3xxx_check_revision Vaibhav Hiremath 2012-01-27 0:25 ` Tony Lindgren 2012-01-27 5:32 ` Hiremath, Vaibhav 2012-01-30 8:30 ` Hiremath, Vaibhav 2012-02-29 22:42 ` Tony Lindgren
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).