[meta-intel] [PATCH 2/4] sugarbay.conf: add wifi machine feature

[Kamble, Nitin A]

On 10/1/14, 9:19 AM, "Zanussi, Tom" <tom.zanussi at intel.com> wrote:

>On Wed, 2014-10-01 at 11:16 -0500, Kamble, Nitin A wrote:
>> 
>> On 10/1/14, 7:34 AM, "Zanussi, Tom" <tom.zanussi at intel.com> wrote:
>> 
>> >On Tue, 2014-09-30 at 19:27 -0700, nitin.a.kamble at intel.com wrote:
>> >> From: Nitin A Kamble <nitin.a.kamble at intel.com>
>> >> 
>> >> The Huron River platform targeted by this BSP has wifi capability.
>> >> The common kernel used by the BSP already has enabled the wifi
>>drivers.
>> >> 
>> >> Enabling the user space wifi tools, which gives functional wifi
>> >>networking
>> >> on the Huron River platform.
>> >> 
>> >> Fixes Bug:
>> >> [YOCTO #6342]
>> >> 
>> >> Signed-off-by: Nitin A Kamble <nitin.a.kamble at intel.com>
>> >> ---
>> >>  meta-sugarbay/conf/machine/sugarbay.conf | 3 +++
>> >>  1 file changed, 3 insertions(+)
>> >> 
>> >> diff --git a/meta-sugarbay/conf/machine/sugarbay.conf
>> >>b/meta-sugarbay/conf/machine/sugarbay.conf
>> >> index de3b002..7d1b0ea 100644
>> >> --- a/meta-sugarbay/conf/machine/sugarbay.conf
>> >> +++ b/meta-sugarbay/conf/machine/sugarbay.conf
>> >> @@ -15,6 +15,9 @@ require
>> >>conf/machine/include/intel-corei7-64-common.inc
>> >>  require conf/machine/include/intel-common-pkgarch.inc
>> >>  
>> >>  MACHINE_HWCODECS ?= "va-intel gst-va-intel"
>> >> +MACHINE_EXTRA_RRECOMMENDS += "linux-firmware"
>> >> +
>> >
>> >This patch also seem to include linux-firmware, which the message says
>> >nothing about.
>> >
>> >Tom
>> 
>> I was counting it in the user space wifi tools as the firmware for the
>> wifi chip is coming from the user space. If you want it mentioned in the
>
>So the firmware for the wifi chip comed from the linux-firmware feature?

Yes. That is why it is included in the BSP. Linux-firmware is a recipe and
not a feature though.

Nitin

>
>Tom
>
>> commit message, it can be done easily.
>> 
>> Nitin
>> 
>> >
>> >
>> >> +MACHINE_FEATURES += "wifi"
>> >>  
>> >>  XSERVER ?= "${XSERVER_X86_BASE} \
>> >>             ${XSERVER_X86_EXT} \
>> >
>> >
>> 
>
>