* [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
@ 2012-06-11 13:22 ` Vaibhav Hiremath
0 siblings, 0 replies; 26+ messages in thread
From: Vaibhav Hiremath @ 2012-06-11 13:22 UTC (permalink / raw)
To: linux-omap; +Cc: tony, linux-arm-kernel, Vaibhav Hiremath
As omap1 and omap2 will never be compiled together, due to
different compiler flags, so we can simply make
cpu_class_is_omap2() = true, for all non-omap1 platforms.
Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
---
arch/arm/plat-omap/include/plat/cpu.h | 3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/arch/arm/plat-omap/include/plat/cpu.h b/arch/arm/plat-omap/include/plat/cpu.h
index 14f050f..6a42558 100644
--- a/arch/arm/plat-omap/include/plat/cpu.h
+++ b/arch/arm/plat-omap/include/plat/cpu.h
@@ -374,8 +374,7 @@ IS_OMAP_TYPE(3430, 0x3430)
/* Macros to detect if we have OMAP1 or OMAP2 */
#define cpu_class_is_omap1() (cpu_is_omap7xx() || cpu_is_omap15xx() || \
cpu_is_omap16xx())
-#define cpu_class_is_omap2() (cpu_is_omap24xx() || cpu_is_omap34xx() || \
- cpu_is_omap44xx())
+#define cpu_class_is_omap2() !cpu_class_is_omap1()
/* Various silicon revisions for omap2 */
#define OMAP242X_CLASS 0x24200024
--
1.7.0.4
^ permalink raw reply related [flat|nested] 26+ messages in thread
* [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
@ 2012-06-11 13:22 ` Vaibhav Hiremath
0 siblings, 0 replies; 26+ messages in thread
From: Vaibhav Hiremath @ 2012-06-11 13:22 UTC (permalink / raw)
To: linux-arm-kernel
As omap1 and omap2 will never be compiled together, due to
different compiler flags, so we can simply make
cpu_class_is_omap2() = true, for all non-omap1 platforms.
Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
---
arch/arm/plat-omap/include/plat/cpu.h | 3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/arch/arm/plat-omap/include/plat/cpu.h b/arch/arm/plat-omap/include/plat/cpu.h
index 14f050f..6a42558 100644
--- a/arch/arm/plat-omap/include/plat/cpu.h
+++ b/arch/arm/plat-omap/include/plat/cpu.h
@@ -374,8 +374,7 @@ IS_OMAP_TYPE(3430, 0x3430)
/* Macros to detect if we have OMAP1 or OMAP2 */
#define cpu_class_is_omap1() (cpu_is_omap7xx() || cpu_is_omap15xx() || \
cpu_is_omap16xx())
-#define cpu_class_is_omap2() (cpu_is_omap24xx() || cpu_is_omap34xx() || \
- cpu_is_omap44xx())
+#define cpu_class_is_omap2() !cpu_class_is_omap1()
/* Various silicon revisions for omap2 */
#define OMAP242X_CLASS 0x24200024
--
1.7.0.4
^ permalink raw reply related [flat|nested] 26+ messages in thread
* RE: [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
2012-06-11 13:22 ` Vaibhav Hiremath
@ 2012-06-14 14:33 ` Hiremath, Vaibhav
-1 siblings, 0 replies; 26+ messages in thread
From: Hiremath, Vaibhav @ 2012-06-14 14:33 UTC (permalink / raw)
To: Hiremath, Vaibhav, linux-omap; +Cc: tony, linux-arm-kernel
On Mon, Jun 11, 2012 at 18:52:54, Hiremath, Vaibhav wrote:
> As omap1 and omap2 will never be compiled together, due to
> different compiler flags, so we can simply make
> cpu_class_is_omap2() = true, for all non-omap1 platforms.
>
> Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
> ---
> arch/arm/plat-omap/include/plat/cpu.h | 3 +--
> 1 files changed, 1 insertions(+), 2 deletions(-)
>
Tony,
Ping???
Thanks,
Vaibhav
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
@ 2012-06-14 14:33 ` Hiremath, Vaibhav
0 siblings, 0 replies; 26+ messages in thread
From: Hiremath, Vaibhav @ 2012-06-14 14:33 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Jun 11, 2012 at 18:52:54, Hiremath, Vaibhav wrote:
> As omap1 and omap2 will never be compiled together, due to
> different compiler flags, so we can simply make
> cpu_class_is_omap2() = true, for all non-omap1 platforms.
>
> Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
> ---
> arch/arm/plat-omap/include/plat/cpu.h | 3 +--
> 1 files changed, 1 insertions(+), 2 deletions(-)
>
Tony,
Ping???
Thanks,
Vaibhav
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
2012-06-14 14:33 ` Hiremath, Vaibhav
@ 2012-06-27 13:39 ` Tony Lindgren
-1 siblings, 0 replies; 26+ messages in thread
From: Tony Lindgren @ 2012-06-27 13:39 UTC (permalink / raw)
To: Hiremath, Vaibhav; +Cc: linux-omap, linux-arm-kernel
* Hiremath, Vaibhav <hvaibhav@ti.com> [120614 07:37]:
> On Mon, Jun 11, 2012 at 18:52:54, Hiremath, Vaibhav wrote:
> > As omap1 and omap2 will never be compiled together, due to
> > different compiler flags, so we can simply make
> > cpu_class_is_omap2() = true, for all non-omap1 platforms.
> >
> > Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
> > ---
> > arch/arm/plat-omap/include/plat/cpu.h | 3 +--
> > 1 files changed, 1 insertions(+), 2 deletions(-)
> >
>
> Tony,
>
> Ping???
Thanks applying into devel-am33xx branch.
Tony
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
@ 2012-06-27 13:39 ` Tony Lindgren
0 siblings, 0 replies; 26+ messages in thread
From: Tony Lindgren @ 2012-06-27 13:39 UTC (permalink / raw)
To: linux-arm-kernel
* Hiremath, Vaibhav <hvaibhav@ti.com> [120614 07:37]:
> On Mon, Jun 11, 2012 at 18:52:54, Hiremath, Vaibhav wrote:
> > As omap1 and omap2 will never be compiled together, due to
> > different compiler flags, so we can simply make
> > cpu_class_is_omap2() = true, for all non-omap1 platforms.
> >
> > Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
> > ---
> > arch/arm/plat-omap/include/plat/cpu.h | 3 +--
> > 1 files changed, 1 insertions(+), 2 deletions(-)
> >
>
> Tony,
>
> Ping???
Thanks applying into devel-am33xx branch.
Tony
^ permalink raw reply [flat|nested] 26+ messages in thread
* RE: [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
2012-06-27 13:39 ` Tony Lindgren
@ 2012-06-27 13:50 ` Hiremath, Vaibhav
-1 siblings, 0 replies; 26+ messages in thread
From: Hiremath, Vaibhav @ 2012-06-27 13:50 UTC (permalink / raw)
To: Tony Lindgren; +Cc: linux-omap, linux-arm-kernel
On Wed, Jun 27, 2012 at 19:09:50, Tony Lindgren wrote:
> * Hiremath, Vaibhav <hvaibhav@ti.com> [120614 07:37]:
> > On Mon, Jun 11, 2012 at 18:52:54, Hiremath, Vaibhav wrote:
> > > As omap1 and omap2 will never be compiled together, due to
> > > different compiler flags, so we can simply make
> > > cpu_class_is_omap2() = true, for all non-omap1 platforms.
> > >
> > > Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
> > > ---
> > > arch/arm/plat-omap/include/plat/cpu.h | 3 +--
> > > 1 files changed, 1 insertions(+), 2 deletions(-)
> > >
> >
> > Tony,
> >
> > Ping???
>
> Thanks applying into devel-am33xx branch.
>
Thanks Tony,
There are few cleanup patches pending for your review, it would be good if
you provide your review comments.
Thanks,
Vaibhav
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
@ 2012-06-27 13:50 ` Hiremath, Vaibhav
0 siblings, 0 replies; 26+ messages in thread
From: Hiremath, Vaibhav @ 2012-06-27 13:50 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Jun 27, 2012 at 19:09:50, Tony Lindgren wrote:
> * Hiremath, Vaibhav <hvaibhav@ti.com> [120614 07:37]:
> > On Mon, Jun 11, 2012 at 18:52:54, Hiremath, Vaibhav wrote:
> > > As omap1 and omap2 will never be compiled together, due to
> > > different compiler flags, so we can simply make
> > > cpu_class_is_omap2() = true, for all non-omap1 platforms.
> > >
> > > Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
> > > ---
> > > arch/arm/plat-omap/include/plat/cpu.h | 3 +--
> > > 1 files changed, 1 insertions(+), 2 deletions(-)
> > >
> >
> > Tony,
> >
> > Ping???
>
> Thanks applying into devel-am33xx branch.
>
Thanks Tony,
There are few cleanup patches pending for your review, it would be good if
you provide your review comments.
Thanks,
Vaibhav
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
2012-06-27 13:50 ` Hiremath, Vaibhav
@ 2012-06-27 13:58 ` Tony Lindgren
-1 siblings, 0 replies; 26+ messages in thread
From: Tony Lindgren @ 2012-06-27 13:58 UTC (permalink / raw)
To: Hiremath, Vaibhav; +Cc: linux-omap, linux-arm-kernel
* Hiremath, Vaibhav <hvaibhav@ti.com> [120627 06:54]:
>
> There are few cleanup patches pending for your review, it would be good if
> you provide your review comments.
Hmm care to post the Subject: lines for those? I may have marked
something as read accidentally..
Tony
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
@ 2012-06-27 13:58 ` Tony Lindgren
0 siblings, 0 replies; 26+ messages in thread
From: Tony Lindgren @ 2012-06-27 13:58 UTC (permalink / raw)
To: linux-arm-kernel
* Hiremath, Vaibhav <hvaibhav@ti.com> [120627 06:54]:
>
> There are few cleanup patches pending for your review, it would be good if
> you provide your review comments.
Hmm care to post the Subject: lines for those? I may have marked
something as read accidentally..
Tony
^ permalink raw reply [flat|nested] 26+ messages in thread
* RE: [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
2012-06-27 13:58 ` Tony Lindgren
@ 2012-06-27 15:27 ` Hiremath, Vaibhav
-1 siblings, 0 replies; 26+ messages in thread
From: Hiremath, Vaibhav @ 2012-06-27 15:27 UTC (permalink / raw)
To: Tony Lindgren; +Cc: linux-omap, linux-arm-kernel
On Wed, Jun 27, 2012 at 19:28:17, Tony Lindgren wrote:
> * Hiremath, Vaibhav <hvaibhav@ti.com> [120627 06:54]:
> >
> > There are few cleanup patches pending for your review, it would be good if
> > you provide your review comments.
>
> Hmm care to post the Subject: lines for those? I may have marked
> something as read accidentally..
>
I will resubmit the patches again ontop of linux-omap/master, makes more sense.
These patches are,
ARM: OMAP2+: CLEANUP: All OMAP2PLUS uses omap-device.o target so add one entry
ARM: OMAP2+: CLEANUP: Move omap3 dpll ops to dpll3xxx.c
ARM: OMAP2+: CLEANUP: Remove ARCH_OMAPx ifdef from struct dpll_data
Thanks,
Vaibhav
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
@ 2012-06-27 15:27 ` Hiremath, Vaibhav
0 siblings, 0 replies; 26+ messages in thread
From: Hiremath, Vaibhav @ 2012-06-27 15:27 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Jun 27, 2012 at 19:28:17, Tony Lindgren wrote:
> * Hiremath, Vaibhav <hvaibhav@ti.com> [120627 06:54]:
> >
> > There are few cleanup patches pending for your review, it would be good if
> > you provide your review comments.
>
> Hmm care to post the Subject: lines for those? I may have marked
> something as read accidentally..
>
I will resubmit the patches again ontop of linux-omap/master, makes more sense.
These patches are,
ARM: OMAP2+: CLEANUP: All OMAP2PLUS uses omap-device.o target so add one entry
ARM: OMAP2+: CLEANUP: Move omap3 dpll ops to dpll3xxx.c
ARM: OMAP2+: CLEANUP: Remove ARCH_OMAPx ifdef from struct dpll_data
Thanks,
Vaibhav
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
2012-06-27 13:39 ` Tony Lindgren
@ 2012-06-27 16:17 ` Tony Lindgren
-1 siblings, 0 replies; 26+ messages in thread
From: Tony Lindgren @ 2012-06-27 16:17 UTC (permalink / raw)
To: Hiremath, Vaibhav; +Cc: linux-omap, linux-arm-kernel
* Tony Lindgren <tony@atomide.com> [120627 06:44]:
> * Hiremath, Vaibhav <hvaibhav@ti.com> [120614 07:37]:
> > On Mon, Jun 11, 2012 at 18:52:54, Hiremath, Vaibhav wrote:
> > > As omap1 and omap2 will never be compiled together, due to
> > > different compiler flags, so we can simply make
> > > cpu_class_is_omap2() = true, for all non-omap1 platforms.
> > >
> > > Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
> > > ---
> > > arch/arm/plat-omap/include/plat/cpu.h | 3 +--
> > > 1 files changed, 1 insertions(+), 2 deletions(-)
> > >
> >
> > Tony,
> >
> > Ping???
>
> Thanks applying into devel-am33xx branch.
Actually looks like this fails for the dynamic detection
case with omap1_defconfig.. Looks like you did not build
test your patch with omap1_defconfig? Please do it for
every patch!
It seems we need something better though for multi zImage
too, we can't really rely on CONFIG_ARCH_OMAP1 or
CONFIG_ARCH_OMAP2PLUS here to determine anything.
Anyways, dropping this for now, let's try to come up
with something better.
Tony
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
@ 2012-06-27 16:17 ` Tony Lindgren
0 siblings, 0 replies; 26+ messages in thread
From: Tony Lindgren @ 2012-06-27 16:17 UTC (permalink / raw)
To: linux-arm-kernel
* Tony Lindgren <tony@atomide.com> [120627 06:44]:
> * Hiremath, Vaibhav <hvaibhav@ti.com> [120614 07:37]:
> > On Mon, Jun 11, 2012 at 18:52:54, Hiremath, Vaibhav wrote:
> > > As omap1 and omap2 will never be compiled together, due to
> > > different compiler flags, so we can simply make
> > > cpu_class_is_omap2() = true, for all non-omap1 platforms.
> > >
> > > Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
> > > ---
> > > arch/arm/plat-omap/include/plat/cpu.h | 3 +--
> > > 1 files changed, 1 insertions(+), 2 deletions(-)
> > >
> >
> > Tony,
> >
> > Ping???
>
> Thanks applying into devel-am33xx branch.
Actually looks like this fails for the dynamic detection
case with omap1_defconfig.. Looks like you did not build
test your patch with omap1_defconfig? Please do it for
every patch!
It seems we need something better though for multi zImage
too, we can't really rely on CONFIG_ARCH_OMAP1 or
CONFIG_ARCH_OMAP2PLUS here to determine anything.
Anyways, dropping this for now, let's try to come up
with something better.
Tony
^ permalink raw reply [flat|nested] 26+ messages in thread
* RE: [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
2012-06-27 16:17 ` Tony Lindgren
@ 2012-06-27 20:33 ` Hiremath, Vaibhav
-1 siblings, 0 replies; 26+ messages in thread
From: Hiremath, Vaibhav @ 2012-06-27 20:33 UTC (permalink / raw)
To: Tony Lindgren; +Cc: linux-omap, linux-arm-kernel
On Wed, Jun 27, 2012 at 21:47:45, Tony Lindgren wrote:
> * Tony Lindgren <tony@atomide.com> [120627 06:44]:
> > * Hiremath, Vaibhav <hvaibhav@ti.com> [120614 07:37]:
> > > On Mon, Jun 11, 2012 at 18:52:54, Hiremath, Vaibhav wrote:
> > > > As omap1 and omap2 will never be compiled together, due to
> > > > different compiler flags, so we can simply make
> > > > cpu_class_is_omap2() = true, for all non-omap1 platforms.
> > > >
> > > > Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
> > > > ---
> > > > arch/arm/plat-omap/include/plat/cpu.h | 3 +--
> > > > 1 files changed, 1 insertions(+), 2 deletions(-)
> > > >
> > >
> > > Tony,
> > >
> > > Ping???
> >
> > Thanks applying into devel-am33xx branch.
>
> Actually looks like this fails for the dynamic detection
> case with omap1_defconfig.. Looks like you did not build
> test your patch with omap1_defconfig? Please do it for
> every patch!
>
> It seems we need something better though for multi zImage
> too, we can't really rely on CONFIG_ARCH_OMAP1 or
> CONFIG_ARCH_OMAP2PLUS here to determine anything.
>
> Anyways, dropping this for now, let's try to come up
> with something better.
>
Yes Tony, you are right I did not test omap1 for this patch. My bad...
How about something like, considering the fact that, omap_type() is the
culprit here and is not applicable to omap1 at all. In case of omap1, we
don't use omap_type() function.
diff --git a/arch/arm/plat-omap/include/plat/cpu.h b/arch/arm/plat-omap/include/plat/cpu.h
index 6a42558..5900a3d 100644
--- a/arch/arm/plat-omap/include/plat/cpu.h
+++ b/arch/arm/plat-omap/include/plat/cpu.h
@@ -42,7 +42,14 @@
#define OMAP2_DEVICE_TYPE_GP 3
#define OMAP2_DEVICE_TYPE_BAD 4
+#ifdef CONFIG_ARCH_OMAP2PLUS
int omap_type(void);
+#else
+inline static int omap_type(void)
+{
+ /* Always return GP, since it is not being used anyway for omap1 */
+ return OMAP2_DEVICE_TYPE_GP;
+}
+#endif
/*
* omap_rev bits:
Thanks,
Vaibhav
^ permalink raw reply related [flat|nested] 26+ messages in thread
* [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
@ 2012-06-27 20:33 ` Hiremath, Vaibhav
0 siblings, 0 replies; 26+ messages in thread
From: Hiremath, Vaibhav @ 2012-06-27 20:33 UTC (permalink / raw)
To: linux-arm-kernel
On Wed, Jun 27, 2012 at 21:47:45, Tony Lindgren wrote:
> * Tony Lindgren <tony@atomide.com> [120627 06:44]:
> > * Hiremath, Vaibhav <hvaibhav@ti.com> [120614 07:37]:
> > > On Mon, Jun 11, 2012 at 18:52:54, Hiremath, Vaibhav wrote:
> > > > As omap1 and omap2 will never be compiled together, due to
> > > > different compiler flags, so we can simply make
> > > > cpu_class_is_omap2() = true, for all non-omap1 platforms.
> > > >
> > > > Signed-off-by: Vaibhav Hiremath <hvaibhav@ti.com>
> > > > ---
> > > > arch/arm/plat-omap/include/plat/cpu.h | 3 +--
> > > > 1 files changed, 1 insertions(+), 2 deletions(-)
> > > >
> > >
> > > Tony,
> > >
> > > Ping???
> >
> > Thanks applying into devel-am33xx branch.
>
> Actually looks like this fails for the dynamic detection
> case with omap1_defconfig.. Looks like you did not build
> test your patch with omap1_defconfig? Please do it for
> every patch!
>
> It seems we need something better though for multi zImage
> too, we can't really rely on CONFIG_ARCH_OMAP1 or
> CONFIG_ARCH_OMAP2PLUS here to determine anything.
>
> Anyways, dropping this for now, let's try to come up
> with something better.
>
Yes Tony, you are right I did not test omap1 for this patch. My bad...
How about something like, considering the fact that, omap_type() is the
culprit here and is not applicable to omap1 at all. In case of omap1, we
don't use omap_type() function.
diff --git a/arch/arm/plat-omap/include/plat/cpu.h b/arch/arm/plat-omap/include/plat/cpu.h
index 6a42558..5900a3d 100644
--- a/arch/arm/plat-omap/include/plat/cpu.h
+++ b/arch/arm/plat-omap/include/plat/cpu.h
@@ -42,7 +42,14 @@
#define OMAP2_DEVICE_TYPE_GP 3
#define OMAP2_DEVICE_TYPE_BAD 4
+#ifdef CONFIG_ARCH_OMAP2PLUS
int omap_type(void);
+#else
+inline static int omap_type(void)
+{
+ /* Always return GP, since it is not being used anyway for omap1 */
+ return OMAP2_DEVICE_TYPE_GP;
+}
+#endif
/*
* omap_rev bits:
Thanks,
Vaibhav
^ permalink raw reply related [flat|nested] 26+ messages in thread
* Re: [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
2012-06-27 20:33 ` Hiremath, Vaibhav
@ 2012-07-10 8:41 ` Tony Lindgren
-1 siblings, 0 replies; 26+ messages in thread
From: Tony Lindgren @ 2012-07-10 8:41 UTC (permalink / raw)
To: Hiremath, Vaibhav; +Cc: linux-omap, linux-arm-kernel
* Hiremath, Vaibhav <hvaibhav@ti.com> [120627 13:37]:
> --- a/arch/arm/plat-omap/include/plat/cpu.h
> +++ b/arch/arm/plat-omap/include/plat/cpu.h
> @@ -42,7 +42,14 @@
> #define OMAP2_DEVICE_TYPE_GP 3
> #define OMAP2_DEVICE_TYPE_BAD 4
>
> +#ifdef CONFIG_ARCH_OMAP2PLUS
> int omap_type(void);
> +#else
> +inline static int omap_type(void)
> +{
> + /* Always return GP, since it is not being used anyway for omap1 */
> + return OMAP2_DEVICE_TYPE_GP;
> +}
> +#endif
>
> /*
> * omap_rev bits:
Just to follow-up on this, looks like we need to postpone this
for v3.7. While at it, we need to make sure all soc_is_omapxxxx()
are false by default.
What you're suggesting above would also depend on some MULTI_ARCH
option set if multiple ARM architectures are selected. Above patch
would only optimize things when ARCH_OMAP2PLUS alone is selected.
Regards,
Tony
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
@ 2012-07-10 8:41 ` Tony Lindgren
0 siblings, 0 replies; 26+ messages in thread
From: Tony Lindgren @ 2012-07-10 8:41 UTC (permalink / raw)
To: linux-arm-kernel
* Hiremath, Vaibhav <hvaibhav@ti.com> [120627 13:37]:
> --- a/arch/arm/plat-omap/include/plat/cpu.h
> +++ b/arch/arm/plat-omap/include/plat/cpu.h
> @@ -42,7 +42,14 @@
> #define OMAP2_DEVICE_TYPE_GP 3
> #define OMAP2_DEVICE_TYPE_BAD 4
>
> +#ifdef CONFIG_ARCH_OMAP2PLUS
> int omap_type(void);
> +#else
> +inline static int omap_type(void)
> +{
> + /* Always return GP, since it is not being used anyway for omap1 */
> + return OMAP2_DEVICE_TYPE_GP;
> +}
> +#endif
>
> /*
> * omap_rev bits:
Just to follow-up on this, looks like we need to postpone this
for v3.7. While at it, we need to make sure all soc_is_omapxxxx()
are false by default.
What you're suggesting above would also depend on some MULTI_ARCH
option set if multiple ARM architectures are selected. Above patch
would only optimize things when ARCH_OMAP2PLUS alone is selected.
Regards,
Tony
^ permalink raw reply [flat|nested] 26+ messages in thread
* RE: [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
2012-07-10 8:41 ` Tony Lindgren
@ 2012-07-10 9:10 ` Hiremath, Vaibhav
-1 siblings, 0 replies; 26+ messages in thread
From: Hiremath, Vaibhav @ 2012-07-10 9:10 UTC (permalink / raw)
To: Tony Lindgren; +Cc: linux-omap, linux-arm-kernel
On Tue, Jul 10, 2012 at 14:11:08, Tony Lindgren wrote:
> * Hiremath, Vaibhav <hvaibhav@ti.com> [120627 13:37]:
> > --- a/arch/arm/plat-omap/include/plat/cpu.h
> > +++ b/arch/arm/plat-omap/include/plat/cpu.h
> > @@ -42,7 +42,14 @@
> > #define OMAP2_DEVICE_TYPE_GP 3
> > #define OMAP2_DEVICE_TYPE_BAD 4
> >
> > +#ifdef CONFIG_ARCH_OMAP2PLUS
> > int omap_type(void);
> > +#else
> > +inline static int omap_type(void)
> > +{
> > + /* Always return GP, since it is not being used anyway for omap1 */
> > + return OMAP2_DEVICE_TYPE_GP;
> > +}
> > +#endif
> >
> > /*
> > * omap_rev bits:
>
> Just to follow-up on this, looks like we need to postpone this
> for v3.7. While at it, we need to make sure all soc_is_omapxxxx()
> are false by default.
>
Just FYI, I had submitted V2 of this patch.
> What you're suggesting above would also depend on some MULTI_ARCH
> option set if multiple ARM architectures are selected. Above patch
> would only optimize things when ARCH_OMAP2PLUS alone is selected.
>
Isn't this only applicable for ARCH_OMAP2PLUS and OMAP1?? And above patch
take cares of both, as OMAP1 and 2 will not be build together, since
compiler flags itself will be different.
Can you elaborate on how things will break if multiple ARM architectures are
selected?
I am not sure whether I am following you completely.
Thanks,
Vaibhav
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
@ 2012-07-10 9:10 ` Hiremath, Vaibhav
0 siblings, 0 replies; 26+ messages in thread
From: Hiremath, Vaibhav @ 2012-07-10 9:10 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Jul 10, 2012 at 14:11:08, Tony Lindgren wrote:
> * Hiremath, Vaibhav <hvaibhav@ti.com> [120627 13:37]:
> > --- a/arch/arm/plat-omap/include/plat/cpu.h
> > +++ b/arch/arm/plat-omap/include/plat/cpu.h
> > @@ -42,7 +42,14 @@
> > #define OMAP2_DEVICE_TYPE_GP 3
> > #define OMAP2_DEVICE_TYPE_BAD 4
> >
> > +#ifdef CONFIG_ARCH_OMAP2PLUS
> > int omap_type(void);
> > +#else
> > +inline static int omap_type(void)
> > +{
> > + /* Always return GP, since it is not being used anyway for omap1 */
> > + return OMAP2_DEVICE_TYPE_GP;
> > +}
> > +#endif
> >
> > /*
> > * omap_rev bits:
>
> Just to follow-up on this, looks like we need to postpone this
> for v3.7. While at it, we need to make sure all soc_is_omapxxxx()
> are false by default.
>
Just FYI, I had submitted V2 of this patch.
> What you're suggesting above would also depend on some MULTI_ARCH
> option set if multiple ARM architectures are selected. Above patch
> would only optimize things when ARCH_OMAP2PLUS alone is selected.
>
Isn't this only applicable for ARCH_OMAP2PLUS and OMAP1?? And above patch
take cares of both, as OMAP1 and 2 will not be build together, since
compiler flags itself will be different.
Can you elaborate on how things will break if multiple ARM architectures are
selected?
I am not sure whether I am following you completely.
Thanks,
Vaibhav
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
2012-07-10 9:10 ` Hiremath, Vaibhav
@ 2012-07-10 9:18 ` Tony Lindgren
-1 siblings, 0 replies; 26+ messages in thread
From: Tony Lindgren @ 2012-07-10 9:18 UTC (permalink / raw)
To: Hiremath, Vaibhav; +Cc: linux-omap, linux-arm-kernel
* Hiremath, Vaibhav <hvaibhav@ti.com> [120710 02:15]:
> On Tue, Jul 10, 2012 at 14:11:08, Tony Lindgren wrote:
> > * Hiremath, Vaibhav <hvaibhav@ti.com> [120627 13:37]:
> > > --- a/arch/arm/plat-omap/include/plat/cpu.h
> > > +++ b/arch/arm/plat-omap/include/plat/cpu.h
> > > @@ -42,7 +42,14 @@
> > > #define OMAP2_DEVICE_TYPE_GP 3
> > > #define OMAP2_DEVICE_TYPE_BAD 4
> > >
> > > +#ifdef CONFIG_ARCH_OMAP2PLUS
> > > int omap_type(void);
> > > +#else
> > > +inline static int omap_type(void)
> > > +{
> > > + /* Always return GP, since it is not being used anyway for omap1 */
> > > + return OMAP2_DEVICE_TYPE_GP;
> > > +}
> > > +#endif
> > >
> > > /*
> > > * omap_rev bits:
> >
> > Just to follow-up on this, looks like we need to postpone this
> > for v3.7. While at it, we need to make sure all soc_is_omapxxxx()
> > are false by default.
> >
>
> Just FYI, I had submitted V2 of this patch.
>
> > What you're suggesting above would also depend on some MULTI_ARCH
> > option set if multiple ARM architectures are selected. Above patch
> > would only optimize things when ARCH_OMAP2PLUS alone is selected.
> >
>
> Isn't this only applicable for ARCH_OMAP2PLUS and OMAP1?? And above patch
> take cares of both, as OMAP1 and 2 will not be build together, since
> compiler flags itself will be different.
Right, but we also need to deal with
> Can you elaborate on how things will break if multiple ARM architectures are
> selected?
>
> I am not sure whether I am following you completely.
We have quite a few initcalls that exit early with various soc_is, cpu_is,
cpu_class_is tests that would fail for multi zImage kernels. Some of these
have been patched away with additional calls in MACHINE_START, but some
still remain.
We probably always need to return false for all soc_is_omapxxxx() unless
the SoC detection is initialized in id.c.
Regards,
Tony
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
@ 2012-07-10 9:18 ` Tony Lindgren
0 siblings, 0 replies; 26+ messages in thread
From: Tony Lindgren @ 2012-07-10 9:18 UTC (permalink / raw)
To: linux-arm-kernel
* Hiremath, Vaibhav <hvaibhav@ti.com> [120710 02:15]:
> On Tue, Jul 10, 2012 at 14:11:08, Tony Lindgren wrote:
> > * Hiremath, Vaibhav <hvaibhav@ti.com> [120627 13:37]:
> > > --- a/arch/arm/plat-omap/include/plat/cpu.h
> > > +++ b/arch/arm/plat-omap/include/plat/cpu.h
> > > @@ -42,7 +42,14 @@
> > > #define OMAP2_DEVICE_TYPE_GP 3
> > > #define OMAP2_DEVICE_TYPE_BAD 4
> > >
> > > +#ifdef CONFIG_ARCH_OMAP2PLUS
> > > int omap_type(void);
> > > +#else
> > > +inline static int omap_type(void)
> > > +{
> > > + /* Always return GP, since it is not being used anyway for omap1 */
> > > + return OMAP2_DEVICE_TYPE_GP;
> > > +}
> > > +#endif
> > >
> > > /*
> > > * omap_rev bits:
> >
> > Just to follow-up on this, looks like we need to postpone this
> > for v3.7. While at it, we need to make sure all soc_is_omapxxxx()
> > are false by default.
> >
>
> Just FYI, I had submitted V2 of this patch.
>
> > What you're suggesting above would also depend on some MULTI_ARCH
> > option set if multiple ARM architectures are selected. Above patch
> > would only optimize things when ARCH_OMAP2PLUS alone is selected.
> >
>
> Isn't this only applicable for ARCH_OMAP2PLUS and OMAP1?? And above patch
> take cares of both, as OMAP1 and 2 will not be build together, since
> compiler flags itself will be different.
Right, but we also need to deal with
> Can you elaborate on how things will break if multiple ARM architectures are
> selected?
>
> I am not sure whether I am following you completely.
We have quite a few initcalls that exit early with various soc_is, cpu_is,
cpu_class_is tests that would fail for multi zImage kernels. Some of these
have been patched away with additional calls in MACHINE_START, but some
still remain.
We probably always need to return false for all soc_is_omapxxxx() unless
the SoC detection is initialized in id.c.
Regards,
Tony
^ permalink raw reply [flat|nested] 26+ messages in thread
* RE: [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
2012-07-10 9:18 ` Tony Lindgren
@ 2012-07-10 9:25 ` Hiremath, Vaibhav
-1 siblings, 0 replies; 26+ messages in thread
From: Hiremath, Vaibhav @ 2012-07-10 9:25 UTC (permalink / raw)
To: Tony Lindgren; +Cc: linux-omap, linux-arm-kernel
On Tue, Jul 10, 2012 at 14:48:21, Tony Lindgren wrote:
> * Hiremath, Vaibhav <hvaibhav@ti.com> [120710 02:15]:
> > On Tue, Jul 10, 2012 at 14:11:08, Tony Lindgren wrote:
> > > * Hiremath, Vaibhav <hvaibhav@ti.com> [120627 13:37]:
> > > > --- a/arch/arm/plat-omap/include/plat/cpu.h
> > > > +++ b/arch/arm/plat-omap/include/plat/cpu.h
> > > > @@ -42,7 +42,14 @@
> > > > #define OMAP2_DEVICE_TYPE_GP 3
> > > > #define OMAP2_DEVICE_TYPE_BAD 4
> > > >
> > > > +#ifdef CONFIG_ARCH_OMAP2PLUS
> > > > int omap_type(void);
> > > > +#else
> > > > +inline static int omap_type(void)
> > > > +{
> > > > + /* Always return GP, since it is not being used anyway for omap1 */
> > > > + return OMAP2_DEVICE_TYPE_GP;
> > > > +}
> > > > +#endif
> > > >
> > > > /*
> > > > * omap_rev bits:
> > >
> > > Just to follow-up on this, looks like we need to postpone this
> > > for v3.7. While at it, we need to make sure all soc_is_omapxxxx()
> > > are false by default.
> > >
> >
> > Just FYI, I had submitted V2 of this patch.
> >
> > > What you're suggesting above would also depend on some MULTI_ARCH
> > > option set if multiple ARM architectures are selected. Above patch
> > > would only optimize things when ARCH_OMAP2PLUS alone is selected.
> > >
> >
> > Isn't this only applicable for ARCH_OMAP2PLUS and OMAP1?? And above patch
> > take cares of both, as OMAP1 and 2 will not be build together, since
> > compiler flags itself will be different.
>
> Right, but we also need to deal with
>
> > Can you elaborate on how things will break if multiple ARM architectures are
> > selected?
> >
> > I am not sure whether I am following you completely.
>
> We have quite a few initcalls that exit early with various soc_is, cpu_is,
> cpu_class_is tests that would fail for multi zImage kernels. Some of these
> have been patched away with additional calls in MACHINE_START, but some
> still remain.
>
Let me check on this inside code but if you have any quick reference, please
let me know, I can fix it and submit the patch.
> We probably always need to return false for all soc_is_omapxxxx() unless
> the SoC detection is initialized in id.c.
>
Ditto...
Thanks,
Vaibhav
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
@ 2012-07-10 9:25 ` Hiremath, Vaibhav
0 siblings, 0 replies; 26+ messages in thread
From: Hiremath, Vaibhav @ 2012-07-10 9:25 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Jul 10, 2012 at 14:48:21, Tony Lindgren wrote:
> * Hiremath, Vaibhav <hvaibhav@ti.com> [120710 02:15]:
> > On Tue, Jul 10, 2012 at 14:11:08, Tony Lindgren wrote:
> > > * Hiremath, Vaibhav <hvaibhav@ti.com> [120627 13:37]:
> > > > --- a/arch/arm/plat-omap/include/plat/cpu.h
> > > > +++ b/arch/arm/plat-omap/include/plat/cpu.h
> > > > @@ -42,7 +42,14 @@
> > > > #define OMAP2_DEVICE_TYPE_GP 3
> > > > #define OMAP2_DEVICE_TYPE_BAD 4
> > > >
> > > > +#ifdef CONFIG_ARCH_OMAP2PLUS
> > > > int omap_type(void);
> > > > +#else
> > > > +inline static int omap_type(void)
> > > > +{
> > > > + /* Always return GP, since it is not being used anyway for omap1 */
> > > > + return OMAP2_DEVICE_TYPE_GP;
> > > > +}
> > > > +#endif
> > > >
> > > > /*
> > > > * omap_rev bits:
> > >
> > > Just to follow-up on this, looks like we need to postpone this
> > > for v3.7. While at it, we need to make sure all soc_is_omapxxxx()
> > > are false by default.
> > >
> >
> > Just FYI, I had submitted V2 of this patch.
> >
> > > What you're suggesting above would also depend on some MULTI_ARCH
> > > option set if multiple ARM architectures are selected. Above patch
> > > would only optimize things when ARCH_OMAP2PLUS alone is selected.
> > >
> >
> > Isn't this only applicable for ARCH_OMAP2PLUS and OMAP1?? And above patch
> > take cares of both, as OMAP1 and 2 will not be build together, since
> > compiler flags itself will be different.
>
> Right, but we also need to deal with
>
> > Can you elaborate on how things will break if multiple ARM architectures are
> > selected?
> >
> > I am not sure whether I am following you completely.
>
> We have quite a few initcalls that exit early with various soc_is, cpu_is,
> cpu_class_is tests that would fail for multi zImage kernels. Some of these
> have been patched away with additional calls in MACHINE_START, but some
> still remain.
>
Let me check on this inside code but if you have any quick reference, please
let me know, I can fix it and submit the patch.
> We probably always need to return false for all soc_is_omapxxxx() unless
> the SoC detection is initialized in id.c.
>
Ditto...
Thanks,
Vaibhav
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
2012-07-10 9:25 ` Hiremath, Vaibhav
@ 2012-07-10 9:43 ` Tony Lindgren
-1 siblings, 0 replies; 26+ messages in thread
From: Tony Lindgren @ 2012-07-10 9:43 UTC (permalink / raw)
To: Hiremath, Vaibhav; +Cc: linux-omap, linux-arm-kernel
* Hiremath, Vaibhav <hvaibhav@ti.com> [120710 02:29]:
> On Tue, Jul 10, 2012 at 14:48:21, Tony Lindgren wrote:
> >
> > We have quite a few initcalls that exit early with various soc_is, cpu_is,
> > cpu_class_is tests that would fail for multi zImage kernels. Some of these
> > have been patched away with additional calls in MACHINE_START, but some
> > still remain.
> >
> Let me check on this inside code but if you have any quick reference, please
> let me know, I can fix it and submit the patch.
Any initcall that returns early if (!cpu_class_is_omap2()) would continue
running on multi zImage kernel on other SoCs compiled in unless we check
for each cpu_is_omapxxxx() || soc_is_omapxxxx() tests like we currently
do.
Regards,
Tony
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms
@ 2012-07-10 9:43 ` Tony Lindgren
0 siblings, 0 replies; 26+ messages in thread
From: Tony Lindgren @ 2012-07-10 9:43 UTC (permalink / raw)
To: linux-arm-kernel
* Hiremath, Vaibhav <hvaibhav@ti.com> [120710 02:29]:
> On Tue, Jul 10, 2012 at 14:48:21, Tony Lindgren wrote:
> >
> > We have quite a few initcalls that exit early with various soc_is, cpu_is,
> > cpu_class_is tests that would fail for multi zImage kernels. Some of these
> > have been patched away with additional calls in MACHINE_START, but some
> > still remain.
> >
> Let me check on this inside code but if you have any quick reference, please
> let me know, I can fix it and submit the patch.
Any initcall that returns early if (!cpu_class_is_omap2()) would continue
running on multi zImage kernel on other SoCs compiled in unless we check
for each cpu_is_omapxxxx() || soc_is_omapxxxx() tests like we currently
do.
Regards,
Tony
^ permalink raw reply [flat|nested] 26+ messages in thread
end of thread, other threads:[~2012-07-10 9:43 UTC | newest]
Thread overview: 26+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-06-11 13:22 [PATCH] ARM: OMAP2+: cpu: Make cpu_class_is_omap2 true for all non-omap1 platforms Vaibhav Hiremath
2012-06-11 13:22 ` Vaibhav Hiremath
2012-06-14 14:33 ` Hiremath, Vaibhav
2012-06-14 14:33 ` Hiremath, Vaibhav
2012-06-27 13:39 ` Tony Lindgren
2012-06-27 13:39 ` Tony Lindgren
2012-06-27 13:50 ` Hiremath, Vaibhav
2012-06-27 13:50 ` Hiremath, Vaibhav
2012-06-27 13:58 ` Tony Lindgren
2012-06-27 13:58 ` Tony Lindgren
2012-06-27 15:27 ` Hiremath, Vaibhav
2012-06-27 15:27 ` Hiremath, Vaibhav
2012-06-27 16:17 ` Tony Lindgren
2012-06-27 16:17 ` Tony Lindgren
2012-06-27 20:33 ` Hiremath, Vaibhav
2012-06-27 20:33 ` Hiremath, Vaibhav
2012-07-10 8:41 ` Tony Lindgren
2012-07-10 8:41 ` Tony Lindgren
2012-07-10 9:10 ` Hiremath, Vaibhav
2012-07-10 9:10 ` Hiremath, Vaibhav
2012-07-10 9:18 ` Tony Lindgren
2012-07-10 9:18 ` Tony Lindgren
2012-07-10 9:25 ` Hiremath, Vaibhav
2012-07-10 9:25 ` Hiremath, Vaibhav
2012-07-10 9:43 ` Tony Lindgren
2012-07-10 9:43 ` Tony Lindgren
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.