2013/4/14 Alex Deucher <alexdeucher at gmail.com>: > On Sun, Apr 14, 2013 at 11:55 AM, Rafa? Mi?ecki <zajec5 at gmail.com> wrote: >> 2013/4/14 Alex Deucher <alexdeucher at gmail.com>: >>> On Sat, Apr 13, 2013 at 7:26 PM, Rafa? Mi?ecki <zajec5 at gmail.com> wrote: >>>> We need interrupts on format change for R6xx only, where hardware seems >>>> to be somehow bugged and requires setting audio info manually. >>> >>> Can you confirm that this is actually needed on older chips? AFAIK, >>> it shouldn't be required for any chips. It's mainly for debugging. >> >> I can't really right now :( My notebook with RV620 died (hard disk >> ended it's life and power cable got broken). I hope to resurrect him >> in about a week. >> >> If that isn't needed on R6xx, I'm not sure why we implemented it in >> first place at all. Christian? Do you have idea why this was required? >> I remember than in first place we were using timer, then we switched >> to the interrupts. But why we needed it at all? > > I suspect it was just assumed to be necessary due to the original RE.
I'm OK with removing that from R6xx too, if it's not needed. I just want to check that first, to don't break audio accidentally. In case of Evergreen I was able to test it, so I dares to submit this patch ;) -- Rafa?
