Re: [PATCH 1/2] ASoC: Check for exact register match in wm97xx_reset()

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

 



On Fri, Jun 13, 2008 at 05:11:06PM +0200, Takashi Iwai wrote:
> Mark Brown wrote:

> >  	soc_ac97_ops.reset(codec->ac97);
> > -	if (ac97_read(codec, 0) & 0x8000)
> > +	if (ac97_read(codec, 0) != 0x6174)

> Can we define this magic value more understandable?

It's the default value for the (non-modifiable) register - building it
up from a bitmask with semantics isn't particularly meaningful.  The
only thing I can think is to just reference the register cache default
values, I'll resend in a minute or two.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux