VIDIOC_QUERYMENU error when using VideoCapture and Video4Linux on RaspberryPi (Bugfix #3554)

Added by blair a about 11 years ago. Updated about 11 years ago.

Status:Done Start date:2014-02-16
Priority:Normal Due date:
Assignee:- % Done:

0%
Category:highgui-video
Target version:2.4.9
Affected version:branch '2.4' (2.4-dev) Operating System:Mobile/Embedded Linux
Difficulty: HW Platform:ARM
Pull request:https://github.com/Itseez/opencv/pull/2443

Description

v4l2_scan_controls_enumerate_menu is unused and causes ioctl error on RaspberryPi and possibly other Video4Linux variants. See http://www.raspberrypi.org/forum/viewtopic.php?f=43&t=65026 for more detail. I have tried pushing my commits to my fork but can't seem to get it to work (would be useful if there was a step by step instruction manual for forking/cloning/editing/commiting/pushing-to-fork/pull-requesting somewhere just for opencv!). A pull request I had somehow managed to request but is no longer valid is https://github.com/Itseez/opencv/pull/2298 (I provide just for context).

Attached are two patched files cap_libv4l.cpp and cap_v4l.cpp. The patched files do not cause any regressions on my build using the opencv_test_* binaries.

cap_libv4l.cpp (57.2 kB) blair a, 2014-02-16 10:36 pm

cap_v4l.cpp (84.2 kB) blair a, 2014-02-16 10:36 pm

Associated revisions

Revision b184d7f2
Added by Dave Jones about 11 years ago

Fix for #3554

v4l2_scan_controls_enumerate_menu is unused and causes ioctl error on
RaspberryPi and possibly other Video4Linux variants. See
http://www.raspberrypi.org/forum/viewtopic.php?f=43&t=65026 for more
detail.

Revision 5cc4aa0e
Added by Vadim Pisarevsky about 10 years ago

Merge pull request #3554 from wangyan42164:match_template_mask

History

#1
Updated by Kirill Kornyakov about 11 years ago

Here is our instruction: http://code.opencv.org/projects/opencv/wiki/How_to_contribute. Do you think it is not sufficient?

Also, better to try again with a pull request. This way it will be automatically tested and faster reviewed.

  • Tracker changed from Bug to Bugfix
  • Category set to highgui-video
  • Difficulty deleted (Easy)

#2
Updated by Kirill Kornyakov about 11 years ago

  • Assignee set to blair a

#3
Updated by Kirill Kornyakov about 11 years ago

  • Status changed from New to Open

#4
Updated by blair a about 11 years ago

Kirill Kornyakov wrote:

Do you think it is not sufficient?

Yep tried that 20 times over and over again. I just couldn't make it work. I must be be stupid so no point assigning it back to me.

#5
Updated by blair a about 11 years ago

  • Assignee deleted (blair a)

#6
Updated by Dave Jones about 11 years ago

I've submitted this on behalf of "blair a" as pull request #2443 against the 2.4 branch

#7
Updated by Roman Donchenko about 11 years ago

  • Status changed from Open to Done
  • Pull request set to https://github.com/Itseez/opencv/pull/2443

Also available in: Atom PDF