On Thu, Mar 05, 2015 at 01:56:53PM +0100, Daniel Vetter wrote: > On Thu, Mar 05, 2015 at 02:51:28PM +0530, Sonika Jindal wrote: > > @@ -1519,16 +1550,7 @@ intel_plane_init(struct drm_device *dev, enum pipe > > pipe, int plane) > > goto out; > > } > > > > - if (!dev->mode_config.rotation_property) > > - dev->mode_config.rotation_property = > > - drm_mode_create_rotation_property(dev, > > - BIT(DRM_ROTATE_0) | > > - BIT(DRM_ROTATE_180)); > > - > > - if (dev->mode_config.rotation_property) > > - drm_object_attach_property(&intel_plane->base.base, > > - dev->mode_config.rotation_property, > > - state->base.rotation); > > + intel_create_rotation_property(dev, intel_plane); > > I think back from the original rotation work we've had the leftover task > to move this into common code so that we do create the property just once > without this check. > > I think this should be done now.
Someone should also make it so we can again have different supported rotation bits on different planes. I'll have need for it on CHV I think. > -Daniel > > > > drm_plane_helper_add(&intel_plane->base, &intel_plane_helper_funcs); > > > > -- > > 1.7.10.4 > > > > _______________________________________________ > > 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 -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
