Re: [PATCH 2/2] wm8750: add missing VREF output

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

 



On Mon, Aug 11, 2008 at 12:45:32PM +0200, Takashi Iwai wrote:
> Mark Brown wrote:

> > The long description is exactly what's broken - the names are wrong or
> > undeclared and since DAPM works based on the control names this causes
> > the drivers to fail to initialise DAPM for the device concerned. 

> Exactly such a text should appear in the changelog.

Sure, my intention was to provide some appropriate text.

> > It's
> > pretty much equivalent to C code referencing an undeclared variable.

> Everyone knows C syntax but not ASoC implementation details.
> If the patch is pushed to stable tree, it should be described well so
> that other reviewers can understand it well.

Sorry, there may have been a miscommunication here - this should only
*need* to go to 2.6.27 since the bulk registration functions are more
careful about checking error values than the hand rolled loops tended to
be.  For earlier kernels people will either not notice or have already
have local patches so the bugs are not quite so serious.

Obviously, it'd be good to fix for the earlier kernels but it's not
clear it is critical enough for them.
_______________________________________________
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