[alsa-devel] [PATCH] ASoC: rt5640: ifdef for ACPI module table

Mark Brown broonie at kernel.org
Wed Sep 18 20:34:01 CEST 2013


On Wed, Sep 18, 2013 at 10:36:20AM -0700, Olof Johansson wrote:
> On Wed, Sep 18, 2013 at 06:28:58PM +0100, Mark Brown wrote:

> > I'd have expected the folks doing the allyesconfig builds to have
> > complained then...

> Allyes will probably turn on CONFIG_ACPI and thus reference the variable
> in the struct device.

Yeah, I was thinking of OF which isn't available on all architectures.
Perhaps it is on every architecture people tend to do this stuff on
though.

> With the new of_match_ptr() and ACPI_PTR() you get rid of the
> ifdef in struct device for those, but you still want it around
> MODULE_DEVICE_TABLE() and the actual definition of the match table.

Hrm, perhaps that was what was supposed to happen - have
MODULE_DEVICE_TABLE() create the appropriate eliminateable static
reference so the compiler can figure things out and we don't need the
ifdefs.  It'd certainly be nice to elimiante more of these ifdefs.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20130918/bef22ecf/attachment.sig>


More information about the Alsa-devel mailing list