drm/qxl: Fix qxl_noop_get_vblank_counter()
authorMario Kleiner <mario.kleiner.de@gmail.com>
Mon, 4 May 2015 04:29:47 +0000 (06:29 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Mon, 4 May 2015 09:14:55 +0000 (11:14 +0200)
This breaks under the vblank timestamp cleanup patch
by Daniel Vetter. Also it is pointless to return anything
but zero (or any other constant) if the function doesn't
actually query a hw vblank counter. The bogus return of
the current drm vblank counter via direct readout or via
drm_vblank_count() is found in many of the new kms drivers,
but it does exactly nothing different from returning any
arbitrary constant - it's a no operation.

Let's simply return 0 - Easy and fast.

Signed-off-by: Mario Kleiner <mario.kleiner.de@gmail.com>
Cc: Dave Airlie <airlied@redhat.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/qxl/qxl_drv.c

index e2d07085b6a55ab506d51935a25c6ea29ec32e5d..83f6f0b5e9efa292e83fae0c2b9ea45d4c7241b2 100644 (file)
@@ -198,7 +198,7 @@ static int qxl_pm_restore(struct device *dev)
 
 static u32 qxl_noop_get_vblank_counter(struct drm_device *dev, int crtc)
 {
-       return dev->vblank[crtc].count.counter;
+       return 0;
 }
 
 static int qxl_noop_enable_vblank(struct drm_device *dev, int crtc)