lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 29 Sep 2021 11:17:21 +0100 From: Will Deacon <will@...nel.org> To: Arnd Bergmann <arnd@...nel.org> Cc: Bjorn Andersson <bjorn.andersson@...aro.org>, Arnd Bergmann <arnd@...db.de>, Mark Brown <broonie@...nel.org>, Liam Girdwood <lgirdwood@...il.com>, Charles Keepax <ckeepax@...nsource.cirrus.com>, Simon Trimmer <simont@...nsource.cirrus.com>, Michael Ellerman <mpe@...erman.id.au>, Russell King <linux@...linux.org.uk>, Catalin Marinas <catalin.marinas@....com>, Thomas Bogendoerfer <tsbogend@...ha.franken.de>, "James E.J. Bottomley" <James.Bottomley@...senPartnership.com>, Helge Deller <deller@....de>, Paul Walmsley <paul.walmsley@...ive.com>, Palmer Dabbelt <palmer@...belt.com>, Albert Ou <aou@...s.berkeley.edu>, Thomas Gleixner <tglx@...utronix.de>, Ingo Molnar <mingo@...hat.com>, Borislav Petkov <bp@...en8.de>, x86@...nel.org, "H. Peter Anvin" <hpa@...or.com>, Geert Uytterhoeven <geert+renesas@...der.be>, Linus Walleij <linus.walleij@...aro.org>, Andrew Morton <akpm@...ux-foundation.org>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>, linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org, linux-ia64@...r.kernel.org, linux-mips@...r.kernel.org, linux-parisc@...r.kernel.org, linux-riscv@...ts.infradead.org Subject: Re: [PATCH 1/2] firmware: include drivers/firmware/Kconfig unconditionally On Tue, Sep 28, 2021 at 09:50:26AM +0200, Arnd Bergmann wrote: > From: Arnd Bergmann <arnd@...db.de> > > Compile-testing drivers that require access to a firmware layer > fails when that firmware symbol is unavailable. This happened > twice this week: > > - My proposed to change to rework the QCOM_SCM firmware symbol > broke on ppc64 and others. > > - The cs_dsp firmware patch added device specific firmware loader > into drivers/firmware, which broke on the same set of > architectures. > > We should probably do the same thing for other subsystems as well, > but fix this one first as this is a dependency for other patches > getting merged. > > Cc: Mark Brown <broonie@...nel.org> > Cc: Liam Girdwood <lgirdwood@...il.com> > Cc: Charles Keepax <ckeepax@...nsource.cirrus.com> > Cc: Simon Trimmer <simont@...nsource.cirrus.com> > Cc: Arnd Bergmann <arnd@...db.de> > Cc: Michael Ellerman <mpe@...erman.id.au> > Signed-off-by: Arnd Bergmann <arnd@...db.de> > --- > Not sure how we'd want to merge this patch, if two other things > need it. I'd prefer to merge it along with the QCOM_SCM change > through the soc tree, but that leaves the cirrus firmware broken > unless we also merge it the same way (rather than through ASoC > as it is now). > > Alternatively, we can try to find a different home for the Cirrus > firmware to decouple the two problems. I'd argue that it's actually > misplaced here, as drivers/firmware is meant for kernel code that > interfaces with system firmware, not for device drivers to load > their own firmware blobs from user space. > --- > arch/arm/Kconfig | 2 -- > arch/arm64/Kconfig | 2 -- > arch/ia64/Kconfig | 2 -- > arch/mips/Kconfig | 2 -- > arch/parisc/Kconfig | 2 -- > arch/riscv/Kconfig | 2 -- > arch/x86/Kconfig | 2 -- > drivers/Kconfig | 2 ++ > 8 files changed, 2 insertions(+), 14 deletions(-) For arm64: Acked-by: Will Deacon <will@...nel.org> Will
Powered by blists - more mailing lists