| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-Id: <201212271255.11159.hverkuil@xs4all.nl>
Date: Thu, 27 Dec 2012 12:55:11 +0100
From: Hans Verkuil <hverkuil@...all.nl>
To: Kirill Smelkov <kirr@...ytux.spb.ru>
Cc: Mauro Carvalho Chehab <mchehab@...hat.com>,
linux-media@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] [media] vivi: Constify structures
On Wed December 26 2012 16:29:43 Kirill Smelkov wrote:
> Most of *_ops and other structures in vivi.c were already declared const
> but some have not. Constify and code/data will take less space:
>
> $ size drivers/media/platform/vivi.o
> text data bss dec hex filename
> before: 12569 248 8 12825 3219 drivers/media/platform/vivi.o
> after: 12308 20 8 12336 3030 drivers/media/platform/vivi.o
>
> i.e. vivi.o is now ~500 bytes less.
>
> Signed-off-by: Kirill Smelkov <kirr@...ytux.spb.ru>
> ---
> drivers/media/platform/vivi.c | 26 +++++++++++++-------------
> 1 file changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/media/platform/vivi.c b/drivers/media/platform/vivi.c
> index ec65089..bed04e1 100644
> --- a/drivers/media/platform/vivi.c
> +++ b/drivers/media/platform/vivi.c
> @@ -91,13 +91,13 @@ static const struct v4l2_fract
> ------------------------------------------------------------------*/
>
> struct vivi_fmt {
> - char *name;
> + const char *name;
Just use one space before '*' since it no longer lines up to anything.
> u32 fourcc; /* v4l2 format id */
> u8 depth;
> bool is_yuv;
> };
>
> -static struct vivi_fmt formats[] = {
> +static const struct vivi_fmt formats[] = {
> {
> .name = "4:2:2, packed, YUYV",
> .fourcc = V4L2_PIX_FMT_YUYV,
> @@ -164,9 +164,9 @@ static struct vivi_fmt formats[] = {
> },
> };
>
> -static struct vivi_fmt *__get_format(u32 pixelformat)
> +static const struct vivi_fmt *__get_format(u32 pixelformat)
> {
> - struct vivi_fmt *fmt;
> + const struct vivi_fmt *fmt;
> unsigned int k;
>
> for (k = 0; k < ARRAY_SIZE(formats); k++) {
> @@ -181,7 +181,7 @@ static struct vivi_fmt *__get_format(u32 pixelformat)
> return &formats[k];
> }
>
> -static struct vivi_fmt *get_format(struct v4l2_format *f)
> +static const struct vivi_fmt *get_format(struct v4l2_format *f)
> {
> return __get_format(f->fmt.pix.pixelformat);
> }
> @@ -191,7 +191,7 @@ struct vivi_buffer {
> /* common v4l buffer stuff -- must be first */
> struct vb2_buffer vb;
> struct list_head list;
> - struct vivi_fmt *fmt;
> + struct vivi_fmt const *fmt;
> };
>
> struct vivi_dmaqueue {
> @@ -250,7 +250,7 @@ struct vivi_dev {
> int input;
>
> /* video capture */
> - struct vivi_fmt *fmt;
> + struct vivi_fmt const *fmt;
'const' should be before 'struct' for consistency reasons.
> struct v4l2_fract timeperframe;
> unsigned int width, height;
> struct vb2_queue vb_vidq;
> @@ -297,7 +297,7 @@ struct bar_std {
>
> /* Maximum number of bars are 10 - otherwise, the input print code
> should be modified */
> -static struct bar_std bars[] = {
> +static const struct bar_std bars[] = {
> { /* Standard ITU-R color bar sequence */
> { COLOR_WHITE, COLOR_AMBER, COLOR_CYAN, COLOR_GREEN,
> COLOR_MAGENTA, COLOR_RED, COLOR_BLUE, COLOR_BLACK, COLOR_BLACK }
> @@ -926,7 +926,7 @@ static void vivi_unlock(struct vb2_queue *vq)
> }
>
>
> -static struct vb2_ops vivi_video_qops = {
> +static const struct vb2_ops vivi_video_qops = {
> .queue_setup = queue_setup,
> .buf_prepare = buffer_prepare,
> .buf_queue = buffer_queue,
> @@ -957,7 +957,7 @@ static int vidioc_querycap(struct file *file, void *priv,
> static int vidioc_enum_fmt_vid_cap(struct file *file, void *priv,
> struct v4l2_fmtdesc *f)
> {
> - struct vivi_fmt *fmt;
> + struct vivi_fmt const *fmt;
Ditto.
>
> if (f->index >= ARRAY_SIZE(formats))
> return -EINVAL;
> @@ -993,7 +993,7 @@ static int vidioc_try_fmt_vid_cap(struct file *file, void *priv,
> struct v4l2_format *f)
> {
> struct vivi_dev *dev = video_drvdata(file);
> - struct vivi_fmt *fmt;
> + struct vivi_fmt const *fmt;
Ditto.
>
> fmt = get_format(f);
> if (!fmt) {
> @@ -1102,7 +1102,7 @@ static int vidioc_s_input(struct file *file, void *priv, unsigned int i)
> static int vidioc_enum_frameintervals(struct file *file, void *priv,
> struct v4l2_frmivalenum *fival)
> {
> - struct vivi_fmt *fmt;
> + struct vivi_fmt const *fmt;
Ditto.
>
> if (fival->index)
> return -EINVAL;
> @@ -1330,7 +1330,7 @@ static const struct v4l2_ioctl_ops vivi_ioctl_ops = {
> .vidioc_unsubscribe_event = v4l2_event_unsubscribe,
> };
>
> -static struct video_device vivi_template = {
> +static const struct video_device vivi_template = {
> .name = "vivi",
> .fops = &vivi_fops,
> .ioctl_ops = &vivi_ioctl_ops,
>
Regards,
Hans
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists