> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Monday, August 26, 2013 9:48 AM
> To: Cooper Jr., Franklin
> Cc: [email protected]
> Subject: Re: [meta-arago] [PATCHv2 3/4] linux-ti-staging: enable LOCALVERSION
> and AUTOREV
>
> On Fri, Aug 23, 2013 at 09:54:44PM +0000, Cooper Jr., Franklin wrote:
> >
> >
> > > -----Original Message-----
> > > From: Dmytriyenko, Denys
> > > Sent: Friday, August 23, 2013 4:15 PM
> > > To: Cooper Jr., Franklin
> > > Cc: Denys Dmytriyenko; [email protected]
> > > Subject: Re: [meta-arago] [PATCHv2 3/4] linux-ti-staging: enable
> > > LOCALVERSION and AUTOREV
> > >
> > > On Fri, Aug 23, 2013 at 09:08:06PM +0000, Cooper Jr., Franklin wrote:
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: [email protected] [mailto:meta-arago-
> > > > > [email protected]] On Behalf Of Denys Dmytriyenko
> > > > > Sent: Friday, August 23, 2013 4:04 PM
> > > > > To: [email protected]
> > > > > Subject: [meta-arago] [PATCHv2 3/4] linux-ti-staging: enable
> > > > > LOCALVERSION and AUTOREV
> > > > >
> > > > > From: Denys Dmytriyenko <[email protected]>
> > > > >
> > > > > Signed-off-by: Denys Dmytriyenko <[email protected]>
> > > > > ---
> > > > > v2 - removed the comment about upstreaming LOCALVERSION, may not
> > > > > happen
> > > > >
> > > > > .../recipes-kernel/linux/linux-ti-staging_git.bbappend | 17
> > > > > +++++++++++++++++
> > > > > 1 file changed, 17 insertions(+) create mode 100644
> > > > > meta-arago-distro/recipes-kernel/linux/linux-ti-
> > > > > staging_git.bbappend
> > > > >
> > > > > diff --git
> > > > > a/meta-arago-distro/recipes-kernel/linux/linux-ti-staging_git.bb
> > > > > appe
> > > > > nd
> > > > > b/meta-arago-distro/recipes-kernel/linux/linux-ti-staging_git.bb
> > > > > appe
> > > > > nd
> > > > > new file mode 100644
> > > > > index 0000000..f99482c
> > > > > --- /dev/null
> > > > > +++ b/meta-arago-distro/recipes-kernel/linux/linux-ti-staging_gi
> > > > > +++ t.bb
> > > > > +++ append
> > > > > @@ -0,0 +1,17 @@
> > > > > +MACHINE_KERNEL_PR_append = "-arago0"
> > > > > +
> > > > > +FILESEXTRAPATHS_prepend := "${THISDIR}/${P}:"
> > > > > +
> > > > > +require copy-defconfig.inc
> > >
> > > > [Franklin] Are we really planning on have different defconfigs for
> > > > each machine? I thought with DT all machines will share the same
> defconfig.
> > >
> > > Yes, linux-ti-staging uses a blank defconfig that points to
> omap2plus_defconfig.
> > >
> > > But copy-defconfig just saves the current config for SDK use - they
> > > will be the same for machines using this recipe. But still, there
> > > are currently only 2 OMAP machines supported by this kernel and will
> > > be 2 Sitara machines with this patchset. If/when we switch remaining
> > > SDK machines to use the latest kernel, we can revisit this matter, I
> > > guess.
> > [Franklin]
> > I guess my point was that copy-defconfig.inc renames the .config to
> > tisdk_${MACHINE}_defconfig. Can we set CONFIG_NAME to
> "omap2plus_defconfig"
> > since we should stick with that defconfig name for all machines?
>
> And what I was trying to say before, is that "all machines" is currently only
> 2 out
> of several and even then, omap2plus_defconfig in the kernel may not match
> exactly the one produced by the SDK - there may be some mangling involved to
> adjust some config options to sane defaults (although we no longer include
> linux.inc with that code), plus we run "make oldconfig" on it, which checks
> dependecies between config options and may change some os them. In other
> words: omap2plus_defconfig != .config which we package in the SDK. It is like
> the difference between pristine sources and sources that were patched and
> configured...
[Franklin] I was hoping we would get rid of all the tweaks to the defconfig so
that omap2plus_defconfig == .config . This would mean we push all modifications
we needed to mainline. Similar to my response to Chase if that isn't the plan
then I have no issue with the approach.
>
> --
> Denys
>
>
> > > > > +SRCREV = "${AUTOREV}"
> > > > > +
> > > > > +KERNEL_LOCALVERSION = "-g${@d.getVar('SRCPV',
> > > > > True).partition('+')[2][0:7]}"
> > > > > +
> > > > > +kernel_do_configure_prepend() {
> > > > > + if [ ! -e ${B}/.scmversion -a ! -e ${S}/.scmversion ]
> > > > > + then
> > > > > + echo ${KERNEL_LOCALVERSION} > ${B}/.scmversion
> > > > > + echo ${KERNEL_LOCALVERSION} > ${S}/.scmversion
> > > > > + fi
> > > > > +}
> > > > > --
> > > > > 1.8.3.2
> > > > >
> > > > > _______________________________________________
> > > > > meta-arago mailing list
> > > > > [email protected]
> > > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > > > _______________________________________________
> > > > meta-arago mailing list
> > > > [email protected]
> > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> >
_______________________________________________
meta-arago mailing list
[email protected]
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
