Re: [PATCH v2] [media] vb2: core: Lower the log level of debug outputs

2017-06-07 Thread Joe Perches
On Thu, 2017-06-08 at 14:24 +0900, Tomasz Figa wrote: > On Thu, Jun 8, 2017 at 2:16 PM, Joe Perches wrote: [] > > If there automated systems that rely on specific levels, then > > changing the levels of individual messages could also cause > > those automated systems to fail. >

Re: [PATCH v2] [media] vb2: core: Lower the log level of debug outputs

2017-06-07 Thread Tomasz Figa
On Thu, Jun 8, 2017 at 2:16 PM, Joe Perches wrote: > On Thu, 2017-06-08 at 13:39 +0900, Tomasz Figa wrote: >> On Thu, Jun 8, 2017 at 12:24 PM, Hirokazu Honda wrote: >> > Hi, >> > >> > I completely understand bitmask method now. >> > I agree to the idea, but

Re: [PATCH v2] [media] vb2: core: Lower the log level of debug outputs

2017-06-07 Thread Joe Perches
On Thu, 2017-06-08 at 13:39 +0900, Tomasz Figa wrote: > On Thu, Jun 8, 2017 at 12:24 PM, Hirokazu Honda wrote: > > Hi, > > > > I completely understand bitmask method now. > > I agree to the idea, but it is necessary to change the specification of > > a debug parameter. > >

Re: [PATCH v2] [media] vb2: core: Lower the log level of debug outputs

2017-06-07 Thread Tomasz Figa
On Thu, Jun 8, 2017 at 12:24 PM, Hirokazu Honda wrote: > Hi, > > I completely understand bitmask method now. > I agree to the idea, but it is necessary to change the specification of > a debug parameter. > (We probably need to change a document about that?) > For example,

cron job: media_tree daily build: ERRORS

2017-06-07 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: Thu Jun 8 05:00:18 CEST 2017 media-tree git hash:47b586f66a9e78c91586b9c363603a52c75840d7 media_build

Re: [PATCH v2] [media] vb2: core: Lower the log level of debug outputs

2017-06-07 Thread Hirokazu Honda
Hi, I completely understand bitmask method now. I agree to the idea, but it is necessary to change the specification of a debug parameter. (We probably need to change a document about that?) For example, there is maybe a user who set a debug parameter 3. The user assume that logs whose levels

Re: [PATCH] [media] mtk-mdp: Fix g_/s_selection capture/compose logic

2017-06-07 Thread Minghsiu Tsai
Hi, Stanimir Varbanov, I have upload patch v2. https://patchwork.kernel.org/patch/9723463/ Could you review it? Thanks On Thu, 2017-04-27 at 17:42 +0300, Stanimir Varbanov wrote: > Hi, > > On 04/13/2017 07:18 AM, Minghsiu Tsai wrote: > > From: Daniel Kurtz > > > >

Re: [PATCH 03/12] intel-ipu3: Add DMA API implementation

2017-06-07 Thread Tomasz Figa
Hi Alan, On Thu, Jun 8, 2017 at 2:45 AM, Alan Cox wrote: >> > + struct ipu3_mmu *mmu = to_ipu3_mmu(dev); >> > + dma_addr_t daddr = iommu_iova_to_phys(mmu->domain, dma_handle); >> > + >> > + clflush_cache_range(phys_to_virt(daddr), size); >> >> You

Re: Unknown symbol put_vaddr_frames when using media_build

2017-06-07 Thread Mauro Carvalho Chehab
Em Wed, 7 Jun 2017 22:17:50 +0200 Matthias Schwarzott escreveu: > Am 07.06.2017 um 20:23 schrieb Mauro Carvalho Chehab: > > Em Tue, 9 May 2017 06:56:25 +0200 > > Matthias Schwarzott escreveu: > > > >> Hi! > >> > >> Whenever I compile the media drivers using

Re: [PATCH 01/11] [media] dvb-core/dvb_ca_en50221.c: Rename STATUSREG_??

2017-06-07 Thread Mauro Carvalho Chehab
Em Wed, 7 Jun 2017 21:37:02 +0200 "Jasmin J." escreveu: > Hello Mauro! > > > If you want it applied, this is needed anyway, as the patch doesn't apply > > cleanly: > Because you didn't apply the first series! > In the first series >[PATCH 0/7] Add block read/write to

Re: [PATCH 0/7] Add block read/write to en50221 CAM functions

2017-06-07 Thread Mauro Carvalho Chehab
Em Wed, 7 Jun 2017 21:18:14 +0200 "Jasmin J." escreveu: > Hello Mauro! > > THX for looking into this! > > > Hmm... from what I understood, the original author for those patches > > is Ralph, right? > Yes. I re-formatted them into smaller pieces to be easier reviewed. > The

Re: [PATCH 04/12] intel-ipu3: Add user space ABI definitions

2017-06-07 Thread Sakari Ailus
Hi Hans, On Tue, Jun 06, 2017 at 10:28:26AM +0200, Hans Verkuil wrote: > On 05/06/17 22:39, Yong Zhi wrote: > > Commit message missing. > > > Signed-off-by: Yong Zhi > > --- > > include/uapi/linux/intel-ipu3.h | 2182 > > +++ > > 1 file

Re: [PATCH] [media] sir_ir: annotate hardware config module parameters

2017-06-07 Thread kbuild test robot
Hi Sean, [auto build test ERROR on linuxtv-media/master] [also build test ERROR on v4.12-rc4 next-20170607] [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/Sean-Young/sir_ir-annotate-hardware

Re: [PATCH 02/12] intel-ipu3: mmu: implement driver

2017-06-07 Thread Sakari Ailus
Hi Tomasz, On Tue, Jun 06, 2017 at 07:13:19PM +0900, Tomasz Figa wrote: > Hi Yong, Tuukka, > > +CC IOMMU ML and Joerg. (Technically you should resend this patch > including them.) Thanks! > > On Tue, Jun 6, 2017 at 5:39 AM, Yong Zhi wrote: ... > > diff --git

Re: [PATCH v6 1/2] dt-bindings: media: stm32 cec driver

2017-06-07 Thread Rob Herring
On Wed, May 31, 2017 at 02:13:55PM +0200, Benjamin Gaignard wrote: > Add bindings documentation for stm32 CEC driver. > > Signed-off-by: Benjamin Gaignard > --- > .../devicetree/bindings/media/st,stm32-cec.txt| 19 > +++ > 1 file changed,

Re: [RFC v4 00/18] vb2: Handle user cache hints, allow drivers to choose cache coherency

2017-06-07 Thread Sakari Ailus
Hi Mauro, Mauro Carvalho Chehab wrote: Hi Sakari, Em Mon, 8 May 2017 18:03:12 +0300 Sakari Ailus escreveu: Hello, This is a rebased and partially reworked version of the vb2 cache hints support patch series posted by first myself, then Laurent and then myself

Re: Unknown symbol put_vaddr_frames when using media_build

2017-06-07 Thread Matthias Schwarzott
Am 07.06.2017 um 20:23 schrieb Mauro Carvalho Chehab: > Em Tue, 9 May 2017 06:56:25 +0200 > Matthias Schwarzott escreveu: > >> Hi! >> >> Whenever I compile the media drivers using media_build against a recent >> kernel, I get this message when loading them: >> >> [5.848537]

Re: [PATCH 01/11] [media] dvb-core/dvb_ca_en50221.c: Rename STATUSREG_??

2017-06-07 Thread Jasmin J.
Hello Mauro! > If you want it applied, this is needed anyway, as the patch doesn't apply > cleanly: Because you didn't apply the first series! In the first series [PATCH 0/7] Add block read/write to en50221 CAM functions I wrote: There is another patch series coming soon "Fix coding style in

Re: [PATCH 0/7] Add block read/write to en50221 CAM functions

2017-06-07 Thread Jasmin J.
Hello Mauro! THX for looking into this! > Hmm... from what I understood, the original author for those patches > is Ralph, right? Yes. I re-formatted them into smaller pieces to be easier reviewed. The goal of this series is to make the Kernel version equal to the official DD version. But I

Re: [PATCH v8 00/34] i.MX Media Driver

2017-06-07 Thread Steve Longerbeam
On 06/07/2017 12:02 PM, Hans Verkuil wrote: We're still waiting for an Ack for patch 02/34, right? Hi Hans, Yes still waiting for an ack for the imx-media bindings. Other than that everything is ready AFAICT. Agreed. Steve Regards, Hans On 07/06/17 20:33, Steve Longerbeam

Re: [PATCH v8 00/34] i.MX Media Driver

2017-06-07 Thread Hans Verkuil
We're still waiting for an Ack for patch 02/34, right? Other than that everything is ready AFAICT. Regards, Hans On 07/06/17 20:33, Steve Longerbeam wrote: > In version 8: > > - Switched to v4l2_fwnode APIs. > > - Always pass a valid CSI id to ipu_set_ic_src_mux() in imx-ic-prp, even

Re: [PATCH 9/9] ARM: dts: exynos: add needs-hpd to for Odroid-U3

2017-06-07 Thread Hans Verkuil
On 07/06/17 20:36, Krzysztof Kozlowski wrote: > On Wed, Jun 07, 2017 at 04:46:16PM +0200, Hans Verkuil wrote: >> From: Hans Verkuil >> >> The Odroid-U3 board has an IP4791CZ12 level shifter that is >> disabled if the HPD is low, which means that the CEC pin is >> disabled

[PATCH v8 01/34] dt-bindings: Add bindings for video-multiplexer device

2017-06-07 Thread Steve Longerbeam
From: 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: Sakari

[PATCH v8 03/34] [media] dt/bindings: Add bindings for OV5640

2017-06-07 Thread Steve Longerbeam
Add device tree binding documentation for the OV5640 camera sensor. Signed-off-by: Steve Longerbeam Acked-by: Rob Herring --- .../devicetree/bindings/media/i2c/ov5640.txt | 45 ++ 1 file changed, 45 insertions(+) create

[PATCH v8 02/34] [media] dt-bindings: Add bindings for i.MX media driver

2017-06-07 Thread Steve Longerbeam
Add bindings documentation for the i.MX media driver. Signed-off-by: Steve Longerbeam --- Documentation/devicetree/bindings/media/imx.txt | 47 + 1 file changed, 47 insertions(+) create mode 100644

[PATCH v8 04/34] ARM: dts: imx6qdl: add multiplexer controls

2017-06-07 Thread Steve Longerbeam
From: Philipp Zabel The IOMUXC General Purpose Register space contains various bitfields that control video bus multiplexers. Describe them using a mmio-mux node. The placement of the IPU CSI video mux controls differs between i.MX6D/Q and i.MX6S/DL. Signed-off-by:

[PATCH v8 05/34] ARM: dts: imx6qdl: Add compatible, clocks, irqs to MIPI CSI-2 node

2017-06-07 Thread Steve Longerbeam
Add to the MIPI CSI2 receiver node: compatible strings, interrupt sources, and clocks. Signed-off-by: Steve Longerbeam --- arch/arm/boot/dts/imx6qdl.dtsi | 7 +++ 1 file changed, 7 insertions(+) diff --git a/arch/arm/boot/dts/imx6qdl.dtsi

[PATCH v8 08/34] ARM: dts: imx6qdl-sabrelite: remove erratum ERR006687 workaround

2017-06-07 Thread Steve Longerbeam
There is a pin conflict with GPIO_6. This pin functions as a power input pin to the OV5642 camera sensor, but ENET uses it as the h/w workaround for erratum ERR006687, to wake-up the ARM cores on normal RX and TX packet done events. So we need to remove the h/w workaround to support the OV5642.

[PATCH v8 10/34] ARM: dts: imx6-sabresd: add OV5642 and OV5640 camera sensors

2017-06-07 Thread Steve Longerbeam
Enables the OV5642 parallel-bus sensor, and the OV5640 MIPI CSI-2 sensor. The OV5642 connects to the parallel-bus mux input port on ipu1_csi0_mux. The OV5640 connects to the input port on the MIPI CSI-2 receiver on mipi_csi. Until the OV5652 sensor module compatible with the SabreSD becomes

[PATCH v8 09/34] ARM: dts: imx6-sabrelite: add OV5642 and OV5640 camera sensors

2017-06-07 Thread Steve Longerbeam
Adds the OV5642 parallel-bus sensor, and the OV5640 MIPI CSI-2 sensor. Both hang off the same i2c2 bus, so they require different (and non- default) i2c slave addresses. The OV5642 connects to the parallel-bus mux input port on ipu1_csi0_mux. The OV5640 connects to the input port on the MIPI

[PATCH v8 07/34] ARM: dts: imx6qdl: add capture-subsystem device

2017-06-07 Thread Steve Longerbeam
Signed-off-by: Steve Longerbeam --- arch/arm/boot/dts/imx6dl.dtsi | 5 + arch/arm/boot/dts/imx6q.dtsi | 5 + 2 files changed, 10 insertions(+) diff --git a/arch/arm/boot/dts/imx6dl.dtsi b/arch/arm/boot/dts/imx6dl.dtsi index 4049af7..8475e6c 100644 ---

[PATCH v8 06/34] ARM: dts: imx6qdl: Add video multiplexers, mipi_csi, and their connections

2017-06-07 Thread Steve Longerbeam
From: Philipp Zabel This patch adds the device tree graph connecting the input multiplexers to the IPU CSIs and the MIPI-CSI2 gasket on i.MX6. The MIPI_IPU multiplexers are added as children of the iomuxc-gpr syscon device node. On i.MX6Q/D two two-input multiplexers in

[PATCH v8 15/34] add mux and video interface bridge entity functions

2017-06-07 Thread Steve Longerbeam
From: Philipp Zabel Add two new media entity function definitions for video multiplexers and video interface bridges. Signed-off-by: Philipp Zabel - renamed MEDIA_ENT_F_MUX to MEDIA_ENT_F_VID_MUX Signed-off-by: Steve Longerbeam

[PATCH v8 16/34] platform: add video-multiplexer subdevice driver

2017-06-07 Thread Steve Longerbeam
From: 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

[PATCH v8 13/34] ARM: dts: imx6-sabreauto: add pinctrl for gpt input capture

2017-06-07 Thread Steve Longerbeam
Add pinctrl groups for both GPT input capture channels. Signed-off-by: Steve Longerbeam --- arch/arm/boot/dts/imx6qdl-sabreauto.dtsi | 12 1 file changed, 12 insertions(+) diff --git a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi

[PATCH v8 11/34] ARM: dts: imx6-sabreauto: create i2cmux for i2c3

2017-06-07 Thread Steve Longerbeam
The sabreauto uses a steering pin to select between the SDA signal on i2c3 bus, and a data-in pin for an SPI NOR chip. Use i2cmux to control this steering pin. Idle state of the i2cmux selects SPI NOR. This is not a classic way to use i2cmux, since one side of the mux selects something other than

[PATCH v8 14/34] ARM: dts: imx6-sabreauto: add the ADV7180 video decoder

2017-06-07 Thread Steve Longerbeam
Enables the ADV7180 decoder sensor. The ADV7180 connects to the parallel-bus mux input on ipu1_csi0_mux. The ADV7180 power pin is via max7310_b port expander. Signed-off-by: Steve Longerbeam - Use IRQ_TYPE_LEVEL_LOW instead of 0x8 for interrupt type for clarity. -

[PATCH v8 12/34] ARM: dts: imx6-sabreauto: add reset-gpios property for max7310_b

2017-06-07 Thread Steve Longerbeam
The reset pin to the port expander chip (MAX7310) is controlled by a gpio, so define a reset-gpios property to control it. There are three MAX7310's on the SabreAuto CPU card (max7310_[abc]), but all use the same pin for their reset. Since all can't acquire the same pin, assign it to max7310_b,

[PATCH v8 17/34] [media] add Omnivision OV5640 sensor driver

2017-06-07 Thread Steve Longerbeam
This driver is based on ov5640_mipi.c from Freescale imx_3.10.17_1.0.0_beta branch, modified heavily to bring forward to latest interfaces and code cleanup. Signed-off-by: Steve Longerbeam Acked-by: Hans Verkuil --- drivers/media/i2c/Kconfig

[PATCH v8 19/34] media: Add i.MX media core driver

2017-06-07 Thread Steve Longerbeam
Add the core media driver for i.MX SOC. Signed-off-by: Steve Longerbeam Switch from the v4l2_of_ APIs to the v4l2_fwnode_ APIs. Signed-off-by: Philipp Zabel Add the bayer formats to imx-media's list of supported pixel and bus formats.

[PATCH v8 23/34] media: imx: Add VDIC subdev driver

2017-06-07 Thread Steve Longerbeam
This is a media entity subdevice driver for the i.MX Video De-Interlacing or Combining Block. So far this entity does not implement the Combining function but only motion compensated deinterlacing. Video frames are received from the CSI and are routed to the IC PRPVF entity. Signed-off-by: Steve

[PATCH v8 18/34] media: Add userspace header file for i.MX

2017-06-07 Thread Steve Longerbeam
This adds a header file for use by userspace programs wanting to interact with the i.MX media driver. It defines custom events and v4l2 controls for the i.MX v4l2 subdevices. Signed-off-by: Steve Longerbeam --- include/linux/imx-media.h | 29

[PATCH v8 21/34] media: imx: Add Capture Device Interface

2017-06-07 Thread Steve Longerbeam
This is the capture device interface driver that provides the v4l2 user interface. Frames can be received from various sources: - directly from CSI for capturing unconverted images directly from camera sensors. - from the IC pre-process encode task. - from the IC pre-process viewfinder task.

[PATCH v8 20/34] media: imx: Add a TODO file

2017-06-07 Thread Steve Longerbeam
Add a TODO file. Signed-off-by: Steve Longerbeam --- drivers/staging/media/imx/TODO | 23 +++ 1 file changed, 23 insertions(+) create mode 100644 drivers/staging/media/imx/TODO diff --git a/drivers/staging/media/imx/TODO

[PATCH v8 22/34] media: imx: Add CSI subdev driver

2017-06-07 Thread Steve Longerbeam
This is a media entity subdevice for the i.MX Camera Sensor Interface module. Signed-off-by: Steve Longerbeam - Added support for negotiation of frame intervals. Signed-off-by: Russell King - Fixed cropping rectangle negotiation at

[PATCH v8 24/34] media: imx: Add IC subdev drivers

2017-06-07 Thread Steve Longerbeam
This is a set of three media entity subdevice drivers for the i.MX Image Converter: - Pre-process Router: Takes input frames from CSI0, CSI1, or VDIC. Two output pads enable either or both of the preprocess tasks below. If the input is from one of the CSIs, both proprocess task links can be

[PATCH v8 27/34] media: imx: csi: increase burst size for YUV formats

2017-06-07 Thread Steve Longerbeam
From: Philipp Zabel The IDMAC supports burst sizes of up to 32 pixels for interleaved YUV formats and up to 64 pixels for planar YUV formats. Signed-off-by: Philipp Zabel --- drivers/staging/media/imx/imx-media-csi.c | 17 + 1

[PATCH v8 26/34] media: imx: csi: add support for bayer formats

2017-06-07 Thread Steve Longerbeam
From: Russell King Bayer formats must be treated as generic data and passthrough mode must be used. Add the correct setup for these formats. Signed-off-by: Russell King - added check to csi_link_validate() to verify that destination is

[PATCH v8 31/34] media: imx: capture: add frame sizes/interval enumeration

2017-06-07 Thread Steve Longerbeam
From: Russell King Add support for enumerating frame sizes and frame intervals from the first subdev via the V4L2 interfaces. Signed-off-by: Russell King Signed-off-by: Steve Longerbeam ---

[PATCH v8 25/34] media: imx: Add MIPI CSI-2 Receiver subdev driver

2017-06-07 Thread Steve Longerbeam
Adds MIPI CSI-2 Receiver subdev driver. This subdev is required for sensors with a MIPI CSI2 interface. Signed-off-by: Steve Longerbeam - Switch from the v4l2_of_ APIs to the v4l2_fwnode_ APIs. Signed-off-by: Philipp Zabel - Add the

[PATCH v8 28/34] media: imx: csi: add frame skipping support

2017-06-07 Thread Steve Longerbeam
From: Philipp Zabel The CSI can skip any out of up to 6 input frames, allowing to reduce the frame rate at the output pads by small fractions. Signed-off-by: Philipp Zabel Signed-off-by: Steve Longerbeam

[PATCH v8 32/34] media: imx: set and propagate default field, colorimetry

2017-06-07 Thread Steve Longerbeam
This patch adds a call to imx_media_fill_default_mbus_fields() in the *_try_fmt() functions at the sink pads, to set empty field order and colorimetry parameters. If the field order is set to ANY, choose the currently set field order at the sink pad. If the colorspace is set to DEFAULT, choose

[PATCH v8 34/34] ARM: imx_v6_v7_defconfig: Enable staging video4linux drivers

2017-06-07 Thread Steve Longerbeam
Enable i.MX v4l2 media staging driver. For video capture on i.MX, the video multiplexer subdev is required. On the SabreAuto, the ADV7180 video decoder is required along with i2c-mux-gpio. The Sabrelite and SabreSD require the OV5640 and the SabreLite requires PWM clocks for the OV5640. Increase

Re: [PATCH 9/9] ARM: dts: exynos: add needs-hpd to for Odroid-U3

2017-06-07 Thread Krzysztof Kozlowski
On Wed, Jun 07, 2017 at 04:46:16PM +0200, Hans Verkuil wrote: > From: Hans Verkuil > > The Odroid-U3 board has an IP4791CZ12 level shifter that is > disabled if the HPD is low, which means that the CEC pin is > disabled as well. > > Signed-off-by: Hans Verkuil

[PATCH v8 29/34] media: imx: csi: add sink selection rectangles

2017-06-07 Thread Steve Longerbeam
From: Philipp Zabel Move the crop rectangle to the sink pad and add a sink compose rectangle to configure scaling. Also propagate rectangles from sink pad to crop rectangle, to compose rectangle, and to the source pads both in ACTIVE and TRY variants of set_fmt/selection,

[PATCH v8 30/34] media: imx: csi: add frame size/interval enumeration

2017-06-07 Thread Steve Longerbeam
From: Russell King Add frame size and frame interval enumeration to CSI. CSI can downscale the image independently horizontally and vertically by a factor of two, which enumerates to four different frame sizes at the output pads. The input pad supports a range of

[PATCH v8 33/34] media: imx: Drop warning upon multiple S_STREAM disable calls

2017-06-07 Thread Steve Longerbeam
From: Marek Vasut Calling S_STREAM OFF multiple times on a video device is valid, although dubious, practice. Instead of warning about it and setting stream count lower than zero, just ignore the subsequent S_STREAM calls and correct the stream count to zero. Signed-off-by: Marek

[PATCH v8 00/34] i.MX Media Driver

2017-06-07 Thread Steve Longerbeam
In version 8: - Switched to v4l2_fwnode APIs. - Always pass a valid CSI id to ipu_set_ic_src_mux() in imx-ic-prp, even if the IC is receiving from the VDIC. The reason is due to a bug in the i.MX6 reference manual: from experiment it is determined that the CSI id select bit in IPU_CONF

Re: Unknown symbol put_vaddr_frames when using media_build

2017-06-07 Thread Mauro Carvalho Chehab
Em Tue, 9 May 2017 06:56:25 +0200 Matthias Schwarzott escreveu: > Hi! > > Whenever I compile the media drivers using media_build against a recent > kernel, I get this message when loading them: > > [5.848537] media: Linux media interface: v0.10 > [5.881440] Linux video

Re: [PATCH] [media] sir_ir: annotate hardware config module parameters

2017-06-07 Thread Sean Young
On Wed, Jun 07, 2017 at 09:38:57AM -0300, Mauro Carvalho Chehab wrote: > Em Tue, 6 Jun 2017 16:18:19 +0100 > Sean Young escreveu: > > > This module was merged after commit 5a8fc6a3cebb ("Annotate hardware > > config module parameters in drivers/media/"), so add add the missing >

Re: [PATCH v2 2/3] [media] doc-rst: add IPU3 raw10 bayer pixel format definitions

2017-06-07 Thread Alan Cox
> + > +10-bit Bayer formats > + > +Description > +=== > + > +These four pixel formats are used by Intel IPU3 driver, Are the same formats present in IPUv2, will they ever be present in other hardware. If so (and I think it is so...) then it's not a good idea to encode ipu3 in the name.

Re: [PATCH 03/12] intel-ipu3: Add DMA API implementation

2017-06-07 Thread Alan Cox
> > + struct ipu3_mmu *mmu = to_ipu3_mmu(dev); > > + dma_addr_t daddr = iommu_iova_to_phys(mmu->domain, dma_handle); > > + > > + clflush_cache_range(phys_to_virt(daddr), size); > > You might need to consider another IOMMU on the way here. Generally, > given that daddr is your

[PATCH for 4.12] cec: race fix: don't return -ENONET in cec_receive()

2017-06-07 Thread Hans Verkuil
When calling CEC_RECEIVE do not check if the adapter is configured. Typically CEC_RECEIVE is called after a select() and if that indicates that there are messages in the receive queue, then you should always be able to dequeue a message. The race condition here is that a message has been received

Re: [RFC v4 00/18] vb2: Handle user cache hints, allow drivers to choose cache coherency

2017-06-07 Thread Mauro Carvalho Chehab
Hi Sakari, Em Mon, 8 May 2017 18:03:12 +0300 Sakari Ailus escreveu: > Hello, > > This is a rebased and partially reworked version of the vb2 cache hints > support patch series posted by first myself, then Laurent and then myself > again. > > I'm still posting

Re: [PATCH v2 1/2] v4l: vsp1: Add support for colorkey alpha blending

2017-06-07 Thread Mauro Carvalho Chehab
Em Sun, 7 May 2017 13:13:26 +0300 Alexandru Gheorghe escreveu: > The vsp2 hw supports changing of the alpha of pixels that match a color > key, this patch adds support for this feature in order to be used by > the rcar-du driver. > The colorkey is interpreted

Re: [PATCH 01/11] [media] dvb-core/dvb_ca_en50221.c: Rename STATUSREG_??

2017-06-07 Thread Mauro Carvalho Chehab
Em Mon, 8 May 2017 19:28:48 +0200 "Jasmin J." escreveu: > Hello Mauro! > > >> Rename STATUSREG_?? -> STATREG_?? to reduce the line length. > > That sounds a bad idea. We use "stat" on the DVB subsystem as an > > alias for statistics. > At the beginning of the style fixes,

Re: [PATCH 0/7] Add block read/write to en50221 CAM functions

2017-06-07 Thread Mauro Carvalho Chehab
Em Sun, 7 May 2017 22:51:46 +0200 "Jasmin J." escreveu: > From: Jasmin Jessich > > These patch series implement a block read/write interface to the en50221 > CAM control functions. The origin of this patches can be found in the > Digital Devices Git on

Re: [PATCH v2 4/4] [media] davinci: vpif: adaptions for DT support

2017-06-07 Thread Mauro Carvalho Chehab
Em Tue, 6 Jun 2017 16:37:41 -0700 Kevin Hilman escreveu: > The davinci VPIF is a single hardware block, but the existing driver > is broken up into a common library (vpif.c), output (vpif_display.c) and > intput (vpif_capture.c). > > When migrating to DT, to better model

Re: [PATCH 6/9] [media] s5p-jpeg: Add support for resolution change event

2017-06-07 Thread Thierry Escande
Hi Jacek, On 02/06/2017 23:53, Jacek Anaszewski wrote: Hi Thierry, On 06/02/2017 06:02 PM, Thierry Escande wrote: From: henryhsu This patch adds support for resolution change event to notify clients so they can prepare correct output buffer. When resolution change

Problem using libdvbv5

2017-06-07 Thread Russel Winder
I am having some issues with dvb_scan_transponder in the Debian Sid distribution of libdvbv5. I am sure the arguments I am giving it are fine, but a function called within dvb_scan_transponder is causing a SIGSEGV. The Debian Sid package appears to be only of a production version, there is no

[PATCH 9/9] ARM: dts: exynos: add needs-hpd to for Odroid-U3

2017-06-07 Thread Hans Verkuil
From: Hans Verkuil The Odroid-U3 board has an IP4791CZ12 level shifter that is disabled if the HPD is low, which means that the CEC pin is disabled as well. Signed-off-by: Hans Verkuil Cc: Krzysztof Kozlowski Cc: Andrzej Hajda

[PATCH 8/9] s5p_cec: set the CEC_CAP_NEEDS_HPD flag if needed

2017-06-07 Thread Hans Verkuil
From: Hans Verkuil Use the needs-hpd DT property to determine if the CEC_CAP_NEEDS_HPD should be set. Signed-off-by: Hans Verkuil Cc: Krzysztof Kozlowski Cc: Andrzej Hajda Cc: devicet...@vger.kernel.org ---

[PATCH 1/9] cec: add cec_s_phys_addr_from_edid helper function

2017-06-07 Thread Hans Verkuil
From: Hans Verkuil This function simplifies the integration of CEC in DRM drivers. Signed-off-by: Hans Verkuil --- Documentation/media/kapi/cec-core.rst | 8 drivers/media/cec/cec-adap.c | 14 ++

[PATCH 2/9] cec: add cec_phys_addr_invalidate() helper function

2017-06-07 Thread Hans Verkuil
From: Hans Verkuil Simplifies setting the physical address to CEC_PHYS_ADDR_INVALID. Signed-off-by: Hans Verkuil --- include/media/cec.h | 13 + 1 file changed, 13 insertions(+) diff --git a/include/media/cec.h b/include/media/cec.h

[PATCH 0/9] cec improvements

2017-06-07 Thread Hans Verkuil
From: Hans Verkuil This patch series adds several helper functions to ease writing drm or media CEC drivers. It also adds support for the CEC_CAP_NEEDS_HPD capability which is needed for hardware that turns off the CEC support if the hotplug detect signal is low. Usually

[PATCH 4/9] stih-cec/vivid/pulse8/rainshadow: use cec_transmit_attempt_done

2017-06-07 Thread Hans Verkuil
From: Hans Verkuil Use the helper function cec_transmit_attempt_done instead of cec_transmit_done to simplify the code. Signed-off-by: Hans Verkuil Cc: Benjamin Gaignard ---

[PATCH 7/9] dt-bindings: media/s5p-cec.txt: document needs-hpd property

2017-06-07 Thread Hans Verkuil
From: Hans Verkuil Needed for boards that wire the CEC pin in such a way that it is unavailable when the HPD is low. Signed-off-by: Hans Verkuil Cc: Krzysztof Kozlowski Cc: Andrzej Hajda Cc:

[PATCH 3/9] cec: add cec_transmit_attempt_done helper function

2017-06-07 Thread Hans Verkuil
A simpler variant of cec_transmit_done to be used where the HW does just a single attempt at a transmit. So if the status indicates an error, then the corresponding error count will always be 1 and this function figures that out based on the status argument. Signed-off-by: Hans Verkuil

[PATCH 5/9] cec: add CEC_CAP_NEEDS_HPD

2017-06-07 Thread Hans Verkuil
From: Hans Verkuil Add a new capability CEC_CAP_NEEDS_HPD. If this capability is set then the hardware can only use CEC if the HDMI Hotplug Detect pin is high. Such hardware cannot handle the corner case in the CEC specification where it is possible to transmit messages

[PATCH 6/9] cec-ioc-adap-g-caps.rst: document CEC_CAP_NEEDS_HPD

2017-06-07 Thread Hans Verkuil
From: Hans Verkuil Document the new CEC_CAP_NEEDS_HPD capability. Signed-off-by: Hans Verkuil Cc: Krzysztof Kozlowski Cc: Andrzej Hajda --- Documentation/media/uapi/cec/cec-ioc-adap-g-caps.rst | 8

[PATCH] Staging: media: davinci_vpfe Fix resource leaks in error paths.

2017-06-07 Thread Arvind Yadav
Free memory, if ipipe_s_config and ipipe_g_config are not successful. Signed-off-by: Arvind Yadav --- drivers/staging/media/davinci_vpfe/dm365_ipipe.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git

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

2017-06-07 Thread Mauro Carvalho Chehab
Em Wed, 31 May 2017 09:44:53 +0100 Ramesh Shanmugasundaram escreveu: > +++ b/Documentation/media/v4l-drivers/max2175.rst > @@ -0,0 +1,60 @@ > +Maxim Integrated MAX2175 RF to bits tuner driver > + > + > +The

Re: [PATCH] [media] sir_ir: annotate hardware config module parameters

2017-06-07 Thread Mauro Carvalho Chehab
Em Tue, 6 Jun 2017 16:18:19 +0100 Sean Young escreveu: > This module was merged after commit 5a8fc6a3cebb ("Annotate hardware > config module parameters in drivers/media/"), so add add the missing > hardware annotations. This patch seems wrong: drivers/media/rc/sir_ir.c:403:1:

Re: [PATCH 1/9] [media] s5p-jpeg: Reset the Codec before doing a soft reset

2017-06-07 Thread Thierry Escande
Hi Jacek, On 02/06/2017 21:50, Jacek Anaszewski wrote: Hi Thierry, On 06/02/2017 06:02 PM, Thierry Escande wrote: From: Abhilash Kesavan This patch resets the encoding and decoding register bits before doing a soft reset. Signed-off-by: Tony K Nadackal

Re: [PATCH v7 16/34] [media] add Omnivision OV5640 sensor driver

2017-06-07 Thread Sakari Ailus
On Sun, Jun 04, 2017 at 11:00:14AM -0700, Steve Longerbeam wrote: > > > On 06/03/2017 11:02 AM, Steve Longerbeam wrote: > >Hi Sakari, > > > > > >On 05/29/2017 11:56 PM, Sakari Ailus wrote: > >>Hi Steve, > >> > >>On Mon, May 29, 2017 at 02:50:34PM -0700, Steve Longerbeam wrote: > > >

Re: [PATCH v7 16/34] [media] add Omnivision OV5640 sensor driver

2017-06-07 Thread Sakari Ailus
On Sat, Jun 03, 2017 at 09:46:36PM -0700, Steve Longerbeam wrote: > > > On 06/03/2017 02:57 PM, Sakari Ailus wrote: > >On Sat, Jun 03, 2017 at 09:51:39PM +0200, Pavel Machek wrote: > >>Hi! > >> > >>+ /* Auto/manual exposure */ > >>+ ctrls->auto_exp =

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

2017-06-07 Thread Sergei Shtylyov
Hello! On 06/07/2017 10:26 AM, Hans Verkuil wrote: Any progress on this? Yes. :-) There where a bunch of comments, I think I've addresses most of those (except those I didn't quite understand). but I haven't seen anything since. I'm still busy documenting the driver. :-(

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

2017-06-07 Thread Mauro Carvalho Chehab
Em Tue, 9 May 2017 18:36:01 +0300 Stanimir Varbanov escreveu: > This adds Venus driver Makefile and changes v4l2 platform > Makefile/Kconfig in order to enable building of the driver. > > Note that in this initial version the COMPILE_TEST-ing is not > supported

Re: [PATCH v4] v4l: subdev: tolerate null in media_entity_to_v4l2_subdev

2017-06-07 Thread Sakari Ailus
On Wed, Jun 07, 2017 at 10:52:07AM +0100, Kieran Bingham wrote: > From: Kieran Bingham > > Return NULL, if a null entity is parsed for it's v4l2_subdev > > Signed-off-by: Kieran Bingham Reviewed-by: Sakari Ailus

[GIT PULL FOR v4.13] Various fixes

2017-06-07 Thread Hans Verkuil
Hi Mauro, Another set of fixes, including all (I hope) pending atomisp-related patches. Regards, Hans The following changes since commit 6fb05e0dd32e566facb96ea61a48c7488daa5ac3: [media] saa7164: fix double fetch PCIe access condition (2017-06-06 16:55:50 -0300) are available in

Re: [PATCH v2] [media] v4l2: Add support for go2001 PCI codec driver

2017-06-07 Thread Hans Verkuil
Hi Thierry, On 25/05/17 16:42, Thierry Escande wrote: > This patch adds support for the go2001 PCI codec driver. This hardware > is present on ChromeOS based devices like the Acer ChromeBox and Acer/LG > ChromeBase 24 devices. What hardware is this exactly? Who makes it? Is it a mini-pci card?

[PATCH v4] v4l: subdev: tolerate null in media_entity_to_v4l2_subdev

2017-06-07 Thread Kieran Bingham
From: Kieran Bingham Return NULL, if a null entity is parsed for it's v4l2_subdev Signed-off-by: Kieran Bingham --- Not sure if this patch ever made it out of my mailbox: Here's the respin with the parameter

Re: [PATCH v3 5/7] docs-rst: media: Sort topic list alphabetically

2017-06-07 Thread Mauro Carvalho Chehab
Em Tue, 6 Jun 2017 23:57:55 +0300 Sakari Ailus escreveu: > Hi Mauro, > > Mauro Carvalho Chehab wrote: > > Em Mon, 10 Apr 2017 16:02:54 +0300 > > Sakari Ailus escreveu: > > > >> Bring some order by alphabetically ordering the list of

Re: [PATCH 03/12] intel-ipu3: Add DMA API implementation

2017-06-07 Thread Tomasz Figa
Hi Yong, +Robin, Joerg, IOMMU ML Please see my comments inline. On Tue, Jun 6, 2017 at 5:39 AM, Yong Zhi wrote: > IPU3 mmu based DMA mapping driver > > Signed-off-by: Yong Zhi > --- > drivers/media/pci/intel/ipu3/Kconfig | 6 + >

Re: [PATCH] [media] imx: switch from V4L2 OF to V4L2 fwnode API

2017-06-07 Thread Philipp Zabel
On Tue, 2017-06-06 at 18:00 -0700, Steve Longerbeam wrote: > Hi Philipp, > > v4l2_fwnode patch has been merged to mediatree, so I've applied this > to my imx-media-staging-md-v16 branch, thanks for the patch! > > However before I can submit version 8 of the patchset, the video-mux > driver also

[PATCH v8 3/3] [media] platform: add video-multiplexer subdevice driver

2017-06-07 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. Since the mux

[PATCH v8 2/3] [media] add mux and video interface bridge entity functions

2017-06-07 Thread Philipp Zabel
Add two new media entity function definitions for video multiplexers and video interface bridges. Signed-off-by: Philipp Zabel - renamed MEDIA_ENT_F_MUX to MEDIA_ENT_F_VID_MUX Signed-off-by: Steve Longerbeam Acked-by: Sakari Ailus

[PATCH] [media] media-ioc-g-topology.rst: fix typos

2017-06-07 Thread Alexandre Courbot
Fix what seems to be a few typos induced by copy/paste. Signed-off-by: Alexandre Courbot --- Documentation/media/uapi/mediactl/media-ioc-g-topology.rst | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH v8 1/3] dt-bindings: Add bindings for video-multiplexer device

2017-06-07 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: Sakari Ailus Reviewed-by:

RE: Grant

2017-06-07 Thread Mayrhofer Family
-- Good Day, My wife and I have awarded you with a donation of $ 1,000,000.00 Dollars from part of our Jackpot Lottery of 50 Million Dollars, respond with your details for claims. We await your earliest response and God Bless you. Friedrich And Annand Mayrhofer.

  1   2   >