Hi Bruce,

On Wed, 2014-04-02 at 11:44 -0400, Bruce Ashfield wrote:
> On 14-04-02 11:11 AM, Stanacar, StefanX wrote:
> >
> >
> >
> > On Wed, 2014-04-02 at 00:19 -0700, Darren Hart wrote:
> >> CONFIG_X86_32 is a non-selectable configuration item, automatically set
> >> by ARCH and !CONFIG_64BIT. There are no users of the .cfg nor the .scc.
> >> Delete them.
> >>
> >> Signed-off-by: Darren Hart <dvh...@linux.intel.com>
> >> ---
> >>   meta/cfg/kernel-cache/cfg/x32.cfg |    1 -
> >>   meta/cfg/kernel-cache/cfg/x32.scc |    4 ----
> >>   2 files changed, 5 deletions(-)
> >>   delete mode 100644 meta/cfg/kernel-cache/cfg/x32.cfg
> >>   delete mode 100644 meta/cfg/kernel-cache/cfg/x32.scc
> >>
> >
> > Does this mean oe-core will need an update on linux-yocto*.bb which has:
> >
> > KERNEL_FEATURES_append = " ${@bb.utils.contains("TUNE_FEATURES", "mx32",
> > " cfg/x32.scc", "" ,d)}"
> 
> Yes it does. I even grepped/checked for this explicitly and missed it.
> To top it off, I pushed the change by mistake (long story) this morning.
> 
> So if we update SRCREVs, this breaks the build for 3.14 based kernels.
> 
> We could keep an empty .scc file around to keep the interface consistent
> (and comment in the .scc file why it is empty), or do the updates.
> 
> preferences ?
> 

I think we need to revert this commit, before updating the SRCREVs. See
my other reply and Darren's reply. He wanted to remove X86_32 not
X86_X32. If we remove this config we won't be able (by default) to build
images for the x86-64-32 tune. (and nightly-x32 on the AB will get red
and I know someone who doesn't like that :))

Cheers,
Stefan


> Bruce
> 
> >
> > ?
> >
> > Cheers,
> > Stefan
> >
> >
> >> diff --git a/meta/cfg/kernel-cache/cfg/x32.cfg 
> >> b/meta/cfg/kernel-cache/cfg/x32.cfg
> >> deleted file mode 100644
> >> index 725e05f..0000000
> >> --- a/meta/cfg/kernel-cache/cfg/x32.cfg
> >> +++ /dev/null
> >> @@ -1 +0,0 @@
> >> -CONFIG_X86_X32=y
> >> diff --git a/meta/cfg/kernel-cache/cfg/x32.scc 
> >> b/meta/cfg/kernel-cache/cfg/x32.scc
> >> deleted file mode 100644
> >> index e4b954a..0000000
> >> --- a/meta/cfg/kernel-cache/cfg/x32.scc
> >> +++ /dev/null
> >> @@ -1,4 +0,0 @@
> >> -define KFEATURE_DESCRIPTION "x86 x32 support"
> >> -define KFEATURE_COMPATIBILITY arch
> >> -
> >> -kconf non-hardware x32.cfg
> >> --
> >> 1.7.9.5
> >>
> >
> 

-- 
_______________________________________________
linux-yocto mailing list
linux-yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/linux-yocto

Reply via email to