Re: Jack reporting v2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



At Tue, 3 Jun 2008 11:40:48 +0100,
Mark Brown wrote:
> 
> The following couple of patches update the jack reporting API I posted
> last week with the feedback I have received so far.  The changes since
> the last submision are:
> 
>  - Addition of snd_jack_set_parent(), allowing drivers to override the
>    default parent for the input device prior to registration.
>  - The jack detection support is now included unconditionally in ALSA,
>    requiring a dependency on the input subsystem.
> 
> I'm not entirely happy with making ALSA depend on input but given that
> it is only optional on embedded systems and that the input core is not
> that large it seemed the best way of ensuring that the input subsystem
> is available for the jack API without hassle for drivers using it,
> especially given the lack of dependency propagation from select.

I also feel uneasy that it's built in unconditionally and requires an 
additional dependency on CONFIG_INPUT.  The fact that both SND and
INPUT can be modules makes things a bit complicated.

As a workaround, how about to select with "if" conditions?
For example, in sound/core/Kconfig,

config SND_JACK
	bool

while the driver requires SND_JACK has the below:

config SND_FOO
	tristate "..."
	...
	select SND_JACK if INPUT=y || INPUT=SND

Yeah, it looks horrible at a first glance, but it's a known workaround
for reverse selections...

In sound/core/Makefile, just add the line:

snd-$(CONFIG_SND_JACK) += jack.o

so that the code is built into snd module only when necessary.

Also, sound/jack.h should provide empty inline functions for the case 
CONFIG_SND_JACK=n.  Then you'll be able to reduce ifdefs in the driver
codes.


thanks,

Takashi
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux