[oe] [PATCHv3 1/3] kernel bbclass: split do_compile into do_compile and do_compile_modules
Koen Kooi
k.kooi at student.utwente.nl
Tue Oct 12 13:02:16 UTC 2010
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 12-10-10 14:53, Frans Meulenbroeks wrote:
> 2010/10/12 Frans Meulenbroeks <fransmeulenbroeks at gmail.com>:
>> 2010/10/12 Koen Kooi <k.kooi at student.utwente.nl>:
> On 12-10-10 12:08, Frans Meulenbroeks wrote:
>>>>> 2010/10/12 Koen Kooi <k.kooi at student.utwente.nl>:
>
>>>>>>> +}
>>>>>>> +
>>>>>>> +do_compile_kernelmodules() {
>>>>>>> + unset CFLAGS CPPFLAGS CXXFLAGS LDFLAGS MACHINE
>>>>>
>>>>> Is it needed/required?
>
> It is, and it's in kernel.bbclass already:
>
> fe938bbe (Michael Lauer 2004-12-14) kernel_do_compile() {
> 0448ec0d (Richard Purdie 2006-11-20) unset CFLAGS CPPFLAGS CXXFLAGS
> LDFLAGS MACHINE
> fe938bbe (Michael Lauer 2004-12-14) oe_runmake
> include/linux/version.h CC="${KERNEL_CC}" LD="${KERNEL_LD}"
>
>
>>>>> I'm a little bit concerned by this.
>>>>> (actually this was the only worry I had after reading the patch, don't
>>>>> want to loose additional CFLAGS settings here
>
> You actually want to do that, since the kernel is bare-metal, not userspace.
>>>
>>
>> Thanks for the answer.
>>
>> Did a quick peek on CFLAGS in the recipes, saw there is a CFLAGS_KERNEL.
>> That one is only used on one place:
>> linux-wrt_2.4.30.bb:export CFLAGS_KERNEL="-fno-delayed-branch "
>>
>> Haven't tested this, but my understanding on how things work is good
>> enough and the change simple enough for to give an ack
>> Acked-by: Frans Meulenbroeks <fransmeulenbroeks at gmail.com>
>>
> Actually on 2nd thought one tiny remark left:
> after seeing patch 2:
> what about renaming:
> do_compile_kernelmodules
> to something with a kernel prefix
> e.g. kernel_do_compile_modules.
> That makes the naming scheme somewhat more consistent.
Namespacing the class is only used for methods overridden from
base.bbclass. Again, look at the rest of kernel.bbclass methods like
uboot-mkimage, menuconfig and friends don't use that namespace.
And it doesn't actually work since _modules is a local override.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)
iD8DBQFMtFxYMkyGM64RGpERArFyAKCr5HipFKmy8xsYIoF/U0NIXFfDlwCfferR
yhTvF+MOCupbbMMstKVvkmk=
=6zMs
-----END PGP SIGNATURE-----
More information about the Openembedded-devel
mailing list