From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from arroyo.ext.ti.com (arroyo.ext.ti.com [192.94.94.40]) by arago-project.org (Postfix) with ESMTPS id 6FC2D52AB0 for ; Wed, 26 Jun 2013 16:16:43 +0000 (UTC) Received: from dlelxv90.itg.ti.com ([172.17.2.17]) by arroyo.ext.ti.com (8.13.7/8.13.7) with ESMTP id r5QGGgrN025177 for ; Wed, 26 Jun 2013 11:16:42 -0500 Received: from DFLE72.ent.ti.com (dfle72.ent.ti.com [128.247.5.109]) by dlelxv90.itg.ti.com (8.14.3/8.13.8) with ESMTP id r5QGGgOe006909 for ; Wed, 26 Jun 2013 11:16:42 -0500 Received: from dlelxv22.itg.ti.com (172.17.1.197) by DFLE72.ent.ti.com (128.247.5.109) with Microsoft SMTP Server id 14.2.342.3; Wed, 26 Jun 2013 11:16:42 -0500 Received: from localhost (gtudedge.gt.design.ti.com [158.218.102.158]) by dlelxv22.itg.ti.com (8.13.8/8.13.8) with ESMTP id r5QGGgTF021878; Wed, 26 Jun 2013 11:16:42 -0500 Date: Wed, 26 Jun 2013 12:16:42 -0400 From: Denys Dmytriyenko To: "Maupin, Chase" Message-ID: <20130626161641.GB28379@edge> References: <1372244597-11600-1-git-send-email-heroor@ti.com> <1372244597-11600-3-git-send-email-heroor@ti.com> <8F29D6B095ED194EA1980491A5E029710C4DE84B@DFLE08.ent.ti.com> <7D46E86EC0A8354091174257B2FED10159618630@DLEE11.ent.ti.com> MIME-Version: 1.0 In-Reply-To: <7D46E86EC0A8354091174257B2FED10159618630@DLEE11.ent.ti.com> User-Agent: Mutt/1.5.20 (2009-06-14) Cc: "meta-arago@arago-project.org" , "Cooper Jr., Franklin" Subject: Re: [PATCH 2/2] glsdk: add the initial GLSDK branding policy X-BeenThere: meta-arago@arago-project.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Arago metadata layer for TI SDKs - OE-Core/Yocto compatible List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 26 Jun 2013 16:16:43 -0000 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline On Wed, Jun 26, 2013 at 04:11:52PM +0000, Maupin, Chase wrote: > >-----Original Message----- > >From: meta-arago-bounces@arago-project.org [mailto:meta-arago- > >bounces@arago-project.org] On Behalf Of Cooper Jr., Franklin > >Sent: Wednesday, June 26, 2013 10:31 AM > >To: Heroor, Siddharth; meta-arago@arago-project.org > >Subject: Re: [meta-arago] [PATCH 2/2] glsdk: add the initial GLSDK > >branding policy > > > >> -----Original Message----- > >> From: meta-arago-bounces@arago-project.org [mailto:meta-arago- > >> bounces@arago-project.org] On Behalf Of Heroor, Siddharth > >> Sent: Wednesday, June 26, 2013 6:03 AM > >> To: meta-arago@arago-project.org > >> Subject: [meta-arago] [PATCH 2/2] glsdk: add the initial GLSDK > >branding policy > >> > >> * Choose ti-linux-glsdk as the preferred provider > >> > >> Signed-off-by: Siddharth Heroor > >> cc: Mrinmayee Hingolikar > >> --- > >> .../conf/distro/include/branding-glsdk.inc | 4 ++++ > >> 1 files changed, 4 insertions(+), 0 deletions(-) create mode > >100644 meta- > >> arago-distro/conf/distro/include/branding-glsdk.inc > >> > >> diff --git a/meta-arago-distro/conf/distro/include/branding- > >glsdk.inc b/meta- > >> arago-distro/conf/distro/include/branding-glsdk.inc > >> new file mode 100644 > >> index 0000000..d1c30f3 > >> --- /dev/null > >> +++ b/meta-arago-distro/conf/distro/include/branding-glsdk.inc > >> @@ -0,0 +1,4 @@ > >> +# GLSDK branding configuration > >> + > >> +# Use the SDK supported kernel and kernel version > >> +PREFERRED_PROVIDER_virtual/kernel = "linux-ti-glsdk" > > > >I am not familiar with the GLSDK but I am not sure why you need > >your own branding include file. This patch shows you want to > >change the kernel but this really should be done in meta-ti within > >each of the machine configuration files. I will let Denys comment > >but if you really can't put this within meta-ti then you should at > >least make this statement machine or at least SOC specific. > > > >Branding-amsdk.inc only exist due to the mainline effort for > >am335x. Currently the Sitara team supports am335x on the 3.2 > >kernel while the kernel team is working on getting am335x in > >mainline. So satisfy both kernels the branding include file was > >created. > > Another example would be if you want to use QT5 instead of QT4 to set > preferred versions and providers for your SDK. So, how this is different? Until we see the actual kernel recipe, I'm inclined to believe this patch is valid and correct. Especially if it's altering one of our existing machines and kernel recipes specifically for GLSDK. That is what branding was created for in the first place. > >Unless your team's work falls in a similar situation there isn't a > >need to create this file. I personally don't like the branding > >include file and plan on getting rid of the amsdk file once we > >release a mainline am335x SDK. > > > >Also I don't see a recipe for linux-ti-glsdk within meta-ti. > > > >> -- > >> 1.7.0.4 > >> > >> _______________________________________________ > >> meta-arago mailing list > >> meta-arago@arago-project.org > >> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago > >_______________________________________________ > >meta-arago mailing list > >meta-arago@arago-project.org > >http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago > _______________________________________________ > meta-arago mailing list > meta-arago@arago-project.org > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago