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. thanks, Takashi _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel