Re: [PATCH 3/4] drm: Only lastclose on unload for legacy drivers

2017-08-11 Thread Daniel Vetter
On Thu, Aug 03, 2017 at 01:52:55PM -0400, Alex Deucher wrote: > On Thu, Aug 3, 2017 at 9:54 AM, Daniel Vetter wrote: > > On Thu, Aug 3, 2017 at 1:17 AM, Daniel Vetter > > wrote: > >> On Wed, Aug 2, 2017 at 10:50 PM, Alex Deucher > >> wrote: > >>> On Wed, Aug 2, 2017 at 7:56 AM, Daniel Vetter

Re: [PATCH 3/4] drm: Only lastclose on unload for legacy drivers

2017-08-04 Thread Daniel Vetter
On Fri, Aug 4, 2017 at 6:12 AM, Michel Dänzer wrote: > On 03/08/17 10:54 PM, Daniel Vetter wrote: >> On Thu, Aug 3, 2017 at 1:17 AM, Daniel Vetter wrote: >>> On Wed, Aug 2, 2017 at 10:50 PM, Alex Deucher wrote: On Wed, Aug 2, 2017 at 7:56 AM, Daniel Vetter wrote: > The only thing

Re: [PATCH 3/4] drm: Only lastclose on unload for legacy drivers

2017-08-03 Thread Michel Dänzer
On 03/08/17 10:54 PM, Daniel Vetter wrote: > On Thu, Aug 3, 2017 at 1:17 AM, Daniel Vetter wrote: >> On Wed, Aug 2, 2017 at 10:50 PM, Alex Deucher wrote: >>> On Wed, Aug 2, 2017 at 7:56 AM, Daniel Vetter >>> wrote: The only thing modern drivers are supposed to do in lastclose is resto

Re: [PATCH 3/4] drm: Only lastclose on unload for legacy drivers

2017-08-03 Thread Alex Deucher
On Thu, Aug 3, 2017 at 9:54 AM, Daniel Vetter wrote: > On Thu, Aug 3, 2017 at 1:17 AM, Daniel Vetter wrote: >> On Wed, Aug 2, 2017 at 10:50 PM, Alex Deucher wrote: >>> On Wed, Aug 2, 2017 at 7:56 AM, Daniel Vetter >>> wrote: The only thing modern drivers are supposed to do in lastclose is

Re: [PATCH 3/4] drm: Only lastclose on unload for legacy drivers

2017-08-03 Thread Daniel Vetter
On Thu, Aug 3, 2017 at 1:17 AM, Daniel Vetter wrote: > On Wed, Aug 2, 2017 at 10:50 PM, Alex Deucher wrote: >> On Wed, Aug 2, 2017 at 7:56 AM, Daniel Vetter wrote: >>> The only thing modern drivers are supposed to do in lastclose is >>> restore the fb emulation state. Which is entirely optional,

Re: [PATCH 3/4] drm: Only lastclose on unload for legacy drivers

2017-08-02 Thread Daniel Vetter
On Wed, Aug 2, 2017 at 10:50 PM, Alex Deucher wrote: > On Wed, Aug 2, 2017 at 7:56 AM, Daniel Vetter wrote: >> The only thing modern drivers are supposed to do in lastclose is >> restore the fb emulation state. Which is entirely optional, and >> there's really no reason to do that. So restrict it

Re: [PATCH 3/4] drm: Only lastclose on unload for legacy drivers

2017-08-02 Thread Alex Deucher
On Wed, Aug 2, 2017 at 7:56 AM, Daniel Vetter wrote: > The only thing modern drivers are supposed to do in lastclose is > restore the fb emulation state. Which is entirely optional, and > there's really no reason to do that. So restrict it to legacy drivers > (where the driver cleanup essentially

[PATCH 3/4] drm: Only lastclose on unload for legacy drivers

2017-08-02 Thread Daniel Vetter
The only thing modern drivers are supposed to do in lastclose is restore the fb emulation state. Which is entirely optional, and there's really no reason to do that. So restrict it to legacy drivers (where the driver cleanup essentially happens in lastclose). This will also allow us to share the u