[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <4077998.GJIlik9CKQ@phil>
Date: Sat, 24 Jan 2015 16:14:54 +0100
From: Heiko Stübner <heiko@...ech.de>
To: dri-devel@...ts.freedesktop.org
Cc: Mark Yao <mark.yao@...k-chips.com>,
David Airlie <airlied@...ux.ie>,
Daniel Vetter <daniel@...ll.ch>,
Rob Clark <robdclark@...il.com>,
Philipp Zabel <p.zabel@...gutronix.de>,
Daniel Kurtz <djkurtz@...omium.org>,
linux-kernel@...r.kernel.org, linux-rockchip@...ts.infradead.org
Subject: Re: [PATCH v2 0/2] drm/rockchip: Optimization vop dpms control
Am Donnerstag, 22. Januar 2015, 18:29:55 schrieb Mark Yao:
> drm dpms have many power modes, ON,OFF,SUSPEND,STANDBY, etc.
> but vop only have enable/disable mode, maybe case such bug:
> --> DRM_DPMS_ON: power on vop
> --> DRM_DPMS_SUSPEND: power off vop
> --> DRM_DPMS_OFF: already power off at SUSPEND, crash
> so use a bool val is more suitable.
>
> another problem at vop_crtc_dpms:
> vop_enable()->drm_vblank_on, drm_vblank_on may call vop
> enable vblank. if it happen, vblank enable would failed,
> then cause irq status error. because is_enabled value is set
> after drm_vblank_on.
This series, on a rk3288-firefly board with a sda7123 vga connector
Tested-by: Heiko Stuebner <heiko@...ech.de>
Heiko
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists