From: Hans Verkuil <hansverk@cisco.com>
To: Federico Vaga <federico.vaga@gmail.com>
Cc: Kamil Debski <k.debski@samsung.com>,
Mauro Carvalho Chehab <mchehab@redhat.com>,
Srinivas Kandagatla <srinivas.kandagatla@st.com>,
Peter Senna Tschudin <peter.senna@gmail.com>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Marek Szyprowski <m.szyprowski@samsung.com>,
devel@driverdev.osuosl.org,
Giancarlo Asnaghi <giancarlo.asnaghi@st.com>,
linux-samsung-soc@vger.kernel.org,
Jonathan Corbet <corbet@lwn.net>,
Sachin Kamat <sachin.kamat@linaro.org>,
Prabhakar Lad <prabhakar.lad@ti.com>,
Hans Verkuil <hans.verkuil@cisco.com>,
Sasha Levin <sasha.levin@oracle.com>,
linux-media@vger.kernel.org,
Tomasz Stanislawski <t.stanislaws@samsung.com>,
Pawel Osciak <pawel@osciak.com>,
davinci-linux-open-source@linux.davincidsp.com,
Mauro Carvalho Chehab <mchehab@infradead.org>,
Jeongtae Park <jtp.park@samsung.com>,
Andrzej Pietrasiewicz <andrzej.p@samsung.com>,
Sylwester
Subject: Re: [PATCH v5 2/3] sta2x11_vip: convert to videobuf2 and control framework
Date: Thu, 10 Jan 2013 15:41:00 +0100 [thread overview]
Message-ID: <201301101541.01082.hansverk@cisco.com> (raw)
In-Reply-To: <3596574.18Q2Rh4jo9@harkonnen>
On Thu 10 January 2013 13:57:19 Federico Vaga wrote:
> @Hans: here the output of v4l2-compliance that you request me for the v4
> version. This is done on the v5 version where I applied your suggestion. I
> have 1 warning but I don't understand where is the problem
To see the warning you need to run v4l2-compliance with the '-v1' option.
Hmm, I think I should probably change v4l2-compliance to show warnings by default
and add an option that surpresses warnings. That might work better.
Regards,
Hans
>
> Driver Info:
> Driver name : sta2x11_vip
> Card type : sta2x11_vip
> Bus info : PCI:0000:06:00.0
> Driver version: 3.6.0
> Capabilities : 0x85000001
> Video Capture
> Read/Write
> Streaming
> Device Capabilities
> Device Caps : 0x05000001
> Video Capture
> Read/Write
> Streaming
>
> Compliance test for device /dev/video0 (not using libv4l2):
>
> Required ioctls:
> test VIDIOC_QUERYCAP: OK
>
> Allow for multiple opens:
> test second video open: OK
> test VIDIOC_QUERYCAP: OK
> test VIDIOC_G/S_PRIORITY: OK
>
> Debug ioctls:
> test VIDIOC_DBG_G_CHIP_IDENT: Not Supported
> test VIDIOC_DBG_G/S_REGISTER: Not Supported
> test VIDIOC_LOG_STATUS: OK
>
> Input ioctls:
> test VIDIOC_G/S_TUNER: Not Supported
> test VIDIOC_G/S_FREQUENCY: Not Supported
> test VIDIOC_S_HW_FREQ_SEEK: OK
> test VIDIOC_ENUMAUDIO: Not Supported
> test VIDIOC_G/S/ENUMINPUT: OK
> test VIDIOC_G/S_AUDIO: Not Supported
> Inputs: 2 Audio Inputs: 0 Tuners: 0
>
> Output ioctls:
> test VIDIOC_G/S_MODULATOR: Not Supported
> test VIDIOC_G/S_FREQUENCY: Not Supported
> test VIDIOC_ENUMAUDOUT: Not Supported
> test VIDIOC_G/S/ENUMOUTPUT: Not Supported
> test VIDIOC_G/S_AUDOUT: Not Supported
> Outputs: 0 Audio Outputs: 0 Modulators: 0
>
> Control ioctls:
> test VIDIOC_QUERYCTRL/MENU: OK
> test VIDIOC_G/S_CTRL: OK
> test VIDIOC_G/S/TRY_EXT_CTRLS: OK
> test VIDIOC_(UN)SUBSCRIBE_EVENT/DQEVENT: OK
> test VIDIOC_G/S_JPEGCOMP: Not Supported
> Standard Controls: 5 Private Controls: 0
>
> Input/Output configuration ioctls:
> test VIDIOC_ENUM/G/S/QUERY_STD: OK
> test VIDIOC_ENUM/G/S/QUERY_DV_PRESETS: Not Supported
> test VIDIOC_ENUM/G/S/QUERY_DV_TIMINGS: Not Supported
> test VIDIOC_DV_TIMINGS_CAP: Not Supported
>
> Format ioctls:
> test VIDIOC_ENUM_FMT/FRAMESIZES/FRAMEINTERVALS: OK
> test VIDIOC_G/S_PARM: OK
> test VIDIOC_G_FBUF: Not Supported
> test VIDIOC_G_FMT: OK
> test VIDIOC_TRY_FMT: OK
> test VIDIOC_S_FMT: OK
> test VIDIOC_G_SLICED_VBI_CAP: Not Supported
>
> Buffer ioctls:
> test VIDIOC_REQBUFS/CREATE_BUFS: OK
> test read/write: OK
>
> Total: 36, Succeeded: 36, Failed: 0, Warnings: 1
>
>
>
WARNING: multiple messages have this Message-ID (diff)
From: hansverk@cisco.com (Hans Verkuil)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v5 2/3] sta2x11_vip: convert to videobuf2 and control framework
Date: Thu, 10 Jan 2013 15:41:00 +0100 [thread overview]
Message-ID: <201301101541.01082.hansverk@cisco.com> (raw)
In-Reply-To: <3596574.18Q2Rh4jo9@harkonnen>
On Thu 10 January 2013 13:57:19 Federico Vaga wrote:
> @Hans: here the output of v4l2-compliance that you request me for the v4
> version. This is done on the v5 version where I applied your suggestion. I
> have 1 warning but I don't understand where is the problem
To see the warning you need to run v4l2-compliance with the '-v1' option.
Hmm, I think I should probably change v4l2-compliance to show warnings by default
and add an option that surpresses warnings. That might work better.
Regards,
Hans
>
> Driver Info:
> Driver name : sta2x11_vip
> Card type : sta2x11_vip
> Bus info : PCI:0000:06:00.0
> Driver version: 3.6.0
> Capabilities : 0x85000001
> Video Capture
> Read/Write
> Streaming
> Device Capabilities
> Device Caps : 0x05000001
> Video Capture
> Read/Write
> Streaming
>
> Compliance test for device /dev/video0 (not using libv4l2):
>
> Required ioctls:
> test VIDIOC_QUERYCAP: OK
>
> Allow for multiple opens:
> test second video open: OK
> test VIDIOC_QUERYCAP: OK
> test VIDIOC_G/S_PRIORITY: OK
>
> Debug ioctls:
> test VIDIOC_DBG_G_CHIP_IDENT: Not Supported
> test VIDIOC_DBG_G/S_REGISTER: Not Supported
> test VIDIOC_LOG_STATUS: OK
>
> Input ioctls:
> test VIDIOC_G/S_TUNER: Not Supported
> test VIDIOC_G/S_FREQUENCY: Not Supported
> test VIDIOC_S_HW_FREQ_SEEK: OK
> test VIDIOC_ENUMAUDIO: Not Supported
> test VIDIOC_G/S/ENUMINPUT: OK
> test VIDIOC_G/S_AUDIO: Not Supported
> Inputs: 2 Audio Inputs: 0 Tuners: 0
>
> Output ioctls:
> test VIDIOC_G/S_MODULATOR: Not Supported
> test VIDIOC_G/S_FREQUENCY: Not Supported
> test VIDIOC_ENUMAUDOUT: Not Supported
> test VIDIOC_G/S/ENUMOUTPUT: Not Supported
> test VIDIOC_G/S_AUDOUT: Not Supported
> Outputs: 0 Audio Outputs: 0 Modulators: 0
>
> Control ioctls:
> test VIDIOC_QUERYCTRL/MENU: OK
> test VIDIOC_G/S_CTRL: OK
> test VIDIOC_G/S/TRY_EXT_CTRLS: OK
> test VIDIOC_(UN)SUBSCRIBE_EVENT/DQEVENT: OK
> test VIDIOC_G/S_JPEGCOMP: Not Supported
> Standard Controls: 5 Private Controls: 0
>
> Input/Output configuration ioctls:
> test VIDIOC_ENUM/G/S/QUERY_STD: OK
> test VIDIOC_ENUM/G/S/QUERY_DV_PRESETS: Not Supported
> test VIDIOC_ENUM/G/S/QUERY_DV_TIMINGS: Not Supported
> test VIDIOC_DV_TIMINGS_CAP: Not Supported
>
> Format ioctls:
> test VIDIOC_ENUM_FMT/FRAMESIZES/FRAMEINTERVALS: OK
> test VIDIOC_G/S_PARM: OK
> test VIDIOC_G_FBUF: Not Supported
> test VIDIOC_G_FMT: OK
> test VIDIOC_TRY_FMT: OK
> test VIDIOC_S_FMT: OK
> test VIDIOC_G_SLICED_VBI_CAP: Not Supported
>
> Buffer ioctls:
> test VIDIOC_REQBUFS/CREATE_BUFS: OK
> test read/write: OK
>
> Total: 36, Succeeded: 36, Failed: 0, Warnings: 1
>
>
>
next prev parent reply other threads:[~2013-01-10 14:41 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-10 12:46 [PATCH v5] modify videobuf2-dma-contig and update sta2x11_vip Federico Vaga
2013-01-10 12:46 ` Federico Vaga
2013-01-10 12:46 ` [PATCH v5 1/3] videobuf2-dma-contig: user can specify GFP flags Federico Vaga
2013-01-10 12:46 ` Federico Vaga
2013-01-10 14:16 ` Laurent Pinchart
2013-01-10 14:32 ` Federico Vaga
2013-01-10 14:32 ` Federico Vaga
2013-01-10 12:46 ` [PATCH v5 2/3] sta2x11_vip: convert to videobuf2 and control framework Federico Vaga
2013-01-10 12:46 ` Federico Vaga
2013-01-10 12:57 ` Federico Vaga
2013-01-10 12:57 ` Federico Vaga
2013-01-10 14:41 ` Hans Verkuil [this message]
2013-01-10 14:41 ` Hans Verkuil
2013-01-10 14:58 ` Federico Vaga
2013-01-10 14:58 ` Federico Vaga
2013-01-10 14:01 ` Dan Carpenter
2013-01-10 14:01 ` Dan Carpenter
2013-01-10 14:43 ` Federico Vaga
2013-01-10 14:43 ` Federico Vaga
2013-01-10 12:46 ` [PATCH v5 3/3] adv7180: remove {query/g_/s_}ctrl Federico Vaga
2013-01-10 12:46 ` Federico Vaga
2013-01-10 15:22 ` [PATCH v5] modify videobuf2-dma-contig and update sta2x11_vip Federico Vaga
2013-01-10 15:22 ` Federico Vaga
2013-01-10 16:06 ` Mauro Carvalho Chehab
2013-01-10 16:06 ` Mauro Carvalho Chehab
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201301101541.01082.hansverk@cisco.com \
--to=hansverk@cisco.com \
--cc=andrzej.p@samsung.com \
--cc=corbet@lwn.net \
--cc=davinci-linux-open-source@linux.davincidsp.com \
--cc=devel@driverdev.osuosl.org \
--cc=federico.vaga@gmail.com \
--cc=giancarlo.asnaghi@st.com \
--cc=hans.verkuil@cisco.com \
--cc=jtp.park@samsung.com \
--cc=k.debski@samsung.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=m.szyprowski@samsung.com \
--cc=mchehab@infradead.org \
--cc=mchehab@redhat.com \
--cc=pawel@osciak.com \
--cc=peter.senna@gmail.com \
--cc=prabhakar.lad@ti.com \
--cc=sachin.kamat@linaro.org \
--cc=sasha.levin@oracle.com \
--cc=srinivas.kandagatla@st.com \
--cc=t.stanislaws@samsung.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.