Re: [PATCH 1/2] firmware: include drivers/firmware/Kconfig unconditionally
From: Mark Brown
Date: Tue Sep 28 2021 - 07:59:48 EST
On Tue, Sep 28, 2021 at 09:50:26AM +0200, Arnd Bergmann wrote:
> Compile-testing drivers that require access to a firmware layer
> fails when that firmware symbol is unavailable. This happened
> twice this week:
> 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.
Reviwed-by: Mark Brown <broonie@xxxxxxxxxx>
Regardless of what we do with the Cirrus DSP this just seems like a good
idea - surprises due to this not being generally available keep coming
up, IIRC with the i.MX firmware in the past for example.
> 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).
We could also merge a tag into both places.
> 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.
Trying to enforce distinctions here feels like it's liable to lead to
trouble at some point...
Description: PGP signature