cron job: media_tree daily build: OK

This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Fri Apr 15 04:00:21 CEST 2016 git branch: test git hash: ecb7b0183a89613c154d1bea48b494907efbf8f9 gcc

[PATCH 2/2] [media] tvp5150: propagate I2C write error in .s_register callback

The tvp5150_write() function can fail so don't return 0 unconditionally in tvp5150_s_register() but propagate what's returned by tvp5150_write(). Signed-off-by: Javier Martinez Canillas --- drivers/media/i2c/tvp5150.c | 3 +-- 1 file changed, 1 insertion(+), 2

[PATCH 1/2] [media] tvp5150: return I2C write operation failure to callers

The tvp5150_write() function calls i2c_smbus_write_byte_data() that can fail but does not propagate the error to the caller. Instead it just prints a debug, so callers can't know if the operation failed. So change the function to return the error code to the caller so it knows that the write

Re: tvp5150 regression after commit 9f924169c035

Hello Tony, On 04/14/2016 11:12 AM, Tony Lindgren wrote: > Hi, > > * Javier Martinez Canillas [160414 07:28]: >> Hello Wofram, >> >> On 04/14/2016 10:19 AM, Wolfram Sang wrote: >>> Yes, I also wonder why I'm the only one facing this issue... maybe no one else

Re: [PATCH] media: saa7134 fix media_dev alloc error path to not free when alloc fails

On 04/14/2016 03:08 PM, Mauro Carvalho Chehab wrote: > Em Thu, 14 Apr 2016 10:31:20 -0600 > Shuah Khan escreveu: > >> media_dev alloc error path does kfree when alloc fails. Fix it to not call >> kfree when media_dev alloc fails. > > No need. kfree(NULL) is OK. Agreed.

Re: [PATCH] media: saa7134 fix media_dev alloc error path to not free when alloc fails

Em Thu, 14 Apr 2016 10:31:20 -0600 Shuah Khan escreveu: > media_dev alloc error path does kfree when alloc fails. Fix it to not call > kfree when media_dev alloc fails. No need. kfree(NULL) is OK. Adding a label inside a conditional block is ugly. > > Signed-off-by:

[PATCH 1/2] [media] mceusb: remove useless debug message

Signed-off-by: Sean Young --- drivers/media/rc/mceusb.c | 21 + 1 file changed, 1 insertion(+), 20 deletions(-) diff --git a/drivers/media/rc/mceusb.c b/drivers/media/rc/mceusb.c index 35155ae..85823e8 100644 --- a/drivers/media/rc/mceusb.c +++

[PATCH 2/2] [media] mceusb: remove pointless mce_flush_rx_buffer function

This function just submits the urb much like mceusb_dev_resume; removing it simplifies mce_request_packet. Also add missing usb_kill_urb to mce_dev_probe. Signed-off-by: Sean Young --- drivers/media/rc/mceusb.c | 77 +++ 1 file

Re: [PATCH] [media] vsp1: make vsp1_drm_frame_end static

Hi Mauro, On Wednesday 13 Apr 2016 17:42:24 Mauro Carvalho Chehab wrote: > As reported by smatch: > drivers/media/platform/vsp1/vsp1_drm.c:39:6: warning: no previous > prototype > for 'vsp1_drm_frame_end' [-Wmissing-prototypes] void > vsp1_drm_frame_end(struct vsp1_pipeline *pipe) > >

Re: [PATCH] uvc: Fix bytesperline calculation for planar YUV

Hi Nicolas, On Wednesday 13 Apr 2016 11:57:34 Nicolas Dufresne wrote: > Le mercredi 13 avril 2016 à 17:36 +0300, Laurent Pinchart a écrit : > > Hi Nicolas, > > > > Thank you for the patch. > > > > On Thursday 07 Jan 2016 15:43:48 Nicolas Dufresne wrote: > > > The formula used to calculate

[PATCH] media: saa7134 fix media_dev alloc error path to not free when alloc fails

media_dev alloc error path does kfree when alloc fails. Fix it to not call kfree when media_dev alloc fails. Signed-off-by: Shuah Khan --- drivers/media/pci/saa7134/saa7134-core.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

Re: [PATCH 1/2] [media] atmel-isc: add the Image Sensor Controller code

Hello Songjun, Thank you for the patch. On Wednesday 13 Apr 2016 15:44:19 Songjun Wu wrote: > Add driver for the Image Sensor Controller. It manages > incoming data from a parallel based CMOS/CCD sensor. > It has an internal image processor, also integrates a > triple channel direct memory

[PATCH v3 4/7] media: rcar-vin: pad-aware driver initialisation

Add detection of source pad number for drivers aware of the media controller API, so that rcar-vin can create device nodes to support modern drivers such as adv7604.c (for HDMI on Lager) and the converted adv7180.c (for composite) underneath. Building rcar_vin gains a dependency on

[PATCH v3 5/7] media: rcar-vin: add DV timings support

Adds ioctls DV_TIMINGS_CAP, ENUM_DV_TIMINGS, G_DV_TIMINGS, S_DV_TIMINGS, and QUERY_DV_TIMINGS. Signed-off-by: Ulrich Hecht --- drivers/media/platform/rcar-vin/rcar-v4l2.c | 69 + 1 file changed, 69 insertions(+) diff --git

[PATCH v3 3/7] media: rcar_vin: Use correct pad number in try_fmt

Fix rcar_vin_try_fmt's use of an inappropriate pad number when calling the subdev set_fmt function - for the ADV7612, IDs should be non-zero. Signed-off-by: William Towle Reviewed-by: Rob Taylor Acked-by: Hans Verkuil

[PATCH v3 7/7] ARM: dts: lager: Add entries for VIN HDMI input support

From: William Towle Add DT entries for vin0, vin0_pins, and adv7612. Sets the 'default-input' property for ADV7612, enabling image and video capture without the need to have userspace specifying routing. Signed-off-by: William Towle

[PATCH v3 6/7] media: rcar-vin: initialize EDID data

Initializes the decoder subdevice with a fixed EDID blob. Signed-off-by: Ulrich Hecht --- drivers/media/platform/rcar-vin/rcar-v4l2.c | 46 + 1 file changed, 46 insertions(+) diff --git a/drivers/media/platform/rcar-vin/rcar-v4l2.c

[PATCH v3 2/7] media: adv7604: automatic "default-input" selection

From: William Towle Add logic such that the "default-input" property becomes unnecessary for chips that only have one suitable input (ADV7611 by design, and ADV7612 due to commit 7111cddd518f ("[media] media: adv7604: reduce support to first (digital) input").

[PATCH v3 1/7] v4l: subdev: Add pad config allocator and init

From: Laurent Pinchart Add a new subdev operation to initialize a subdev pad config array, and a helper function to allocate and initialize the array. This can be used by bridge drivers to implement try format based on subdev pad operations. Signed-off-by: Laurent

[PATCH v3 0/7] Lager board HDMI input support

Hi! This series implements Lager HDMI input support on top of version 4 of Niklas's rcar-vin rewrite ("[PATCHv4] [media] rcar-vin: add Renesas R-Car VIN driver"). Apart from rebasing, this revision removes one patch that has since been picked up, squashes the DT changes into one patch and

Re: [PATCH 2/7] dt-bindings: Add a binding for Mediatek Video Decoder

On Wed, Apr 13, 2016 at 08:01:50PM +0800, Tiffany Lin wrote: > Add a DT binding documentation of Video Decoder for the > MT8173 SoC from Mediatek. > > Signed-off-by: Tiffany Lin > --- > .../devicetree/bindings/media/mediatek-vcodec.txt | 50 > ++-- >

Re: tvp5150 regression after commit 9f924169c035

Hi, * Javier Martinez Canillas [160414 07:28]: > Hello Wofram, > > On 04/14/2016 10:19 AM, Wolfram Sang wrote: > > > >> Yes, I also wonder why I'm the only one facing this issue... maybe no one > >> else is using the tvp5150 driver on an OMAP board with mainline? > > >

Re: tvp5150 regression after commit 9f924169c035

Hello Wofram, On 04/14/2016 10:19 AM, Wolfram Sang wrote: > >> Yes, I also wonder why I'm the only one facing this issue... maybe no one >> else is using the tvp5150 driver on an OMAP board with mainline? > > I wonder why it only affects tvp5150. I don't see the connection yet. > Yes, me

Re: tvp5150 regression after commit 9f924169c035

> Yes, I also wonder why I'm the only one facing this issue... maybe no one > else is using the tvp5150 driver on an OMAP board with mainline? I wonder why it only affects tvp5150. I don't see the connection yet. signature.asc Description: PGP signature

Re: [PATCH 0/2] [media] atmel-isc: add driver for Atmel ISC

Hello Songjun, On Wednesday 13 Apr 2016 15:44:18 Songjun Wu wrote: > The Image Sensor Controller driver includes two parts. > 1) Driver code to implement the ISC function. > 2) Device tree binding documentation, it describes how >to add the ISC in device tree. > > > Songjun Wu (2): >

Re: [PATCH 1/2] [media] atmel-isc: add the Image Sensor Controller code

Hello Songjun, On Thursday 14 Apr 2016 13:44:27 Wu, Songjun wrote: > The option 'CONFIG_COMMON_CLK=y' is needed to add to '.config'. > But I do not validate, '.config' will be generated automatically and > overwritten when it is changed. Your driver's Kconfig entry should then contain "depends

Re: tvp5150 regression after commit 9f924169c035

://patchwork.ozlabs.org/patch/609280/ > Unfortunately it doesn't help, I just tried today's -next (next-20160414) that already has the mentioned patch but the issue is still present. Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America -- To unsubscribe from this list:

Re: tvp5150 regression after commit 9f924169c035

> I'll write what I found so far in case someone with better knowledge about > the runtime PM API and the OMAP I2C controller driver can have an idea of > what could be causing this. Thanks for the summary. I got no other reports like this, I wonder about that. That being said, can you try this

Re: [PATCH] [media] exynos-gsc: remove an always false condition

On 04/13/2016 09:32 PM, Mauro Carvalho Chehab wrote: > As reported by smatch: > drivers/media/platform/exynos-gsc/gsc-core.c:1073 gsc_probe() warn: > impossible condition '(gsc->id < 0) => (0-65535 < 0)' > drivers/media/platform/exynos-gsc/gsc-core.c: In function 'gsc_probe': >