[Intel-gfx] [PATCH] drm/crtc: constify drm_crtc_index parameter

2016-10-10 Thread Ville Syrjälä
On Mon, Oct 10, 2016 at 06:10:42PM +0200, Daniel Vetter wrote: > On Mon, Oct 10, 2016 at 6:04 PM, Ville Syrjälä > wrote: > > On Mon, Oct 10, 2016 at 06:26:10PM +0300, Jani Nikula wrote: > >> Signed-off-by: Jani Nikula > >> > >> --- > >> > >> I needed this for some stuff that turned out to be a

[Intel-gfx] [PATCH] drm/crtc: constify drm_crtc_index parameter

2016-10-10 Thread Daniel Vetter
On Mon, Oct 10, 2016 at 6:30 PM, Ville Syrjälä wrote: >> You can't hotplug CRTCs, like you can't hotplug anything else than >> connectors. And for connectors we have an ida to make sure you never >> end up with duplicated indizes. So I think we're safe. > > Except in i915 we don't abort the driv

[Intel-gfx] [PATCH] drm/crtc: constify drm_crtc_index parameter

2016-10-10 Thread Daniel Vetter
On Mon, Oct 10, 2016 at 6:04 PM, Ville Syrjälä wrote: > On Mon, Oct 10, 2016 at 06:26:10PM +0300, Jani Nikula wrote: >> Signed-off-by: Jani Nikula >> >> --- >> >> I needed this for some stuff that turned out to be a dead end. But this >> seems to be the right thing to do anyway. >> --- >> incl

[Intel-gfx] [PATCH] drm/crtc: constify drm_crtc_index parameter

2016-10-10 Thread Daniel Vetter
On Mon, Oct 10, 2016 at 06:26:10PM +0300, Jani Nikula wrote: > Signed-off-by: Jani Nikula > > --- > > I needed this for some stuff that turned out to be a dead end. But this > seems to be the right thing to do anyway. Applied to drm-misc. There's also the connector and plane versions of this, c

[Intel-gfx] [PATCH] drm/crtc: constify drm_crtc_index parameter

2016-10-10 Thread Chris Wilson
On Mon, Oct 10, 2016 at 06:26:10PM +0300, Jani Nikula wrote: > Signed-off-by: Jani Nikula Plus equivalents in drm_plane.h, drm_encoder.h ... Reviewed-by: Chris Wilson -Chris -- Chris Wilson, Intel Open Source Technology Centre