From mboxrd@z Thu Jan 1 00:00:00 1970 From: John Keeping Subject: [PATCH 0/3] drm/rockchip: fix cursor performance with atomic Date: Thu, 14 Jan 2016 14:39:39 +0000 Message-ID: References: <20160114142047.GD19130@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20160114142047.GD19130@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Mark yao Cc: Linux Kernel Mailing List , dri-devel , "open list:ARM/Rockchip SoC..." , "linux-arm-kernel@lists.infradead.org" List-Id: linux-rockchip.vger.kernel.org T24gVGh1LCAxNCBKYW4gMjAxNiAxNToyMDo0NyArMDEwMCwgRGFuaWVsIFZldHRlciB3cm90ZToK Cj4gVWdoLiBPaCB3ZWxsLCB0aGVyZSdzIG5vdCByZWFsbHkgYW55dGhpbmcgd2UgY2FuIGRvIGlu IGNvcmUgbm9yZSBoZWxwZXJzCj4gdG8gbWFrZSB0aGlzIGVhc2llciBmb3IgZHJpdmVycy4gVGhp cyByZWFsbHkgb25seSBjYW4gYmUgZml4ZWQgc2Vuc2libHkgYXQKPiB0aGUgaGFyZHdhcmUgbGV2 ZWwuCj4gCj4gU28geWVhaCBJIHRoaW5rIGV4cG9zaW5nIGZyYW1lYnVmZmVyX2NoYW5nZWQgYXMg YSBoZWxwZXIgaXMgdGhlIHdheSB0byBnbwo+IGhlcmUuCgpPSywgaGVyZSdzIGEgc2VyaWVzIHRv IGRvIHRoYXQuICBJIGFsc28gYWRkZWQgYSBjb21tZW50IHRvCnJvY2tjaGlwX2F0b21pY193YWl0 X2Zvcl9jb21wbGV0ZSgpIGV4cGxhaW5pbmcgd2h5IHdlIGNhbid0IHVzZQpkcm1fYXRvbWljX2hl bHBlcl93YWl0X2Zvcl92YmxhbmtzKCkuCgpKb2huIEtlZXBpbmcgKDMpOgogIGRybS9hdG9taWMt aGVscGVyOiBFeHBvcnQgZnJhbWVidWZmZXJfY2hhbmdlZCgpCiAgZHJtL3JvY2tjaGlwOiBkb24n dCB3YWl0IGZvciB2YmxhbmsgaWYgZmIgaGFzbid0IGNoYW5nZWQKICBkcm0vcm9ja2NoaXA6IGV4 cGxhaW4gd2h5IHdlIGNhbid0IHdhaXRfZm9yX3ZibGFua3MKCiBkcml2ZXJzL2dwdS9kcm0vZHJt X2F0b21pY19oZWxwZXIuYyAgICAgICAgfCAyNCArKysrKysrKysrKysrKysrKysrKy0tLS0KIGRy aXZlcnMvZ3B1L2RybS9yb2NrY2hpcC9yb2NrY2hpcF9kcm1fZmIuYyB8IDE0ICsrKysrKysrKysr Ky0tCiBpbmNsdWRlL2RybS9kcm1fYXRvbWljX2hlbHBlci5oICAgICAgICAgICAgfCAgNCArKysr CiAzIGZpbGVzIGNoYW5nZWQsIDM2IGluc2VydGlvbnMoKyksIDYgZGVsZXRpb25zKC0pCgotLSAK Mi43LjAuMjI2LmdmZTk4NmZlCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVz a3RvcC5vcmcKaHR0cDovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2Ry aS1kZXZlbAo= From mboxrd@z Thu Jan 1 00:00:00 1970 From: john@metanate.com (John Keeping) Date: Thu, 14 Jan 2016 14:39:39 +0000 Subject: [PATCH 0/3] drm/rockchip: fix cursor performance with atomic In-Reply-To: <20160114142047.GD19130@phenom.ffwll.local> References: <20160114142047.GD19130@phenom.ffwll.local> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, 14 Jan 2016 15:20:47 +0100, Daniel Vetter wrote: > Ugh. Oh well, there's not really anything we can do in core nore helpers > to make this easier for drivers. This really only can be fixed sensibly at > the hardware level. > > So yeah I think exposing framebuffer_changed as a helper is the way to go > here. OK, here's a series to do that. I also added a comment to rockchip_atomic_wait_for_complete() explaining why we can't use drm_atomic_helper_wait_for_vblanks(). John Keeping (3): drm/atomic-helper: Export framebuffer_changed() drm/rockchip: don't wait for vblank if fb hasn't changed drm/rockchip: explain why we can't wait_for_vblanks drivers/gpu/drm/drm_atomic_helper.c | 24 ++++++++++++++++++++---- drivers/gpu/drm/rockchip/rockchip_drm_fb.c | 14 ++++++++++++-- include/drm/drm_atomic_helper.h | 4 ++++ 3 files changed, 36 insertions(+), 6 deletions(-) -- 2.7.0.226.gfe986fe From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755278AbcANOkd (ORCPT ); Thu, 14 Jan 2016 09:40:33 -0500 Received: from dougal.metanate.com ([90.155.101.14]:35692 "EHLO metanate.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1755226AbcANOk3 (ORCPT ); Thu, 14 Jan 2016 09:40:29 -0500 From: John Keeping To: Mark yao Cc: John Keeping , Daniel Vetter , Linux Kernel Mailing List , dri-devel , "open list:ARM/Rockchip SoC..." , "linux-arm-kernel@lists.infradead.org" Subject: [PATCH 0/3] drm/rockchip: fix cursor performance with atomic Date: Thu, 14 Jan 2016 14:39:39 +0000 Message-Id: X-Mailer: git-send-email 2.7.0.226.gfe986fe In-Reply-To: <20160114142047.GD19130@phenom.ffwll.local> References: <20160114142047.GD19130@phenom.ffwll.local> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 14 Jan 2016 15:20:47 +0100, Daniel Vetter wrote: > Ugh. Oh well, there's not really anything we can do in core nore helpers > to make this easier for drivers. This really only can be fixed sensibly at > the hardware level. > > So yeah I think exposing framebuffer_changed as a helper is the way to go > here. OK, here's a series to do that. I also added a comment to rockchip_atomic_wait_for_complete() explaining why we can't use drm_atomic_helper_wait_for_vblanks(). John Keeping (3): drm/atomic-helper: Export framebuffer_changed() drm/rockchip: don't wait for vblank if fb hasn't changed drm/rockchip: explain why we can't wait_for_vblanks drivers/gpu/drm/drm_atomic_helper.c | 24 ++++++++++++++++++++---- drivers/gpu/drm/rockchip/rockchip_drm_fb.c | 14 ++++++++++++-- include/drm/drm_atomic_helper.h | 4 ++++ 3 files changed, 36 insertions(+), 6 deletions(-) -- 2.7.0.226.gfe986fe