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

[Ng, Wei Tee]

Hi Darren,

Thanks for the catch up!. This patch is intend to be applied to master and backported
to fido and dizzy. After getting input from your side, I will resubmit the patch to include
the linux-firmware as a whole. 

Regards,
Wei Tee

> -----Original Message-----
> From: Darren Hart [mailto:dvhart at linux.intel.com]
> Sent: Thursday, August 6, 2015 7:15 AM
> To: Ng, Wei Tee; meta-intel at yoctoproject.org
> Subject: Re: [meta-intel] [PATCH 1/1] meta-valleyisland: Add linux-firmware-
> pcie8897 support in valleyisland-32.conf and valleyisland-64.conf
> 
> 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