[meta-intel] [PATCH 1/1] meta-valleyisland: Add linux-firmware-pcie8897 support in valleyisland-32.conf and valleyisland-64.conf

[Darren Hart]

On 8/4/15 8:08 PM, Ng, Wei Tee wrote:
> Dear maintainers,
> 
> Any updates regarding to this patch?

Hi Wei Tee:

This patch was marked as dizzy. I presume you also intend it to be
applied to master and backported to fido and dizzy?

Is there a reason you have chosen to single out this particular firmware
piece rather than just include linux-firmware as a whole (as intel-core*
BSPs do) so you have it all in case your users choose to use devices you
haven't explicitly accounted for?

Thanks,

Darren

> Thanks.
> 
> Regards,
> Wei Tee
> 
>> -----Original Message-----
>> From: Ong, Boon Leong
>> Sent: Thursday, July 30, 2015 5:42 PM
>> To: Ng, Wei Tee
>> Cc: meta-intel at yoctoproject.org; Darren Hart (dvhart at linux.intel.com)
>> Subject: RE: [meta-intel] [PATCH 1/1] meta-valleyisland: Add linux-firmware-
>> pcie8897 support in valleyisland-32.conf and valleyisland-64.conf
>>
>>> -----Original Message-----
>>> From: meta-intel-bounces at yoctoproject.org [mailto:meta-intel-
>>> bounces at yoctoproject.org] On Behalf Of wei.tee.ng at intel.com
>>> Sent: Thursday, July 30, 2015 1:30 PM
>>> To: meta-intel at yoctoproject.org
>>> Subject: [meta-intel] [PATCH 1/1] meta-valleyisland: Add
>>> linux-firmware-
>>> pcie8897 support in valleyisland-32.conf and valleyisland-64.conf
>>>
>>> From: Ng Wei Tee <wei.tee.ng at intel.com>
>>>
>>> Add linux-firmware-pcie8897 support for both valleyisland-32.conf and
>>> valleyisland-64.conf in order to enable the Marvell 8897 card.
>>>
>>> Signed-off-by: Ng Wei Tee <wei.tee.ng at intel.com>
>> Acked-by: Ong Boon Leong <boon.leong.ong at intel.com>
>>
>> I am giving you my acked as I believe that Marvell card is essential to be a
>> package needed for booting the system. So, that is why instead of using
>> MACHINE_ESSENTIAL_EXTRA_RRECOMMENDS below, you intentionally are
>> using MACHINE_EXTRA_RRECOMMENDS
>>
>> BL
>>
>>> ---
>>> .../conf/machine/valleyisland-32.conf              |    2 ++
>>> .../conf/machine/valleyisland-64.conf              |    1 +
>>> 2 files changed, 3 insertions(+)
>>>
>>> diff --git
>>> a/meta-isg/meta-valleyisland/conf/machine/valleyisland-32.conf
>>> b/meta-isg/meta-valleyisland/conf/machine/valleyisland-32.conf
>>> index 6bbae7a..f989e7d 100644
>>> --- a/meta-isg/meta-valleyisland/conf/machine/valleyisland-32.conf
>>> +++ b/meta-isg/meta-valleyisland/conf/machine/valleyisland-32.conf
>>> @@ -14,6 +14,8 @@ require conf/machine/include/meta-intel.inc
>>>
>>> MACHINE_FEATURES += "pcbios efi"
>>>
>>> +MACHINE_EXTRA_RRECOMMENDS += "linux-firmware-pcie8897"
>>> +
>>> XSERVER ?= "${XSERVER_X86_BASE} \
>>>             ${XSERVER_X86_EXT} \
>>>             ${XSERVER_X86_FBDEV} \
>>> diff --git
>>> a/meta-isg/meta-valleyisland/conf/machine/valleyisland-64.conf
>>> b/meta-isg/meta-valleyisland/conf/machine/valleyisland-64.conf
>>> index a13cc17..9158232 100644
>>> --- a/meta-isg/meta-valleyisland/conf/machine/valleyisland-64.conf
>>> +++ b/meta-isg/meta-valleyisland/conf/machine/valleyisland-64.conf
>>> @@ -15,6 +15,7 @@ require conf/machine/include/meta-intel.inc
>>>
>>> MACHINE_FEATURES += "pcbios efi"
>>>
>>> +MACHINE_EXTRA_RRECOMMENDS += "linux-firmware-pcie8897"
>>>
>>> XSERVER ?= "${XSERVER_X86_BASE} \
>>>             ${XSERVER_X86_EXT} \
>>> --
>>> 1.7.9.5
>>>
>>> --
>>> _______________________________________________
>>> meta-intel mailing list
>>> meta-intel at yoctoproject.org
>>> https://lists.yoctoproject.org/listinfo/meta-intel

-- 
Darren Hart
Intel Open Source Technology Center