Re: [PATCH 0/8] Cleanup CRTC .enable()/.disable() cargo-cult

2017-06-29 Thread Emil Velikov
On 28 June 2017 at 14:10, Laurent Pinchart wrote: > Hi Emil, > > On Wednesday 28 Jun 2017 12:27:06 Emil Velikov wrote: >> On 27 June 2017 at 21:38, Laurent Pinchart wrote: >> > Hello, >> > >> > The atomic helpers favour the .enable() and .atomic_disable() CRTC helper >> > operations, but still sup

Re: [PATCH 0/8] Cleanup CRTC .enable()/.disable() cargo-cult

2017-06-28 Thread Laurent Pinchart
Hi Emil, On Wednesday 28 Jun 2017 12:27:06 Emil Velikov wrote: > On 27 June 2017 at 21:38, Laurent Pinchart wrote: > > Hello, > > > > The atomic helpers favour the .enable() and .atomic_disable() CRTC helper > > operations, but still support the legacy .prepare(), .commit(), .disable() > > and .d

Re: [PATCH 0/8] Cleanup CRTC .enable()/.disable() cargo-cult

2017-06-28 Thread Emil Velikov
Hi Laurent, On 27 June 2017 at 21:38, Laurent Pinchart wrote: > Hello, > > The atomic helpers favour the .enable() and .atomic_disable() CRTC helper > operations, but still support the legacy .prepare(), .commit(), .disable() and > .dpms() operations. Some drivers have been updated, but most stil

Re: [PATCH 0/8] Cleanup CRTC .enable()/.disable() cargo-cult

2017-06-28 Thread Laurent Pinchart
Hi Daniel, On Wednesday 28 Jun 2017 10:20:33 Daniel Vetter wrote: > On Wed, Jun 28, 2017 at 10:16:14AM +0200, Daniel Vetter wrote: > > On Tue, Jun 27, 2017 at 11:38:35PM +0300, Laurent Pinchart wrote: > >> Hello, > >> > >> The atomic helpers favour the .enable() and .atomic_disable() CRTC > >> he

Re: [PATCH 0/8] Cleanup CRTC .enable()/.disable() cargo-cult

2017-06-28 Thread Daniel Vetter
On Wed, Jun 28, 2017 at 10:16:14AM +0200, Daniel Vetter wrote: > On Tue, Jun 27, 2017 at 11:38:35PM +0300, Laurent Pinchart wrote: > > Hello, > > > > The atomic helpers favour the .enable() and .atomic_disable() CRTC helper > > operations, but still support the legacy .prepare(), .commit(), .disab

Re: [PATCH 0/8] Cleanup CRTC .enable()/.disable() cargo-cult

2017-06-28 Thread Daniel Vetter
On Tue, Jun 27, 2017 at 11:38:35PM +0300, Laurent Pinchart wrote: > Hello, > > The atomic helpers favour the .enable() and .atomic_disable() CRTC helper > operations, but still support the legacy .prepare(), .commit(), .disable() and > .dpms() operations. Some drivers have been updated, but most s

[PATCH 0/8] Cleanup CRTC .enable()/.disable() cargo-cult

2017-06-27 Thread Laurent Pinchart
Hello, The atomic helpers favour the .enable() and .atomic_disable() CRTC helper operations, but still support the legacy .prepare(), .commit(), .disable() and .dpms() operations. Some drivers have been updated, but most still use various combination of new and legacy operations, leading to confus