2009/12/19 Luca Tettamanti <kronos...@gmail.com>: > Hello, > I'm currently testing kernel 2.6.32-rc1 and the first thing I noticed is: > > [drm] Enabling audio support > > ..that, and 10 wakeup/s ;-) I'd like to share a couple comments: > > - first of all, I have a M56 (RV630) which lacks the PCI audio > function, I guess in this case the audio shouldn't be enabled at all > - the r600_audio_update_hdmi callback: is it really necessary to call > it 10 times/sec, even if no audio is playing? In a "normal" audio > driver that would be done only on ALSA ->open (or maybe ->prepare) > functions, but in this case it's not clear to me how the two pieces > (drm and alsa) interacts (can someone point me to the relevant docs?). > Maybe a tighter integration between them is needed?
We have to check what sound comes to GPU and program registers to support it. That's what mentioned update function does. In future we will use IRQ for that, GPU will just tell us that audio has changed and we need to update. For now we could try to stop that timer where there is not HDMI, but not sure if this is worth of that, as we have IRQ working on Dave's local (not public) tree. -- Rafał ------------------------------------------------------------------------------ This SF.Net email is sponsored by the Verizon Developer Community Take advantage of Verizon's best-in-class app development support A streamlined, 14 day to market process makes app distribution fast and easy Join now and get one step closer to millions of Verizon customers http://p.sf.net/sfu/verizon-dev2dev -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel
