Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH] linux-firmware: make the main package depend on all sub-packages
@ 2013-04-05 17:06 Ross Burton
  2013-04-05 20:11 ` Otavio Salvador
  2013-04-06 10:26 ` Koen Kooi
  0 siblings, 2 replies; 5+ messages in thread
From: Ross Burton @ 2013-04-05 17:06 UTC (permalink / raw)
  To: openembedded-core

Whilst splitting out specific large firmware blobs is a good move for space
saving, it makes installing "all the firmware" tricky.

Make linux-firmware depend on all of the separated packages so that installing
that pulls in all of the sub-packages.

Signed-off-by: Ross Burton <ross.burton@intel.com>
---
 meta/recipes-kernel/linux-firmware/linux-firmware_git.bb |    6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/meta/recipes-kernel/linux-firmware/linux-firmware_git.bb b/meta/recipes-kernel/linux-firmware/linux-firmware_git.bb
index 5ee3b73..ef036cf 100644
--- a/meta/recipes-kernel/linux-firmware/linux-firmware_git.bb
+++ b/meta/recipes-kernel/linux-firmware/linux-firmware_git.bb
@@ -162,3 +162,9 @@ ALTERNATIVE_linux-firmware-bcm4334 = "brcmfmac-sdio.bin"
 ALTERNATIVE_TARGET_linux-firmware-bcm4334[brcmfmac-sdio.bin] = "/lib/firmware/brcm/brcmfmac4334.bin"
 
 FILES_${PN} += "/lib/firmware/*"
+
+# Make linux-firmware depend on all of the split-out packages.
+python populate_packages_prepend () {
+    firmware_pkgs = oe.utils.packages_filter_out_system(d)
+    d.appendVar('RDEPENDS_linux-firmware', ' ' + ' '.join(firmware_pkgs))
+}
-- 
1.7.10.4




^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] linux-firmware: make the main package depend on all sub-packages
  2013-04-05 17:06 [PATCH] linux-firmware: make the main package depend on all sub-packages Ross Burton
@ 2013-04-05 20:11 ` Otavio Salvador
  2013-04-06 10:26 ` Koen Kooi
  1 sibling, 0 replies; 5+ messages in thread
From: Otavio Salvador @ 2013-04-05 20:11 UTC (permalink / raw)
  To: Ross Burton; +Cc: Patches and discussions about the oe-core layer

On Fri, Apr 5, 2013 at 2:06 PM, Ross Burton <ross.burton@intel.com> wrote:
> Whilst splitting out specific large firmware blobs is a good move for space
> saving, it makes installing "all the firmware" tricky.
>
> Make linux-firmware depend on all of the separated packages so that installing
> that pulls in all of the sub-packages.
>
> Signed-off-by: Ross Burton <ross.burton@intel.com>

+1 Looks great, indeed.

--
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] linux-firmware: make the main package depend on all sub-packages
  2013-04-05 17:06 [PATCH] linux-firmware: make the main package depend on all sub-packages Ross Burton
  2013-04-05 20:11 ` Otavio Salvador
@ 2013-04-06 10:26 ` Koen Kooi
  2013-04-06 10:36   ` Martin Jansa
  1 sibling, 1 reply; 5+ messages in thread
From: Koen Kooi @ 2013-04-06 10:26 UTC (permalink / raw)
  To: Ross Burton; +Cc: openembedded-core


Op 5 apr. 2013, om 19:06 heeft Ross Burton <ross.burton@intel.com> het volgende geschreven:

> Whilst splitting out specific large firmware blobs is a good move for space
> saving, it makes installing "all the firmware" tricky.
> 
> Make linux-firmware depend on all of the separated packages so that installing
> that pulls in all of the sub-packages.

Speaking of firmwares, is anybody working on fixing the clashes between kernel-firmware-foo and linux-firmware-foo? It should only be a matter of adding RREPLACES to the linux-firmware subpackages.

Koen


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] linux-firmware: make the main package depend on all sub-packages
  2013-04-06 10:26 ` Koen Kooi
@ 2013-04-06 10:36   ` Martin Jansa
  2013-04-06 11:44     ` Richard Purdie
  0 siblings, 1 reply; 5+ messages in thread
From: Martin Jansa @ 2013-04-06 10:36 UTC (permalink / raw)
  To: Koen Kooi; +Cc: Patches and discussions about the oe-core layer

[-- Attachment #1: Type: text/plain, Size: 1156 bytes --]

And also staging firmware files just once to resolve

> WARNING: The recipe linux-firmware is trying to install files into a
shared area when those files already exist. Those files are:

when building linux-firmware after kernel was built and vice-versa.


On Sat, Apr 6, 2013 at 12:26 PM, Koen Kooi <koen@dominion.thruhere.net>wrote:

>
> Op 5 apr. 2013, om 19:06 heeft Ross Burton <ross.burton@intel.com> het
> volgende geschreven:
>
> > Whilst splitting out specific large firmware blobs is a good move for
> space
> > saving, it makes installing "all the firmware" tricky.
> >
> > Make linux-firmware depend on all of the separated packages so that
> installing
> > that pulls in all of the sub-packages.
>
> Speaking of firmwares, is anybody working on fixing the clashes between
> kernel-firmware-foo and linux-firmware-foo? It should only be a matter of
> adding RREPLACES to the linux-firmware subpackages.
>
> Koen
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>

[-- Attachment #2: Type: text/html, Size: 1909 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] linux-firmware: make the main package depend on all sub-packages
  2013-04-06 10:36   ` Martin Jansa
@ 2013-04-06 11:44     ` Richard Purdie
  0 siblings, 0 replies; 5+ messages in thread
From: Richard Purdie @ 2013-04-06 11:44 UTC (permalink / raw)
  To: Martin Jansa; +Cc: Koen Kooi, Patches, the oe-core layer

On Sat, 2013-04-06 at 12:36 +0200, Martin Jansa wrote:
> And also staging firmware files just once to resolve
>
> > WARNING: The recipe linux-firmware is trying to install files into a
> shared area when those files already exist. Those files are:
>
> when building linux-firmware after kernel was built and vice-versa.

What is the point in staging firmware files at all? There is probably a
small space/performance gain to be had there...

Cheers,

Richard





^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2013-04-06 12:01 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-05 17:06 [PATCH] linux-firmware: make the main package depend on all sub-packages Ross Burton
2013-04-05 20:11 ` Otavio Salvador
2013-04-06 10:26 ` Koen Kooi
2013-04-06 10:36   ` Martin Jansa
2013-04-06 11:44     ` Richard Purdie

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox