-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Somebody in the thread at some point said: | Hey, | | On 16/06/2008, Andy Green <[EMAIL PROTECTED]> wrote: |> Two issues... we never took care to take down engines in suspend |> and bring them back in resume. This was part of the display |> corruption that could be seen briefly on resume. The other issue |> that made the "noise" corruption was bad ordering of resume steps. |> |> This patch simplifies (removing needless re-init) resume actions |> and makes explicit the suspend and resume steps. It also adds |> code to track which engines are up and push them down in suspend |> and bring them back in resume. |> |> The result is no more corruption of display buffer in suspend, it |> comes back completely clean. | | IIRC previously we thought the corruption was a result of Glamo being | powered down. Now that there's no corruption wouldn't it mean that | video RAM is staying powered up needlessly? (Just wondering)
It was always staying powered up with the Glamo DRAM in "Deep Powerdown" doing self-refresh. In fact we can't switch the Glamo off on GTA02. The differences are around taking care to disable the engines from the clocks before we trash the PLL (which I think made the "lines" type corruption) and reordering the steps to manage the memory private states with selfrefresh and deep powerdown also before we kill the PLLs, which removed the "noise" type corruption. Hey while you're here well done about your MP4 / Glamo work stitching the Harald / Olv stuff actually into mplayer. I saw you had a little trouble with the engine held in reset, it seems we should export something that lets you do bring the engine up in the kernel, what do you think? - -Andy -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.9 (GNU/Linux) Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org iEYEARECAAYFAkhWBrQACgkQOjLpvpq7dMqLTwCfZc3Gn3c4dWwnEyudU6nnkHBG IyMAn2KZagWZB36gIN6Wh0DwaOk1ixQW =iB1/ -----END PGP SIGNATURE-----
