[alsa-devel] [PATCH 2/2] wm8750: add missing VREF output
tiwai at suse.de
Mon Aug 11 13:36:46 CEST 2008
At Mon, 11 Aug 2008 12:01:00 +0100,
Mark Brown wrote:
> 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.
Ah, OK. So it's not for stable. (A detailed changelog would be still
better, though :)
> Obviously, it'd be good to fix for the earlier kernels but it's not
> clear it is critical enough for them.
I see. Thanks for clarification.
More information about the Alsa-devel