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

[Tom Zanussi]

On Wed, 2014-10-01 at 11:28 -0500, Kamble, Nitin A wrote:
> 
> 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.
> 

OK, yeah, never mind, I was confusing this with the recent microcode
thread, sorry.

Tom

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