linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [REVIEW PATCH 0/5] v4l2: constify _IOW ioctls
@ 2013-03-15 10:27 Hans Verkuil
  2013-03-15 10:27 ` [REVIEW PATCH 1/5] v4l2: add const to argument of write-only s_frequency ioctl Hans Verkuil
  0 siblings, 1 reply; 21+ messages in thread
From: Hans Verkuil @ 2013-03-15 10:27 UTC (permalink / raw)
  To: linux-media
  Cc: Mauro Carvalho Chehab, Scott Jiang, Laurent Pinchart,
	Jonathan Corbet, Guennadi Liakhovetski, Andy Walls, Prabhakar Lad,
	Kyungmin Park, Tomasz Stanislawski, Alexey Klimov, Hans de Goede,
	Brian Johnson, Mike Isely, Ezequiel Garcia, Huang Shijie,
	Ismael Luceno, Takashi Iwai, Ondrej Zary

This is the second and last phase of ensuring that the arguments of write-only
ioctls in V4L2 are const. The first phase was 4-5 months ago and added const
to s_crop, s_modulator, s_audio, s_audout, (un)subscribe_event, s_freq_hw_seek,
s_jpegcomp and s_fbuf.

This second phase adds const to s_frequency, s_tuner, s_std and s_register.
Actually, for s_std it doesn't add const but changes it to pass the std by
value which is more consistent in that particular case.

As a result drivers will be aware when they are implementing write-only ioctls
(and I saw a few drivers attempting to return data back to the user), and the
v4l2 core will know that drivers won't change the argument of a write-only
ioctls which simplifies the core debug code.

The changes have been compile-tested with the linux-media daily build but
I may have missed some more exotic architectures.

Ideally I would like to have this merged fairly early on so we have enough
time to shake out any remaining compile problems.

Regards,

	Hans


^ permalink raw reply	[flat|nested] 21+ messages in thread

end of thread, other threads:[~2013-03-18 13:53 UTC | newest]

Thread overview: 21+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-15 10:27 [REVIEW PATCH 0/5] v4l2: constify _IOW ioctls Hans Verkuil
2013-03-15 10:27 ` [REVIEW PATCH 1/5] v4l2: add const to argument of write-only s_frequency ioctl Hans Verkuil
2013-03-15 10:27   ` [REVIEW PATCH 2/5] v4l2: add const to argument of write-only s_tuner ioctl Hans Verkuil
2013-03-15 12:12     ` Laurent Pinchart
2013-03-16 12:18     ` Alexey Klimov
2013-03-15 10:27   ` [REVIEW PATCH 3/5] v4l2: pass std by value to the write-only s_std ioctl Hans Verkuil
2013-03-15 12:17     ` Laurent Pinchart
2013-03-15 13:38     ` Jonathan Corbet
2013-03-15 13:58     ` Mauro Carvalho Chehab
2013-03-15 14:14       ` Hans Verkuil
2013-03-16  6:46     ` Guennadi Liakhovetski
2013-03-16  7:26     ` Prabhakar Lad
2013-03-15 10:27   ` [REVIEW PATCH 4/5] v4l2: add const to argument of write-only s_register ioctl Hans Verkuil
2013-03-15 12:22     ` Laurent Pinchart
2013-03-16  6:50     ` Guennadi Liakhovetski
2013-03-16  7:43     ` Prabhakar Lad
2013-03-15 10:27   ` [REVIEW PATCH 5/5] v4l2-ioctl: simplify debug code Hans Verkuil
2013-03-15 12:25     ` Laurent Pinchart
2013-03-18 13:52       ` Hans Verkuil
2013-03-15 12:08   ` [REVIEW PATCH 1/5] v4l2: add const to argument of write-only s_frequency ioctl Laurent Pinchart
2013-03-15 12:18     ` Hans Verkuil

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).