Re: [PATCH v7 2/2] media: Add a driver for the ov5645 camera sensor.

2016-11-14 Thread Laurent Pinchart
Hi Todor, Thank you for the patch. On Monday 14 Nov 2016 12:24:36 Todor Tomov wrote: > The ov5645 sensor from Omnivision supports up to 2592x1944 > and CSI2 interface. > > The driver adds support for the following modes: > - 1280x960 > - 1920x1080 > - 2592x1944 > > Output format is packed 8bit

Re: [PATCH v4 7/8] v4l: Add signal lock status to source change events

2016-11-14 Thread Devin Heitmueller
> OK, but what can the application do with that event? If the glitch didn't > affect the video, then it is pointless. > > If the lock is lost, then normally you loose video as well. If not, then > applications are not interested in the event. What about free running mode (where some decoders

Re: [PATCH v3 0/9] Qualcomm video decoder/encoder driver

2016-11-14 Thread Hans Verkuil
On 11/14/2016 03:59 PM, Stanimir Varbanov wrote: > Hi, > > On 11/11/2016 02:11 PM, Javier Martinez Canillas wrote: >> Hello Hans, >> >> On Fri, Nov 11, 2016 at 8:49 AM, Hans Verkuil wrote: >>> Hi Stanimir, >>> >>> Overall it looks good. As you saw, I do have some comments but

Re: [PATCH v3 3/9] media: venus: adding core part and helper functions

2016-11-14 Thread Hans Verkuil
On 11/14/2016 10:42 AM, Stanimir Varbanov wrote: > Hi Hans, > > Thanks for the comments! > > On 11/11/2016 01:32 PM, Hans Verkuil wrote: >> Hi Stanimir, >> >> Some comments: >> >> On 11/07/2016 06:33 PM, Stanimir Varbanov wrote: >>> * core.c has implemented the platform dirver methods, file >>>

[RFCv2 PATCH] tc358840: add Toshiba HDMI-to-CSI bridge driver

2016-11-14 Thread Hans Verkuil
Second RFC version. This driver was created by Armin Weiss and cleaned up, rebased and 4k and CEC support finalized by Hans Verkuil. It's almost ready for mainlining. The main TODO is to clean up and document the device tree bindings. Signed-off-by: Armin Weiss Signed-off-by:

Re: [RFC v4 08/21] media: Enable allocating the media device dynamically

2016-11-14 Thread Sakari Ailus
Hi Shuah, On Tue, Nov 08, 2016 at 12:20:29PM -0700, Shuah Khan wrote: > On Tue, Nov 8, 2016 at 6:55 AM, Sakari Ailus > wrote: > > From: Sakari Ailus > > > > Allow allocating the media device dynamically. As the struct media_device > > embeds

Re: [PATCH 1/3] [media] dvb-usb: move data_mutex to struct dvb_usb_device

2016-11-14 Thread Michael Ira Krufky
On Sat, Nov 12, 2016 at 9:46 AM, Mauro Carvalho Chehab wrote: > From: Mauro Carvalho Chehab > > The data_mutex is initialized too late, as it is needed for > each device driver's power control, causing an OOPS: > > dvb-usb: found a

[bug report] [media] dib0700_core: don't use stack on I2C reads

2016-11-14 Thread Dan Carpenter
Hello Mauro Carvalho Chehab, The patch fa1ecd8dc454: "[media] dib0700_core: don't use stack on I2C reads" from Oct 7, 2016, leads to the following static checker warning: drivers/media/usb/dvb-usb/dib0700_core.c:273 dib0700_i2c_xfer_new() warn: inconsistent returns 'mutex:>i2c_mutex'. Locked

Re: Including images on Sphinx documents

2016-11-14 Thread Mauro Carvalho Chehab
Em Sun, 13 Nov 2016 12:52:50 -0700 Jonathan Corbet escreveu: > On Mon, 7 Nov 2016 07:55:24 -0200 > Mauro Carvalho Chehab wrote: > > > So, we have a few alternatives: > > > > 1) copy (or symlink) all rst files to Documentation/output (or to the > >

Re: [bug report] [media] dib0700_core: don't use stack on I2C reads

2016-11-14 Thread Dan Carpenter
Same for the legacy functions as well. drivers/media/usb/dvb-usb/dib0700_core.c:338 dib0700_i2c_xfer_legacy() warn: inconsistent returns 'mutex:>i2c_mutex'. Locked on: line 301 line 322 Unlocked on: line 287 line 291 line 338

Re: [PATCH v6 2/2] media: Add a driver for the ov5645 camera sensor.

2016-11-14 Thread Mark Brown
On Mon, Nov 14, 2016 at 02:18:49PM +0200, Laurent Pinchart wrote: > On Wednesday 26 Oct 2016 12:51:49 Mark Brown wrote: > > Why would this be guaranteed by the API given that it's not documented > > and why would many drivers break? It's fairly rare for devices other > > than SoCs to have strict

[PATCH 0/2] Add support for the DW IP Prototyping Kits for MIPI CSI-2 Host

2016-11-14 Thread Ramiro Oliveira
This patchset adds basic support for the DW CSI-2 Host IPK. There are some parts of the kit that aren't currently supported by this media platform driver but will be in the future. Ramiro Oliveira (2): Add Documentation for Media Device, Video Device, and Synopsys DW MIPI CSI-2 Host Add

[PATCH 2/2] Add basic support for DW CSI-2 Host IPK

2016-11-14 Thread Ramiro Oliveira
Add support for basic configuration of the DW CSI-2 Host IPK Signed-off-by: Ramiro Oliveira --- MAINTAINERS | 7 + drivers/media/platform/Kconfig | 1 + drivers/media/platform/Makefile | 2 +

[PATCH 1/2] Add Documentation for Media Device, Video Device, and Synopsys DW MIPI CSI-2 Host

2016-11-14 Thread Ramiro Oliveira
Add documentation for Media and Video Device, as well as the DW MIPI CSI-2 Host. Signed-off-by: Ramiro Oliveira --- .../devicetree/bindings/media/snps,dw-mipi-csi.txt | 27 ++ .../devicetree/bindings/media/snps,plat-ipk.txt| 9

[PATCH] uvcvideo: Add bayer 16-bit format patterns

2016-11-14 Thread Edgar Thier
>From aec97c931cb4b91f91dd0ed38f74d866d4f13347 Mon Sep 17 00:00:00 2001 From: Edgar Thier Date: Mon, 14 Nov 2016 14:17:57 +0100 Subject: [PATCH] uvcvideo: Add bayer 16-bit format patterns Add bayer 16-bit GUIDs to uvcvideo and associated them with the corresponding V4L2

Re: [PATCH] uvcvideo: Add bayer 16-bit format patterns

2016-11-14 Thread Sakari Ailus
Hi Edgar, On Mon, Nov 14, 2016 at 02:26:56PM +0100, Edgar Thier wrote: > From aec97c931cb4b91f91dd0ed38f74d866d4f13347 Mon Sep 17 00:00:00 2001 > From: Edgar Thier > Date: Mon, 14 Nov 2016 14:17:57 +0100 > Subject: [PATCH] uvcvideo: Add bayer 16-bit format patterns > > Add

Re: [PATCH 1/2] Add Documentation for Media Device, Video Device, and Synopsys DW MIPI CSI-2 Host

2016-11-14 Thread Laurent Pinchart
Hi Ramiro, Thank you for the patch. On Monday 14 Nov 2016 14:20:22 Ramiro Oliveira wrote: > Add documentation for Media and Video Device, as well as the DW MIPI CSI-2 > Host. > > Signed-off-by: Ramiro Oliveira > --- > .../devicetree/bindings/media/snps,dw-mipi-csi.txt |

Re: [PATCH v3 0/9] Qualcomm video decoder/encoder driver

2016-11-14 Thread Stanimir Varbanov
Hi, On 11/11/2016 02:11 PM, Javier Martinez Canillas wrote: > Hello Hans, > > On Fri, Nov 11, 2016 at 8:49 AM, Hans Verkuil wrote: >> Hi Stanimir, >> >> Overall it looks good. As you saw, I do have some comments but nothing major. >> >> One question: you use qcom as the

Re: [PATCH v6 2/2] media: Add a driver for the ov5645 camera sensor.

2016-11-14 Thread Laurent Pinchart
Hi Mark, On Wednesday 26 Oct 2016 12:51:49 Mark Brown wrote: > On Wed, Oct 26, 2016 at 02:27:23PM +0300, Todor Tomov wrote: > > And using Mark Brown's correct address... > > This is an *enormous* e-mail quoted to multiple levels with top posting > and very little editing which makes it

Re: [PATCH v7 1/2] media: i2c/ov5645: add the device tree binding document

2016-11-14 Thread Laurent Pinchart
Hi Todor, Thank you for the patch. On Monday 14 Nov 2016 12:24:35 Todor Tomov wrote: > Add the document for ov5645 device tree binding. > > Signed-off-by: Todor Tomov Reviewed-by: Laurent Pinchart > --- >

Re: [PATCH v7 2/2] media: Add a driver for the ov5645 camera sensor.

2016-11-14 Thread Todor Tomov
Hi Laurent, Thank you once again for the review. On 11/14/2016 02:38 PM, Laurent Pinchart wrote: > Hi Todor, > > Thank you for the patch. > > On Monday 14 Nov 2016 12:24:36 Todor Tomov wrote: >> The ov5645 sensor from Omnivision supports up to 2592x1944 >> and CSI2 interface. >> >> The driver

[PATCH v4 2/2] Add support for OV5647 sensor

2016-11-14 Thread Ramiro Oliveira
Add support for OV5647 sensor. Modes supported: - 640x480 RAW 8 Signed-off-by: Ramiro Oliveira --- MAINTAINERS| 7 + drivers/media/i2c/Kconfig | 12 + drivers/media/i2c/Makefile | 1 + drivers/media/i2c/ov5647.c | 861

[PATCH v4 0/2] Add support for Omnivision OV5647

2016-11-14 Thread Ramiro Oliveira
Hello, This patch adds support for the Omnivision OV5647 sensor. At the moment it only supports 640x480 in Raw 8. This is the fourth version of the OV5647 camera driver patchset. v4: - Add correct license - Revert debugging info to generic infrastructure - Turn defines into enums -

[PATCH v4 1/2] Add OV5647 device tree documentation

2016-11-14 Thread Ramiro Oliveira
Add device tree documentation. Signed-off-by: Ramiro Oliveira --- .../devicetree/bindings/media/i2c/ov5647.txt | 19 +++ 1 file changed, 19 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/i2c/ov5647.txt diff --git

Re: [RFC v3 00/21] Make use of kref in media device, grab references as needed

2016-11-14 Thread Sakari Ailus
Hi Mauro, I'm replying below but let me first summarise the remaining problem area that this patchset addresses. The problems you and Shuah have seen and partially addressed are related to a larger picture which is the lifetime of (mostly) memory resources related to various objects used by as

[PATCH] s5p-mfc: Fix clock management in s5p_mfc_release function

2016-11-14 Thread Marek Szyprowski
Clock control indirectly requires access to MFC device, so call it only if we are sure that the device exists in s5p_mfc_release function. s5p_mfc_remove() calls s5p_mfc_final_pm(), which releases all PM related resources, including clocks, so any call to clocks related functions is not valid

Re: Including images on Sphinx documents

2016-11-14 Thread Mauro Carvalho Chehab
Em Sun, 13 Nov 2016 14:00:27 -0700 Jonathan Corbet escreveu: > On Mon, 7 Nov 2016 09:46:48 -0200 > Mauro Carvalho Chehab wrote: > > > When running LaTeX in interactive mode, building just the media > > PDF file with: > > > > $ cls;make cleandocs;

Re: [PATCH] usb: core: urb make use of usb_endpoint_maxp_mult

2016-11-14 Thread Greg KH
On Sun, Nov 13, 2016 at 01:31:16PM +0300, Mike Krinkin wrote: > Since usb_endpoint_maxp now returns only lower 11 bits mult > calculation here isn't correct anymore and that breaks webcam > for me. Patch make use of usb_endpoint_maxp_mult instead of > direct calculation. > > Fixes: abb621844f6a

Re: [PATCH v3 3/9] media: venus: adding core part and helper functions

2016-11-14 Thread Stanimir Varbanov
Hi Hans, >>> +void vidc_vb2_stop_streaming(struct vb2_queue *q) +{ + struct venus_inst *inst = vb2_get_drv_priv(q); + struct venus_core *core = inst->core; + struct device *dev = core->dev; + struct vb2_queue *other_queue; + struct vidc_buffer *buf, *n;

Re: [PATCH v3 4/9] media: venus: vdec: add video decoder files

2016-11-14 Thread Stanimir Varbanov
Hi Hans, Thanks for the comments! On 11/11/2016 01:39 PM, Hans Verkuil wrote: > I made some comments about start_streaming in my review of patch 3/9, so > I am not going to repeat that here. Sure. > > On 11/07/2016 06:33 PM, Stanimir Varbanov wrote: >> This consists of video decoder

Re: [RFC PATCH 6/7] atmel-isi: remove dependency of the soc-camera framework

2016-11-14 Thread Wu, Songjun
On 11/14/2016 16:19, Hans Verkuil wrote: Hi Songjun, On 10/19/2016 09:48 AM, Wu, Songjun wrote: On 10/19/2016 15:46, Hans Verkuil wrote: On 10/19/2016 09:36 AM, Wu, Songjun wrote: On 10/18/2016 18:58, Hans Verkuil wrote: On 10/18/16 11:21, Wu, Songjun wrote: Hi Hans, Do you have any

Re: [PATCH v3 3/9] media: venus: adding core part and helper functions

2016-11-14 Thread Stanimir Varbanov
Hi Hans, Thanks for the comments! On 11/11/2016 01:32 PM, Hans Verkuil wrote: > Hi Stanimir, > > Some comments: > > On 11/07/2016 06:33 PM, Stanimir Varbanov wrote: >> * core.c has implemented the platform dirver methods, file >> operations and v4l2 registration. >> >> * helpers.c has

Re: [RFC 07/10] sunxi-cedrus: Add a MPEG 2 codec

2016-11-14 Thread Hans Verkuil
On 08/25/2016 11:39 AM, Florent Revest wrote: > This patch introduces the support of MPEG2 video decoding to the > sunxi-cedrus video decoder driver. > > Signed-off-by: Florent Revest > --- > drivers/media/platform/sunxi-cedrus/Makefile | 2 +- >

Re: [PATCH v4 3/8] media: adv7180: add support for NEWAVMODE

2016-11-14 Thread Hans Verkuil
On 08/03/2016 08:03 PM, Steve Longerbeam wrote: > Parse the optional v4l2 endpoint DT node. If the bus type is > V4L2_MBUS_BT656 and the endpoint node specifies "newavmode", > configure the BT.656 bus in NEWAVMODE. > > Signed-off-by: Steve Longerbeam > > --- > >

Re: [PATCH] usb: core: urb make use of usb_endpoint_maxp_mult

2016-11-14 Thread Felipe Balbi
Hi, Greg KH writes: > On Sun, Nov 13, 2016 at 01:31:16PM +0300, Mike Krinkin wrote: >> Since usb_endpoint_maxp now returns only lower 11 bits mult >> calculation here isn't correct anymore and that breaks webcam >> for me. Patch make use of usb_endpoint_maxp_mult

Re: [RFC 03/10] v4l: Add sunxi Video Engine pixel format

2016-11-14 Thread Hans Verkuil
On 08/25/2016 11:39 AM, Florent Revest wrote: > Add support for the allwinner's proprietary pixel format described in > details here: http://linux-sunxi.org/File:Ve_tile_format_v1.pdf > > This format is similar to V4L2_PIX_FMT_NV12M but the planes are divided > in tiles of 32x32px. > >

Re: [RFC 07/10] sunxi-cedrus: Add a MPEG 2 codec

2016-11-14 Thread Hans Verkuil
On 11/14/2016 11:18 AM, Hans Verkuil wrote: > On 08/25/2016 11:39 AM, Florent Revest wrote: >> This patch introduces the support of MPEG2 video decoding to the >> sunxi-cedrus video decoder driver. >> >> Signed-off-by: Florent Revest >> --- >>

[PATCH v7 2/2] media: Add a driver for the ov5645 camera sensor.

2016-11-14 Thread Todor Tomov
The ov5645 sensor from Omnivision supports up to 2592x1944 and CSI2 interface. The driver adds support for the following modes: - 1280x960 - 1920x1080 - 2592x1944 Output format is packed 8bit UYVY. Signed-off-by: Todor Tomov --- drivers/media/i2c/Kconfig | 12 +

[PATCH v7 0/2] OV5645 camera sensor driver

2016-11-14 Thread Todor Tomov
This is the seventh version of the OV5645 camera sensor driver patchset. Changes since version 6 include: - keep a pointer to the current sensor mode and remove enum ov5645_mode; - do not keep v4l2 control values and xclk frequency in the main data struct; - add caching variables in then main

[PATCH v7 1/2] media: i2c/ov5645: add the device tree binding document

2016-11-14 Thread Todor Tomov
Add the document for ov5645 device tree binding. Signed-off-by: Todor Tomov --- .../devicetree/bindings/media/i2c/ov5645.txt | 54 ++ 1 file changed, 54 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/i2c/ov5645.txt

Re: [PATCH v3 5/9] media: venus: venc: add video encoder files

2016-11-14 Thread Stanimir Varbanov
Hi Hans, Thanks for the comments! On 11/11/2016 01:43 PM, Hans Verkuil wrote: > The comments I made before about start_streaming and the use of struct > venus_ctrl > apply here as well and I won't repeat them. > > On 11/07/2016 06:33 PM, Stanimir Varbanov wrote: >> This adds encoder part of

[PATCH 04/12 v2] exynos-gsc: Make PM callbacks available conditionally

2016-11-14 Thread Marek Szyprowski
From: Ulf Hansson There are no need to set up the PM callbacks (runtime and system) unless they are being used. It also causes compiler warnings about unused functions. Silence the warnings by making them available for CONFIG_PM (runtime callbacks) and CONFIG_PM_SLEEP

Re: [PATCH v4 1/8] media: adv7180: fix field type

2016-11-14 Thread Hans Verkuil
On 08/03/2016 08:03 PM, Steve Longerbeam wrote: > From: Steve Longerbeam > > The ADV7180 and ADV7182 transmit whole fields, bottom field followed > by top (or vice-versa, depending on detected video standard). So > for chips that do not have support for explicitly setting

Re: [Intel-gfx] [PATCH 1/4] lib/scatterlist: Fix offset type in sg_alloc_table_from_pages

2016-11-14 Thread Chris Wilson
On Fri, Nov 11, 2016 at 08:50:17AM +, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin > > Scatterlist entries have an unsigned int for the offset so > correct the sg_alloc_table_from_pages function accordingly. > > Since these are offsets withing a page, unsigned int

Re: [PATCH v4 7/8] v4l: Add signal lock status to source change events

2016-11-14 Thread Hans Verkuil
On 08/03/2016 08:03 PM, Steve Longerbeam wrote: > Add a signal lock status change to the source changes bitmask. > This indicates there was a signal lock or unlock event detected > at the input of a video decoder. > > Signed-off-by: Steve Longerbeam > Cc: Mauro

Re: [RFC] V4L2 unified low-level decoder API

2016-11-14 Thread Hans Verkuil
On 10/27/2016 09:42 AM, Hugues FRUCHET wrote: > Hi, > > This RFC aims to start discussions in order to define the codec specific > controls structures to fulfill the low-level decoder API needed by non > "Stream API" based decoders ("stateless" or "Frame API" based decoders). Let's refer to

Re: [PATCH v3 3/9] media: venus: adding core part and helper functions

2016-11-14 Thread Hans Verkuil
On 11/14/2016 11:11 AM, Stanimir Varbanov wrote: > Hi Hans, > > > > +void vidc_vb2_stop_streaming(struct vb2_queue *q) > +{ > + struct venus_inst *inst = vb2_get_drv_priv(q); > + struct venus_core *core = inst->core; > + struct device *dev = core->dev; > + struct

Re: [PATCHv4] media: rcar-csi2: add Renesas R-Car MIPI CSI-2 driver

2016-11-14 Thread Geert Uytterhoeven
On Sat, Nov 12, 2016 at 1:29 PM, Niklas Söderlund wrote: > +Example: > + > +/* SoC properties */ > + > +csi20: csi2@fea8 { > +compatible = "renesas,r8a7795-csi2"; 7795 > +reg = <0 0xfea8 0 0x1>; > +

Re: [PATCH v4 5/8] media: adv7180: implement g_parm

2016-11-14 Thread Hans Verkuil
On 08/03/2016 08:03 PM, Steve Longerbeam wrote: > Implement g_parm to return the current standard's frame period. > > Signed-off-by: Steve Longerbeam > Tested-by: Tim Harvey > Acked-by: Tim Harvey > > --- > v4: no

Re: [RFC PATCH 6/7] atmel-isi: remove dependency of the soc-camera framework

2016-11-14 Thread Hans Verkuil
Hi Songjun, On 10/19/2016 09:48 AM, Wu, Songjun wrote: > > > On 10/19/2016 15:46, Hans Verkuil wrote: >> On 10/19/2016 09:36 AM, Wu, Songjun wrote: >>> >>> >>> On 10/18/2016 18:58, Hans Verkuil wrote: On 10/18/16 11:21, Wu, Songjun wrote: > Hi Hans, > > Do you have any issue on

Re: [RFC 06/10] media: platform: Add Sunxi Cedrus decoder driver

2016-11-14 Thread Hans Verkuil
On 08/25/2016 11:39 AM, Florent Revest wrote: > This patch adds a "sunxi-cedrus" v4l2 m2m decoder driver for > Allwinner's Video Processing Unit. This VPU has a low-level interface > which requires manual registers writing for frame headers. Hence, it > depends on the Request API to synchronize

[PATCH] ARM: shmobile: dts: Switch to panel-lvds bindings for Mitsubishi panels

2016-11-14 Thread Laurent Pinchart
The aa104xd12 and aa121td01 panels are LVDS panels, not DPI panels. Use the correct DT bindings. Signed-off-by: Laurent Pinchart --- arch/arm/boot/dts/r8a77xx-aa104xd12-panel.dtsi | 3 ++- arch/arm/boot/dts/r8a77xx-aa121td01-panel.dtsi | 3 ++- 2 files

Re: [RFC v4 08/21] media: Enable allocating the media device dynamically

2016-11-14 Thread Shuah Khan
On 11/14/2016 06:40 AM, Sakari Ailus wrote: > Hi Shuah, > > On Tue, Nov 08, 2016 at 12:20:29PM -0700, Shuah Khan wrote: >> On Tue, Nov 8, 2016 at 6:55 AM, Sakari Ailus >> wrote: >>> From: Sakari Ailus >>> >>> Allow allocating the media device

Re: [PATCH 1/2] devicetree/bindings: display: Add bindings for LVDS panels

2016-11-14 Thread Rob Herring
On Mon, Oct 17, 2016 at 7:42 AM, Laurent Pinchart wrote: > Hi Rob, > > On Friday 14 Oct 2016 07:40:14 Rob Herring wrote: >> On Sun, Oct 9, 2016 at 11:33 AM, Laurent Pinchart wrote: >> > On Saturday 08 Oct 2016 20:29:39 Rob Herring wrote: >> >> On Tue, Oct 04,

Re: [PATCH v4] media: Driver for Toshiba et8ek8 5MP sensor

2016-11-14 Thread Sebastian Reichel
Hi Sakari, On Mon, Nov 14, 2016 at 11:58:28PM +0200, Sakari Ailus wrote: > [...] > > On Fri, Nov 04, 2016 at 01:05:25AM +0100, Sebastian Reichel wrote: > > I'm not sure what part relevant for video-bus-switch is currently > > not supported? > > > > video-bus-switch registers its own async

Re: [PATCH] [media] mtk-mdp: allocate video_device dynamically

2016-11-14 Thread Minghsiu Tsai
On Fri, 2016-11-11 at 11:51 +0100, Hans Verkuil wrote: > Almost correct: > > On 11/07/2016 01:42 PM, Minghsiu Tsai wrote: > > It can fix known problems with embedded video_device structs. > > > > Signed-off-by: Minghsiu Tsai > > --- > >

[PATCH v2] [media] mtk-mdp: allocate video_device dynamically

2016-11-14 Thread Minghsiu Tsai
It can fix known problems with embedded video_device structs. Signed-off-by: Minghsiu Tsai --- Changes in v2: . Call video_device_release() instead of video_unregister_device() in mtk_mdp_register_m2m_device() --- drivers/media/platform/mtk-mdp/mtk_mdp_core.h |

Re: [PATCH 1/2] devicetree/bindings: display: Add bindings for LVDS panels

2016-11-14 Thread Laurent Pinchart
Hi Rob, On Monday 14 Nov 2016 19:40:26 Rob Herring wrote: > On Mon, Oct 17, 2016 at 7:42 AM, Laurent Pinchart wrote: > > On Friday 14 Oct 2016 07:40:14 Rob Herring wrote: > >> On Sun, Oct 9, 2016 at 11:33 AM, Laurent Pinchart wrote: > >>> On Saturday 08 Oct 2016 20:29:39 Rob Herring wrote: >

Re: [PATCH 1/2] Add Documentation for Media Device, Video Device, and Synopsys DW MIPI CSI-2 Host

2016-11-14 Thread Ramiro Oliveira
Hi Laurent, Thanks for the feedback. On 11/14/2016 2:49 PM, Laurent Pinchart wrote: > Hi Ramiro, > > Thank you for the patch. > > On Monday 14 Nov 2016 14:20:22 Ramiro Oliveira wrote: >> Add documentation for Media and Video Device, as well as the DW MIPI CSI-2 >> Host. >> >> Signed-off-by:

Re: [PATCH 3/5] media: Add new SDR formats SC16, SC18 & SC20

2016-11-14 Thread Hans Verkuil
On 11/14/2016 05:20 PM, Ramesh Shanmugasundaram wrote: > Hi Hans, > > Thanks for the review comments. > >> Subject: Re: [PATCH 3/5] media: Add new SDR formats SC16, SC18 & SC20 >> >> On 11/09/2016 04:44 PM, Ramesh Shanmugasundaram wrote: >>> This patch adds support for the three new SDR formats.

[RFCv2 PATCH 2/5] drm/bridge: dw_hdmi: remove CEC engine register definitions

2016-11-14 Thread Hans Verkuil
From: Russell King We don't need the CEC engine register definitions, so let's remove them. Signed-off-by: Russell King --- drivers/gpu/drm/bridge/dw-hdmi.h | 45 1 file changed, 45 deletions(-)

[RFCv2 PATCH 4/5] drm/bridge: add dw-hdmi cec driver using Hans Verkuil's CEC code

2016-11-14 Thread Hans Verkuil
From: Russell King Add a CEC driver for the dw-hdmi hardware using Hans Verkuil's CEC implementation. Signed-off-by: Russell King --- drivers/gpu/drm/bridge/Kconfig| 7 + drivers/gpu/drm/bridge/Makefile | 1 +

[RFCv2 PATCH 3/5] drm/bridge: dw_hdmi: add HDMI notifier support

2016-11-14 Thread Hans Verkuil
From: Russell King Add HDMI notifiers to the HDMI bridge driver. Signed-off-by: Russell King --- drivers/gpu/drm/bridge/Kconfig | 1 + drivers/gpu/drm/bridge/dw-hdmi.c | 25 - 2 files changed, 25

[RFCv2 PATCH 5/5] drm/i2c: add tda998x/tda9950 CEC driver

2016-11-14 Thread Hans Verkuil
From: Russell King Add a CEC driver for the TDA9950, which is a stand-alone I2C CEC device. The TDA9950 contains a command processor which handles retransmissions and the low level bus protocol. The driver just has to read and write the messages, and handle error

[RFCv2 PATCH 0/5] CEC drivers for iMX6 and TDA9950

2016-11-14 Thread Hans Verkuil
From: Hans Verkuil This patch series is an update to this RFC series from Russell: https://lists.freedesktop.org/archives/dri-devel/2016-August/115733.html I have not seen any updates to this, so I hope that that series is still the latest version. The main problem

Re: [PATCH v3 5/6] Documentation: bindings: add documentation for ir-spi device driver

2016-11-14 Thread Rob Herring
On Thu, Nov 10, 2016 at 4:02 AM, Andi Shyti wrote: > Hi Jacek, > >> > > > > Only DT bindings of LED class drivers should be placed in >> > > > > Documentation/devicetree/bindings/leds. Please move it to the >> > > > > media bindings. >> > > > >> > > > that's where I placed

Re: [RFCv2 PATCH 2/5] drm/bridge: dw_hdmi: remove CEC engine register definitions

2016-11-14 Thread Hans Verkuil
You're CC-ed for all, so if you don't receive it in the next 15 minutes let me know and I'll forward it to you. But my guess is that the mails were delayed for some reason and that they simply haven't arrived yet. Hans On 11/14/2016 04:39 PM, Russell King - ARM Linux wrote: > I can't

Re: [PATCH v5] media: et8ek8: add device tree binding documentation

2016-11-14 Thread Rob Herring
On Mon, Nov 07, 2016 at 11:46:48AM +0100, Pavel Machek wrote: > Add device tree binding documentation for toshiba et8ek8 sensor. > > Signed-off-by: Ivaylo Dimitrov > Signed-off-by: Pavel Machek > > diff --git

Re: [RFCv2 PATCH 2/5] drm/bridge: dw_hdmi: remove CEC engine register definitions

2016-11-14 Thread Russell King - ARM Linux
I can't comment on these, you've not included me in patch 1 nor the covering message. On Mon, Nov 14, 2016 at 04:22:45PM +0100, Hans Verkuil wrote: > From: Russell King > > We don't need the CEC engine register definitions, so let's remove them. > > Signed-off-by:

RE: [PATCH 5/5] media: platform: rcar_drif: Add DRIF support

2016-11-14 Thread Ramesh Shanmugasundaram
Hi Hans, Thanks for the review comments. > Subject: Re: [PATCH 5/5] media: platform: rcar_drif: Add DRIF support > > On 11/09/2016 04:44 PM, Ramesh Shanmugasundaram wrote: > > This patch adds Digital Radio Interface (DRIF) support to R-Car Gen3 > SoCs. > > The driver exposes each instance of

RE: [PATCH 2/5] media: i2c: max2175: Add MAX2175 support

2016-11-14 Thread Ramesh Shanmugasundaram
Hi Hans, Thank you for the review comments. > On 11/09/2016 04:44 PM, Ramesh Shanmugasundaram wrote: > > This patch adds driver support for MAX2175 chip. This is Maxim > > Integrated's RF to Bits tuner front end chip designed for > > software-defined radio solutions. This driver exposes the

RE: [RFC 5/5] doc_rst: media: New SDR formats SC16, SC18 & SC20

2016-11-14 Thread Ramesh Shanmugasundaram
Hello Laurent, Antti, Hans, > Subject: Re: [RFC 5/5] doc_rst: media: New SDR formats SC16, SC18 & SC20 > > On 11/11/2016 02:57 PM, Laurent Pinchart wrote: > > Hi Hans, > > > > On Friday 11 Nov 2016 14:53:58 Hans Verkuil wrote: > >> On 11/10/2016 09:08 AM, Laurent Pinchart wrote: > >>> Antti,

RE: [PATCH 3/5] media: Add new SDR formats SC16, SC18 & SC20

2016-11-14 Thread Ramesh Shanmugasundaram
Hi Hans, Thanks for the review comments. > Subject: Re: [PATCH 3/5] media: Add new SDR formats SC16, SC18 & SC20 > > On 11/09/2016 04:44 PM, Ramesh Shanmugasundaram wrote: > > This patch adds support for the three new SDR formats. These formats > > were prefixed with "sliced" indicating I data

Re: [RFC] [media] Add Synopsys Designware HDMI RX PHY e405 driver

2016-11-14 Thread Jose Abreu
Hi Hans, On 11-11-2016 14:52, Hans Verkuil wrote: > Hi Jose, > > On 11/09/2016 06:43 PM, Jose Abreu wrote: >> Hi All, >> >> This is a RFC patch for Synopsys Designware HDMI RX PHY e405. >> This phy receives and decodes HDMI video that is delivered to >> a controller. The controller bit is not

[RFCv2 PATCH 1/5] video: add HDMI state notifier support

2016-11-14 Thread Hans Verkuil
From: Hans Verkuil Add support for HDMI hotplug and EDID notifiers, which is used to convey information from HDMI drivers to their CEC and audio counterparts. Based on an earlier version from Russell King: https://patchwork.kernel.org/patch/9277043/ The hdmi_notifier

[PATCH] uvcvideo: Add bayer 16-bit format patterns

2016-11-14 Thread Edgar Thier
>From 10ce06db4ab3c037758b3cb5264007f59801f1a1 Mon Sep 17 00:00:00 2001 From: Edgar Thier Date: Tue, 15 Nov 2016 06:33:10 +0100 Subject: [PATCH] uvcvideo: Add bayer 16-bit format patterns Signed-off-by: Edgar Thier ---

cron job: media_tree daily build: ERRORS

2016-11-14 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: Tue Nov 15 05:00:21 CET 2016 media-tree git hash:bd676c0c04ec94bd830b9192e2c33f2c4532278d media_build

[PATCH] [media] ir-hix5hd2: make hisilicon,power-syscon property deprecated

2016-11-14 Thread Jiancheng Xue
From: Ruqiang Ju The clock of IR can be provided by the clock provider and controlled by common clock framework APIs. Signed-off-by: Ruqiang Ju Signed-off-by: Jiancheng Xue ---

Re: [PATCH] uvcvideo: Add bayer 16-bit format patterns

2016-11-14 Thread Edgar Thier
Hi Sakari, Sorry for the waiting. I hope the patch I just submitted is alright. I took the freedom and uploaded the lsusb -v output for 3 cameras with bayer 16-bit patterns. You can find them here: dfk23up1300_16bitbayer_RG.lsusb: http://pastebin.com/PDdY7rs0 dfk23ux249_16bitbayer_GB.lsusb:

Re: [PATCH v4 3/8] media: adv7180: add support for NEWAVMODE

2016-11-14 Thread Steve Longerbeam
On 11/14/2016 03:28 AM, Hans Verkuil wrote: On 08/03/2016 08:03 PM, Steve Longerbeam wrote: Parse the optional v4l2 endpoint DT node. If the bus type is V4L2_MBUS_BT656 and the endpoint node specifies "newavmode", configure the BT.656 bus in NEWAVMODE. Signed-off-by: Steve Longerbeam

Re: [PATCH v3 1/9] doc: DT: vidc: binding document for Qualcomm video driver

2016-11-14 Thread Rob Herring
On Mon, Nov 07, 2016 at 07:33:55PM +0200, Stanimir Varbanov wrote: > Add binding document for Venus video encoder/decoder driver > > Cc: Rob Herring > Cc: Mark Rutland > Cc: devicet...@vger.kernel.org > Signed-off-by: Stanimir Varbanov

Re: [PATCH v5 1/3] dt-bindings: mediatek: Add a binding for Mediatek JPEG Decoder

2016-11-14 Thread Rob Herring
On Tue, Nov 08, 2016 at 02:34:38PM +0800, Rick Chang wrote: > Add a DT binding documentation for Mediatek JPEG Decoder of > MT2701 SoC. > > Signed-off-by: Rick Chang > Signed-off-by: Minghsiu Tsai > --- >

[GIT PULL] Samsung SoC driver updates for 4.10

2016-11-14 Thread Sylwester Nawrocki
Hi Mauro, This includes mostly clean up and fixes of the s5p-mfc and exynos-gsc drivers. Support for the Exynos5433 (64-bit ARM) SoC is added to the exynos-gsc driver. The following changes since commit 669c6141ea78dff885b5bf025456c7dffb669a61: [media] mtk-mdp: fix double mutex_unlock

[PATCH v6] media: et8ek8: add device tree binding documentation

2016-11-14 Thread Pavel Machek
Add device tree binding documentation for toshiba et8ek8 sensor. Signed-off-by: Ivaylo Dimitrov Signed-off-by: Pavel Machek --- v6: added missing article, fixed signal polarity. diff --git

Re: [PATCH v6] media: et8ek8: add device tree binding documentation

2016-11-14 Thread Rob Herring
On Mon, Nov 14, 2016 at 12:30 PM, Pavel Machek wrote: > Add device tree binding documentation for toshiba et8ek8 sensor. > > Signed-off-by: Ivaylo Dimitrov > Signed-off-by: Pavel Machek Acked-by: Rob Herring -- To

Re: [PATCH 2/2] Add basic support for DW CSI-2 Host IPK

2016-11-14 Thread kbuild test robot
Hi Ramiro, [auto build test ERROR on linuxtv-media/master] [also build test ERROR on v4.9-rc5 next-20161114] [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/Ramiro-Oliveira/Add-support

Re: [PATCH v4 7/8] v4l: Add signal lock status to source change events

2016-11-14 Thread Steve Longerbeam
On 11/14/2016 11:19 AM, Steve Longerbeam wrote: On 11/14/2016 03:36 AM, Hans Verkuil wrote: On 08/03/2016 08:03 PM, Steve Longerbeam wrote: Add a signal lock status change to the source changes bitmask. This indicates there was a signal lock or unlock event detected at the input of a video

Re: [PATCH 2/5] media: i2c: max2175: Add MAX2175 support

2016-11-14 Thread Rob Herring
On Wed, Nov 09, 2016 at 03:44:41PM +, Ramesh Shanmugasundaram wrote: > This patch adds driver support for MAX2175 chip. This is Maxim > Integrated's RF to Bits tuner front end chip designed for software-defined > radio solutions. This driver exposes the tuner as a sub-device instance > with

Re: [PATCH 2/2] Add basic support for DW CSI-2 Host IPK

2016-11-14 Thread kbuild test robot
Hi Ramiro, [auto build test WARNING on linuxtv-media/master] [also build test WARNING on v4.9-rc5] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH v4 7/8] v4l: Add signal lock status to source change events

2016-11-14 Thread Steve Longerbeam
On 11/14/2016 03:36 AM, Hans Verkuil wrote: On 08/03/2016 08:03 PM, Steve Longerbeam wrote: Add a signal lock status change to the source changes bitmask. This indicates there was a signal lock or unlock event detected at the input of a video decoder. Signed-off-by: Steve Longerbeam

Re: [PATCH 5/5] media: platform: rcar_drif: Add DRIF support

2016-11-14 Thread Rob Herring
On Thu, Nov 10, 2016 at 11:22:20AM +0200, Laurent Pinchart wrote: > Hi Ramesh, > > Thank you for the patch. > > On Wednesday 09 Nov 2016 15:44:44 Ramesh Shanmugasundaram wrote: > > This patch adds Digital Radio Interface (DRIF) support to R-Car Gen3 SoCs. > > The driver exposes each instance of

[PATCH dtv-scan-tables] Rename pl-Krosno_Sucha_Gora with only ASCII characters

2016-11-14 Thread Thomas Petazzoni
The pl-Krosno_Sucha_Gora file, added in commit 4cb113fd15e562f0629000fcad9f41405595198d, is the only file that contains non-ASCII characters in the tree. This causes a number of build issues with other packages that don't necessarily handle very well non-ASCII file name encodings. Since no other

Re: [PATCH v4] media: Driver for Toshiba et8ek8 5MP sensor

2016-11-14 Thread Sakari Ailus
Hi Sebastian, On Fri, Nov 04, 2016 at 01:05:25AM +0100, Sebastian Reichel wrote: > Hi, > > On Fri, Nov 04, 2016 at 01:05:01AM +0200, Sakari Ailus wrote: > > On Thu, Nov 03, 2016 at 11:48:43PM +0100, Sebastian Reichel wrote: > > > On Tue, Nov 01, 2016 at 12:54:08AM +0200, Sakari Ailus wrote: > >

Re: [PATCH 5/5] media: platform: rcar_drif: Add DRIF support

2016-11-14 Thread Geert Uytterhoeven
Hi Rob, On Mon, Nov 14, 2016 at 8:52 PM, Rob Herring wrote: > On Thu, Nov 10, 2016 at 11:22:20AM +0200, Laurent Pinchart wrote: >> On Wednesday 09 Nov 2016 15:44:44 Ramesh Shanmugasundaram wrote: >> > --- /dev/null >> > +++

Re: [PATCH 1/2] devicetree/bindings: display: Add bindings for LVDS panels

2016-11-14 Thread Laurent Pinchart
Hi Rob, Ping ? On Monday 17 Oct 2016 15:42:56 Laurent Pinchart wrote: > On Friday 14 Oct 2016 07:40:14 Rob Herring wrote: > > On Sun, Oct 9, 2016 at 11:33 AM, Laurent Pinchart wrote: > >> On Saturday 08 Oct 2016 20:29:39 Rob Herring wrote: > >>> On Tue, Oct 04, 2016 at 07:23:29PM +0300, Laurent

Re: [PATCH 01/12] [media] rc-main: clear rc_map.name in ir_free_table()

2016-11-14 Thread Shuah Khan
On 08/09/2016 03:32 PM, Max Kellermann wrote: > rc_unregister_device() will first call ir_free_table(), and later > device_del(); however, the latter causes a call to rc_dev_uevent(), > which prints rc_map.name, which at this point has already bee freed. > > This fixes a use-after-free bug found