7 Oct
2011
7 Oct
'11
4:33 p.m.
On Fri, Oct 07, 2011 at 05:18:19PM +0300, Péter Ujfalusi wrote:
On Friday 07 October 2011 14:34:02 Mark Brown wrote:
card->instantiated = 1;
- snd_soc_dapm_sync(&card->dapm);
Without prior snd_soc_dapm_new_widgets() call we just moved the kernel crash here.
As I keep saying new_wigets() is orthogonal to the syncing. They are two totally separate issues. There's a good solid reason why I didn't mention any issues with uninitialized widgets in the changelog which is that this patch is not intended to help with that in any way, it's intended to stop us wasting our time (and potentially introducing pops and clicks from bouncing power) performing needless syncs while starting up.
The syncs that are being suppressed are totally pointless regardless of anything else.