On Thu, Jan 22, 2015 at 08:04:33AM +0100, Daniel Vetter wrote: [...] > diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c [...] > @@ -1391,6 +1395,12 @@ static int drm_mode_create_standard_properties(struct > drm_device *dev) > return -ENOMEM; > dev->mode_config.prop_crtc_id = prop; > > + prop = drm_property_create_bool(dev, DRM_MODE_PROP_ATOMIC, > + "ACTIVE");
We seem to have a weird mix of property names. Some are all caps, others all lowercase. I guess we can't really make it consistent anymore since the only one so far that stands out is "type" and we've already exposed that to userspace, so ABI... Otherwise, this looks good, but I don't feel like I understand this good enough to give my Reviewed-by. I'll go give this a spin on Tegra, see if that improves my understanding. Thierry
pgpYUynmP1czG.pgp
Description: PGP signature
_______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
