Commit c0d51204 authored by Sachin Kamat's avatar Sachin Kamat Committed by Mauro Carvalho Chehab

[media] s5p-tv: Add missing braces around sizeof in mixer_video.c

Silences several checkpatch warnings of the type:
WARNING: sizeof *out should be sizeof(*out)
FILE: media/platform/s5p-tv/mixer_video.c:98:
		out = kzalloc(sizeof *out, GFP_KERNEL);
Signed-off-by: default avatarSachin Kamat <sachin.kamat@linaro.org>
Acked-by: default avatarTomasz Stanislawski <t.stanislaws@samsung.com>
Signed-off-by: default avatarSylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent 80f0dee2
...@@ -95,7 +95,7 @@ int __devinit mxr_acquire_video(struct mxr_device *mdev, ...@@ -95,7 +95,7 @@ int __devinit mxr_acquire_video(struct mxr_device *mdev,
/* trying to register next output */ /* trying to register next output */
if (sd == NULL) if (sd == NULL)
continue; continue;
out = kzalloc(sizeof *out, GFP_KERNEL); out = kzalloc(sizeof(*out), GFP_KERNEL);
if (out == NULL) { if (out == NULL) {
mxr_err(mdev, "no memory for '%s'\n", mxr_err(mdev, "no memory for '%s'\n",
conf->output_name); conf->output_name);
...@@ -127,7 +127,7 @@ int __devinit mxr_acquire_video(struct mxr_device *mdev, ...@@ -127,7 +127,7 @@ int __devinit mxr_acquire_video(struct mxr_device *mdev,
/* kfree is NULL-safe */ /* kfree is NULL-safe */
for (i = 0; i < mdev->output_cnt; ++i) for (i = 0; i < mdev->output_cnt; ++i)
kfree(mdev->output[i]); kfree(mdev->output[i]);
memset(mdev->output, 0, sizeof mdev->output); memset(mdev->output, 0, sizeof(mdev->output));
fail_vb2_allocator: fail_vb2_allocator:
/* freeing allocator context */ /* freeing allocator context */
...@@ -160,8 +160,8 @@ static int mxr_querycap(struct file *file, void *priv, ...@@ -160,8 +160,8 @@ static int mxr_querycap(struct file *file, void *priv,
mxr_dbg(layer->mdev, "%s:%d\n", __func__, __LINE__); mxr_dbg(layer->mdev, "%s:%d\n", __func__, __LINE__);
strlcpy(cap->driver, MXR_DRIVER_NAME, sizeof cap->driver); strlcpy(cap->driver, MXR_DRIVER_NAME, sizeof(cap->driver));
strlcpy(cap->card, layer->vfd.name, sizeof cap->card); strlcpy(cap->card, layer->vfd.name, sizeof(cap->card));
sprintf(cap->bus_info, "%d", layer->idx); sprintf(cap->bus_info, "%d", layer->idx);
cap->device_caps = V4L2_CAP_STREAMING | V4L2_CAP_VIDEO_OUTPUT_MPLANE; cap->device_caps = V4L2_CAP_STREAMING | V4L2_CAP_VIDEO_OUTPUT_MPLANE;
cap->capabilities = cap->device_caps | V4L2_CAP_DEVICE_CAPS; cap->capabilities = cap->device_caps | V4L2_CAP_DEVICE_CAPS;
...@@ -192,7 +192,7 @@ static void mxr_layer_default_geo(struct mxr_layer *layer) ...@@ -192,7 +192,7 @@ static void mxr_layer_default_geo(struct mxr_layer *layer)
struct mxr_device *mdev = layer->mdev; struct mxr_device *mdev = layer->mdev;
struct v4l2_mbus_framefmt mbus_fmt; struct v4l2_mbus_framefmt mbus_fmt;
memset(&layer->geo, 0, sizeof layer->geo); memset(&layer->geo, 0, sizeof(layer->geo));
mxr_get_mbus_fmt(mdev, &mbus_fmt); mxr_get_mbus_fmt(mdev, &mbus_fmt);
...@@ -425,7 +425,7 @@ static int mxr_s_selection(struct file *file, void *fh, ...@@ -425,7 +425,7 @@ static int mxr_s_selection(struct file *file, void *fh,
struct mxr_geometry tmp; struct mxr_geometry tmp;
struct v4l2_rect res; struct v4l2_rect res;
memset(&res, 0, sizeof res); memset(&res, 0, sizeof(res));
mxr_dbg(layer->mdev, "%s: rect: %dx%d@%d,%d\n", __func__, mxr_dbg(layer->mdev, "%s: rect: %dx%d@%d,%d\n", __func__,
s->r.width, s->r.height, s->r.left, s->r.top); s->r.width, s->r.height, s->r.left, s->r.top);
...@@ -464,7 +464,7 @@ static int mxr_s_selection(struct file *file, void *fh, ...@@ -464,7 +464,7 @@ static int mxr_s_selection(struct file *file, void *fh,
/* apply change and update geometry if needed */ /* apply change and update geometry if needed */
if (target) { if (target) {
/* backup current geometry if setup fails */ /* backup current geometry if setup fails */
memcpy(&tmp, geo, sizeof tmp); memcpy(&tmp, geo, sizeof(tmp));
/* apply requested selection */ /* apply requested selection */
target->x_offset = s->r.left; target->x_offset = s->r.left;
...@@ -496,7 +496,7 @@ static int mxr_s_selection(struct file *file, void *fh, ...@@ -496,7 +496,7 @@ static int mxr_s_selection(struct file *file, void *fh,
fail: fail:
/* restore old geometry, which is not touched if target is NULL */ /* restore old geometry, which is not touched if target is NULL */
if (target) if (target)
memcpy(geo, &tmp, sizeof tmp); memcpy(geo, &tmp, sizeof(tmp));
return -ERANGE; return -ERANGE;
} }
...@@ -1071,7 +1071,7 @@ struct mxr_layer *mxr_base_layer_create(struct mxr_device *mdev, ...@@ -1071,7 +1071,7 @@ struct mxr_layer *mxr_base_layer_create(struct mxr_device *mdev,
{ {
struct mxr_layer *layer; struct mxr_layer *layer;
layer = kzalloc(sizeof *layer, GFP_KERNEL); layer = kzalloc(sizeof(*layer), GFP_KERNEL);
if (layer == NULL) { if (layer == NULL) {
mxr_err(mdev, "not enough memory for layer.\n"); mxr_err(mdev, "not enough memory for layer.\n");
goto fail; goto fail;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment