On Wed, Dec 8, 2010 at 11:10 AM, Santosh Shilimkar santosh.shilimkar@ti.com wrote:
-----Original Message----- From: Dave Martin [mailto:dave.martin@linaro.org] Sent: Wednesday, December 08, 2010 4:35 PM To: Santosh Shilimkar Cc: Tony Lindgren; linux-omap@vger.kernel.org; linaro- dev@lists.linaro.org; linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v2 2/3] ARM: omap4: Correct definition of do_wfi() forCONFIG_THUMB2_KERNEL
On Wed, Dec 8, 2010 at 10:45 AM, Santosh Shilimkar santosh.shilimkar@ti.com wrote:
-----Original Message----- From: Dave Martin [mailto:dave.martin@linaro.org] Sent: Wednesday, December 08, 2010 4:11 PM To: Santosh Shilimkar Cc: linux-arm-kernel@lists.infradead.org; Tony Lindgren; linux- omap@vger.kernel.org; linaro-dev@lists.linaro.org Subject: Re: [PATCH v2 2/3] ARM: omap4: Correct definition of
do_wfi()
forCONFIG_THUMB2_KERNEL
On Wed, Dec 8, 2010 at 5:59 AM, Santosh Shilimkar santosh.shilimkar@ti.com wrote:
Dave,
-----Original Message----- From: Santosh Shilimkar [mailto:santosh.shilimkar@ti.com] Sent: Wednesday, December 08, 2010 11:27 AM To: Dave Martin Cc: linux-arm-kernel@lists.infradead.org; Tony Lindgren; linux- omap@vger.kernel.org; linaro-dev@lists.linaro.org Subject: RE: [PATCH v2 2/3] ARM: omap4: Correct definition of
do_wfi()
forCONFIG_THUMB2_KERNEL
> -----Original Message----- > From: Dave Martin [mailto:dave.martin@linaro.org] > Sent: Tuesday, December 07, 2010 10:21 PM > To: Santosh Shilimkar > Cc: linux-arm-kernel@lists.infradead.org; Tony Lindgren; linux- > omap@vger.kernel.org; linaro-dev@lists.linaro.org > Subject: Re: [PATCH v2 2/3] ARM: omap4: Correct definition of
do_wfi()
> forCONFIG_THUMB2_KERNEL > > Hi, > > On Tue, Dec 7, 2010 at 6:28 AM, Santosh Shilimkar > santosh.shilimkar@ti.com wrote: > > Dave,
[.....]
> So anything inside #ifdef CONFIG_THUMB2_KERNEL can assume
v7/Thumb-2
> capable (and hence reasonably new) tools. > > I'll follow up shortly with a patch to the generic ARM Kconfig
to
make
> this explicit, so that ARCH_OMAP2 and THUMB2_KERNEL can't
accidentally
> be configured together. > sure
When you are doing the changes can you please check if you could
build
the THUMB2 kernel with omap2plus_defconfig. I suspect the build
will
fail.
With my Kconfig patch, kconfig won't let you turn on Thumb-2 in that configuration:
http://lists.arm.linux.org.uk/lurker/message/20101207.165737.0897658f.en.h
tml
If you want to turn on Thumb-2, you must disable ARCH_OMAP2 first.
If
my understanding is correct, this is the right behaviour.
Ofcourse it will build with ARCH_OMAP2 disabled :) (ARMv6 dependency) In other words, I wanted to say that "omap2plus_defconfig" can't be used as is to build THUMB kernel binary.
Well, yes, that's another way of looking at it. Anyway, I think this is the intended result -- is it OK for you?
Should be ok considering you can't do much with it. But I let Tony comment on it.
Well, it depends on what you're trying to do. We can't combine Thumb-2 with a combined omap kernel intended to support omap2. But in linaro for example we are targeting v7+ only for the kernel and userspace, so we already build the kernel for v7 and don't include the OMAP2 support; instead we target OMAP3/4. I believe the situation for Ubuntu is similar. In our case, including the omap2 support wouldn't be useful, since the userspace won't run on that platform anyway...
So really, my concerns are that a) the generic rules for CONFIG_THUMB2_KERNEL are correct, and b) the omap tree works well with this configuration to the maximum extent possible. Obviously, mixing ARCH_OMAP2 with THUMB2_KERNEL _isn't_ possible even in theory, so I don't consider it a failure if that combination isn't supported.
Cheers ---Dave