[meta-intel] [PATCH 6/6] meta-valleyisland: add kernel recipe for valleyisland BSP

[Kamble, Nitin A]

On 2/5/2014 1:25 AM, rebecca.swee.fun.chang at intel.com wrote:
> From: "Chang, Rebecca Swee Fun" <rebecca.swee.fun.chang at intel.com>
>
> Provide kernel v3.8 recipe for valleyisland BSP.
>
> Signed-off-by: Chang, Rebecca Swee Fun <rebecca.swee.fun.chang at intel.com>
> ---
>   .../recipes-kernel/linux/linux-yocto_3.8.bbappend  |   32 ++++++++++++++++++++
>   1 file changed, 32 insertions(+)
>   create mode 100644 meta-isg/meta-valleyisland/recipes-kernel/linux/linux-yocto_3.8.bbappend
>
> diff --git a/meta-isg/meta-valleyisland/recipes-kernel/linux/linux-yocto_3.8.bbappend b/meta-isg/meta-valleyisland/recipes-kernel/linux/linux-yocto_3.8.bbappend
> new file mode 100644
> index 0000000..385da96
> --- /dev/null
> +++ b/meta-isg/meta-valleyisland/recipes-kernel/linux/linux-yocto_3.8.bbappend
> @@ -0,0 +1,32 @@
> +FILESEXTRAPATHS_prepend := "${THISDIR}/${PN}:"
> +
> +KMETA = "meta"
> +LINUX_VERSION = "3.8.0"
Darren has pointed out,  this is not set correct.

> +
> +#############################
> +# MACHINE = valleyisland-32 #
> +#############################
> +COMPATIBLE_MACHINE_valleyisland-32 = "valleyisland-32"
> +KMACHINE_valleyisland-32 = "valleyisland-32"
> +KBRANCH_valleyisland-32 = "standard/common-pc/base"
> +KERNEL_FEATURES_valleyisland-32 = "features/valleyisland-io/valleyisland-io"
> +
> +SRCREV_machine_pn-linux-yocto_valleyisland-32 ?= "6f3e338aa9496cf68ad03a98f66c2e98975829c7"
> +SRCREV_meta_pn-linux-yocto_valleyisland-32 ?= "f87180481b12aebd16b9234d922d5f4a25a32dca"
> +
> +SRC_URI_valleyisland-32 = "git://git.yoctoproject.org/linux-yocto-3.8.git;protocol=git;nocheckout=1;branch=${KBRANCH},${KMETA};name=machine,meta"
This line is not necessary. the .bb file in poky layer already has 
defined this correctly.
> +
> +#############################
> +# MACHINE = valleyisland-64 #
> +#############################
> +COMPATIBLE_MACHINE_valleyisland-64 = "valleyisland-64"
> +KMACHINE_valleyisland-64 = "valleyisland"
> +KBRANCH_valleyisland-64 = "standard/common-pc-64/base"
> +KERNEL_FEATURES_valleyisland-32 = "features/valleyisland-io/valleyisland-io"
> +
> +SRCREV_machine_pn-linux-yocto_valleyisland-64 ?= "6f3e338aa9496cf68ad03a98f66c2e98975829c7"
> +SRCREV_meta_pn-linux-yocto_valleyisland-64 ?= "f87180481b12aebd16b9234d922d5f4a25a32dca"
> +
> +SRC_URI_valleyisland-64 = "git://git.yoctoproject.org/linux-yocto-3.8.git;protocol=git;nocheckout=1;branch=${KBRANCH},${KMETA};name=machine,meta"
This line is not necessary.
> +
> +module_autoload_i2c-dev = "i2c-dev"
Can you provide a comment in the recipe why this line is needed?

Thanks,
Nitin