Re: [PATCHv2 4/4] ti43x, omap-a15: switch KERNEL_IMAGETYPE to zImage by default

[Denys Dmytriyenko <denys@ti.com>]

On Thu, Sep 26, 2013 at 08:34:27PM +0530, Siddharth Heroor wrote:
> On 9/26/2013 5:29 PM, Maupin, Chase wrote:
> > Acked-by: Chase Maupin <chase.maupin@ti.com>
> 
> Just a question - is it possible to override the KERNEL_IMAGETYPE in a
> branding.inc file? In general, we want to move to zImage for GLSDK but
> some customers who might want to stay on uImage as they ramp.

It won't be very easy to do from a branding file. That's why I had to rev 
the other patch to keep GLSDK 3.8 and AMSDK 3.2 kernel recipes on uImage, but 
switch other platforms to zImage which already have recent enough u-boot...

When you have a new kernel/u-boot that prefers zImage (like the new 
linux-ti-staging_git.bb we have for 3.12 integration), you can set the image 
type to zImage in the kernel recipe, or rather not specify it at all, since 
it will be picked up from machine config.

In other words, after the setting in the machine config, you have one more 
option to override it in a specific kernel recipe. That way if your branding 
selects latest kernel/u-boot combo, you get zImage by default from the machine 
config. But if branding selects 3.2 kernel with early u-boot, it gets 
overwritten back to uImage...

-- 
Denys


> Otherwise, no comments.
> 
> Acked-by: Siddharth Heroor <heroor@ti.com>
> 
> > 
> > 
> >> -----Original Message-----
> >> From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
> >> bounces@yoctoproject.org] On Behalf Of Denys Dmytriyenko
> >> Sent: Wednesday, September 25, 2013 2:50 PM
> >> To: meta-ti@yoctoproject.org
> >> Cc: Rini, Tom
> >> Subject: [meta-ti] [PATCHv2 4/4] ti43x, omap-a15: switch
> >> KERNEL_IMAGETYPE to zImage by default
> >>
> >> From: Denys Dmytriyenko <denys@ti.com>
> >>
> >> * To be used with latest u-boot, that prefers zImage over uImage
> >> * ti33x will follow as soon as u-boot changes its default for that
> >> platform
> >>
> >> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> >> Cc: Tom Rini <trini@ti.com>
> >> ---
> >> v2 - don't change the default for ti33x yet, wait for u-boot
> >>
> >> conf/machine/include/omap-a15.inc | 2 +-
> >> conf/machine/include/ti43x.inc    | 2 +-
> >> 2 files changed, 2 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/conf/machine/include/omap-a15.inc
> >> b/conf/machine/include/omap-a15.inc
> >> index 623f2df..aba122a 100644
> >> --- a/conf/machine/include/omap-a15.inc
> >> +++ b/conf/machine/include/omap-a15.inc
> >> @@ -10,7 +10,7 @@ PREFERRED_PROVIDER_virtual/kernel = "linux-ti-
> >> staging"
> >> PREFERRED_PROVIDER_virtual/bootloader = "u-boot-ti-staging"
> >> PREFERRED_PROVIDER_u-boot = "u-boot-ti-staging"
> >>
> >> -KERNEL_IMAGETYPE = "uImage"
> >> +KERNEL_IMAGETYPE = "zImage"
> >>
> >> UBOOT_ARCH = "arm"
> >> UBOOT_ENTRYPOINT = "0x80008000"
> >> diff --git a/conf/machine/include/ti43x.inc
> >> b/conf/machine/include/ti43x.inc
> >> index 62da105..adb66e3 100644
> >> --- a/conf/machine/include/ti43x.inc
> >> +++ b/conf/machine/include/ti43x.inc
> >> @@ -23,7 +23,7 @@ PREFERRED_PROVIDER_virtual/kernel = "linux-ti-
> >> staging"
> >> PREFERRED_PROVIDER_virtual/bootloader = "u-boot-ti-staging"
> >> PREFERRED_PROVIDER_u-boot = "u-boot-ti-staging"
> >>
> >> -KERNEL_IMAGETYPE = "uImage"
> >> +KERNEL_IMAGETYPE = "zImage"
> >>
> >> UBOOT_ARCH = "arm"
> >> UBOOT_MACHINE = "am43xx_evm_config"
> >> --
> >> 1.8.3.2
> >>
> >> _______________________________________________
> >> meta-ti mailing list
> >> meta-ti@yoctoproject.org
> >> https://lists.yoctoproject.org/listinfo/meta-ti
> > _______________________________________________
> > meta-ti mailing list
> > meta-ti@yoctoproject.org
> > https://lists.yoctoproject.org/listinfo/meta-ti
> > 
> 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti