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
