[Intel-gfx] [PULL] topic/vblank-rework

2014-09-11 Thread Daniel Vetter
Hi Mario, Can you please take a look at the patches I've submitted and review them (at least the first 2)? Dave will close the 3.18 drm-next merge window at the end of this week and I'd like to really get this in. Thanks, Daniel On Wed, Sep 10, 2014 at 5:45 PM, Mario Kleiner wrote: > On Wed,

[Intel-gfx] [PULL] topic/vblank-rework

2014-09-10 Thread Mario Kleiner
On Wed, Sep 10, 2014 at 5:29 PM, Daniel Vetter wrote: > On Wed, Sep 10, 2014 at 4:19 PM, Mario Kleiner > wrote: >> Hmm, not quite an ack from my side for the pull in its current form. I >> said if the two remaining issues i mentioned are addressed, then i'm >> happy with it and can have my

[Intel-gfx] [PULL] topic/vblank-rework

2014-09-10 Thread Daniel Vetter
On Wed, Sep 10, 2014 at 4:19 PM, Mario Kleiner wrote: > Hmm, not quite an ack from my side for the pull in its current form. I > said if the two remaining issues i mentioned are addressed, then i'm > happy with it and can have my reviewed/acked-by. Looking at the code > they haven't been

[Intel-gfx] [PULL] topic/vblank-rework

2014-09-10 Thread Mario Kleiner
e-mail snafu, sent it too early by accident, and from a gmail web interface which i'm apparently incapable of using properly... The second fix should look like this: > A simple fix in static void vblank_disable_and_save() would be to > replace the new... > > if (!vblank->enabled) { > > ... check

[Intel-gfx] [PULL] topic/vblank-rework

2014-09-10 Thread Mario Kleiner
Hmm, not quite an ack from my side for the pull in its current form. I said if the two remaining issues i mentioned are addressed, then i'm happy with it and can have my reviewed/acked-by. Looking at the code they haven't been adressed. However, this is easily fixable on top of the current