diff mbox

[1/2] drm/vc4: Fix get_vblank_counter with proper no-op for Linux 4.4+

Message ID 1462555566-27612-2-git-send-email-mario.kleiner.de@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Mario Kleiner May 6, 2016, 5:26 p.m. UTC
get_vblank_counter hooked up to drm_vblank_count() which alway was
non-sensical but didn't hurt in the past. Since Linux 4.4 it
triggers a WARN_ON_ONCE in drm_update_vblank_count on first vblank
irq disable, so fix it by hooking to drm_vblank_no_hw_counter().

Tested against Raspian kernel 4.4.8 tree on RPi 2B.

Signed-off-by: Mario Kleiner <mario.kleiner.de@gmail.com>
Cc: Eric Anholt <eric@anholt.net>
---
 drivers/gpu/drm/vc4/vc4_drv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Eric Anholt May 10, 2016, 12:42 a.m. UTC | #1
Mario Kleiner <mario.kleiner.de@gmail.com> writes:

> get_vblank_counter hooked up to drm_vblank_count() which alway was
> non-sensical but didn't hurt in the past. Since Linux 4.4 it
> triggers a WARN_ON_ONCE in drm_update_vblank_count on first vblank
> irq disable, so fix it by hooking to drm_vblank_no_hw_counter().
>
> Tested against Raspian kernel 4.4.8 tree on RPi 2B.

These are both:

Reviewed-by: Eric Anholt <eric@anholt.net>
diff mbox

Patch

diff --git a/drivers/gpu/drm/vc4/vc4_drv.c b/drivers/gpu/drm/vc4/vc4_drv.c
index b7d2ff0..962d0d7 100644
--- a/drivers/gpu/drm/vc4/vc4_drv.c
+++ b/drivers/gpu/drm/vc4/vc4_drv.c
@@ -90,7 +90,7 @@  static struct drm_driver vc4_drm_driver = {
 
 	.enable_vblank = vc4_enable_vblank,
 	.disable_vblank = vc4_disable_vblank,
-	.get_vblank_counter = drm_vblank_count,
+	.get_vblank_counter = drm_vblank_no_hw_counter,
 
 #if defined(CONFIG_DEBUG_FS)
 	.debugfs_init = vc4_debugfs_init,