From: Ville Syrjälä <ville.syrj...@linux.intel.com>

The global mode_config.rotation_property is going away, switch over to
per-plane rotation_property.

v2: Propagate error upwards (Boris)

Cc: Boris Brezillon <boris.brezillon at free-electrons.com>
Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
---
 drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 37 +++++++++++++------------
 1 file changed, 20 insertions(+), 17 deletions(-)

diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c 
b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
index f3350c80704d..ee9bd7a938c3 100644
--- a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
+++ b/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c
@@ -883,9 +883,9 @@ static int atmel_hlcdc_plane_atomic_get_property(struct 
drm_plane *p,
        return 0;
 }

-static void atmel_hlcdc_plane_init_properties(struct atmel_hlcdc_plane *plane,
-                               const struct atmel_hlcdc_layer_desc *desc,
-                               struct atmel_hlcdc_plane_properties *props)
+static int atmel_hlcdc_plane_init_properties(struct atmel_hlcdc_plane *plane,
+                                            const struct 
atmel_hlcdc_layer_desc *desc,
+                                            struct 
atmel_hlcdc_plane_properties *props)
 {
        struct regmap *regmap = plane->layer.hlcdc->regmap;

@@ -902,10 +902,18 @@ static void atmel_hlcdc_plane_init_properties(struct 
atmel_hlcdc_plane *plane,
                                ATMEL_HLCDC_LAYER_GA_MASK);
        }

-       if (desc->layout.xstride && desc->layout.pstride)
-               drm_object_attach_property(&plane->base.base,
-                               plane->base.dev->mode_config.rotation_property,
-                               BIT(DRM_ROTATE_0));
+       if (desc->layout.xstride && desc->layout.pstride) {
+               int ret;
+
+               ret = drm_plane_create_rotation_property(&plane->base,
+                                                        BIT(DRM_ROTATE_0),
+                                                        BIT(DRM_ROTATE_0) |
+                                                        BIT(DRM_ROTATE_90) |
+                                                        BIT(DRM_ROTATE_180) |
+                                                        BIT(DRM_ROTATE_270));
+               if (ret)
+                       return ret;
+       }

        if (desc->layout.csc) {
                /*
@@ -925,6 +933,8 @@ static void atmel_hlcdc_plane_init_properties(struct 
atmel_hlcdc_plane *plane,
                             ATMEL_HLCDC_LAYER_CSC_CFG(&plane->layer, 2),
                             0x40040890);
        }
+
+       return 0;
 }

 static struct drm_plane_helper_funcs atmel_hlcdc_layer_plane_helper_funcs = {
@@ -1036,7 +1046,9 @@ atmel_hlcdc_plane_create(struct drm_device *dev,
                             &atmel_hlcdc_layer_plane_helper_funcs);

        /* Set default property values*/
-       atmel_hlcdc_plane_init_properties(plane, desc, props);
+       ret = atmel_hlcdc_plane_init_properties(plane, desc, props);
+       if (ret)
+               return ERR_PTR(ret);

        return plane;
 }
@@ -1054,15 +1066,6 @@ atmel_hlcdc_plane_create_properties(struct drm_device 
*dev)
        if (!props->alpha)
                return ERR_PTR(-ENOMEM);

-       dev->mode_config.rotation_property =
-                       drm_mode_create_rotation_property(dev,
-                                                         BIT(DRM_ROTATE_0) |
-                                                         BIT(DRM_ROTATE_90) |
-                                                         BIT(DRM_ROTATE_180) |
-                                                         BIT(DRM_ROTATE_270));
-       if (!dev->mode_config.rotation_property)
-               return ERR_PTR(-ENOMEM);
-
        return props;
 }

-- 
2.7.4

Reply via email to