* 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