drm/omap: dss: Remove display ordering from dss/display.c
authorPeter Ujfalusi <peter.ujfalusi@ti.com>
Mon, 12 Feb 2018 09:44:40 +0000 (11:44 +0200)
committerTomi Valkeinen <tomi.valkeinen@ti.com>
Mon, 3 Sep 2018 13:13:25 +0000 (16:13 +0300)
As ordering of the dss_devices based on DT aliases is now implemented in
omap_drm.c, there is no need to do the ordering in dss/display.c
anymore.

At the same time remove the alias member of the omap_dss_device struct
since it is no longer needed. The only place it was used is in the
omapdss_register_display() function.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Sebastian Reichel <sebastian.reichel@collabora.co.uk>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
drivers/gpu/drm/omapdrm/dss/display.c
drivers/gpu/drm/omapdrm/dss/omapdss.h

index 5063274..e1ab159 100644 (file)
@@ -41,7 +41,6 @@ static int disp_num_counter;
 int omapdss_register_display(struct omap_dss_device *dssdev)
 {
        struct omap_dss_driver *drv = dssdev->driver;
-       struct list_head *cur;
        int id;
 
        /*
@@ -54,26 +53,18 @@ int omapdss_register_display(struct omap_dss_device *dssdev)
 
        dssdev->alias_id = id;
 
-       snprintf(dssdev->alias, sizeof(dssdev->alias), "display%d", id);
-
        /* Use 'label' property for name, if it exists */
        of_property_read_string(dssdev->dev->of_node, "label", &dssdev->name);
 
        if (dssdev->name == NULL)
-               dssdev->name = dssdev->alias;
+               dssdev->name = devm_kasprintf(dssdev->dev, GFP_KERNEL,
+                                             "display%u", id);
 
        if (drv && drv->get_timings == NULL)
                drv->get_timings = omapdss_default_get_timings;
 
        mutex_lock(&panel_list_mutex);
-       list_for_each(cur, &panel_list) {
-               struct omap_dss_device *ldev = list_entry(cur,
-                                                        struct omap_dss_device,
-                                                        panel_list);
-               if (strcmp(ldev->alias, dssdev->alias) > 0)
-                       break;
-       }
-       list_add_tail(&dssdev->panel_list, cur);
+       list_add_tail(&dssdev->panel_list, &panel_list);
        mutex_unlock(&panel_list_mutex);
        return 0;
 }
index eae105b..8d53005 100644 (file)
@@ -465,8 +465,6 @@ struct omap_dss_device {
 
        struct list_head panel_list;
 
-       /* alias in the form of "display%d" */
-       char alias[16];
        unsigned int alias_id;
 
        enum omap_display_type type;