>-----Original Message-----
>From: [email protected] [mailto:meta-ti-
>[email protected]] On Behalf Of Heroor, Siddharth
>Sent: Thursday, June 27, 2013 5:13 AM
>To: [email protected]
>Subject: [meta-ti] [PATCH] recipes-kernel: Add GLSDK specific
>kernel tree for omap-a15
>
>* Machine compatible with only omap-a15 devices.
>* The TI Staging tree will not be used to maintain K3.8. Instead,
>  this new tree is set up as a production tree. Pending features
>  for 3.8 will be implemented on this tree.
>  - Backported features from linux mainline
>  - Backported bug-fixes from linux mainline and linux-stable.
>  - Features implemented by the GLSDK team which maybe product
>    specific.
>  This is the tree on which GLSDK kernel releases will be made.
>* This recipe is based off the Core SDK 2013.04.02 released kernel
>  recipe - recipes-kernel/linux/linux-ti-staging_3.8.bb
>

Can you try using --find-copies-harder when making this patch or I think Denys 
likes -m.  That will help to minimize the actual patch difference in cases like 
this where you are based on an existing recipe.

>Signed-off-by: Siddharth Heroor <[email protected]>
>cc: Mrinmayee Hingolikar <[email protected]>
>---
> recipes-kernel/linux/linux-ti-glsdk/defconfig |    1 +
> recipes-kernel/linux/linux-ti-glsdk_3.8.bb    |   30
>+++++++++++++++++++++++++
> 2 files changed, 31 insertions(+), 0 deletions(-)
> create mode 100644 recipes-kernel/linux/linux-ti-glsdk/defconfig
> create mode 100644 recipes-kernel/linux/linux-ti-glsdk_3.8.bb
>
>diff --git a/recipes-kernel/linux/linux-ti-glsdk/defconfig
>b/recipes-kernel/linux/linux-ti-glsdk/defconfig
>new file mode 100644
>index 0000000..926b35a
>--- /dev/null
>+++ b/recipes-kernel/linux/linux-ti-glsdk/defconfig
>@@ -0,0 +1 @@
>+use-kernel-config=omap2plus_defconfig
>diff --git a/recipes-kernel/linux/linux-ti-glsdk_3.8.bb b/recipes-
>kernel/linux/linux-ti-glsdk_3.8.bb
>new file mode 100644
>index 0000000..8d66f84
>--- /dev/null
>+++ b/recipes-kernel/linux/linux-ti-glsdk_3.8.bb
>@@ -0,0 +1,30 @@
>+SECTION = "kernel"
>+DESCRIPTION = "Linux kernel for TI devices supported by the GLSDK
>product"
>+LICENSE = "GPLv2"
>+LIC_FILES_CHKSUM =
>"file://COPYING;md5=d7810fab7487fb0aad327b76f1be7cd7"
>+KERNEL_IMAGETYPE = "uImage"
>+
>+inherit kernel
>+
>+require recipes-kernel/linux/linux-dtb.inc
>+require recipes-kernel/linux/setup-defconfig.inc
>+
>+KERNEL_DEVICETREE_omap5-evm = "arch/arm/boot/dts/omap5-sevm.dts
>arch/arm/boot/dts/omap5-uevm.dts"
>+KERNEL_DEVICETREE_dra7xx-evm = "arch/arm/boot/dts/dra7-evm.dts"
>+
>+COMPATIBLE_MACHINE = "omap-a15"

I think you might want to set a DEFAULT_PREFERRENCE = -1 here.  Since you are 
going to use your branding config to select this kernel vs. the 
linux-ti-staging kernel this should be no issue.  That way there isn't a 
confusion about with basic meta-ti what kernel is being picked up.

>+
>+S = "${WORKDIR}/git"
>+
>+BRANCH = "master"
>+
>+# This commit corresponds to ti2013.04.02 release tag

Sorry to nit-pick here, but any reason to re-use the tag name from the 
linux-ti-staging tree?  The reason I ask is back to it being confusing if I 
have two trees with the same tag name (including version) it becomes kind of 
confusing.  Especially since this tree is part of the same git.ti.com project 
as the other one.  Thoughts?

>+SRCREV = "f3cb62a800ae37f17a3382b10d155ee02dbc83fc"
>+PV = "3.8.13"
>+
>+# Append to the MACHINE_KERNEL_PR so that a new SRCREV will cause
>a rebuild
>+MACHINE_KERNEL_PR_append = "f+gitr${SRCPV}"

Go ahead and reset your append here to a+ gitr${SRCPV}.  No need to start from 
f.

Thanks for the submission.  Overall looks pretty good, just a few things to fix 
up.

>+
>+SRC_URI = "git://git.ti.com/ti-linux-kernel/ti-linux-3-8-y-
>kernel.git;protocol=git;branch=${BRANCH} \
>+           file://defconfig \
>+          "
>--
>1.7.0.4
>
>_______________________________________________
>meta-ti mailing list
>[email protected]
>https://lists.yoctoproject.org/listinfo/meta-ti
_______________________________________________
meta-ti mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/meta-ti

Reply via email to