UbiquitousOS
/
OpenCloudOS-Kernel
mirror of https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git
11
0
Fork 0
Code Issues Projects Releases Wiki Activity
OpenCloudOS-Kernel/drivers/media/usb/uvc
History
Rafael J. Wysocki 9c016d6109 Partly revert "[media] uvcvideo: Set error_idx properly for extended controls API failures" 
Commit f0ed2ce840 ("[media] uvcvideo: Set error_idx properly for
extended controls API failures") causes user space to behave incorrectly
on one of my test machines (there is no sound under KDE 4.9.4 using
pulseaudio and there is a knotify4 process occupying one of the CPU
cores 100% of the time).  Reverting that commit entirely fixes the
problem for me.

However, commit f0ed2ce840 appears to do more than it follows from its
changelog, because the changelog only says about the changes related to
ctrls->error_idx, while the commit additionally changes error codes
returned by various functions in uvc_ctrl.c and uvc_v4l2.c.  It turns
out that the changes of the returned error codes confuse the user spce,
so it is sufficient to revert the part of commit f0ed2ce840 not
mentioned in its changelog to fix the problem.

[ 'ENOENT' is not a valid error return from an ioctl to begin with, and
  I don't understand how anybody ever even thought it would be.  - Linus ]

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Mauro Carvalho Chehab <mchehab@redhat.com>
Cc: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
 		2012-12-23 09:39:14 -08:00
..
Kconfig [media] rename most media/video usb drivers to media/usb 2012-08-15 16:25:07 -03:00
Makefile [media] rename most media/video usb drivers to media/usb 2012-08-15 16:25:07 -03:00
uvc_ctrl.c Partly revert "[media] uvcvideo: Set error_idx properly for extended controls API failures" 2012-12-23 09:39:14 -08:00
uvc_debugfs.c [media] rename most media/video usb drivers to media/usb 2012-08-15 16:25:07 -03:00
uvc_driver.c [media] uvcvideo: Mark first output terminal as default video node 2012-11-28 10:30:00 -02:00
uvc_entity.c [media] uvcvideo: Mark first output terminal as default video node 2012-11-28 10:30:00 -02:00
uvc_isight.c [media] rename most media/video usb drivers to media/usb 2012-08-15 16:25:07 -03:00
uvc_queue.c [media] v4l: uvc: add support for DMABUF importing 2012-11-25 17:19:31 -02:00
uvc_status.c [media] rename most media/video usb drivers to media/usb 2012-08-15 16:25:07 -03:00
uvc_v4l2.c Partly revert "[media] uvcvideo: Set error_idx properly for extended controls API failures" 2012-12-23 09:39:14 -08:00
uvc_video.c [media] uvcvideo: Don't fail when an unsupported format is requested 2012-11-28 10:27:49 -02:00
uvcvideo.h [media] uvcvideo: Mark first output terminal as default video node 2012-11-28 10:30:00 -02:00