drm: omapdrm: Use sizeof(*var) instead of sizeof(type) for structures

By linking the sizeof to a variable type the code will be less prone to
bugs due to future type changes of variables.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com>

diff --git a/drivers/gpu/drm/omapdrm/displays/panel-dsi-cm.c b/drivers/gpu/drm/omapdrm/displays/panel-dsi-cm.c
index dc026a8437123f44df161e57eb8ec7045439c25f..a2bb855a2851f7c8df668a9669ac4b50afc36273 100644 (file)
--- a/drivers/gpu/drm/omapdrm/displays/panel-dsi-cm.c
+++ b/drivers/gpu/drm/omapdrm/displays/panel-dsi-cm.c
@@ -1253,7 +1253,7 @@ static int dsicm_probe(struct platform_device *pdev)
        dsicm_hw_reset(ddata);
 
        if (ddata->use_dsi_backlight) {
-               memset(&props, 0, sizeof(struct backlight_properties));
+               memset(&props, 0, sizeof(props));
                props.max_brightness = 255;
 
                props.type = BACKLIGHT_RAW;

diff --git a/drivers/gpu/drm/omapdrm/dss/omapdss-boot-init.c b/drivers/gpu/drm/omapdrm/dss/omapdss-boot-init.c
index 136d30484d02345430a1875d49acffa836c8a059..bf626acae2712b3a7eff53ec148a210f910f5c6d 100644 (file)
--- a/drivers/gpu/drm/omapdrm/dss/omapdss-boot-init.c
+++ b/drivers/gpu/drm/omapdrm/dss/omapdss-boot-init.c
@@ -119,8 +119,7 @@ static void __init omapdss_omapify_node(struct device_node *node)
 
 static void __init omapdss_add_to_list(struct device_node *node, bool root)
 {
-       struct dss_conv_node *n = kmalloc(sizeof(struct dss_conv_node),
-               GFP_KERNEL);
+       struct dss_conv_node *n = kmalloc(sizeof(*n), GFP_KERNEL);
        if (n) {
                n->node = node;
                n->root = root;

diff --git a/drivers/gpu/drm/omapdrm/omap_connector.c b/drivers/gpu/drm/omapdrm/omap_connector.c
index 2580e8673908a8b72847aefe4e17954b71315c9a..691cffebb76ef4b1d3e51929afdef547a546750a 100644 (file)
--- a/drivers/gpu/drm/omapdrm/omap_connector.c
+++ b/drivers/gpu/drm/omapdrm/omap_connector.c
@@ -162,7 +162,7 @@ static int omap_connector_mode_valid(struct drm_connector *connector,
 
                dssdrv->get_timings(dssdev, &t);
 
-               if (memcmp(&vm, &t, sizeof(struct videomode)))
+               if (memcmp(&vm, &t, sizeof(vm)))
                        r = -EINVAL;
                else
                        r = 0;
@@ -217,7 +217,7 @@ struct drm_connector *omap_connector_init(struct drm_device *dev,
 
        omap_dss_get_device(dssdev);
 
-       omap_connector = kzalloc(sizeof(struct omap_connector), GFP_KERNEL);
+       omap_connector = kzalloc(sizeof(*omap_connector), GFP_KERNEL);
        if (!omap_connector)
                goto fail;
 

diff --git a/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c b/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c
index 4ceed7a9762f1a8309f84441154550878a8e4bbc..3cab06661a0830f802f89b70b9162cc2eb5b4859 100644 (file)
--- a/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c
+++ b/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c
@@ -224,7 +224,7 @@ static void dmm_txn_append(struct dmm_txn *txn, struct pat_area *area,
        int rows = (1 + area->y1 - area->y0);
        int i = columns*rows;
 
-       pat = alloc_dma(txn, sizeof(struct pat), &pat_pa);
+       pat = alloc_dma(txn, sizeof(*pat), &pat_pa);
 
        if (txn->last_pat)
                txn->last_pat->next_pa = (uint32_t)pat_pa;
@@ -735,7 +735,7 @@ static int omap_dmm_probe(struct platform_device *dev)
 
        /* alloc engines */
        omap_dmm->engines = kcalloc(omap_dmm->num_engines,
-                                   sizeof(struct refill_engine), GFP_KERNEL);
+                                   sizeof(*omap_dmm->engines), GFP_KERNEL);
        if (!omap_dmm->engines) {
                ret = -ENOMEM;
                goto fail;

diff --git a/drivers/gpu/drm/omapdrm/omap_encoder.c b/drivers/gpu/drm/omapdrm/omap_encoder.c
index a20f30039aee6c158c9ca7e89e6324a09a3ca667..86c977b7189a567afc2314c7bb4c140a40181a77 100644 (file)
--- a/drivers/gpu/drm/omapdrm/omap_encoder.c
+++ b/drivers/gpu/drm/omapdrm/omap_encoder.c
@@ -117,7 +117,7 @@ static int omap_encoder_update(struct drm_encoder *encoder,
 
                dssdrv->get_timings(dssdev, &t);
 
-               if (memcmp(vm, &t, sizeof(struct videomode)))
+               if (memcmp(vm, &t, sizeof(*vm)))
                        ret = -EINVAL;
                else
                        ret = 0;