On Tue, Mar 11, 2014 at 04:45:00PM -0700, Bob Paauwe wrote: > On Fri, 7 Mar 2014 16:03:17 -0800 > Matt Roper <matthew.d.roper@xxxxxxxxx> wrote: ... > > +static int drm_mode_create_standard_plane_properties(struct drm_device *dev) > > +{ > > + struct drm_property *type; > > + > > + /* > > + * Standard properties (apply to all planes) > > + */ > > + type = drm_property_create_enum(dev, 0, > > + "TYPE", drm_plane_type_enum_list, > > + ARRAY_SIZE(drm_plane_type_enum_list)); > > Should this property be created with DRM_MODE_PROP_IMMUTABLE set? What > happens if user space tries to change this property to a different type? > Yep, good point. I'll add that flag in my next revision of the patchset. Thanks! Matt -- Matt Roper Graphics Software Engineer ISG Platform Enabling & Development Intel Corporation (916) 356-2795 _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel