There are some old broken definitions of ioctl's, where the
read/write arguments were marked wrong. The last one were added
on 2.6.6 kernel. Remove them, in order to cleanup some
copy_from_user/copy_to_user logic done inside V4L core.

Signed-off-by: Mauro Carvalho Chehab <mche...@redhat.com>

diff --git a/Documentation/feature-removal-schedule.txt 
b/Documentation/feature-removal-schedule.txt
index e348b7e..f2742e1 100644
--- a/Documentation/feature-removal-schedule.txt
+++ b/Documentation/feature-removal-schedule.txt
@@ -112,6 +112,27 @@ Who:       Mauro Carvalho Chehab <mche...@infradead.org>
 
 ---------------------------
 
+What:  Video4Linux: Remove obsolete ioctl's
+When:  kernel 2.6.39
+Files: include/media/videodev2.h
+Why:   Some ioctl's were defined wrong on 2.6.2 and 2.6.6, using the wrong
+       type of R/W arguments. They were fixed, but the old ioctl names are
+       still there, maintained to avoid breaking binary compatibility:
+         #define VIDIOC_OVERLAY_OLD    _IOWR('V', 14, int)
+         #define VIDIOC_S_PARM_OLD     _IOW('V', 22, struct v4l2_streamparm)
+         #define VIDIOC_S_CTRL_OLD     _IOW('V', 28, struct v4l2_control)
+         #define VIDIOC_G_AUDIO_OLD    _IOWR('V', 33, struct v4l2_audio)
+         #define VIDIOC_G_AUDOUT_OLD   _IOWR('V', 49, struct v4l2_audioout)
+         #define VIDIOC_CROPCAP_OLD    _IOR('V', 58, struct v4l2_cropcap)
+       There's no sense on preserving those forever, as it is very doubtful
+       that someone would try to use a such old binary with a modern kernel.
+       Removing them will allow us to remove some magic done at the V4L ioctl
+       handler.
+
+Who:   Mauro Carvalho Chehab <mche...@infradead.org>
+
+---------------------------
+
 What:  sys_sysctl
 When:  September 2010
 Option: CONFIG_SYSCTL_SYSCALL
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to