Re: [PATCH] [rfc] drm/radeon/kms: pm debugging check for vbl.

2010-02-23 Thread Luca Tettamanti
2010/2/18 Dave Airlie airl...@gmail.com: 2010/2/18 Rafał Miłecki zaj...@gmail.com: W dniu 18 lutego 2010 09:28 użytkownik Dave Airlie airl...@gmail.com napisał: It adds some reading printing steps before every reclock, while we really want it to happen as soon as possible. Maybe you could

Re: [PATCH] [rfc] drm/radeon/kms: pm debugging check for vbl.

2010-02-18 Thread Rafał Miłecki
2010/2/18 Dave Airlie airl...@gmail.com: From: Dave Airlie airl...@redhat.com This patch adds a check on avivo chips to see if we are in the VBL region for the active crtcs when we trigger the engine change. I appear to have glitches locally on pm transistion (not sure all fixes are in yet)

Re: [PATCH] [rfc] drm/radeon/kms: pm debugging check for vbl.

2010-02-18 Thread Dave Airlie
2010/2/18 Rafał Miłecki zaj...@gmail.com: 2010/2/18 Dave Airlie airl...@gmail.com: From: Dave Airlie airl...@redhat.com This patch adds a check on avivo chips to see if we are in the VBL region for the active crtcs when we trigger the engine change. I appear to have glitches locally on pm

Re: [PATCH] [rfc] drm/radeon/kms: pm debugging check for vbl.

2010-02-18 Thread Dave Airlie
It adds some reading printing steps before every reclock, while we really want it to happen as soon as possible. Maybe you could execute this only on some btw you won't get the print if you are in vblank, and if you aren't well the print doesn't matter, I'm also thinking the engine change

Re: [PATCH] [rfc] drm/radeon/kms: pm debugging check for vbl.

2010-02-18 Thread Rafał Miłecki
W dniu 18 lutego 2010 09:28 użytkownik Dave Airlie airl...@gmail.com napisał: It adds some reading printing steps before every reclock, while we really want it to happen as soon as possible. Maybe you could execute this only on some btw you won't get the print if you are in vblank, and if

Re: [PATCH] [rfc] drm/radeon/kms: pm debugging check for vbl.

2010-02-18 Thread Dave Airlie
2010/2/18 Rafał Miłecki zaj...@gmail.com: W dniu 18 lutego 2010 09:28 użytkownik Dave Airlie airl...@gmail.com napisał: It adds some reading printing steps before every reclock, while we really want it to happen as soon as possible. Maybe you could execute this only on some btw you won't