diff options
author | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2016-12-12 04:57:24 -0500 |
---|---|---|
committer | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2016-12-19 04:25:05 -0500 |
commit | d501b12999eb9cdfc488127fabaf794171dd9bbf (patch) | |
tree | 4b97a912c76f6ac6d2cf6c62fc82540d8be47f58 /drivers/gpu/drm/omapdrm/omap_encoder.c | |
parent | 84e1d4578fe9de486b367cc22a405f4a299ab17c (diff) |
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>
Diffstat (limited to 'drivers/gpu/drm/omapdrm/omap_encoder.c')
-rw-r--r-- | drivers/gpu/drm/omapdrm/omap_encoder.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/omapdrm/omap_encoder.c b/drivers/gpu/drm/omapdrm/omap_encoder.c index a20f30039aee..86c977b7189a 100644 --- 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, | |||
117 | 117 | ||
118 | dssdrv->get_timings(dssdev, &t); | 118 | dssdrv->get_timings(dssdev, &t); |
119 | 119 | ||
120 | if (memcmp(vm, &t, sizeof(struct videomode))) | 120 | if (memcmp(vm, &t, sizeof(*vm))) |
121 | ret = -EINVAL; | 121 | ret = -EINVAL; |
122 | else | 122 | else |
123 | ret = 0; | 123 | ret = 0; |