Oh I should probably have dropped all the audio bits, I didn't even see this reply before I updated to v11.
The r600 audio code is a bit of disaster area hopefully we can clean it up, like the timer was firing after the device was suspended. I'll repost with all that r600 audio ripped out and you can fix the mess. > > Don't mess with r600_audio_* there. You call if for all chipsets which > is not needed and break nice S&R layout, which is chipset specific. > > I guess you needed that because you didn't work on branch containing my patch: > http://git.kernel.org/?p=linux/kernel/git/airlied/drm-2.6.git;a=commitdiff;h=38fd2c6ff526e6a59edfa8e08f6f0724646784c4 > (you commited it to drm-linus) This patch is now based on Linus tree, but yeah the original version predates your patches. >> -int radeon_audio = 1; >> +int radeon_audio = 0; > > Why?! What for we disable this feature by default?! If you see some > reason for that, explain it to others please. I can change my mind, > but for now I don't like this. It makes audio an option from just > working-out-of-box. > Oversights, the audio was just broken in the presence of suspend/resume when I wrote it. Dave. ------------------------------------------------------------------------------ Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications for parallel performance. See why Intel Parallel Studio got high marks during beta. http://p.sf.net/sfu/intel-sw-dev -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel
