[alsa-devel] [PATCH 1/2] ALSA: Add jack reporting API

Mark Brown broonie at opensource.wolfsonmicro.com
Mon Jul 28 19:47:25 CEST 2008

On Mon, Jul 28, 2008 at 07:01:19PM +0200, Takashi Iwai wrote:

> They are bitmask, so HEADSET should be better defined as


> > +void snd_jack_set_parent(struct snd_jack *jack, struct device *parent)
> > +{
> > +	BUG_ON(jack->registered);

> I'm afraid BUG_ON() is brutal for this kind of check.

I'll change it down to a WARN_ON() but I do feel that this is something
that should be complained about loudly, it's a clear error to try to
reparent the instantiated device.

More information about the Alsa-devel mailing list