Request your Partnership for a legal business Deal, reply if interested

2017-01-31 Thread Ms Chiang Lai Yuen JP
-- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

cron job: media_tree daily build: ERRORS

2017-01-31 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 Feb 1 05:00:23 CET 2017 media-tree git hash:e7b3a2b22176d01db0c3b31d6389ccf542ba1967 media_build git

Re: [PATCH v3 00/24] i.MX Media Driver

2017-01-31 Thread Steve Longerbeam
On 01/31/2017 04:23 PM, Russell King - ARM Linux wrote: On Tue, Jan 31, 2017 at 03:43:22PM -0800, Steve Longerbeam wrote: On 01/31/2017 03:00 AM, Russell King - ARM Linux wrote: Just like smiapp, the camera sensor block (which is the very far end of the pipeline) is marked with MEDIA_ENT_F_CA

Re: [PATCH v3 17/24] media: imx: Add CSI subdev driver

2017-01-31 Thread Steve Longerbeam
On 01/31/2017 08:48 AM, Philipp Zabel wrote: On Tue, 2017-01-31 at 16:51 +0100, Philipp Zabel wrote: On Fri, 2017-01-06 at 18:11 -0800, Steve Longerbeam wrote: [...] +static int csi_set_fmt(struct v4l2_subdev *sd, + struct v4l2_subdev_pad_config *cfg, +

Re: rtl2832_sdr and /dev/swradio0

2017-01-31 Thread Nicolas Dufresne
Le mardi 31 janvier 2017 à 17:50 +, Russel Winder a écrit : > Hi, > > Is anyone actively working on the rtl2832_sdr driver? > > I am particularly interested in anyone who has code for turning the > byte stream from /dev/swradio0 into an ETI stream. Or failing that > getting enough data about

Re: [PATCH v3 00/24] i.MX Media Driver

2017-01-31 Thread Steve Longerbeam
On 01/31/2017 05:54 AM, Philipp Zabel wrote: Hi Steve, I have just tested the imx-media-staging-md-wip branch on a Nitrogen6X with a tc358743 (BD_HDMI_MIPI HDMI to MIPI CSI-2 receiver board). Some observations: # Link pipeline media-ctl -l "'tc358743 1-000f':0->'imx6-mipi-csi2':0[1]" media-ct

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

2017-01-31 Thread Russell King - ARM Linux
Hi Steve, On Tue, Jan 31, 2017 at 05:02:40PM -0800, Steve Longerbeam wrote: > But this also puts a requirement on MIPI sensors that s_power(ON) > should only place the D_PHY in LP-11, and _not_ start the clock lane. > But perhaps that is correct behavior anyway. If the CSI2 DPHY is held in reset

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

2017-01-31 Thread Steve Longerbeam
On 01/31/2017 01:49 AM, Philipp Zabel wrote: On Tue, 2017-01-31 at 00:01 +, Russell King - ARM Linux wrote: [...] The iMX6 manuals call for a very specific seven sequence of initialisation for CSI2, which begins with: 1. reset the D-PHY. 2. place MIPI sensor in LP-11 state 3. perform D-PH

Re: [PATCH v3 17/24] media: imx: Add CSI subdev driver

2017-01-31 Thread Russell King - ARM Linux
On Tue, Jan 31, 2017 at 04:31:55PM -0800, Steve Longerbeam wrote: > > > On 01/31/2017 03:20 AM, Russell King - ARM Linux wrote: > >On Fri, Jan 06, 2017 at 06:11:35PM -0800, Steve Longerbeam wrote: > >>+static int csi_link_validate(struct v4l2_subdev *sd, > >>+struct media_

Re: [PATCH v3 17/24] media: imx: Add CSI subdev driver

2017-01-31 Thread Steve Longerbeam
On 01/31/2017 03:20 AM, Russell King - ARM Linux wrote: On Fri, Jan 06, 2017 at 06:11:35PM -0800, Steve Longerbeam wrote: +static int csi_link_validate(struct v4l2_subdev *sd, +struct media_link *link, +struct v4l2_subdev_format *source_f

Re: [PATCH v3 00/24] i.MX Media Driver

2017-01-31 Thread Russell King - ARM Linux
On Tue, Jan 31, 2017 at 03:43:22PM -0800, Steve Longerbeam wrote: > > > On 01/31/2017 03:00 AM, Russell King - ARM Linux wrote: > >Just like smiapp, the camera sensor block (which is the very far end > >of the pipeline) is marked with MEDIA_ENT_F_CAM_SENSOR. However, in > >front of that is the b

Re: [PATCH v3 00/24] i.MX Media Driver

2017-01-31 Thread Steve Longerbeam
On 01/31/2017 03:00 AM, Russell King - ARM Linux wrote: On Mon, Jan 30, 2017 at 05:22:01PM -0800, Steve Longerbeam wrote: I'm also having trouble finding a datasheet for it, but from what I've read, it has a MIPI CSI-2 interface. It should work fine as long as it presents a single source pad,

Re: [PATCH v3 20/24] media: imx: Add Camera Interface subdev driver

2017-01-31 Thread Russell King - ARM Linux
On Tue, Jan 31, 2017 at 02:36:53PM -0800, Steve Longerbeam wrote: > On 01/31/2017 02:04 PM, Russell King - ARM Linux wrote: > >I don't want master though, I want v4.10-rc1, and if I ask for that > >it tells me it knows nothing about v4.10-rc1, despite the fact that's > >a tag in the mainline kernel

Re: [PATCH v3 20/24] media: imx: Add Camera Interface subdev driver

2017-01-31 Thread Steve Longerbeam
On 01/31/2017 03:30 PM, Russell King - ARM Linux wrote: On Tue, Jan 31, 2017 at 02:36:53PM -0800, Steve Longerbeam wrote: On 01/31/2017 02:04 PM, Russell King - ARM Linux wrote: I don't want master though, I want v4.10-rc1, and if I ask for that it tells me it knows nothing about v4.10-rc1, d

Re: [PATCH v3 20/24] media: imx: Add Camera Interface subdev driver

2017-01-31 Thread Steve Longerbeam
On 01/31/2017 02:04 PM, Russell King - ARM Linux wrote: On Tue, Jan 31, 2017 at 09:55:29PM +, Ian Arkver wrote: On 31/01/17 20:33, Russell King - ARM Linux wrote: On Tue, Jan 31, 2017 at 10:21:26AM -0800, Steve Longerbeam wrote: On 01/31/2017 05:42 AM, Russell King - ARM Linux wrote: On

Re: [PATCH v3 20/24] media: imx: Add Camera Interface subdev driver

2017-01-31 Thread Ian Arkver
On 31/01/17 22:04, Russell King - ARM Linux wrote: On Tue, Jan 31, 2017 at 09:55:29PM +, Ian Arkver wrote: On 31/01/17 20:33, Russell King - ARM Linux wrote: On Tue, Jan 31, 2017 at 10:21:26AM -0800, Steve Longerbeam wrote: On 01/31/2017 05:42 AM, Russell King - ARM Linux wrote: On Fri, J

Re: [PATCH v3 20/24] media: imx: Add Camera Interface subdev driver

2017-01-31 Thread Russell King - ARM Linux
On Tue, Jan 31, 2017 at 09:55:29PM +, Ian Arkver wrote: > On 31/01/17 20:33, Russell King - ARM Linux wrote: > >On Tue, Jan 31, 2017 at 10:21:26AM -0800, Steve Longerbeam wrote: > >>On 01/31/2017 05:42 AM, Russell King - ARM Linux wrote: > >>>On Fri, Jan 20, 2017 at 03:38:28PM +0100, Hans Verku

Re: [PATCH v3 20/24] media: imx: Add Camera Interface subdev driver

2017-01-31 Thread Ian Arkver
On 31/01/17 20:33, Russell King - ARM Linux wrote: On Tue, Jan 31, 2017 at 10:21:26AM -0800, Steve Longerbeam wrote: On 01/31/2017 05:42 AM, Russell King - ARM Linux wrote: On Fri, Jan 20, 2017 at 03:38:28PM +0100, Hans Verkuil wrote: Should be set to something like 'platform:imx-media-camif'.

Re: [PATCH] [media] media: fix semicolon.cocci warnings

2017-01-31 Thread Pavel Machek
On Tue 2017-01-31 22:20:27, kbuild test robot wrote: > drivers/media/i2c/et8ek8/et8ek8_driver.c:1112:3-4: Unneeded semicolon > > > Remove unneeded semicolon. > > Generated by: scripts/coccinelle/misc/semicolon.cocci > Acked-by: Pavel Machek -- (english) http://www.livejournal.com/~pavelmac

[PATCH] Documentation: devicetree: meson-ir: "linux,rc-map-name" is supported

2017-01-31 Thread Martin Blumenstingl
The driver already parses the "linux,rc-map-name" property. Add this information to the documentation so .dts maintainers don't have to look it up in the source-code. Signed-off-by: Martin Blumenstingl --- Documentation/devicetree/bindings/media/meson-ir.txt | 3 +++ 1 file changed, 3 insertions

[PATCH] Documentation: devicetree: add the RC map name of the geekbox remote

2017-01-31 Thread Martin Blumenstingl
Signed-off-by: Martin Blumenstingl --- The geekbox keymap was added while the documentation patch was not applied yet (and I wasn't aware of this pending patch). This ensures that the documentation is in sync with the actual keymaps. Documentation/devicetree/bindings/media/rc.txt | 1 + 1 file c

Re: [PATCH v3 20/24] media: imx: Add Camera Interface subdev driver

2017-01-31 Thread Russell King - ARM Linux
On Tue, Jan 31, 2017 at 10:21:26AM -0800, Steve Longerbeam wrote: > On 01/31/2017 05:42 AM, Russell King - ARM Linux wrote: > >On Fri, Jan 20, 2017 at 03:38:28PM +0100, Hans Verkuil wrote: > >>Should be set to something like 'platform:imx-media-camif'. v4l2-compliance > >>should complain about this

Re: [PATCH 6/6] staging: bcm2835-v4l2: Apply spelling fixes from checkpatch.

2017-01-31 Thread Joe Perches
On Tue, 2017-01-31 at 10:30 -0800, Eric Anholt wrote: > Joe Perches writes: > > > On Mon, 2017-01-30 at 12:05 -0800, Eric Anholt wrote: > > > Joe Perches writes: > > > > > > > On Fri, 2017-01-27 at 13:55 -0800, Eric Anholt wrote: > > > > > Generated with checkpatch.pl --fix-inplace and git add

Re: [PATCH 6/6] staging: bcm2835-v4l2: Apply spelling fixes from checkpatch.

2017-01-31 Thread Eric Anholt
Joe Perches writes: > On Mon, 2017-01-30 at 12:05 -0800, Eric Anholt wrote: >> Joe Perches writes: >> >> > On Fri, 2017-01-27 at 13:55 -0800, Eric Anholt wrote: >> > > Generated with checkpatch.pl --fix-inplace and git add -p out of the >> > > results. >> > >> > Maybe another. >> > >> > > dif

Re: [PATCH v3 20/24] media: imx: Add Camera Interface subdev driver

2017-01-31 Thread Steve Longerbeam
On 01/31/2017 05:42 AM, Russell King - ARM Linux wrote: On Fri, Jan 20, 2017 at 03:38:28PM +0100, Hans Verkuil wrote: Should be set to something like 'platform:imx-media-camif'. v4l2-compliance should complain about this. ... and more. Right, in version 3 that you are working with, no v4l2-

rtl2832_sdr and /dev/swradio0

2017-01-31 Thread Russel Winder
Hi, Is anyone actively working on the rtl2832_sdr driver? I am particularly interested in anyone who has code for turning the byte stream from /dev/swradio0 into an ETI stream. Or failing that getting enough data about the API for using /dev/swradio0 so as to write a byte sequence to ETI driver b

Re: [PATCH v3 17/24] media: imx: Add CSI subdev driver

2017-01-31 Thread Philipp Zabel
On Tue, 2017-01-31 at 16:51 +0100, Philipp Zabel wrote: > On Fri, 2017-01-06 at 18:11 -0800, Steve Longerbeam wrote: > [...] > > +static int csi_set_fmt(struct v4l2_subdev *sd, > > + struct v4l2_subdev_pad_config *cfg, > > + struct v4l2_subdev_format *sdformat) > >

[PATCH v5 08/10] [media] st-delta: EOS (End Of Stream) support

2017-01-31 Thread Hugues Fruchet
EOS (End Of Stream) support allows user to get all the potential decoded frames remaining in decoder pipeline after having reached the end of video bitstream. To do so, user calls VIDIOC_DECODER_CMD(V4L2_DEC_CMD_STOP) which will drain the decoder and get the drained frames that are then returned to

[PATCH v5 09/10] [media] st-delta: add mjpeg support

2017-01-31 Thread Hugues Fruchet
Adds support of DELTA MJPEG video decoder back-end, implemented by calling JPEG_DECODER_HW0 firmware using RPMSG IPC communication layer. Signed-off-by: Hugues Fruchet --- drivers/media/platform/Kconfig | 6 + drivers/media/platform/sti/delta/Makefile | 4 + driv

[PATCH v5 05/10] [media] st-delta: STiH4xx multi-format video decoder v4l2 driver

2017-01-31 Thread Hugues Fruchet
This V4L2 driver enables DELTA multi-format video decoder of STMicroelectronics STiH4xx SoC series. Signed-off-by: Hugues Fruchet --- drivers/media/platform/Kconfig| 20 + drivers/media/platform/Makefile |2 + drivers/media/platform/sti/delta/Makefile |

[PATCH v5 01/10] Documentation: DT: add bindings for ST DELTA

2017-01-31 Thread Hugues Fruchet
This patch adds DT binding documentation for STMicroelectronics DELTA V4L2 video decoder. Signed-off-by: Hugues Fruchet --- Documentation/devicetree/bindings/media/st,st-delta.txt | 17 + 1 file changed, 17 insertions(+) create mode 100644 Documentation/devicetree/bindings/media

[PATCH v5 00/10] Add support for DELTA video decoder of STMicroelectronics STiH4xx SoC series

2017-01-31 Thread Hugues Fruchet
This patchset introduces a basic support for DELTA multi-format video decoder of STMicroelectronics STiH4xx SoC series. DELTA hardware IP is controlled by a remote firmware loaded in a ST231 coprocessor. Communication with firmware is done within an IPC layer using rpmsg kernel framework and a sha

[PATCH v5 07/10] [media] st-delta: rpmsg ipc support

2017-01-31 Thread Hugues Fruchet
IPC (Inter Process Communication) support for communication with DELTA coprocessor firmware using rpmsg kernel framework. Based on 4 services open/set_stream/decode/close and their associated rpmsg messages. The messages structures are duplicated on both host and firmware side and are packed (use o

[PATCH v5 10/10] [media] st-delta: debug: trace stream/frame information & summary

2017-01-31 Thread Hugues Fruchet
Adds some trace points showing input compressed stream or output decoded frame information. Adds an unconditional trace point when streaming starts showing the compressed stream and the decoded frame information. Adds an unconditional trace point at instance closure summarizing into a single line t

[PATCH v5 06/10] [media] st-delta: add memory allocator helper functions

2017-01-31 Thread Hugues Fruchet
Helper functions used by decoder back-ends to allocate physically contiguous memory required by hardware video decoder. Signed-off-by: Hugues Fruchet --- drivers/media/platform/sti/delta/Makefile| 2 +- drivers/media/platform/sti/delta/delta-mem.c | 51 drivers/

[PATCH v5 03/10] ARM: multi_v7_defconfig: enable STMicroelectronics DELTA Support

2017-01-31 Thread Hugues Fruchet
Enables support of STMicroelectronics STiH4xx SoC series DELTA multi-format video decoder V4L2 driver. Signed-off-by: Hugues Fruchet --- arch/arm/configs/multi_v7_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defcon

[PATCH v5 02/10] ARM: dts: STiH410: add DELTA dt node

2017-01-31 Thread Hugues Fruchet
This patch adds DT node for STMicroelectronics DELTA V4L2 video decoder Signed-off-by: Hugues Fruchet --- arch/arm/boot/dts/stih410.dtsi | 10 ++ 1 file changed, 10 insertions(+) diff --git a/arch/arm/boot/dts/stih410.dtsi b/arch/arm/boot/dts/stih410.dtsi index 281a124..42e070c 100644 -

[PATCH v5 04/10] [media] MAINTAINERS: add st-delta driver

2017-01-31 Thread Hugues Fruchet
Add entry for the STMicroelectronics DELTA driver. Signed-off-by: Hugues Fruchet --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index cfff2c9..38cc652 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -2429,6 +2429,14 @@ W: https://linuxtv

Re: DRM Atomic property for color-space conversion

2017-01-31 Thread Brian Starkey
On Tue, Jan 31, 2017 at 05:15:46PM +0200, Ville Syrjälä wrote: On Tue, Jan 31, 2017 at 12:33:29PM +, Brian Starkey wrote: Hi, On Mon, Jan 30, 2017 at 03:35:13PM +0200, Ville Syrjälä wrote: >On Fri, Jan 27, 2017 at 05:23:24PM +, Brian Starkey wrote: >> Hi, >> >> We're looking to enable t

Re: [PATCH v3 17/24] media: imx: Add CSI subdev driver

2017-01-31 Thread Philipp Zabel
On Fri, 2017-01-06 at 18:11 -0800, Steve Longerbeam wrote: [...] > +static int csi_set_fmt(struct v4l2_subdev *sd, > +struct v4l2_subdev_pad_config *cfg, > +struct v4l2_subdev_format *sdformat) > +{ > + struct csi_priv *priv = v4l2_get_subdevdata(sd); > +

[PATCH 10/11] media: rcar-vin: split rvin_s_fmt_vid_cap()

2017-01-31 Thread Niklas Söderlund
To support unbind and rebinding of subdevices rvin_s_fmt_vid_cap() needs to be called from with in the driver itself. Rename the function __rvin_s_fmt_vid_cap() and create a wrapper which can be used by vidioc_s_fmt_vid_cap. Signed-off-by: Niklas Söderlund --- drivers/media/platform/rcar-vin/rca

[PATCH 09/11] media: rcar-vin: use pad information when verifying media bus format

2017-01-31 Thread Niklas Söderlund
Use information about pad index when enumerating mbus codes. Signed-off-by: Niklas Söderlund --- drivers/media/platform/rcar-vin/rcar-core.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/platform/rcar-vin/rcar-core.c b/drivers/media/platform/rcar-vin/rcar-core.c index e9373d

[PATCH 04/11] media: rcar-vin: fix standard in input enumeration

2017-01-31 Thread Niklas Söderlund
If the subdevice supports dv_timings_cap the driver should not fill in the standard. Also don't use the standard from probe time ask the subdevice each time. Signed-off-by: Niklas Söderlund --- drivers/media/platform/rcar-vin/rcar-v4l2.c | 10 -- 1 file changed, 8 insertions(+), 2 deleti

[PATCH 05/11] media: rcar-vin: add wrapper to get rvin_graph_entity

2017-01-31 Thread Niklas Söderlund
Update the driver to retrieve the code and mbus_cfg values from a rvin_graph_entity retrieved from a wrapper function instead of directly accessing the entity for the digital port. This is done to prepare for Gen3 support where the subdeivce might change during runtime, so to directly accesses a sp

[PATCH 01/11] media: rcar-vin: reset bytesperline and sizeimage when resetting format

2017-01-31 Thread Niklas Söderlund
These two fields where forgotten when refactoring the format reset code path. If they are not also reset at the same time as width and hight the format read using G_FMT will not match reality. Signed-off-by: Niklas Söderlund --- drivers/media/platform/rcar-vin/rcar-v4l2.c | 3 +++ 1 file changed

[PATCH 07/11] media: rcar-vin: move pad index discovery to async complete handler

2017-01-31 Thread Niklas Söderlund
To fix support for unbind and rebinding of subdevices the rvin_v4l2_probe() needs to be called before there might be any subdevice bound. Move pad index discovery to when we know the subdevice is present. Signed-off-by: Niklas Söderlund --- drivers/media/platform/rcar-vin/rcar-core.c | 23 ++

[PATCH 00/11] media: rcar-vin: fix OPS and format/pad index issues

2017-01-31 Thread Niklas Söderlund
Hi, This series address issues with the R-Car Gen2 VIN driver. The most serious issue is the OPS when unbind and rebinding the i2c driver for the video source subdevice which have popped up as a blocker for other work. This series is broken out of my much larger R-Car Gen3 enablement series '

[PATCH 06/11] media: rcar-vin: move subdev source and sink pad index to rvin_graph_entity

2017-01-31 Thread Niklas Söderlund
It makes more sens to store the sink and source pad index in struct rvin_graph_entity since that contains other subdevice related information. Add complete documentation for struct rvin_graph_entity while we are at it. Signed-off-by: Niklas Söderlund --- drivers/media/platform/rcar-vin/rcar-v4l2

[PATCH 03/11] media: rcar-vin: fix how pads are handled for v4l2 subdeivce operations

2017-01-31 Thread Niklas Söderlund
The rcar-vin driver only uses one pad, pad number 0. All v4l2 operations which did not check that the requested operation was for pad 0 have been updated with a check to enforce this. All v4l2 operations that stored (and later restore) the requested pad before substituting it for the subdeivc

[PATCH 11/11] media: rcar-vin: register the video device early

2017-01-31 Thread Niklas Söderlund
To support unbind and rebinding of video source subdevices while keeping a constant video device the subdevice needs to be attached when the first user opens the video device and detached when the last user closes it. This changes the rcar-vin behavior in such way that the video device is register

[PATCH 02/11] media: rcar-vin: use rvin_reset_format() in S_DV_TIMINGS

2017-01-31 Thread Niklas Söderlund
Use rvin_reset_format() in rvin_s_dv_timings() instead if just resetting a few fields. This fixes an issue where the field format was not properly set after S_DV_TIMINGS. Signed-off-by: Niklas Söderlund --- drivers/media/platform/rcar-vin/rcar-v4l2.c | 8 ++-- 1 file changed, 2 insertions(+)

[PATCH 08/11] media: rcar-vin: refactor pad lookup code

2017-01-31 Thread Niklas Söderlund
If the subdeivce did not supply pad information the driver will return -EINVAL, this is not what we want so remove that check. The code can then be broken out to a helper function reducing duplication. Signed-off-by: Niklas Söderlund --- drivers/media/platform/rcar-vin/rcar-core.c | 29 +

Re: [GIT PULL FOR v4.11] New st-delta driver

2017-01-31 Thread Hugues FRUCHET
On 01/30/2017 08:18 PM, Mauro Carvalho Chehab wrote: > Em Mon, 30 Jan 2017 17:15:36 -0200 > Mauro Carvalho Chehab escreveu: > >> Em Mon, 9 Jan 2017 14:23:33 +0100 >> Hans Verkuil escreveu: >> >>> See the v4 series for details: >>> >>> https://www.spinics.net/lists/linux-media/msg108737.html >>> >

Re: DRM Atomic property for color-space conversion

2017-01-31 Thread Ville Syrjälä
On Tue, Jan 31, 2017 at 12:33:29PM +, Brian Starkey wrote: > Hi, > > On Mon, Jan 30, 2017 at 03:35:13PM +0200, Ville Syrjälä wrote: > >On Fri, Jan 27, 2017 at 05:23:24PM +, Brian Starkey wrote: > >> Hi, > >> > >> We're looking to enable the per-plane color management hardware in > >> Mali-

Re: [PATCH v3 00/24] i.MX Media Driver

2017-01-31 Thread Russell King - ARM Linux
On Tue, Jan 31, 2017 at 03:25:10PM +0100, Philipp Zabel wrote: > On Tue, 2017-01-31 at 14:54 +0100, Philipp Zabel wrote: > > Hi Steve, > > > > I have just tested the imx-media-staging-md-wip branch on a Nitrogen6X > > with a tc358743 (BD_HDMI_MIPI HDMI to MIPI CSI-2 receiver board). Some > > obser

[PATCH] [media] hva-v4l2: hva_dbg_summary() should be static

2017-01-31 Thread Mauro Carvalho Chehab
As reported by gcc: drivers/media/platform/sti/hva/hva-v4l2.c:227:6: warning: no previous prototype for 'hva_dbg_summary' [-Wmissing-prototypes] void hva_dbg_summary(struct hva_ctx *ctx) ^~~ This function is used only internally, so make it static. Signed-off-by: Mauro Carval

Re: [PATCH v3 00/24] i.MX Media Driver

2017-01-31 Thread Philipp Zabel
On Tue, 2017-01-31 at 14:54 +0100, Philipp Zabel wrote: > Hi Steve, > > I have just tested the imx-media-staging-md-wip branch on a Nitrogen6X > with a tc358743 (BD_HDMI_MIPI HDMI to MIPI CSI-2 receiver board). Some > observations: > > # Link pipeline > media-ctl -l "'tc358743 1-000f':0->'imx6-mi

[PATCH] [media] media: fix semicolon.cocci warnings

2017-01-31 Thread kbuild test robot
drivers/media/i2c/et8ek8/et8ek8_driver.c:1112:3-4: Unneeded semicolon Remove unneeded semicolon. Generated by: scripts/coccinelle/misc/semicolon.cocci CC: Pavel Machek Signed-off-by: Fengguang Wu --- et8ek8_driver.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/drivers/me

[linuxtv-media:master 1136/1158] drivers/media/i2c/et8ek8/et8ek8_driver.c:1112:3-4: Unneeded semicolon

2017-01-31 Thread kbuild test robot
tree: git://linuxtv.org/media_tree.git master head: 0df03379c541ec027c94b025f2ef259f243eeb58 commit: c5254e72b8edc2ca0a98703e92e8c34959343d2c [1136/1158] [media] media: Driver for Toshiba et8ek8 5MP sensor coccinelle warnings: (new ones prefixed by >>) >> drivers/media/i2c/et8ek8/et8ek8_dri

Re: [PATCH v3 00/24] i.MX Media Driver

2017-01-31 Thread Russell King - ARM Linux
On Tue, Jan 31, 2017 at 02:35:00PM +0100, Philipp Zabel wrote: > On Tue, 2017-01-31 at 13:14 +, Russell King - ARM Linux wrote: > > This isn't limited to the serial side - the parallel bus side between > > the CSI2 interface and CSI2IPU wrapper, and the CSI2IPU wrapper and > > the CS0/1 interfa

Re: [PATCH v4 1/2] st-hva: encoding summary at instance release

2017-01-31 Thread Hans Verkuil
On 31/01/17 11:37, Jean-Christophe Trotin wrote: This patch adds a short summary about the encoding operation at each instance closing, for debug purpose (through dev_dbg()): - information about the frame (format, resolution) - information about the stream (format, profile, level, resolution) - n

Re: [PATCH v3 00/24] i.MX Media Driver

2017-01-31 Thread Philipp Zabel
Hi Steve, I have just tested the imx-media-staging-md-wip branch on a Nitrogen6X with a tc358743 (BD_HDMI_MIPI HDMI to MIPI CSI-2 receiver board). Some observations: # Link pipeline media-ctl -l "'tc358743 1-000f':0->'imx6-mipi-csi2':0[1]" media-ctl -l "'imx6-mipi-csi2':1->'ipu1_csi0_mux':0[1]" m

Re: [PATCH v3 1/3] st-hva: encoding summary at instance release

2017-01-31 Thread Mauro Carvalho Chehab
Em Tue, 31 Jan 2017 08:50:38 + Jean Christophe TROTIN escreveu: > On 01/30/2017 06:28 PM, Mauro Carvalho Chehab wrote: > > Em Mon, 28 Nov 2016 11:30:52 +0100 > > Jean-Christophe Trotin escreveu: > > > >> This patch prints unconditionnaly a short summary > > > > Why? Is this driver so bro

Re: [PATCH v3 20/24] media: imx: Add Camera Interface subdev driver

2017-01-31 Thread Russell King - ARM Linux
On Fri, Jan 20, 2017 at 03:38:28PM +0100, Hans Verkuil wrote: > Should be set to something like 'platform:imx-media-camif'. v4l2-compliance > should complain about this. ... and more. Driver Info: Driver name : imx-media-camif Card type : imx-media-camif Bus info

Re: [PATCH v3 00/24] i.MX Media Driver

2017-01-31 Thread Philipp Zabel
On Tue, 2017-01-31 at 13:14 +, Russell King - ARM Linux wrote: > On Tue, Jan 31, 2017 at 11:09:24AM +0100, Philipp Zabel wrote: > > On Mon, 2017-01-30 at 13:06 +, Russell King - ARM Linux wrote: > > > To help illustrate my point, consider the difference between > > > MEDIA_BUS_FMT_RGB565_1X

Re: [PATCH v3 00/24] i.MX Media Driver

2017-01-31 Thread Russell King - ARM Linux
On Tue, Jan 31, 2017 at 11:09:24AM +0100, Philipp Zabel wrote: > On Mon, 2017-01-30 at 13:06 +, Russell King - ARM Linux wrote: > > To help illustrate my point, consider the difference between > > MEDIA_BUS_FMT_RGB565_1X16 and MEDIA_BUS_FMT_RGB565_2X8_BE or > > MEDIA_BUS_FMT_RGB565_2X8_LE. RGB

Re: [GIT PULL FOR v4.11] Add et8ek8 driver

2017-01-31 Thread Mauro Carvalho Chehab
Em Tue, 31 Jan 2017 14:45:34 +0200 Sakari Ailus escreveu: > Hi Mauro, > > On Tue, Jan 31, 2017 at 10:42:48AM -0200, Mauro Carvalho Chehab wrote: > > That added a new warning: > > > > drivers/media/i2c/et8ek8/et8ek8_driver.c: In function 'et8ek8_registered': > > drivers/media/i2c/et8ek8/et8ek8_d

[linuxtv-media:master 1123/1134] drivers/media/platform/coda/imx-vdoa.c:333:1: note: in expansion of macro 'module_platform_driver'

2017-01-31 Thread kbuild test robot
tree: git://linuxtv.org/media_tree.git master head: a2fafda66dccb84592d4b9e42e429471c356c4fc commit: 126f52b02e6ec6a25f0b32058a91648304922d4a [1123/1134] [media] coda/imx-vdoa: constify structs config: arm-allmodconfig (attached as .config) compiler: arm-linux-gnueabi-gcc (Debian 6.1.1-9) 6.1.

Re: [GIT PULL FOR v4.11] Add et8ek8 driver

2017-01-31 Thread Sakari Ailus
Hi Mauro, On Tue, Jan 31, 2017 at 10:42:48AM -0200, Mauro Carvalho Chehab wrote: > That added a new warning: > > drivers/media/i2c/et8ek8/et8ek8_driver.c: In function 'et8ek8_registered': > drivers/media/i2c/et8ek8/et8ek8_driver.c:1262:29: warning: variable 'format' > set but not used [-Wunused-

Re: DRM Atomic property for color-space conversion

2017-01-31 Thread Brian Starkey
Hi, On Mon, Jan 30, 2017 at 03:35:13PM +0200, Ville Syrjälä wrote: On Fri, Jan 27, 2017 at 05:23:24PM +, Brian Starkey wrote: Hi, We're looking to enable the per-plane color management hardware in Mali-DP with atomic properties, which has sparked some conversation around how to handle YCbC

Re: [GIT PULL FOR v4.11] Add et8ek8 driver

2017-01-31 Thread Mauro Carvalho Chehab
Em Wed, 25 Jan 2017 16:07:45 +0200 Sakari Ailus escreveu: > Hi Mauro, > > This pull request adds the sensor et8ek8 driver which is used on the Nokia > N900. Please pull. > > > The following changes since commit 40eca140c404505c09773d1c6685d818cb55ab1a: > > [media] mn88473: add DVB-T2 PLP su

[PATCH] media: exynos4-is: add flags to dummy Exynos IS i2c adapter

2017-01-31 Thread Marek Szyprowski
0 Internal error: Oops: 8005 [#1] PREEMPT SMP ARM Modules linked in: CPU: 1 PID: 100 Comm: kworker/1:2 Not tainted 4.10.0-rc6-next-20170131-00054-g39e6e4233de6 #1921 Hardware name: SAMSUNG EXYNOS (Flattened Device Tree) Workqueue: events deferred_probe_work_func task: ef2e task.stack: ef2ec00

Re: [PATCHv2] v4l: of: check for unique lanes in data-lanes and clock-lanes

2017-01-31 Thread Sakari Ailus
On Tue, Jan 31, 2017 at 01:08:31PM +0100, Niklas Söderlund wrote: > All lanes in data-lanes and clock-lanes properties should be unique. Add > a check for this in v4l2_of_parse_csi_bus() and print a warning if > duplicated lanes are found. > > Signed-off-by: Niklas Söderlund Acked-by: Sakari Ail

[PATCHv2] v4l: of: check for unique lanes in data-lanes and clock-lanes

2017-01-31 Thread Niklas Söderlund
All lanes in data-lanes and clock-lanes properties should be unique. Add a check for this in v4l2_of_parse_csi_bus() and print a warning if duplicated lanes are found. Signed-off-by: Niklas Söderlund --- Changes since v1: - Do not return -EINVAL if a duplicate is found. Sakari pointed out th

Re: [PATCH v3 17/24] media: imx: Add CSI subdev driver

2017-01-31 Thread Russell King - ARM Linux
On Fri, Jan 06, 2017 at 06:11:35PM -0800, Steve Longerbeam wrote: > +static int csi_link_validate(struct v4l2_subdev *sd, > + struct media_link *link, > + struct v4l2_subdev_format *source_fmt, > + struct v4l2_subdev_format

Re: [PATCH v3 00/24] i.MX Media Driver

2017-01-31 Thread Russell King - ARM Linux
On Mon, Jan 30, 2017 at 05:22:01PM -0800, Steve Longerbeam wrote: > I'm also having trouble finding a datasheet for it, but from what > I've read, it has a MIPI CSI-2 interface. It should work fine as long > as it presents a single source pad, registers asynchronously, and > sets its entity functio

Re: [PATCH 01/11] [media] s5p-mfc: Rename IS_MFCV8 macro

2017-01-31 Thread Smitha T Murthy
On Wed, 2017-01-18 at 15:51 +0100, Andrzej Hajda wrote: > On 18.01.2017 11:01, Smitha T Murthy wrote: > > This patch renames macro IS_MFCV8 to IS_MFCV8_PLUS so that the MFCv8 > > code can be resued for MFCv10.10 support. Since the MFCv8 specific code > > holds good for MFC v10.10 also. > > > > Sign

Re: [PATCH] Staging: omap4iss: fix coding style issues

2017-01-31 Thread Laurent Pinchart
On Tuesday 31 Jan 2017 12:42:51 Sakari Ailus wrote: > On Mon, Jan 30, 2017 at 07:47:40PM +0200, Laurent Pinchart wrote: > > > @@ -678,8 +679,8 @@ iss_video_get_selection(struct file *file, void *fh, > > > struct v4l2_selection *sel) if (subdev == NULL) > > > > > > return -EINVAL; > > >

Re: [PATCH] Staging: omap4iss: fix coding style issues

2017-01-31 Thread Sakari Ailus
On Mon, Jan 30, 2017 at 07:47:40PM +0200, Laurent Pinchart wrote: > > @@ -678,8 +679,8 @@ iss_video_get_selection(struct file *file, void *fh, > > struct v4l2_selection *sel) if (subdev == NULL) > > return -EINVAL; > > > > - /* Try the get selection operation first and fallback to ge

[PATCH v4 0/2] add debug capabilities to v4l2 encoder for STMicroelectronics SOC

2017-01-31 Thread Jean-Christophe Trotin
version 4: - As suggested by Mauro, the encoding summary (first patch) is no more unconditionallly added: dev_dbg() is used instead of dev_info(). version 3: - the encoding summary (first patch) is moved from hva-debug.c to hva-v4l2.c. As suggested by Hans, dev_info() is used instead of snprin

[PATCH v4 1/2] st-hva: encoding summary at instance release

2017-01-31 Thread Jean-Christophe Trotin
This patch adds a short summary about the encoding operation at each instance closing, for debug purpose (through dev_dbg()): - information about the frame (format, resolution) - information about the stream (format, profile, level, resolution) - number of encoded frames - potential (system, encodi

[PATCH v4 2/2] st-hva: add debug file system

2017-01-31 Thread Jean-Christophe Trotin
This patch creates 4 static debugfs entries to dump: - the device-related information ("st-hva/device") - the list of registered encoders ("st-hva/encoders") - the current values of the hva registers ("st-hva/regs") - the information about the last closed instance ("st-hva/last") It also creates d

Re: [PATCHv2 10/16] ov2640: enable clock and fix power/reset

2017-01-31 Thread Sakari Ailus
Hi Hans, Thank you for the patchset! On Mon, Jan 30, 2017 at 03:06:22PM +0100, Hans Verkuil wrote: > From: Hans Verkuil > > Convert v4l2_clk to normal clk, enable the clock and fix the power/reset > handling. > > Signed-off-by: Hans Verkuil > --- > drivers/media/i2c/ov2640.c | 80 >

Re: [PATCH v3 17/24] media: imx: Add CSI subdev driver

2017-01-31 Thread Russell King - ARM Linux
On Fri, Jan 06, 2017 at 06:11:35PM -0800, Steve Longerbeam wrote: > This is a media entity subdevice for the i.MX Camera > Serial Interface module. > > Signed-off-by: Steve Longerbeam > --- The lack of s_frame_interval/g_frame_interval in this driver means: media-ctl -v -d /dev/media1 --set-v4l

Re: [PATCH v3 00/24] i.MX Media Driver

2017-01-31 Thread Russell King - ARM Linux
On Mon, Jan 30, 2017 at 05:22:01PM -0800, Steve Longerbeam wrote: > Edit: I see a subdev that is missing: the video mux. Did you enable > CONFIG_VIDEO_MULTIPLEXER? Yes, and that's where the problem is - the video-multiplexer is missing the module aliases to allow it to be automatically loaded. -

Re: [PATCH 2/8] [media] v4l2-async: Delete an error message for a failed memory allocation in v4l2_async_notifier_unregister()

2017-01-31 Thread Mauro Carvalho Chehab
Em Mon, 26 Dec 2016 21:45:50 +0100 SF Markus Elfring escreveu: > From: Markus Elfring > Date: Mon, 26 Dec 2016 19:19:49 +0100 > > The script "checkpatch.pl" pointed information out like the following. > > WARNING: Possible unnecessary 'out of memory' message > > Thus fix the affected source c

[PATCH] [media] coda/imx-vdoa: constify structs

2017-01-31 Thread Mauro Carvalho Chehab
As warned by checkpatch: WARNING: struct of_device_id should normally be const #318: FILE: drivers/media/platform/coda/imx-vdoa.c:318: +static struct of_device_id vdoa_dt_ids[] = { So, constify structs. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/platform/cod

Re: [PATCH v3 00/24] i.MX Media Driver

2017-01-31 Thread Philipp Zabel
On Mon, 2017-01-30 at 13:06 +, Russell King - ARM Linux wrote: > > The central issue seems to be that I think media pad links / media bus > > formats should describe physical links, such as parallel or serial > > buses, and the formats of pixels flowing through them, whereas Steve > > would lik

[PATCH] doc-rst: fixed cleandoc target when used with O=dir

2017-01-31 Thread Markus Heiser
The cleandocs target won't work if I use a different output folder:: $ make O=/tmp/kernel SPHINXDIRS="process" cleandocs make[1]: Entering directory '/tmp/kernel' make[3]: *** No rule to make target 'clean'. Stop. ... Documentation/Makefile.sphinx:100: recipe for target 'cleandocs' failed

Re: [PATCH v3 00/24] i.MX Media Driver

2017-01-31 Thread Philipp Zabel
On Mon, 2017-01-30 at 17:22 -0800, Steve Longerbeam wrote: > > On 01/30/2017 04:45 PM, Russell King - ARM Linux wrote: > > > > Hi, > > > > Trying this driver with an imx219 camera (which works with Philipp's > > driver) results in not much happening... no /dev/media* node for it, > > no subdevs, n

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

2017-01-31 Thread Philipp Zabel
On Tue, 2017-01-31 at 00:01 +, Russell King - ARM Linux wrote: [...] > The iMX6 manuals call for a very specific seven sequence of initialisation > for CSI2, which begins with: > > 1. reset the D-PHY. > 2. place MIPI sensor in LP-11 state > 3. perform D-PHY initialisation > 4. configure CSI2 l

Re: [PATCH] [media] s5p-mfc: Align stream buffer and CPB buffer to 512

2017-01-31 Thread Smitha T Murthy
On Wed, 2017-01-18 at 15:37 +0100, Andrzej Hajda wrote: > Hi Smitha, > > On 18.01.2017 10:37, Smitha T Murthy wrote: > > >From MFCv6 onwards encoder stream buffer and decoder CPB buffer > > Unexpected char at the beginning. > > > need to be aligned with 512. > > Patch below adds checks only if

Re: [PATCH 02/11] [media] s5p-mfc: Adding initial support for MFC v10.10

2017-01-31 Thread Smitha T Murthy
On Wed, 2017-01-18 at 16:10 +0100, Andrzej Hajda wrote: > On 18.01.2017 11:02, Smitha T Murthy wrote: > > Adding the support for MFC v10.10, with new register file and > > necessary hw control, decoder, encoder and structural changes. > > > > CC: Rob Herring > > CC: devicet...@vger.kernel.org > >

Re: [PATCH v3 1/3] st-hva: encoding summary at instance release

2017-01-31 Thread Jean Christophe TROTIN
On 01/30/2017 06:28 PM, Mauro Carvalho Chehab wrote: > Em Mon, 28 Nov 2016 11:30:52 +0100 > Jean-Christophe Trotin escreveu: > >> This patch prints unconditionnaly a short summary > > Why? Is this driver so broken that everyone would need an > unconditional "short summary" about what happened th