> -----Original Message-----
> From: Maupin, Chase
> Sent: Monday, August 26, 2013 8:11 AM
> To: Cooper Jr., Franklin; Dmytriyenko, Denys
> Cc: [email protected]
> Subject: RE: [meta-arago] [PATCHv2 3/4] linux-ti-staging: enable LOCALVERSION
> and AUTOREV
>
> >-----Original Message-----
> >From: [email protected] [mailto:meta-arago-
> >[email protected]] On Behalf Of Cooper Jr., Franklin
> >Sent: Friday, August 23, 2013 4:55 PM
> >To: Dmytriyenko, Denys
> >Cc: [email protected]
> >Subject: Re: [meta-arago] [PATCHv2 3/4] linux-ti-staging: enable
> >LOCALVERSION and AUTOREV
> >
> >
> >
> >> -----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.bbappe
> >> > > nd
> >> > > b/meta-arago-distro/recipes-kernel/linux/linux-ti-
> >staging_git.bbappe
> >> > > nd
> >> > > new file mode 100644
> >> > > index 0000000..f99482c
> >> > > --- /dev/null
> >> > > +++ b/meta-arago-distro/recipes-kernel/linux/linux-ti-
> >staging_git.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?
>
> Franklin this would have some of the following impacts:
>
> 1. You would need to update all your documentation 2. If you ever do decide
> you need a tweak for individual machines in the defconfig such as having
> something turned on for other machines that you do not want turned on for a
> particular machine due to size or boot time impacts, you will end up needing
> this
> functionality back.
> 3. You will probably get some questions about why AMxxx is using an
> "omap2plus" configuration.
> 4. I'm not sure how much of the kernel config magic is still used in oe-core,
> where they would modify the config with certain options (and this is probably
> bypassed right now) but again, those kinds of changes would be lost.
>
> Currently you can just say find the config matching a simple pattern and
> people
> know exactly what you built with.
[Franklin]
Ok. I wasn't sure if we were going to put in the effort to unify defconfigs and
deal with the tradeoffs. But if that's not the case then I have no issue.
>
> >>
> >>
> >> > > +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
_______________________________________________
meta-arago mailing list
[email protected]
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
