* [PATCH 0/2] Add GLSDK Branding
@ 2013-06-26 11:03 Siddharth Heroor
2013-06-26 11:03 ` [PATCH 1/2] meta-arago-distro: recipes-kernel - Overlay Arago info Siddharth Heroor
2013-06-26 11:03 ` [PATCH 2/2] glsdk: add the initial GLSDK branding policy Siddharth Heroor
0 siblings, 2 replies; 12+ messages in thread
From: Siddharth Heroor @ 2013-06-26 11:03 UTC (permalink / raw)
To: meta-arago
* Create a GLSDK Branding configuration.
* Overlay arago information for the GLSDK kernel
Siddharth Heroor (2):
meta-arago-distro: recipes-kernel - Overlay Arago info
glsdk: add the initial GLSDK branding policy
.../conf/distro/include/branding-glsdk.inc | 4 ++++
.../linux/linux-ti-glsdk_3.8.bbappend | 17 +++++++++++++++++
2 files changed, 21 insertions(+), 0 deletions(-)
create mode 100644 meta-arago-distro/conf/distro/include/branding-glsdk.inc
create mode 100644 meta-arago-distro/recipes-kernel/linux/linux-ti-glsdk_3.8.bbappend
cc: Mrinmayee Hingolikar <mrinmayee@ti.com>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 1/2] meta-arago-distro: recipes-kernel - Overlay Arago info
2013-06-26 11:03 [PATCH 0/2] Add GLSDK Branding Siddharth Heroor
@ 2013-06-26 11:03 ` Siddharth Heroor
2013-06-26 12:53 ` Maupin, Chase
2013-06-26 11:03 ` [PATCH 2/2] glsdk: add the initial GLSDK branding policy Siddharth Heroor
1 sibling, 1 reply; 12+ messages in thread
From: Siddharth Heroor @ 2013-06-26 11:03 UTC (permalink / raw)
To: meta-arago
* Based of linux-ti-staging_3.8.bbappend
Signed-off-by: Siddharth Heroor <heroor@ti.com>
cc: Mrinmayee Hingolikar <mrinmayee@ti.com>
---
.../linux/linux-ti-glsdk_3.8.bbappend | 17 +++++++++++++++++
1 files changed, 17 insertions(+), 0 deletions(-)
create mode 100644 meta-arago-distro/recipes-kernel/linux/linux-ti-glsdk_3.8.bbappend
diff --git a/meta-arago-distro/recipes-kernel/linux/linux-ti-glsdk_3.8.bbappend b/meta-arago-distro/recipes-kernel/linux/linux-ti-glsdk_3.8.bbappend
new file mode 100644
index 0000000..b20c63a
--- /dev/null
+++ b/meta-arago-distro/recipes-kernel/linux/linux-ti-glsdk_3.8.bbappend
@@ -0,0 +1,17 @@
+MACHINE_KERNEL_PR_append = "-arago6"
+
+FILESEXTRAPATHS_prepend := "${THISDIR}/${P}:"
+
+require copy-defconfig.inc
+
+KERNEL_LOCALVERSION = "-g${@d.getVar('SRCPV', True).partition('+')[2][0:7]}"
+
+# The below lines are overlayed until the LOCALVERSION change is merged into
+# the oe-core danny branch. Update: not coming to danny, keep until dylan.
+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.7.0.4
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/2] glsdk: add the initial GLSDK branding policy
2013-06-26 11:03 [PATCH 0/2] Add GLSDK Branding Siddharth Heroor
2013-06-26 11:03 ` [PATCH 1/2] meta-arago-distro: recipes-kernel - Overlay Arago info Siddharth Heroor
@ 2013-06-26 11:03 ` Siddharth Heroor
2013-06-26 15:31 ` Cooper Jr., Franklin
1 sibling, 1 reply; 12+ messages in thread
From: Siddharth Heroor @ 2013-06-26 11:03 UTC (permalink / raw)
To: meta-arago
* Choose ti-linux-glsdk as the preferred provider
Signed-off-by: Siddharth Heroor <heroor@ti.com>
cc: Mrinmayee Hingolikar <mrinmayee@ti.com>
---
.../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"
--
1.7.0.4
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] meta-arago-distro: recipes-kernel - Overlay Arago info
2013-06-26 11:03 ` [PATCH 1/2] meta-arago-distro: recipes-kernel - Overlay Arago info Siddharth Heroor
@ 2013-06-26 12:53 ` Maupin, Chase
2013-06-26 13:48 ` Denys Dmytriyenko
0 siblings, 1 reply; 12+ messages in thread
From: Maupin, Chase @ 2013-06-26 12:53 UTC (permalink / raw)
To: Heroor, Siddharth, meta-arago
>-----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 1/2] meta-arago-distro: recipes-
>kernel - Overlay Arago info
>
>* Based of linux-ti-staging_3.8.bbappend
>
>Signed-off-by: Siddharth Heroor <heroor@ti.com>
>cc: Mrinmayee Hingolikar <mrinmayee@ti.com>
>---
> .../linux/linux-ti-glsdk_3.8.bbappend | 17
>+++++++++++++++++
> 1 files changed, 17 insertions(+), 0 deletions(-)
> create mode 100644 meta-arago-distro/recipes-kernel/linux/linux-
>ti-glsdk_3.8.bbappend
>
>diff --git a/meta-arago-distro/recipes-kernel/linux/linux-ti-
>glsdk_3.8.bbappend b/meta-arago-distro/recipes-kernel/linux/linux-
>ti-glsdk_3.8.bbappend
>new file mode 100644
>index 0000000..b20c63a
>--- /dev/null
>+++ b/meta-arago-distro/recipes-kernel/linux/linux-ti-
>glsdk_3.8.bbappend
>@@ -0,0 +1,17 @@
>+MACHINE_KERNEL_PR_append = "-arago6"
Maybe I'm missing something, but I didn't remember seeing the patch that added linux-ti-glsdk recipe to be appended in the first place.
Also, why start with arago6?
>+
>+FILESEXTRAPATHS_prepend := "${THISDIR}/${P}:"
>+
>+require copy-defconfig.inc
>+
>+KERNEL_LOCALVERSION = "-g${@d.getVar('SRCPV',
>True).partition('+')[2][0:7]}"
>+
>+# The below lines are overlayed until the LOCALVERSION change is
>merged into
>+# the oe-core danny branch. Update: not coming to danny, keep
>until dylan.
>+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.7.0.4
>
>_______________________________________________
>meta-arago mailing list
>meta-arago@arago-project.org
>http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] meta-arago-distro: recipes-kernel - Overlay Arago info
2013-06-26 12:53 ` Maupin, Chase
@ 2013-06-26 13:48 ` Denys Dmytriyenko
2013-06-26 13:56 ` Denys Dmytriyenko
0 siblings, 1 reply; 12+ messages in thread
From: Denys Dmytriyenko @ 2013-06-26 13:48 UTC (permalink / raw)
To: Maupin, Chase; +Cc: meta-arago
On Wed, Jun 26, 2013 at 12:53:51PM +0000, Maupin, Chase wrote:
> >-----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 1/2] meta-arago-distro: recipes-
> >kernel - Overlay Arago info
> >
> >* Based of linux-ti-staging_3.8.bbappend
> >
> >Signed-off-by: Siddharth Heroor <heroor@ti.com>
> >cc: Mrinmayee Hingolikar <mrinmayee@ti.com>
> >---
> > .../linux/linux-ti-glsdk_3.8.bbappend | 17
> >+++++++++++++++++
> > 1 files changed, 17 insertions(+), 0 deletions(-)
> > create mode 100644 meta-arago-distro/recipes-kernel/linux/linux-
> >ti-glsdk_3.8.bbappend
> >
> >diff --git a/meta-arago-distro/recipes-kernel/linux/linux-ti-
> >glsdk_3.8.bbappend b/meta-arago-distro/recipes-kernel/linux/linux-
> >ti-glsdk_3.8.bbappend
> >new file mode 100644
> >index 0000000..b20c63a
> >--- /dev/null
> >+++ b/meta-arago-distro/recipes-kernel/linux/linux-ti-
> >glsdk_3.8.bbappend
> >@@ -0,0 +1,17 @@
> >+MACHINE_KERNEL_PR_append = "-arago6"
>
> Maybe I'm missing something, but I didn't remember seeing the patch that
> added linux-ti-glsdk recipe to be appended in the first place.
Correct, a dangling bbappend will break the build.
> Also, why start with arago6?
>
> >+
> >+FILESEXTRAPATHS_prepend := "${THISDIR}/${P}:"
> >+
> >+require copy-defconfig.inc
> >+
> >+KERNEL_LOCALVERSION = "-g${@d.getVar('SRCPV',
> >True).partition('+')[2][0:7]}"
> >+
> >+# The below lines are overlayed until the LOCALVERSION change is
> >merged into
> >+# the oe-core danny branch. Update: not coming to danny, keep
> >until dylan.
> >+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.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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] meta-arago-distro: recipes-kernel - Overlay Arago info
2013-06-26 13:48 ` Denys Dmytriyenko
@ 2013-06-26 13:56 ` Denys Dmytriyenko
0 siblings, 0 replies; 12+ messages in thread
From: Denys Dmytriyenko @ 2013-06-26 13:56 UTC (permalink / raw)
To: Maupin, Chase; +Cc: meta-arago
On Wed, Jun 26, 2013 at 09:48:08AM -0400, Denys Dmytriyenko wrote:
> On Wed, Jun 26, 2013 at 12:53:51PM +0000, Maupin, Chase wrote:
> > >-----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 1/2] meta-arago-distro: recipes-
> > >kernel - Overlay Arago info
> > >
> > >* Based of linux-ti-staging_3.8.bbappend
Now, reading this comment also makes me think that it could be the case of
mis-understanding how bbappend works...
The name of bbappend should match the name of the recipe that is being
altered. You cannot have a linux-ti-glsdk.bbappend trying to amend
linux-ti-staging.bb and you would have to provide your own linux-ti-glsdk.bb
recipe.
> > >Signed-off-by: Siddharth Heroor <heroor@ti.com>
> > >cc: Mrinmayee Hingolikar <mrinmayee@ti.com>
> > >---
> > > .../linux/linux-ti-glsdk_3.8.bbappend | 17
> > >+++++++++++++++++
> > > 1 files changed, 17 insertions(+), 0 deletions(-)
> > > create mode 100644 meta-arago-distro/recipes-kernel/linux/linux-
> > >ti-glsdk_3.8.bbappend
> > >
> > >diff --git a/meta-arago-distro/recipes-kernel/linux/linux-ti-
> > >glsdk_3.8.bbappend b/meta-arago-distro/recipes-kernel/linux/linux-
> > >ti-glsdk_3.8.bbappend
> > >new file mode 100644
> > >index 0000000..b20c63a
> > >--- /dev/null
> > >+++ b/meta-arago-distro/recipes-kernel/linux/linux-ti-
> > >glsdk_3.8.bbappend
> > >@@ -0,0 +1,17 @@
> > >+MACHINE_KERNEL_PR_append = "-arago6"
> >
> > Maybe I'm missing something, but I didn't remember seeing the patch that
> > added linux-ti-glsdk recipe to be appended in the first place.
>
> Correct, a dangling bbappend will break the build.
>
>
> > Also, why start with arago6?
> >
> > >+
> > >+FILESEXTRAPATHS_prepend := "${THISDIR}/${P}:"
> > >+
> > >+require copy-defconfig.inc
> > >+
> > >+KERNEL_LOCALVERSION = "-g${@d.getVar('SRCPV',
> > >True).partition('+')[2][0:7]}"
> > >+
> > >+# The below lines are overlayed until the LOCALVERSION change is
> > >merged into
> > >+# the oe-core danny branch. Update: not coming to danny, keep
> > >until dylan.
> > >+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.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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] glsdk: add the initial GLSDK branding policy
2013-06-26 11:03 ` [PATCH 2/2] glsdk: add the initial GLSDK branding policy Siddharth Heroor
@ 2013-06-26 15:31 ` Cooper Jr., Franklin
2013-06-26 16:11 ` Maupin, Chase
0 siblings, 1 reply; 12+ messages in thread
From: Cooper Jr., Franklin @ 2013-06-26 15:31 UTC (permalink / raw)
To: Heroor, Siddharth, meta-arago
> -----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 <heroor@ti.com>
> cc: Mrinmayee Hingolikar <mrinmayee@ti.com>
> ---
> .../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.
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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] glsdk: add the initial GLSDK branding policy
2013-06-26 15:31 ` Cooper Jr., Franklin
@ 2013-06-26 16:11 ` Maupin, Chase
2013-06-26 16:16 ` Denys Dmytriyenko
0 siblings, 1 reply; 12+ messages in thread
From: Maupin, Chase @ 2013-06-26 16:11 UTC (permalink / raw)
To: Cooper Jr., Franklin, Heroor, Siddharth, meta-arago
>-----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 <heroor@ti.com>
>> cc: Mrinmayee Hingolikar <mrinmayee@ti.com>
>> ---
>> .../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.
>
>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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] glsdk: add the initial GLSDK branding policy
2013-06-26 16:11 ` Maupin, Chase
@ 2013-06-26 16:16 ` Denys Dmytriyenko
2013-06-26 16:20 ` Maupin, Chase
0 siblings, 1 reply; 12+ messages in thread
From: Denys Dmytriyenko @ 2013-06-26 16:16 UTC (permalink / raw)
To: Maupin, Chase; +Cc: meta-arago, Cooper Jr., Franklin
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 <heroor@ti.com>
> >> cc: Mrinmayee Hingolikar <mrinmayee@ti.com>
> >> ---
> >> .../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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] glsdk: add the initial GLSDK branding policy
2013-06-26 16:16 ` Denys Dmytriyenko
@ 2013-06-26 16:20 ` Maupin, Chase
2013-06-26 18:04 ` Cooper Jr., Franklin
0 siblings, 1 reply; 12+ messages in thread
From: Maupin, Chase @ 2013-06-26 16:20 UTC (permalink / raw)
To: Dmytriyenko, Denys; +Cc: meta-arago, Cooper Jr., Franklin
>-----Original Message-----
>From: Dmytriyenko, Denys
>Sent: Wednesday, June 26, 2013 11:17 AM
>To: Maupin, Chase
>Cc: Cooper Jr., Franklin; Heroor, Siddharth; meta-arago@arago-
>project.org
>Subject: Re: [meta-arago] [PATCH 2/2] glsdk: add the initial GLSDK
>branding policy
>
>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 <heroor@ti.com>
>> >> cc: Mrinmayee Hingolikar <mrinmayee@ti.com>
>> >> ---
>> >> .../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.
>
+1
>
>> >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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] glsdk: add the initial GLSDK branding policy
2013-06-26 16:20 ` Maupin, Chase
@ 2013-06-26 18:04 ` Cooper Jr., Franklin
2013-06-27 3:31 ` Siddharth Heroor
0 siblings, 1 reply; 12+ messages in thread
From: Cooper Jr., Franklin @ 2013-06-26 18:04 UTC (permalink / raw)
To: Maupin, Chase, Dmytriyenko, Denys; +Cc: meta-arago
> -----Original Message-----
> From: Maupin, Chase
> Sent: Wednesday, June 26, 2013 11:21 AM
> To: Dmytriyenko, Denys
> Cc: Cooper Jr., Franklin; Heroor, Siddharth; meta-arago@arago-project.org
> Subject: RE: [meta-arago] [PATCH 2/2] glsdk: add the initial GLSDK branding
> policy
>
> >-----Original Message-----
> >From: Dmytriyenko, Denys
> >Sent: Wednesday, June 26, 2013 11:17 AM
> >To: Maupin, Chase
> >Cc: Cooper Jr., Franklin; Heroor, Siddharth; meta-arago@arago-
> >project.org
> >Subject: Re: [meta-arago] [PATCH 2/2] glsdk: add the initial GLSDK
> >branding policy
> >
> >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 <heroor@ti.com>
> >> >> cc: Mrinmayee Hingolikar <mrinmayee@ti.com>
> >> >> ---
> >> >> .../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.
> >
>
> +1
I'll wait to see what this kernel recipe is since that is the biggest point of confusion.
>
> >
> >> >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
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] glsdk: add the initial GLSDK branding policy
2013-06-26 18:04 ` Cooper Jr., Franklin
@ 2013-06-27 3:31 ` Siddharth Heroor
0 siblings, 0 replies; 12+ messages in thread
From: Siddharth Heroor @ 2013-06-27 3:31 UTC (permalink / raw)
To: Cooper Jr., Franklin; +Cc: meta-arago
On 6/26/2013 11:34 PM, Cooper Jr., Franklin wrote:
>
>
>> -----Original Message-----
>> From: Maupin, Chase
>> Sent: Wednesday, June 26, 2013 11:21 AM
>> To: Dmytriyenko, Denys
>> Cc: Cooper Jr., Franklin; Heroor, Siddharth; meta-arago@arago-project.org
>> Subject: RE: [meta-arago] [PATCH 2/2] glsdk: add the initial GLSDK branding
>> policy
>>
>>> -----Original Message-----
>>> From: Dmytriyenko, Denys
>>> Sent: Wednesday, June 26, 2013 11:17 AM
>>> To: Maupin, Chase
>>> Cc: Cooper Jr., Franklin; Heroor, Siddharth; meta-arago@arago-
>>> project.org
>>> Subject: Re: [meta-arago] [PATCH 2/2] glsdk: add the initial GLSDK
>>> branding policy
>>>
>>> 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 <heroor@ti.com>
>>>>>> cc: Mrinmayee Hingolikar <mrinmayee@ti.com>
>>>>>> ---
>>>>>> .../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.
>>>
>>
>> +1
> I'll wait to see what this kernel recipe is since that is the biggest point of confusion.
I actually sent the kernel recipe to meta-ti up ahead, but it seems to
have been "lost" i.e never delivered. Looks like meta-ti or TI's SMTP
server doesn't like me sending patches to meta-ti :-).
I'm working to get that sorted out and I agree we need to get the
meta-ti kernel patch in place first.
>
>>
>>>
>>>>> 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
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2013-06-27 3:31 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-26 11:03 [PATCH 0/2] Add GLSDK Branding Siddharth Heroor
2013-06-26 11:03 ` [PATCH 1/2] meta-arago-distro: recipes-kernel - Overlay Arago info Siddharth Heroor
2013-06-26 12:53 ` Maupin, Chase
2013-06-26 13:48 ` Denys Dmytriyenko
2013-06-26 13:56 ` Denys Dmytriyenko
2013-06-26 11:03 ` [PATCH 2/2] glsdk: add the initial GLSDK branding policy Siddharth Heroor
2013-06-26 15:31 ` Cooper Jr., Franklin
2013-06-26 16:11 ` Maupin, Chase
2013-06-26 16:16 ` Denys Dmytriyenko
2013-06-26 16:20 ` Maupin, Chase
2013-06-26 18:04 ` Cooper Jr., Franklin
2013-06-27 3:31 ` Siddharth Heroor
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.