Re: [PATCH 2/4] [media] pxa_camera: Fix incorrect test in the image size generation

2017-05-02 Thread Petr Cvek
Dne 2.5.2017 v 16:43 Robert Jarzmik napsal(a): > Petr Cvek writes: > >> During the transfer from the soc_camera a test in pxa_mbus_image_size() >> got removed. Without it any PXA_MBUS_LAYOUT_PACKED format causes either >> the return of a wrong value (PXA_MBUS_PACKING_2X8_PADHI doubles >> the corr

cron job: media_tree daily build: ERRORS

2017-05-02 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: Wed May 3 05:00:26 CEST 2017 media-tree git hash:3622d3e77ecef090b5111e3c5423313f11711dfa media_build gi

[PATCH v3 2/2] em28xx: add support for new of Terratec H6

2017-05-02 Thread Mauro Carvalho Chehab
There's a new version of Terratec H6 with uses USB ID 0ccd:10b2. This version is similar to the old one (with is supported via the HTC entry), except that this one has the eeprom on the second bus. On this board, one side of this board is labeled with: dvbc v2.0 The other side with:

[PATCH v3 1/2] em28xx: Ignore errors while reading from eeprom

2017-05-02 Thread Mauro Carvalho Chehab
While testing support for Terratec H6 rev. 2, it was noticed that reading from eeprom there causes a timeout error. Apparently, this is due to the need of properly setting GPIOs. In any case, the driver doesn't really require eeprom reading to succeed, as this is currently used only for debug. S

[PATCH v3 0/2] Add support for Terratec H6 version 2

2017-05-02 Thread Mauro Carvalho Chehab
That's the third version of this patch series. It adds support for Terratec H6 version 2. As on the past version, this board is identified as "H5 MKII". There's a typo on the manufacturer's name, though: [ 2970.196999] usb 1-1.5: Product: TERRATCE H5 MKII [ 2970.197011] usb 1-1.5: Manufacturer: T

Re: [PATCH] v4l2-async: add v4l2_async_match()

2017-05-02 Thread Niklas Söderlund
Hej Sakari, Thanks for your feedback. On 2017-05-02 23:23:47 +0300, Sakari Ailus wrote: > Hi Niklas, > > Niklas Söderlund wrote: > > For drivers registering notifiers with more then one subdevices it's > > difficult to know exactly which one is being bound in each call to the > > notifiers bound

Re: [PATCH v5] media: platform: Renesas IMR driver

2017-05-02 Thread Laurent Pinchart
Hi Geert, On Wednesday 22 Mar 2017 10:34:16 Geert Uytterhoeven wrote: > On Thu, Mar 9, 2017 at 9:08 PM, Sergei Shtylyov wrote: > > --- /dev/null > > +++ media_tree/Documentation/devicetree/bindings/media/rcar_imr.txt > > @@ -0,0 +1,27 @@ > > +Renesas R-Car Image Renderer (Distortion Correction Eng

Re: [PATCH] libdvbv5: T2 delivery descriptor: fix wrong size of bandwidth field

2017-05-02 Thread Mauro Carvalho Chehab
Em Tue, 2 May 2017 22:30:29 +0200 Gregor Jasny escreveu: > Hello Clemens, > > On 4/1/17 5:50 PM, Clemens Ladisch wrote: > > ETSI EN 300 468 V1.11.1 § 6.4.4.2 defines the bandwith field as having > > four bits. > > I just used your patch and another to hopefully fix > https://bugs.debian.org/c

Re: [PATCH 2/6] rc-core: cleanup rc_register_device

2017-05-02 Thread Sean Young
On Tue, May 02, 2017 at 08:53:02PM +0200, David Härdeman wrote: > On Mon, May 01, 2017 at 07:47:25PM +0200, David Härdeman wrote: > >On Mon, May 01, 2017 at 05:49:53PM +0100, Sean Young wrote: > >>On Thu, Apr 27, 2017 at 10:34:03PM +0200, David Härdeman wrote: > >>> The device core infrastructure i

Re: [PATCH] libdvbv5: T2 delivery descriptor: fix wrong size of bandwidth field

2017-05-02 Thread Gregor Jasny
Hello Clemens, On 4/1/17 5:50 PM, Clemens Ladisch wrote: > ETSI EN 300 468 V1.11.1 § 6.4.4.2 defines the bandwith field as having > four bits. I just used your patch and another to hopefully fix https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=859008 But I'm a little bit hesitant to merge it to

Re: [PATCH] v4l2-async: add v4l2_async_match()

2017-05-02 Thread Sakari Ailus
Hi Niklas, Niklas Söderlund wrote: For drivers registering notifiers with more then one subdevices it's difficult to know exactly which one is being bound in each call to the notifiers bound callback. Comparing OF nodes became harder after the introduction of fwnode where the two following compa

Re: [PATCH v8 01/10] firmware: qcom_scm: Fix to allow COMPILE_TEST-ing

2017-05-02 Thread Bjorn Andersson
On Fri 28 Apr 02:13 PDT 2017, Stanimir Varbanov wrote: > Unfortunatly previous attempt to allow consumer drivers to > use COMPILE_TEST option in Kconfig is not enough, because in the > past the consumer drivers used 'depends on' Kconfig option but > now they are using 'select' Kconfig option which

[PATCH v2 2/2] em28xx: add support for new of Terratec H6

2017-05-02 Thread Mauro Carvalho Chehab
There's a new version of Terratec H6 with uses USB ID 0ccd:10b2. This version is similar to the old one (with is supported via the HTC entry), except that this one has the eeprom on the second bus. On this board, one side of this board is labeled with: dvbc v2.0 The other side with:

[PATCH v2 1/2] em28xx: Ignore errors while reading from eeprom

2017-05-02 Thread Mauro Carvalho Chehab
On some newer devices (newer Terratec H6 rev. 2), reading from eeprom fails. Ignore such errors, as we don't really need it to succeed. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/usb/em28xx/em28xx-i2c.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/media/usb/em28xx/em

Re: [PATCH v8 05/10] media: venus: adding core part and helper functions

2017-05-02 Thread Bjorn Andersson
On Tue 02 May 01:52 PDT 2017, Stanimir Varbanov wrote: > Hei Sakari, > > On 04/30/2017 01:21 AM, Sakari Ailus wrote: > > Hi, Stan!! > > > > On Fri, Apr 28, 2017 at 12:13:52PM +0300, Stanimir Varbanov wrote: > > ... > >> +int helper_get_bufreq(struct venus_inst *inst, u32 type, > >> +

Re: [PATCH 2/6] rc-core: cleanup rc_register_device

2017-05-02 Thread David Härdeman
On Mon, May 01, 2017 at 07:47:25PM +0200, David Härdeman wrote: >On Mon, May 01, 2017 at 05:49:53PM +0100, Sean Young wrote: >>On Thu, Apr 27, 2017 at 10:34:03PM +0200, David Härdeman wrote: >>> The device core infrastructure is based on the presumption that >>> once a driver calls device_add(), it

Re: [PATCH 15/16] lirc_dev: remove name from struct lirc_driver

2017-05-02 Thread David Härdeman
On Tue, May 02, 2017 at 06:04:18PM +0100, Sean Young wrote: >On Mon, May 01, 2017 at 06:04:52PM +0200, David Härdeman wrote: >> The name is only used for a few debug messages and the name of the parent >> device as well as the name of the lirc device (e.g. "lirc0") are sufficient >> anyway. >>... >

Re: [PATCH 2/2] [media] platform: add video-multiplexer subdevice driver

2017-05-02 Thread Peter Rosin
On 2017-05-02 17:21, Philipp Zabel wrote: > On Sat, 2017-04-29 at 23:42 +0200, Peter Rosin wrote: >> On 2017-04-29 23:29, Peter Rosin wrote: >>> On 2017-04-28 16:13, Philipp Zabel wrote: This driver can handle SoC internal and external video bus multiplexers, controlled by mux controllers

Re: [PATCH 15/16] lirc_dev: remove name from struct lirc_driver

2017-05-02 Thread Sean Young
On Mon, May 01, 2017 at 06:04:52PM +0200, David Härdeman wrote: > The name is only used for a few debug messages and the name of the parent > device as well as the name of the lirc device (e.g. "lirc0") are sufficient > anyway. > > Signed-off-by: David Härdeman > --- > drivers/media/rc/ir-lirc-c

Re: [PATCH 1/2] em28xx: allow setting the eeprom bus at cards struct

2017-05-02 Thread Frank Schäfer
Am 01.05.2017 um 19:54 schrieb Mauro Carvalho Chehab: > Hi Frank, > > Em Mon, 1 May 2017 16:11:51 +0200 > Frank Schäfer escreveu: > >> Am 01.05.2017 um 13:38 schrieb Mauro Carvalho Chehab: >>> Right now, all devices use bus 0 for eeprom. However, newer >>> versions of Terratec H6 use a different

[PATCH] v4l2-async: add v4l2_async_match()

2017-05-02 Thread Niklas Söderlund
For drivers registering notifiers with more then one subdevices it's difficult to know exactly which one is being bound in each call to the notifiers bound callback. Comparing OF nodes became harder after the introduction of fwnode where the two following comparisons are not equal but where so prev

Re: [PATCH v4 2/7] dt-bindings: media: Add MAX2175 binding description

2017-05-02 Thread Geert Uytterhoeven
Hi Ramesh, On Tue, May 2, 2017 at 3:26 PM, Ramesh Shanmugasundaram wrote: > --- a/Documentation/devicetree/bindings/property-units.txt > +++ b/Documentation/devicetree/bindings/property-units.txt > @@ -28,6 +28,7 @@ Electricity > -ohms : Ohms > -micro-ohms: micro Ohms > -microvolt

Re: [PATCH 2/2] [media] platform: add video-multiplexer subdevice driver

2017-05-02 Thread Philipp Zabel
On Sat, 2017-04-29 at 23:42 +0200, Peter Rosin wrote: > On 2017-04-29 23:29, Peter Rosin wrote: > > On 2017-04-28 16:13, Philipp Zabel wrote: > >> This driver can handle SoC internal and external video bus multiplexers, > >> controlled by mux controllers provided by the mux controller framework, >

[PATCH v2 2/2] [media] platform: add video-multiplexer subdevice driver

2017-05-02 Thread Philipp Zabel
This driver can handle SoC internal and external video bus multiplexers, controlled by mux controllers provided by the mux controller framework, such as MMIO register bitfields or GPIOs. The subdevice passes through the mbus configuration of the active input to the output side. Signed-off-by: Sasc

[PATCH v2 1/2] [media] dt-bindings: Add bindings for video-multiplexer device

2017-05-02 Thread Philipp Zabel
Add bindings documentation for the video multiplexer device. Signed-off-by: Sascha Hauer Signed-off-by: Philipp Zabel Signed-off-by: Steve Longerbeam Acked-by: Peter Rosin --- No changes since v1 [1]. This was previously sent as part of Steve's i.MX media series [2]. [1] https://patchwork.ke

Re: [RFC] [PATCH 0/4] [media] pxa_camera: Fixing bugs and missing colorformats

2017-05-02 Thread Robert Jarzmik
Petr Cvek writes: > Dne 1.5.2017 v 06:20 Petr Cvek napsal(a): >> This patchset is just a grouping of a few bugfixes I've found during >> the ov9640 sensor support re-adding. > > P.S. I've manually calculated every format variant for the image size > calculation functions, but still these functio

Re: [PATCH 4/4] [media] pxa_camera: Fix a call with an uninitialized device pointer

2017-05-02 Thread Robert Jarzmik
Petr Cvek writes: > In 'commit 295ab497d6357 ("[media] media: platform: pxa_camera: make > printk consistent")' a pointer to the device structure in > mclk_get_divisor() was changed to pcdev_to_dev(pcdev). The pointer used > by pcdev_to_dev() is still uninitialized during the call to > mclk_get_d

Re: [PATCH 3/4] [media] pxa_camera: Add (un)subscribe_event ioctl

2017-05-02 Thread Robert Jarzmik
Petr Cvek writes: > The v4l2-compliance complains about nonexistent vidioc_subscribe_event > and vidioc_unsubscribe_event calls. Add them to fix the complaints. > > Signed-off-by: Petr Cvek Don't know on that one, let others on the mailing list comment, I'm not familiar with the event interface

Re: [PATCH 2/4] [media] pxa_camera: Fix incorrect test in the image size generation

2017-05-02 Thread Robert Jarzmik
Petr Cvek writes: > During the transfer from the soc_camera a test in pxa_mbus_image_size() > got removed. Without it any PXA_MBUS_LAYOUT_PACKED format causes either > the return of a wrong value (PXA_MBUS_PACKING_2X8_PADHI doubles > the correct value) or EINVAL (PXA_MBUS_PACKING_NONE and > PXA_M

Re: [PATCH 1/4] [media] pxa_camera: Add remaining Bayer 8 formats

2017-05-02 Thread Robert Jarzmik
Petr Cvek writes: > This patch adds Bayer 8 GBRG and RGGB support and move GRBG definition > close to BGGR (so all Bayer 8 variants are together). No other changes are > needed as the driver handles them as RAW data stream. > > The RGGB variant was tested in a modified OV9640 driver. > > Signed-o

[PATCH v4 5/7] doc_rst: media: New SDR formats PC16, PC18 & PC20

2017-05-02 Thread Ramesh Shanmugasundaram
This patch adds documentation for the three new SDR formats V4L2_SDR_FMT_PCU16BE V4L2_SDR_FMT_PCU18BE V4L2_SDR_FMT_PCU20BE Signed-off-by: Ramesh Shanmugasundaram --- .../media/uapi/v4l/pixfmt-sdr-pcu16be.rst | 55 ++ .../media/uapi/v4l/pixfmt-sdr-pcu18be.rst

[PATCH v4 7/7] media: platform: rcar_drif: Add DRIF support

2017-05-02 Thread Ramesh Shanmugasundaram
This patch adds Digital Radio Interface (DRIF) support to R-Car Gen3 SoCs. The driver exposes each instance of DRIF as a V4L2 SDR device. A DRIF device represents a channel and each channel can have one or two sub-channels respectively depending on the target board. DRIF supports only Rx functiona

[PATCH v4 6/7] dt-bindings: media: Add Renesas R-Car DRIF binding

2017-05-02 Thread Ramesh Shanmugasundaram
Add binding documentation for Renesas R-Car Digital Radio Interface (DRIF) controller. Signed-off-by: Ramesh Shanmugasundaram --- v4: - port property description modified as suggested by Laurent. - removed renesas vendor tag from sync-active end-point property (Rob) --- .../devicetree/bindings

[PATCH v4 4/7] media: Add new SDR formats PC16, PC18 & PC20

2017-05-02 Thread Ramesh Shanmugasundaram
This patch adds support for the three new SDR formats. These formats were prefixed with "planar" indicating I & Q data are not interleaved as in other formats. Here, I & Q data constitutes the top half and bottom half of the received buffer respectively. V4L2_SDR_FMT_PCU16BE - 14-bit complex (I &

[PATCH v4 1/7] media: v4l2-ctrls: Reserve controls for MAX217X

2017-05-02 Thread Ramesh Shanmugasundaram
Reserve controls for MAX217X RF to Bits tuner family. These hybrid radio receiver chips are highly programmable and hence reserving 32 controls. Signed-off-by: Ramesh Shanmugasundaram Acked-by: Laurent Pinchart --- include/uapi/linux/v4l2-controls.h | 5 + 1 file changed, 5 insertions(+) d

[PATCH v4 3/7] media: i2c: max2175: Add MAX2175 support

2017-05-02 Thread Ramesh Shanmugasundaram
This patch adds driver support for the 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 standard and custom controls to configure the device. Signed-off-by: Ramesh Sha

[PATCH v4 2/7] dt-bindings: media: Add MAX2175 binding description

2017-05-02 Thread Ramesh Shanmugasundaram
Add device tree binding documentation for MAX2175 RF to bits tuner device. Signed-off-by: Ramesh Shanmugasundaram Acked-by: Rob Herring --- v4: - port property description modified as suggested by Laurent. - Renamed property name slave to master (Laurent). - Renamed property name am-hiz to am

[PATCH v4 0/7] Add V4L2 SDR (DRIF & MAX2175) driver

2017-05-02 Thread Ramesh Shanmugasundaram
Hi Media, DT maintainers, All, This patch set contains two drivers - Renesas R-Car Digital Radio Interface (DRIF) driver - Maxim's MAX2175 RF to Bits tuner driver These patches were based on top of media-next repo commit:6d95b3f24881c0fd0f345eca959a2a803a040930 These two drivers combined toget

Re: [PATCH v4 23/27] rcar-vin: parse Gen3 OF and setup media graph

2017-05-02 Thread Kieran Bingham
Hi Niklas Just a quick comment here, as I've come across it while looking at the fw_node topic. On 27/04/17 23:41, Niklas Söderlund wrote: > Parse the VIN Gen3 OF graph and register all CSI-2 devices in the VIN > group common media device. Once a CSI-2 subdevice is added to the common > media dev

Re: [PATCH 3/3] [media] intel-ipu3: cio2: Add new MIPI-CSI2 driver

2017-05-02 Thread Sakari Ailus
Hi Yong, Thanks for the patches! Some comments below. On Sat, Apr 29, 2017 at 06:34:36PM -0500, Yong Zhi wrote: > This patch adds CIO2 CSI-2 device driver for > Intel's IPU3 camera sub-system support. > > The V4L2 fwnode matching depends on the following work: > > https://git.linuxtv.org/sailus

[GIT PULL FOR v4.12] RC bugfixes

2017-05-02 Thread Sean Young
Hi Mauro, Please pull these two fixes. One is for a repeat regression which was introduced in v4.11, another is for the lirc_buffer being freed on lirc_unregister even though a user can still have an fd open (this fix depends on "74c839b [media] lirc: use refcounting for lirc devices", so it's not

[RFC 2/3] dt: bindings: Add lens-focus binding for image sensors

2017-05-02 Thread Sakari Ailus
The lens-focus property contains a phandle to the lens voice coil driver that is associated to the sensor; typically both are contained in the same camera module. Signed-off-by: Sakari Ailus --- Documentation/devicetree/bindings/media/video-interfaces.txt | 3 +++ 1 file changed, 3 insertions(+)

[RFC 0/3] Document bindings for camera modules and associated flash devices

2017-05-02 Thread Sakari Ailus
Hi folks, This RFC patchset documents properties commonly required by camera modules and associated camera flash devices. The camera module is essentially a package consisting of an image sensor, a lens, possibly a voice coil to move the lens and a number of other things that at least the drivers

[RFC 3/3] dt: bindings: Add a binding for referencing EEPROM from camera sensors

2017-05-02 Thread Sakari Ailus
Many camera sensor devices contain EEPROM chips that describe the properties of a given unit --- the data is specific to a given unit can thus is not stored e.g. in user space or the driver. Some sensors embed the EEPROM chip and it can be accessed through the sensor's I²C interface. This property

[RFC 1/3] dt: bindings: Add a binding for flash devices associated to a sensor

2017-05-02 Thread Sakari Ailus
Camera flash drivers (and LEDs) are separate from the sensor devices in DT. In order to make an association between the two, provide the association information to the software. Signed-off-by: Sakari Ailus --- Documentation/devicetree/bindings/media/video-interfaces.txt | 11 +++ 1 file

Re: [PATCH v8 05/10] media: venus: adding core part and helper functions

2017-05-02 Thread Stanimir Varbanov
Hi, On 04/29/2017 11:22 PM, Bjorn Andersson wrote: > On Fri 28 Apr 15:02 PDT 2017, Jordan Crouse wrote: > >> On Fri, Apr 28, 2017 at 12:13:52PM +0300, Stanimir Varbanov wrote: >>> +int venus_boot(struct device *parent, struct device *fw_dev) >>> +{ >>> + const struct firmware *mdt; >>> + phys

Re: [PATCH v8 05/10] media: venus: adding core part and helper functions

2017-05-02 Thread Stanimir Varbanov
Hei Sakari, On 04/30/2017 01:21 AM, Sakari Ailus wrote: > Hi, Stan!! > > On Fri, Apr 28, 2017 at 12:13:52PM +0300, Stanimir Varbanov wrote: > ... >> +int helper_get_bufreq(struct venus_inst *inst, u32 type, >> + struct hfi_buffer_requirements *req) >> +{ >> +u32 ptype = HFI_P