Re: [PATCH] media: usb: uvc: add a quirk for Generalplus Technology Inc. 808 Camera

2017-02-09 Thread Laurent Pinchart
Hi Neil, Thank you for the patch. On Thursday 09 Feb 2017 14:26:46 Neil Armstrong wrote: > As reported on [1], this device needs this quirk to be able to > reliably initialise the webcam. > > [1] https://sourceforge.net/p/linux-uvc/mailman/message/33791098/ > > Signed-off-by: Neil Armstrong

Re: [PATCH v4 1/2] staging: omap4iss: fix multiline comment style

2017-02-09 Thread Laurent Pinchart
Hi Avraham, Thank you for the patches. On Thursday 09 Feb 2017 18:57:55 Avraham Shukron wrote: > Fixed multi-line comments to their preferred style (First line empty) > > Signed-off-by: Avraham Shukron For both of them, Acked-by: Laurent Pinchart

Re: [PATCH] media: fix s5p_mfc_set_dec_frame_buffer_v6() to print buf size in hex

2017-02-09 Thread Andrzej Hajda
On 09.02.2017 23:10, Shuah Khan wrote: > Fix s5p_mfc_set_dec_frame_buffer_v6() to print buffer size in hex to be > consistent with the rest of the messages in the routine. > > Signed-off-by: Shuah Khan As Nicolas said please fix the subject. After this you can add my:

Re: [PATCH] media: s5p_mfc - remove unneeded io_modes initialzation in s5p_mfc_open()

2017-02-09 Thread Andrzej Hajda
On 09.02.2017 23:11, Shuah Khan wrote: > Remove unneeded io_modes initialzation in s5p_mfc_open(). It gets done > right below in vdev == dev->vfd_dec conditional. > > Signed-off-by: Shuah Khan Acked-by: Andrzej Hajda -- Regards Andrzej > --- >

Re: [PATCH v2 4/4] [media] s5p-mfc: Always check and set 'v4l2_pix_format:field' field

2017-02-09 Thread Andrzej Hajda
On 09.02.2017 21:04, Thibault Saunier wrote: > It is required by the standard that the field order is set by the > driver. > > Signed-off-by: Thibault Saunier > --- > drivers/media/platform/s5p-mfc/s5p_mfc_dec.c | 23 +-- > 1 file changed, 21

Re: [PATCH v2 3/4] [media] s5p-mfc: Set colorspace in VIDIO_{G,TRY}_FMT

2017-02-09 Thread Andrzej Hajda
On 09.02.2017 21:04, Thibault Saunier wrote: > The media documentation says that the V4L2_COLORSPACE_SMPTE170M colorspace > should be used for SDTV and V4L2_COLORSPACE_REC709 for HDTV but the driver > didn't set the colorimetry, also respect usespace setting. > > Use 576p display resolution as a

Re: [PATCH v2 2/4] [media] exynos-gsc: Respect userspace colorspace setting

2017-02-09 Thread Andrzej Hajda
On 09.02.2017 21:04, Thibault Saunier wrote: > If the colorspace is specified by userspace we should respect > it and not reset it ourself if we can support it. > > Signed-off-by: Thibault Saunier > --- > drivers/media/platform/exynos-gsc/gsc-core.c | 25

Re: [PATCH v2 1/4] [media] exynos-gsc: Use 576p instead 720p as a threshold for colorspaces

2017-02-09 Thread Andrzej Hajda
On 09.02.2017 21:04, Thibault Saunier wrote: > From: Javier Martinez Canillas > > The media documentation says that the V4L2_COLORSPACE_SMPTE170M colorspace > should be used for SDTV and V4L2_COLORSPACE_REC709 for HDTV. But drivers > don't agree on the display resolution

Re: [PATCH 4/4] [media] s5p-mfc: Always check and set 'v4l2_pix_format:field' field

2017-02-09 Thread kbuild test robot
Hi Thibault, [auto build test ERROR on linuxtv-media/master] [also build test ERROR on v4.10-rc7 next-20170209] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Thibault-Saunier/exynos-gsc-Use

cron job: media_tree daily build: WARNINGS

2017-02-09 Thread Hans Verkuil
This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Fri Feb 10 05:00:17 CET 2017 media-tree git hash:9eeb0ed0f30938f31a3d9135a88b9502192c18dd media_build

Re: [PATCH] media: fix s5p_mfc_set_dec_frame_buffer_v6() to print buf size in hex

2017-02-09 Thread Nicolas Dufresne
Le jeudi 09 février 2017 à 15:10 -0700, Shuah Khan a écrit : > Fix s5p_mfc_set_dec_frame_buffer_v6() to print buffer size in hex to > be > consistent with the rest of the messages in the routine. Short and long descriptions are miss-leading. This patch will print the buffer pointer as hex and

Re: [PATCH -next] [media] mtk-vcodec: remove redundant return value check of platform_get_resource()

2017-02-09 Thread Tiffany Lin
On Tue, 2017-02-07 at 15:16 +, Wei Yongjun wrote: > From: Wei Yongjun > > Remove unneeded error handling on the result of a call > to platform_get_resource() when the value is passed to > devm_ioremap_resource(). > > Signed-off-by: Wei Yongjun

Re: [PATCH v3 1/3] [media] v4l: add parsed MPEG-2 support

2017-02-09 Thread Nicolas Dufresne
Le jeudi 09 février 2017 à 18:07 +0100, Hugues Fruchet a écrit : > Add "parsed MPEG-2" pixel format & related controls > needed by stateless video decoders. > In order to decode the video bitstream chunk provided > by user on output queue, stateless decoders require > also some extra data

[RFCv4 PATCH 1/1] hdpvr: fix interrupted recording

2017-02-09 Thread Jonathan Sims
This is a reworking of a patch originally submitted by Ryley Angus, modified by Hans Verkuil and then seemingly forgotten before changes suggested by Keith Pyle here: http://www.mail-archive.com/linux-media@vger.kernel.org/msg75163.html were made and tested. I have implemented the suggested

Re: [PATCH v3 21/24] media: imx: Add MIPI CSI-2 Receiver subdev driver

2017-02-09 Thread Steve Longerbeam
On 02/09/2017 03:49 PM, Steve Longerbeam wrote: On 02/08/2017 03:42 PM, Russell King - ARM Linux wrote: On Wed, Feb 08, 2017 at 03:23:53PM -0800, Steve Longerbeam wrote: Actually, this exact function already exists as dw_mipi_dsi_phy_write in drivers/gpu/drm/rockchip/dw-mipi-dsi.c, and it

Re: [PATCH v3 21/24] media: imx: Add MIPI CSI-2 Receiver subdev driver

2017-02-09 Thread Steve Longerbeam
On 02/08/2017 03:42 PM, Russell King - ARM Linux wrote: On Wed, Feb 08, 2017 at 03:23:53PM -0800, Steve Longerbeam wrote: Actually, this exact function already exists as dw_mipi_dsi_phy_write in drivers/gpu/drm/rockchip/dw-mipi-dsi.c, and it looks like the D-PHY register 0x44 might contain a

Re: [PATCH] devicetree: Add video bus switch

2017-02-09 Thread Rob Herring
On Wed, Feb 8, 2017 at 4:34 PM, Pavel Machek wrote: >> > + >> > +This is a binding for a gpio controlled switch for camera interfaces. >> > Such a >> > +device is used on some embedded devices to connect two cameras to the same >> > +interface of a image signal processor. >> > + >>

Re: [PATCH] devicetree: Add video bus switch

2017-02-09 Thread Rob Herring
On Wed, Feb 8, 2017 at 4:30 PM, Pavel Machek wrote: > On Wed 2017-02-08 15:36:09, Rob Herring wrote: >> On Fri, Feb 03, 2017 at 01:35:08PM +0100, Pavel Machek wrote: >> > >> > N900 contains front and back camera, with a switch between the >> > two. This adds support for the switch

Re: [PATCH] devicetree: Add video bus switch

2017-02-09 Thread Rob Herring
On Thu, Feb 9, 2017 at 5:02 PM, Rob Herring wrote: > On Wed, Feb 8, 2017 at 4:30 PM, Pavel Machek wrote: >> On Wed 2017-02-08 15:36:09, Rob Herring wrote: >>> On Fri, Feb 03, 2017 at 01:35:08PM +0100, Pavel Machek wrote: >>> > >>> > N900 contains front and back

Re: [PATCH v2] pci: drop link_reset

2017-02-09 Thread Bjorn Helgaas
On Tue, Jan 24, 2017 at 07:35:56PM +0200, Michael S. Tsirkin wrote: > No hardware seems to actually call link_reset, and > no driver implements it as more than a nop stub. > > This drops the mentions of the callback from everywhere. > It's dropped from the documentation as well, but > the doc

[PATCH 2/4] [media] exynos-gsc: Respect userspace colorspace setting

2017-02-09 Thread Thibault Saunier
If the colorspace is specified by userspace we should respect it and not reset it ourself if we can support it. Signed-off-by: Thibault Saunier --- drivers/media/platform/exynos-gsc/gsc-core.c | 25 + 1 file changed, 17 insertions(+), 8

[PATCH 1/4] [media] exynos-gsc: Use 576p instead 720p as a threshold for colorspaces

2017-02-09 Thread Thibault Saunier
From: Javier Martinez Canillas The media documentation says that the V4L2_COLORSPACE_SMPTE170M colorspace should be used for SDTV and V4L2_COLORSPACE_REC709 for HDTV. But drivers don't agree on the display resolution that should be used as a threshold. Some drivers set

[PATCH 4/4] [media] s5p-mfc: Always check and set 'v4l2_pix_format:field' field

2017-02-09 Thread Thibault Saunier
It is required by the standard that the field order is set by the driver. Signed-off-by: Thibault Saunier --- drivers/media/platform/s5p-mfc/s5p_mfc_dec.c | 23 +-- 1 file changed, 21 insertions(+), 2 deletions(-) diff --git

[PATCH 3/4] [media] s5p-mfc: Set colorspace in VIDIO_{G,TRY}_FMT

2017-02-09 Thread Thibault Saunier
The media documentation says that the V4L2_COLORSPACE_SMPTE170M colorspace should be used for SDTV and V4L2_COLORSPACE_REC709 for HDTV but the driver didn't set the colorimetry, also respect usespace setting. Use 576p display resolution as a threshold to set this. Signed-off-by: Thibault Saunier

[PATCH 0/4] [media] Fixes for colorspace logic in exynos-gsc and s5p-mfc drivers

2017-02-09 Thread Thibault Saunier
Hello, This patchset fixes a few issues on the colorspace logic for the exynos-gsc and s5p-mfc drivers. We now handle the colorspace in those drivers, and make sure to respect user setting if possible. We also now set the 'v4l2_pix_format:field' if userspace passed ANY, avoiding GStreamer

[PATCH] media: fix s5p_mfc_set_dec_frame_buffer_v6() to print buf size in hex

2017-02-09 Thread Shuah Khan
Fix s5p_mfc_set_dec_frame_buffer_v6() to print buffer size in hex to be consistent with the rest of the messages in the routine. Signed-off-by: Shuah Khan --- drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH] media: s5p_mfc - remove unneeded io_modes initialzation in s5p_mfc_open()

2017-02-09 Thread Shuah Khan
Remove unneeded io_modes initialzation in s5p_mfc_open(). It gets done right below in vdev == dev->vfd_dec conditional. Signed-off-by: Shuah Khan --- drivers/media/platform/s5p-mfc/s5p_mfc.c | 1 - 1 file changed, 1 deletion(-) diff --git

[PATCH v3 1/3] [media] v4l: add parsed MPEG-2 support

2017-02-09 Thread Hugues Fruchet
Add "parsed MPEG-2" pixel format & related controls needed by stateless video decoders. In order to decode the video bitstream chunk provided by user on output queue, stateless decoders require also some extra data resulting from this video bitstream chunk parsing. Those parsed extra data have to

[PATCH v2 2/4] [media] exynos-gsc: Respect userspace colorspace setting

2017-02-09 Thread Thibault Saunier
If the colorspace is specified by userspace we should respect it and not reset it ourself if we can support it. Signed-off-by: Thibault Saunier --- drivers/media/platform/exynos-gsc/gsc-core.c | 25 + 1 file changed, 17 insertions(+), 8

[PATCH v2 0/4] [media] Fixes for colorspace logic in exynos-gsc and s5p-mfc drivers

2017-02-09 Thread Thibault Saunier
Changes since v1: - Fix a silly build error that slipped in while rebasing the patches Javier Martinez Canillas (1): [media] exynos-gsc: Use 576p instead 720p as a threshold for colorspaces Thibault Saunier (3): [media] exynos-gsc: Respect userspace colorspace setting [media]

[PATCH v2 3/4] [media] s5p-mfc: Set colorspace in VIDIO_{G,TRY}_FMT

2017-02-09 Thread Thibault Saunier
The media documentation says that the V4L2_COLORSPACE_SMPTE170M colorspace should be used for SDTV and V4L2_COLORSPACE_REC709 for HDTV but the driver didn't set the colorimetry, also respect usespace setting. Use 576p display resolution as a threshold to set this. Signed-off-by: Thibault Saunier

[PATCH v2 4/4] [media] s5p-mfc: Always check and set 'v4l2_pix_format:field' field

2017-02-09 Thread Thibault Saunier
It is required by the standard that the field order is set by the driver. Signed-off-by: Thibault Saunier --- drivers/media/platform/s5p-mfc/s5p_mfc_dec.c | 23 +-- 1 file changed, 21 insertions(+), 2 deletions(-) diff --git

[PATCH v2 1/4] [media] exynos-gsc: Use 576p instead 720p as a threshold for colorspaces

2017-02-09 Thread Thibault Saunier
From: Javier Martinez Canillas The media documentation says that the V4L2_COLORSPACE_SMPTE170M colorspace should be used for SDTV and V4L2_COLORSPACE_REC709 for HDTV. But drivers don't agree on the display resolution that should be used as a threshold. Some drivers set

[PATCH v3 3/3] [media] st-delta: add mpeg2 support

2017-02-09 Thread Hugues Fruchet
Adds support of DELTA MPEG-2 video decoder back-end, implemented by calling MPEG2_TRANSFORMER0 firmware using RPMSG IPC communication layer. MPEG-2 decoder back-end is a stateless decoder which require specific parsing metadata in access unit in order to complete decoding. Signed-off-by: Hugues

[PATCH v3 2/3] [media] st-delta: add parsing metadata controls support

2017-02-09 Thread Hugues Fruchet
Install all metadata controls required by registered decoders. Update the decoding context with the set of metadata received from user through extended control. Set the received metadata in access unit prior to call the decoder decoding ops. Signed-off-by: Hugues Fruchet

[PATCH v3 0/3] Add support for MPEG-2 in DELTA video decoder

2017-02-09 Thread Hugues Fruchet
The patchset implements the MPEG-2 part of V4L2 unified low-level decoder API RFC [0] needed by stateless video decoders, ie decoders which requires specific parsing metadata in addition to video bitstream chunk in order to complete decoding. A reference implementation using STMicroelectronics

[PATCH v4 2/2] staging: omap4iss: fix coding style issue

2017-02-09 Thread Avraham Shukron
Broke argument list so that it won't exceed 80 characters Signed-off-by: Avraham Shukron --- drivers/staging/media/omap4iss/iss_video.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/media/omap4iss/iss_video.c

[PATCH v4 1/2] staging: omap4iss: fix multiline comment style

2017-02-09 Thread Avraham Shukron
Fixed multi-line comments to their preferred style (First line empty) Signed-off-by: Avraham Shukron --- drivers/staging/media/omap4iss/iss_video.c | 38 -- 1 file changed, 25 insertions(+), 13 deletions(-) diff --git

Re: [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm

2017-02-09 Thread Bartosz Golaszewski
2017-02-09 16:23 GMT+01:00 Sekhar Nori : > On Tuesday 07 February 2017 11:51 PM, Kevin Hilman wrote: >> Bartosz Golaszewski writes: >> >>> When we enable vpif capture on the da850-evm we hit a BUG_ON() because >>> the i2c adapter can't be found. The

Re: [PATCH 08/10] ARM: davinci: fix the DT boot on da850-evm

2017-02-09 Thread Sekhar Nori
On Tuesday 07 February 2017 11:51 PM, Kevin Hilman wrote: > Bartosz Golaszewski writes: > >> When we enable vpif capture on the da850-evm we hit a BUG_ON() because >> the i2c adapter can't be found. The board file boot uses i2c adapter 1 >> but in the DT mode it's

[PATCH] [media] coda/imx-vdoa: platform_driver should not be const

2017-02-09 Thread Arnd Bergmann
The device driver platform is actually written to during registration, for setting the owner field, so platform_driver_register() does not take a const pointer: drivers/media/platform/coda/imx-vdoa.c: In function 'vdoa_driver_init': drivers/media/platform/coda/imx-vdoa.c:333:213: error: passing

[PATCH] cec.h: small typo fix

2017-02-09 Thread Hans Verkuil
ad -> as It won't bring about world peace, but every little bit helps :-) Signed-off-by: Hans Verkuil --- diff --git a/include/uapi/linux/cec.h b/include/uapi/linux/cec.h index 14b6f24b189e..a0dfe27bc6c7 100644 --- a/include/uapi/linux/cec.h +++

Re: [PATCH v6 9/9] media: venus: enable building of Venus video driver

2017-02-09 Thread Stanimir Varbanov
On 02/08/2017 12:38 AM, kbuild test robot wrote: > Hi Stanimir, > > [auto build test ERROR on linuxtv-media/master] > [also build test ERROR on v4.10-rc7 next-20170207] The error is not reproducible on v4.10-rc7 next-20170209! -- regards, Stan -- To unsubscribe from this list: s

[PATCH] media: usb: uvc: add a quirk for Generalplus Technology Inc. 808 Camera

2017-02-09 Thread Neil Armstrong
As reported on [1], this device needs this quirk to be able to reliably initialise the webcam. [1] https://sourceforge.net/p/linux-uvc/mailman/message/33791098/ Signed-off-by: Neil Armstrong --- drivers/media/usb/uvc/uvc_driver.c | 9 + 1 file changed, 9

Assalamu`Alaikum.

2017-02-09 Thread mohammad ouattala
Dear Sir/Madam. Assalamu`Alaikum. I am Dr mohammad ouattara, I have ($10.6 Million us dollars) to transfer into your account, I will send you more details about this deal and the procedures to follow when I receive a positive response from you, Have a great day, Dr mohammad ouattara. --

Re: [PATCH v3 1/2] staging: omap4iss: fix multiline comment style

2017-02-09 Thread Greg KH
On Tue, Feb 07, 2017 at 05:40:57PM +0200, Avraham Shukron wrote: > Signed-off-by: Avraham Shukron > --- > drivers/staging/media/omap4iss/iss_video.c | 38 > -- > 1 file changed, 25 insertions(+), 13 deletions(-) Again, I don't take patches

Re: [PATCH RESEND v7 2/2] Add support for OV5647 sensor.

2017-02-09 Thread Ramiro Oliveira
Hi Sakari On 2/9/2017 10:02 AM, Sakari Ailus wrote: > Hi Ramiro, > > On Wed, Feb 08, 2017 at 09:56:12AM +, Ramiro Oliveira wrote: >> Hi Sakari >> >> On 2/7/2017 5:31 PM, Sakari Ailus wrote: >>> Hi Ramiro, >>> >>> On Mon, Feb 06, 2017 at 11:38:28AM +, Ramiro Oliveira wrote: >>> ... >>

Re: [PATCH RESEND v7 2/2] Add support for OV5647 sensor.

2017-02-09 Thread Sakari Ailus
Hi Ramiro, On Wed, Feb 08, 2017 at 09:56:12AM +, Ramiro Oliveira wrote: > Hi Sakari > > On 2/7/2017 5:31 PM, Sakari Ailus wrote: > > Hi Ramiro, > > > > On Mon, Feb 06, 2017 at 11:38:28AM +, Ramiro Oliveira wrote: > > ... > +ret = ov5647_write_array(sd, ov5647_640x480, >

Re: [PATCH v3 21/24] media: imx: Add MIPI CSI-2 Receiver subdev driver

2017-02-09 Thread Philipp Zabel
On Wed, 2017-02-08 at 15:23 -0800, Steve Longerbeam wrote: [...] > >> + > >> +static int imxcsi2_get_fmt(struct v4l2_subdev *sd, > >> + struct v4l2_subdev_pad_config *cfg, > >> + struct v4l2_subdev_format *sdformat) > >> +{ > >> + struct imxcsi2_dev *csi2 =