[PATCH] drm: refcnt drm_display_mode

2014-07-27 Thread Daniel Vetter
On Sun, Jul 27, 2014 at 6:20 PM, Rob Clark wrote: > On Sun, Jul 27, 2014 at 11:17 AM, Daniel Vetter wrote: >> On Sat, Jul 26, 2014 at 12:51 AM, Rob Clark wrote: >>> We're going to need this for atomic. >>> >>> Signed-off-by: Rob Clark >> >> I disagree. Iiui correctly Rob's concern is that the a

[PATCH] drm: refcnt drm_display_mode

2014-07-27 Thread Daniel Vetter
On Sat, Jul 26, 2014 at 12:51 AM, Rob Clark wrote: > We're going to need this for atomic. > > Signed-off-by: Rob Clark I disagree. Iiui correctly Rob's concern is that the additional stuff to keep track of mode lists (list_head and the idr stuff) could confuse driver writers into doing stupid st

[PATCH] drm: refcnt drm_display_mode

2014-07-27 Thread Rob Clark
On Sun, Jul 27, 2014 at 1:38 PM, Daniel Vetter wrote: > On Sun, Jul 27, 2014 at 6:20 PM, Rob Clark wrote: >> On Sun, Jul 27, 2014 at 11:17 AM, Daniel Vetter wrote: >>> On Sat, Jul 26, 2014 at 12:51 AM, Rob Clark wrote: We're going to need this for atomic. Signed-off-by: Rob Clark

[PATCH] drm: refcnt drm_display_mode

2014-07-27 Thread Rob Clark
On Sun, Jul 27, 2014 at 11:17 AM, Daniel Vetter wrote: > On Sat, Jul 26, 2014 at 12:51 AM, Rob Clark wrote: >> We're going to need this for atomic. >> >> Signed-off-by: Rob Clark > > I disagree. Iiui correctly Rob's concern is that the additional stuff > to keep track of mode lists (list_head an

[PATCH] drm: refcnt drm_display_mode

2014-07-25 Thread Rob Clark
We're going to need this for atomic. Signed-off-by: Rob Clark --- drivers/gpu/drm/drm_crtc.c| 4 +-- drivers/gpu/drm/drm_crtc_helper.c | 2 +- drivers/gpu/drm/drm_edid.c| 6 ++-- drivers/gpu/drm/drm_fb_helper.c | 6 ++-- drive