All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] linux-keystone: Create recipe for linux-keystone kernel version 3.10
@ 2014-01-31  5:23 Sam Nelson
  2014-01-31  7:06 ` Denys Dmytriyenko
  0 siblings, 1 reply; 4+ messages in thread
From: Sam Nelson @ 2014-01-31  5:23 UTC (permalink / raw)
  To: meta-ti

- Updated Source URL & version
- Added device tree file

Signed-off-by: Sam Nelson <sam.nelson@ti.com>

-----
Changes from last patch
Renamed recipe from 3.10--> 3.10.10
Removed directory path from dtb file
Removed do_deploy_append
----
---
 .../defconfig                                      |    0
 ...keystone_3.8.4.bb => linux-keystone_3.10.10.bb} |   25 +++++++++++---------
 2 files changed, 14 insertions(+), 11 deletions(-)
 rename recipes-kernel/linux/{linux-keystone-3.8.4 => linux-keystone-3.10.10}/defconfig (100%)
 rename recipes-kernel/linux/{linux-keystone_3.8.4.bb => linux-keystone_3.10.10.bb} (50%)

diff --git a/recipes-kernel/linux/linux-keystone-3.8.4/defconfig b/recipes-kernel/linux/linux-keystone-3.10.10/defconfig
similarity index 100%
rename from recipes-kernel/linux/linux-keystone-3.8.4/defconfig
rename to recipes-kernel/linux/linux-keystone-3.10.10/defconfig
diff --git a/recipes-kernel/linux/linux-keystone_3.8.4.bb b/recipes-kernel/linux/linux-keystone_3.10.10.bb
similarity index 50%
rename from recipes-kernel/linux/linux-keystone_3.8.4.bb
rename to recipes-kernel/linux/linux-keystone_3.10.10.bb
index d176f14..a5c113e 100644
--- a/recipes-kernel/linux/linux-keystone_3.8.4.bb
+++ b/recipes-kernel/linux/linux-keystone_3.10.10.bb
@@ -3,26 +3,29 @@ DESCRIPTION = "Linux kernel for TI Keystone devices"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=d7810fab7487fb0aad327b76f1be7cd7"
 
+COMPATIBLE_MACHINE = "keystone"
+
 inherit kernel
 
 require recipes-kernel/linux/linux-dtb.inc
 require recipes-kernel/linux/setup-defconfig.inc
 
+MACHINE_KERNEL_PR_append = "a"
+
+CORTEXA8FIXUP = "no"
+
 KERNEL_DEVICETREE_keystone-evm = "k2hk-evm.dtb"
 
-COMPATIBLE_MACHINE = "keystone"
+#This commit corresponds to "K2_LINUX_03.10.10_13.12"
+SRCREV = "335d14b1c2ce23585835243126c6acba89067ecf"
 
-S = "${WORKDIR}/git"
+BRANCH = "master"
 
-BRANCH = "sept-2013/master"
+SRC_URI = "git://git.ti.com/keystone-linux/linux.git;protocol=git;branch=${BRANCH}\
+           file://defconfig\
+           "
 
-# This commit corresponds to K2_LINUX_03.08.04_13.09 tag
-SRCREV = "20a48dc5309c4434650baef2b6072b5f1016f946"
+S = "${WORKDIR}/git"
 
-# Append to the MACHINE_KERNEL_PR so that a new SRCREV will cause a rebuild
-MACHINE_KERNEL_PR_append = "a+gitr${SRCPV}"
-PR = "${MACHINE_KERNEL_PR}"
+RDEPENDS_kernel-base = ""
 
-SRC_URI = "git://arago-project.org/git/projects/linux-keystone.git;protocol=git;branch=${BRANCH}\
-           file://defconfig\
-          "
-- 
1.7.9.5



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH v2] linux-keystone: Create recipe for linux-keystone kernel version 3.10
  2014-01-31  5:23 [PATCH v2] linux-keystone: Create recipe for linux-keystone kernel version 3.10 Sam Nelson
@ 2014-01-31  7:06 ` Denys Dmytriyenko
  2014-01-31 14:51   ` Maupin, Chase
  2014-01-31 20:38   ` Nelson, Sam
  0 siblings, 2 replies; 4+ messages in thread
From: Denys Dmytriyenko @ 2014-01-31  7:06 UTC (permalink / raw)
  To: Sam Nelson; +Cc: meta-ti

Sam,

This is much better! And easier to compare against the previous version too. 
I'll take it as is, although there are couple very minor comments below...


On Fri, Jan 31, 2014 at 12:23:09AM -0500, Sam Nelson wrote:
> - Updated Source URL & version
> - Added device tree file
> 
> Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> 
> -----

Should be exactly "---" as the one below or instead of that, so it won't go 
into the commit message.


> Changes from last patch
> Renamed recipe from 3.10--> 3.10.10
> Removed directory path from dtb file
> Removed do_deploy_append
> ----
> ---
>  .../defconfig                                      |    0
>  ...keystone_3.8.4.bb => linux-keystone_3.10.10.bb} |   25 +++++++++++---------
>  2 files changed, 14 insertions(+), 11 deletions(-)
>  rename recipes-kernel/linux/{linux-keystone-3.8.4 => linux-keystone-3.10.10}/defconfig (100%)
>  rename recipes-kernel/linux/{linux-keystone_3.8.4.bb => linux-keystone_3.10.10.bb} (50%)
> 
> diff --git a/recipes-kernel/linux/linux-keystone-3.8.4/defconfig b/recipes-kernel/linux/linux-keystone-3.10.10/defconfig
> similarity index 100%
> rename from recipes-kernel/linux/linux-keystone-3.8.4/defconfig
> rename to recipes-kernel/linux/linux-keystone-3.10.10/defconfig
> diff --git a/recipes-kernel/linux/linux-keystone_3.8.4.bb b/recipes-kernel/linux/linux-keystone_3.10.10.bb
> similarity index 50%
> rename from recipes-kernel/linux/linux-keystone_3.8.4.bb
> rename to recipes-kernel/linux/linux-keystone_3.10.10.bb
> index d176f14..a5c113e 100644
> --- a/recipes-kernel/linux/linux-keystone_3.8.4.bb
> +++ b/recipes-kernel/linux/linux-keystone_3.10.10.bb
> @@ -3,26 +3,29 @@ DESCRIPTION = "Linux kernel for TI Keystone devices"
>  LICENSE = "GPLv2"
>  LIC_FILES_CHKSUM = "file://COPYING;md5=d7810fab7487fb0aad327b76f1be7cd7"
>  
> +COMPATIBLE_MACHINE = "keystone"
> +
>  inherit kernel
>  
>  require recipes-kernel/linux/linux-dtb.inc
>  require recipes-kernel/linux/setup-defconfig.inc
>  
> +MACHINE_KERNEL_PR_append = "a"

1. You might want to also add "+gitr${SRCPV}" so git commit ID gets into 
package version and file name. Useful when you have several builds from 
different points in git which are all 3.10.10 etc. But it's up to you.

2. Having PR = "${MACHINE_KERNEL_PR}" is also useful for others, when not 
using meta-openembedded layer and its machine_kernel_pr.bbclass...


> +
> +CORTEXA8FIXUP = "no"

I don't think this is needed any more...


>  KERNEL_DEVICETREE_keystone-evm = "k2hk-evm.dtb"
>  
> -COMPATIBLE_MACHINE = "keystone"
> +#This commit corresponds to "K2_LINUX_03.10.10_13.12"
> +SRCREV = "335d14b1c2ce23585835243126c6acba89067ecf"
>  
> -S = "${WORKDIR}/git"
> +BRANCH = "master"
>  
> -BRANCH = "sept-2013/master"
> +SRC_URI = "git://git.ti.com/keystone-linux/linux.git;protocol=git;branch=${BRANCH}\
> +           file://defconfig\
> +           "
>  
> -# This commit corresponds to K2_LINUX_03.08.04_13.09 tag
> -SRCREV = "20a48dc5309c4434650baef2b6072b5f1016f946"
> +S = "${WORKDIR}/git"
>  
> -# Append to the MACHINE_KERNEL_PR so that a new SRCREV will cause a rebuild
> -MACHINE_KERNEL_PR_append = "a+gitr${SRCPV}"
> -PR = "${MACHINE_KERNEL_PR}"
> +RDEPENDS_kernel-base = ""
>  
> -SRC_URI = "git://arago-project.org/git/projects/linux-keystone.git;protocol=git;branch=${BRANCH}\
> -           file://defconfig\
> -          "
> -- 
> 1.7.9.5
> 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v2] linux-keystone: Create recipe for linux-keystone kernel version 3.10
  2014-01-31  7:06 ` Denys Dmytriyenko
@ 2014-01-31 14:51   ` Maupin, Chase
  2014-01-31 20:38   ` Nelson, Sam
  1 sibling, 0 replies; 4+ messages in thread
From: Maupin, Chase @ 2014-01-31 14:51 UTC (permalink / raw)
  To: Dmytriyenko, Denys, Nelson, Sam; +Cc: meta-ti@yoctoproject.org

>-----Original Message-----
>From: meta-ti-bounces@yoctoproject.org [mailto:meta-ti-
>bounces@yoctoproject.org] On Behalf Of Dmytriyenko, Denys
>Sent: Friday, January 31, 2014 1:07 AM
>To: Nelson, Sam
>Cc: meta-ti@yoctoproject.org
>Subject: Re: [meta-ti] [PATCH v2] linux-keystone: Create recipe
>for linux-keystone kernel version 3.10
>
>Sam,
>
>This is much better! And easier to compare against the previous
>version too.
>I'll take it as is, although there are couple very minor comments
>below...
>
>
>On Fri, Jan 31, 2014 at 12:23:09AM -0500, Sam Nelson wrote:
>> - Updated Source URL & version
>> - Added device tree file
>>
>> Signed-off-by: Sam Nelson <sam.nelson@ti.com>
>>
>> -----
>
>Should be exactly "---" as the one below or instead of that, so it
>won't go
>into the commit message.
>
>
>> Changes from last patch
>> Renamed recipe from 3.10--> 3.10.10
>> Removed directory path from dtb file
>> Removed do_deploy_append
>> ----
>> ---
>>  .../defconfig                                      |    0
>>  ...keystone_3.8.4.bb => linux-keystone_3.10.10.bb} |   25
>+++++++++++---------
>>  2 files changed, 14 insertions(+), 11 deletions(-)
>>  rename recipes-kernel/linux/{linux-keystone-3.8.4 => linux-
>keystone-3.10.10}/defconfig (100%)
>>  rename recipes-kernel/linux/{linux-keystone_3.8.4.bb => linux-
>keystone_3.10.10.bb} (50%)
>>
>> diff --git a/recipes-kernel/linux/linux-keystone-3.8.4/defconfig
>b/recipes-kernel/linux/linux-keystone-3.10.10/defconfig
>> similarity index 100%
>> rename from recipes-kernel/linux/linux-keystone-3.8.4/defconfig
>> rename to recipes-kernel/linux/linux-keystone-3.10.10/defconfig
>> diff --git a/recipes-kernel/linux/linux-keystone_3.8.4.bb
>b/recipes-kernel/linux/linux-keystone_3.10.10.bb
>> similarity index 50%
>> rename from recipes-kernel/linux/linux-keystone_3.8.4.bb
>> rename to recipes-kernel/linux/linux-keystone_3.10.10.bb
>> index d176f14..a5c113e 100644
>> --- a/recipes-kernel/linux/linux-keystone_3.8.4.bb
>> +++ b/recipes-kernel/linux/linux-keystone_3.10.10.bb

I would comment that since 3.10 is an LTS and you probably expect updates to it maybe having the release version is not a good idea.  For example just call it 3.10 and set the version to 3.10.10 inside the recipe or just leave it 3.10 and set KVER rather than chaning the recipe name for each update.

>> @@ -3,26 +3,29 @@ DESCRIPTION = "Linux kernel for TI Keystone
>devices"
>>  LICENSE = "GPLv2"
>>  LIC_FILES_CHKSUM =
>"file://COPYING;md5=d7810fab7487fb0aad327b76f1be7cd7"
>>
>> +COMPATIBLE_MACHINE = "keystone"
>> +
>>  inherit kernel
>>
>>  require recipes-kernel/linux/linux-dtb.inc
>>  require recipes-kernel/linux/setup-defconfig.inc
>>
>> +MACHINE_KERNEL_PR_append = "a"
>
>1. You might want to also add "+gitr${SRCPV}" so git commit ID
>gets into
>package version and file name. Useful when you have several builds
>from
>different points in git which are all 3.10.10 etc. But it's up to
>you.
>
>2. Having PR = "${MACHINE_KERNEL_PR}" is also useful for others,
>when not
>using meta-openembedded layer and its machine_kernel_pr.bbclass...
>
>
>> +
>> +CORTEXA8FIXUP = "no"
>
>I don't think this is needed any more...
>
>
>>  KERNEL_DEVICETREE_keystone-evm = "k2hk-evm.dtb"
>>
>> -COMPATIBLE_MACHINE = "keystone"
>> +#This commit corresponds to "K2_LINUX_03.10.10_13.12"
>> +SRCREV = "335d14b1c2ce23585835243126c6acba89067ecf"
>>
>> -S = "${WORKDIR}/git"
>> +BRANCH = "master"
>>
>> -BRANCH = "sept-2013/master"
>> +SRC_URI = "git://git.ti.com/keystone-
>linux/linux.git;protocol=git;branch=${BRANCH}\
>> +           file://defconfig\
>> +           "
>>
>> -# This commit corresponds to K2_LINUX_03.08.04_13.09 tag
>> -SRCREV = "20a48dc5309c4434650baef2b6072b5f1016f946"
>> +S = "${WORKDIR}/git"
>>
>> -# Append to the MACHINE_KERNEL_PR so that a new SRCREV will
>cause a rebuild
>> -MACHINE_KERNEL_PR_append = "a+gitr${SRCPV}"
>> -PR = "${MACHINE_KERNEL_PR}"
>> +RDEPENDS_kernel-base = ""
>>
>> -SRC_URI = "git://arago-project.org/git/projects/linux-
>keystone.git;protocol=git;branch=${BRANCH}\
>> -           file://defconfig\
>> -          "
>> --
>> 1.7.9.5
>>
>> _______________________________________________
>> meta-ti mailing list
>> meta-ti@yoctoproject.org
>> https://lists.yoctoproject.org/listinfo/meta-ti
>_______________________________________________
>meta-ti mailing list
>meta-ti@yoctoproject.org
>https://lists.yoctoproject.org/listinfo/meta-ti


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v2] linux-keystone: Create recipe for linux-keystone kernel version 3.10
  2014-01-31  7:06 ` Denys Dmytriyenko
  2014-01-31 14:51   ` Maupin, Chase
@ 2014-01-31 20:38   ` Nelson, Sam
  1 sibling, 0 replies; 4+ messages in thread
From: Nelson, Sam @ 2014-01-31 20:38 UTC (permalink / raw)
  To: Dmytriyenko, Denys; +Cc: meta-ti@yoctoproject.org

Denys,
Thanks for your feedback. Appreciate it.

> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Friday, January 31, 2014 2:07 AM
> To: Nelson, Sam
> Cc: meta-ti@yoctoproject.org
> Subject: Re: [meta-ti] [PATCH v2] linux-keystone: Create recipe for linux-
> keystone kernel version 3.10
> 
> Sam,
> 
> This is much better! And easier to compare against the previous version too.
> I'll take it as is, although there are couple very minor comments below...
> 
> 
> On Fri, Jan 31, 2014 at 12:23:09AM -0500, Sam Nelson wrote:
> > - Updated Source URL & version
> > - Added device tree file
> >
> > Signed-off-by: Sam Nelson <sam.nelson@ti.com>
> >
> > -----
> 
> Should be exactly "---" as the one below or instead of that, so it won't go
> into the commit message.
> 
> 
> > Changes from last patch
> > Renamed recipe from 3.10--> 3.10.10
> > Removed directory path from dtb file
> > Removed do_deploy_append
> > ----
> > ---
> >  .../defconfig                                      |    0
> >  ...keystone_3.8.4.bb => linux-keystone_3.10.10.bb} |   25 +++++++++++-----
> ----
> >  2 files changed, 14 insertions(+), 11 deletions(-)
> >  rename recipes-kernel/linux/{linux-keystone-3.8.4 => linux-keystone-
> 3.10.10}/defconfig (100%)
> >  rename recipes-kernel/linux/{linux-keystone_3.8.4.bb => linux-
> keystone_3.10.10.bb} (50%)
> >
> > diff --git a/recipes-kernel/linux/linux-keystone-3.8.4/defconfig b/recipes-
> kernel/linux/linux-keystone-3.10.10/defconfig
> > similarity index 100%
> > rename from recipes-kernel/linux/linux-keystone-3.8.4/defconfig
> > rename to recipes-kernel/linux/linux-keystone-3.10.10/defconfig
> > diff --git a/recipes-kernel/linux/linux-keystone_3.8.4.bb b/recipes-
> kernel/linux/linux-keystone_3.10.10.bb
> > similarity index 50%
> > rename from recipes-kernel/linux/linux-keystone_3.8.4.bb
> > rename to recipes-kernel/linux/linux-keystone_3.10.10.bb
> > index d176f14..a5c113e 100644
> > --- a/recipes-kernel/linux/linux-keystone_3.8.4.bb
> > +++ b/recipes-kernel/linux/linux-keystone_3.10.10.bb
> > @@ -3,26 +3,29 @@ DESCRIPTION = "Linux kernel for TI Keystone devices"
> >  LICENSE = "GPLv2"
> >  LIC_FILES_CHKSUM =
> "file://COPYING;md5=d7810fab7487fb0aad327b76f1be7cd7"
> >
> > +COMPATIBLE_MACHINE = "keystone"
> > +
> >  inherit kernel
> >
> >  require recipes-kernel/linux/linux-dtb.inc
> >  require recipes-kernel/linux/setup-defconfig.inc
> >
> > +MACHINE_KERNEL_PR_append = "a"
> 
> 1. You might want to also add "+gitr${SRCPV}" so git commit ID gets into
> package version and file name. Useful when you have several builds from
> different points in git which are all 3.10.10 etc. But it's up to you.
> 
> 2. Having PR = "${MACHINE_KERNEL_PR}" is also useful for others, when not
> using meta-openembedded layer and its machine_kernel_pr.bbclass...
> 
> 
> > +
> > +CORTEXA8FIXUP = "no"
> 
> I don't think this is needed any more...
> 
> 
> >  KERNEL_DEVICETREE_keystone-evm = "k2hk-evm.dtb"
> >
> > -COMPATIBLE_MACHINE = "keystone"
> > +#This commit corresponds to "K2_LINUX_03.10.10_13.12"
> > +SRCREV = "335d14b1c2ce23585835243126c6acba89067ecf"
> >
> > -S = "${WORKDIR}/git"
> > +BRANCH = "master"
> >
> > -BRANCH = "sept-2013/master"
> > +SRC_URI = "git://git.ti.com/keystone-
> linux/linux.git;protocol=git;branch=${BRANCH}\
> > +           file://defconfig\
> > +           "
> >
> > -# This commit corresponds to K2_LINUX_03.08.04_13.09 tag
> > -SRCREV = "20a48dc5309c4434650baef2b6072b5f1016f946"
> > +S = "${WORKDIR}/git"
> >
> > -# Append to the MACHINE_KERNEL_PR so that a new SRCREV will cause a
> rebuild
> > -MACHINE_KERNEL_PR_append = "a+gitr${SRCPV}"
> > -PR = "${MACHINE_KERNEL_PR}"
> > +RDEPENDS_kernel-base = ""
> >
> > -SRC_URI = "git://arago-project.org/git/projects/linux-
> keystone.git;protocol=git;branch=${BRANCH}\
> > -           file://defconfig\
> > -          "
> > --
> > 1.7.9.5
> >
> > _______________________________________________
> > meta-ti mailing list
> > meta-ti@yoctoproject.org
> > https://lists.yoctoproject.org/listinfo/meta-ti


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-01-31 20:38 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-01-31  5:23 [PATCH v2] linux-keystone: Create recipe for linux-keystone kernel version 3.10 Sam Nelson
2014-01-31  7:06 ` Denys Dmytriyenko
2014-01-31 14:51   ` Maupin, Chase
2014-01-31 20:38   ` Nelson, Sam

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.