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:[email protected]] > Sent: Thursday, August 6, 2015 7:15 AM > To: Ng, Wei Tee; [email protected] > 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: [email protected]; Darren Hart ([email protected]) > >> 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: [email protected] [mailto:meta-intel- > >>> [email protected]] On Behalf Of [email protected] > >>> Sent: Thursday, July 30, 2015 1:30 PM > >>> To: [email protected] > >>> 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 <[email protected]> > >>> > >>> 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 <[email protected]> > >> Acked-by: Ong Boon Leong <[email protected]> > >> > >> 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 > >>> [email protected] > >>> https://lists.yoctoproject.org/listinfo/meta-intel > > -- > Darren Hart > Intel Open Source Technology Center -- _______________________________________________ meta-intel mailing list [email protected] https://lists.yoctoproject.org/listinfo/meta-intel
