From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans Verkuil Subject: Re: [RFC] [Patch] implement video driver for sur40 Date: Fri, 19 Dec 2014 15:36:10 +0100 Message-ID: <549437DA.6090601@xs4all.nl> References: <5492D7E8.504@butterbrot.org> <5492E091.1060404@xs4all.nl> <54943680.3020007@butterbrot.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: Received: from lb3-smtp-cloud6.xs4all.net ([194.109.24.31]:60554 "EHLO lb3-smtp-cloud6.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752157AbaLSOgR (ORCPT ); Fri, 19 Dec 2014 09:36:17 -0500 In-Reply-To: <54943680.3020007@butterbrot.org> Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Florian Echtler , linux-input , linux-media@vger.kernel.org On 12/19/2014 03:30 PM, Florian Echtler wrote: > On 18.12.2014 15:11, Hans Verkuil wrote: >> Run as 'v4l2-compliance -s' (-s starts streaming tests as well and it >> assumes you have a valid input signal). >> Mail if you have any questions about the v4l2-compliance output. The failure >> messages expect you to look at the v4l2-compliance source code as well, >> but even than it is not always clear what is going on. > Ran the most recent version from git master, got a total of 6 fails, 4 > of which are probably easy fixes: > >> fail: v4l2-compliance.cpp(306): missing bus_info prefix ('USB:1') >> test VIDIOC_QUERYCAP: FAIL > Changed the relevant code to: > usb_make_path(sur40->usbdev, cap->bus_info, sizeof(cap->bus_info)); > >> fail: v4l2-test-input-output.cpp(455): could set input to invalid input 1 >> test VIDIOC_G/S/ENUMINPUT: FAIL > Now returning -EINVAL when S_INPUT called with input != 0. > >> fail: v4l2-test-formats.cpp(322): !colorspace >> fail: v4l2-test-formats.cpp(429): testColorspace(pix.pixelformat, > pix.colorspace, pix.ycbcr_enc, pix.quantization) >> test VIDIOC_G_FMT: FAIL > Setting colorspace in v4l2_pix_format to V4L2_COLORSPACE_SRGB. > >> fail: v4l2-compliance.cpp(365): doioctl(node, VIDIOC_G_PRIORITY, &prio) >> test VIDIOC_G/S_PRIORITY: FAIL > Don't know how to fix this - does this mean VIDIOC_G/S_PRIORITY _must_ > be implemented? > >> fail: v4l2-test-buffers.cpp(500): q.has_expbuf(node) >> test VIDIOC_EXPBUF: FAIL > Also not clear how to fix this one. > > Could you give some hints on the last two? Can you post the driver code you used to run these tests? And which kernel version and git tree did you base your patch on? Regards, Hans > > Thanks & best regards, Florian >