On Mon, Mar 31, 2014 at 06:21:30PM +0300, [email protected] wrote: > From: Ville Syrjälä <[email protected]> > > Change the update_plane() plane checks to use enum plane, and > also fix up the error message to say something that's not total > nonsense. > > FIXME killing the checks entirely is probably a better idea
At least killing the totally outdated SAREA comment would be good ;-) But yeah I really don't see much point in this, especially since we'll happily frob plane C on ilk/snb despite that we're only supporting it on ivb+ really. Looking through git history with git blame it seems like this went defunct somewhere in the large modesetting rewrite. Or even earlier ... in any case very confusing history and ripe for the bin. -Daniel > > Signed-off-by: Ville Syrjälä <[email protected]> > --- > drivers/gpu/drm/i915/intel_display.c | 14 +++++++------- > 1 file changed, 7 insertions(+), 7 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_display.c > b/drivers/gpu/drm/i915/intel_display.c > index add940c..9a50b64 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -2167,11 +2167,11 @@ static int i9xx_update_primary_plane(struct drm_crtc > *crtc, > u32 reg; > > switch (plane) { > - case 0: > - case 1: > + case PLANE_A: > + case PLANE_B: > break; > default: > - DRM_ERROR("Can't update plane %c in SAREA\n", > plane_name(plane)); > + DRM_ERROR("Can't update plane %c\n", plane_name(plane)); > return -EINVAL; > } > > @@ -2268,12 +2268,12 @@ static int ironlake_update_primary_plane(struct > drm_crtc *crtc, > u32 reg; > > switch (plane) { > - case 0: > - case 1: > - case 2: > + case PLANE_A: > + case PLANE_B: > + case PLANE_C: > break; > default: > - DRM_ERROR("Can't update plane %c in SAREA\n", > plane_name(plane)); > + DRM_ERROR("Can't update plane %c\n", plane_name(plane)); > return -EINVAL; > } > > -- > 1.8.3.2 > > _______________________________________________ > Intel-gfx mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
