diff options
author | Sachin Kamat <sachin.kamat@linaro.org> | 2012-11-25 23:49:05 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2013-01-06 06:41:11 -0500 |
commit | d322bb915425b65a51c48b90e912af55762cc745 (patch) | |
tree | 51d348fc12fa214b58ff7289c8d779da368d8752 | |
parent | 45b56d572cd8b4f118ed3a006aa33527fd966389 (diff) |
[media] s5p-tv: Add missing braces around sizeof in hdmi_drv.c
Fixes the following checkpatch warnings:
WARNING: sizeof *fmt should be sizeof(*fmt)
WARNING: sizeof *res should be sizeof(*res)
WARNING: sizeof *res should be sizeof(*res)
WARNING: sizeof sd->name should be sizeof(sd->name)
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Acked-by: Tomasz Stanislawski <t.stanislaws@samsung.com>
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
-rw-r--r-- | drivers/media/platform/s5p-tv/hdmi_drv.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/media/platform/s5p-tv/hdmi_drv.c b/drivers/media/platform/s5p-tv/hdmi_drv.c index 1c48ca5e419f..c0d0f84e5b86 100644 --- a/drivers/media/platform/s5p-tv/hdmi_drv.c +++ b/drivers/media/platform/s5p-tv/hdmi_drv.c | |||
@@ -656,7 +656,7 @@ static int hdmi_g_mbus_fmt(struct v4l2_subdev *sd, | |||
656 | dev_dbg(hdev->dev, "%s\n", __func__); | 656 | dev_dbg(hdev->dev, "%s\n", __func__); |
657 | if (!hdev->cur_conf) | 657 | if (!hdev->cur_conf) |
658 | return -EINVAL; | 658 | return -EINVAL; |
659 | memset(fmt, 0, sizeof *fmt); | 659 | memset(fmt, 0, sizeof(*fmt)); |
660 | fmt->width = t->hact.end - t->hact.beg; | 660 | fmt->width = t->hact.end - t->hact.beg; |
661 | fmt->height = t->vact[0].end - t->vact[0].beg; | 661 | fmt->height = t->vact[0].end - t->vact[0].beg; |
662 | fmt->code = V4L2_MBUS_FMT_FIXED; /* means RGB888 */ | 662 | fmt->code = V4L2_MBUS_FMT_FIXED; /* means RGB888 */ |
@@ -760,7 +760,7 @@ static void hdmi_resources_cleanup(struct hdmi_device *hdev) | |||
760 | clk_put(res->sclk_hdmi); | 760 | clk_put(res->sclk_hdmi); |
761 | if (!IS_ERR_OR_NULL(res->hdmi)) | 761 | if (!IS_ERR_OR_NULL(res->hdmi)) |
762 | clk_put(res->hdmi); | 762 | clk_put(res->hdmi); |
763 | memset(res, 0, sizeof *res); | 763 | memset(res, 0, sizeof(*res)); |
764 | } | 764 | } |
765 | 765 | ||
766 | static int hdmi_resources_init(struct hdmi_device *hdev) | 766 | static int hdmi_resources_init(struct hdmi_device *hdev) |
@@ -777,7 +777,7 @@ static int hdmi_resources_init(struct hdmi_device *hdev) | |||
777 | 777 | ||
778 | dev_dbg(dev, "HDMI resource init\n"); | 778 | dev_dbg(dev, "HDMI resource init\n"); |
779 | 779 | ||
780 | memset(res, 0, sizeof *res); | 780 | memset(res, 0, sizeof(*res)); |
781 | /* get clocks, power */ | 781 | /* get clocks, power */ |
782 | 782 | ||
783 | res->hdmi = clk_get(dev, "hdmi"); | 783 | res->hdmi = clk_get(dev, "hdmi"); |
@@ -955,7 +955,7 @@ static int __devinit hdmi_probe(struct platform_device *pdev) | |||
955 | v4l2_subdev_init(sd, &hdmi_sd_ops); | 955 | v4l2_subdev_init(sd, &hdmi_sd_ops); |
956 | sd->owner = THIS_MODULE; | 956 | sd->owner = THIS_MODULE; |
957 | 957 | ||
958 | strlcpy(sd->name, "s5p-hdmi", sizeof sd->name); | 958 | strlcpy(sd->name, "s5p-hdmi", sizeof(sd->name)); |
959 | hdmi_dev->cur_preset = HDMI_DEFAULT_PRESET; | 959 | hdmi_dev->cur_preset = HDMI_DEFAULT_PRESET; |
960 | /* FIXME: missing fail preset is not supported */ | 960 | /* FIXME: missing fail preset is not supported */ |
961 | hdmi_dev->cur_conf = hdmi_preset2timings(hdmi_dev->cur_preset); | 961 | hdmi_dev->cur_conf = hdmi_preset2timings(hdmi_dev->cur_preset); |