Re: [GIT PULL for v4.6-rc1] media updates

2016-05-03 Thread Stefan Lippers-Hollmann
Hi On 2016-05-03, Linus Torvalds wrote: > On Tue, May 3, 2016 at 2:38 PM, Stefan Lippers-Hollmann wrote: > > Hi > > [...] > >> Mauro Carvalho Chehab (95): > > [...] > >> [media] use v4l2_mc_usb_media_device_init() on most USB devices > > [...] > > > > This change, as

cron job: media_tree daily build: OK

2016-05-03 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 4 04:00:18 CEST 2016 git branch: test git hash: 68af062b5f38510dc96635314461c6bbe1dbf2fe gcc

Re: [GIT PULL for v4.6-rc1] media updates

2016-05-03 Thread Linus Torvalds
On Tue, May 3, 2016 at 2:38 PM, Stefan Lippers-Hollmann wrote: > Hi > [...] >> Mauro Carvalho Chehab (95): > [...] >> [media] use v4l2_mc_usb_media_device_init() on most USB devices > [...] > > This change, as part of v4.6-rc6-85-g1248ded, breaks two systems, each > equipped

Re: [PATCH v7 18/24] i2c-mux: relax locking of the top i2c adapter during mux-locked muxing

2016-05-03 Thread Wolfram Sang
> +static int i2c_mux_trylock_bus(struct i2c_adapter *adapter, int flags) > +{ > + struct i2c_mux_priv *priv = adapter->algo_data; > + struct i2c_adapter *parent = priv->muxc->parent; > + > + if (!rt_mutex_trylock(>mux_lock)) > + return 0; > + if (!(flags &

Re: [GIT PULL for v4.6-rc1] media updates

2016-05-03 Thread Stefan Lippers-Hollmann
Hi On 2016-03-15, Mauro Carvalho Chehab wrote: [...] > The following changes since commit b562e44f507e863c6792946e4e1b1449fbbac85d: > > Linux 4.5 (2016-03-13 21:28:54 -0700) > > are available in the git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media >

Re: [PATCH v7 16/24] i2c: allow adapter drivers to override the adapter locking

2016-05-03 Thread Wolfram Sang
> Yes, they look like reasonable complaints. Thanks for fixing them. I just sent out my latest comments and when you fix those and send V8, I'll apply that right away. I think we are safe to fix the rest incrementally if needed. Note that I didn't review the IIO and media patches, I trust the

Re: [PATCH v7 16/24] i2c: allow adapter drivers to override the adapter locking

2016-05-03 Thread Wolfram Sang
On Wed, Apr 20, 2016 at 05:17:56PM +0200, Peter Rosin wrote: > Add i2c_lock_bus() and i2c_unlock_bus(), which call the new lock_bus and > unlock_bus ops in the adapter. These funcs/ops take an additional flags > argument that indicates for what purpose the adapter is locked. > > There are two

[PATCH 1/3] [media] s5p-mfc: Set device name for reserved memory region devs

2016-05-03 Thread Javier Martinez Canillas
The devices don't have a name set, so makes dev_name() returns NULL which makes harder to identify the devices that are causing issues, for example: WARNING: CPU: 2 PID: 616 at drivers/base/core.c:251 device_release+0x8c/0x90 Device '(null)' does not have a release() function, it is broken and

[PATCH 2/3] [media] s5p-mfc: Add release callback for memory region devs

2016-05-03 Thread Javier Martinez Canillas
When s5p_mfc_remove() calls put_device() for the reserved memory region devs, the driver core warns that the dev doesn't have a release callback: WARNING: CPU: 0 PID: 591 at drivers/base/core.c:251 device_release+0x8c/0x90 Device 's5p-mfc-l' does not have a release() function, it is broken and

[PATCH 0/3] [media] s5p-mfc: Fixes for issues when module is removed

2016-05-03 Thread Javier Martinez Canillas
Hello, This patch series fixes some issues that I noticed when trying to remove the s5p-mfc driver when built as a module. Some of these issues will be fixed once Marek's patches to convert the custom memory region reservation code is replaced by a generic one that supports named memory region

[PATCH 3/3] [media] s5p-mfc: Fix race between s5p_mfc_probe() and s5p_mfc_open()

2016-05-03 Thread Javier Martinez Canillas
The s5p_mfc_probe() function registers the video devices before all the resources needed by s5p_mfc_open() are correctly initalized. So if s5p_mfc_open() function is called before s5p_mfc_probe() finishes (since the video dev is already registered), a NULL pointer dereference will happen due

Re: [RFC PATCH 06/24] smiapp: Add quirk control support

2016-05-03 Thread Ivaylo Dimitrov
Hi, On 1.05.2016 13:46, Sakari Ailus wrote: On Mon, Apr 25, 2016 at 12:08:06AM +0300, Ivaylo Dimitrov wrote: From: Sakari Ailus Quirk controls can be set up in the init quirk. Signed-off-by: Sakari Ailus Do you need quirk

Re: [RFC PATCH 04/24] smiapp-pll: Take existing divisor into account in minimum divisor check

2016-05-03 Thread Ivaylo Dimitrov
Hi, On 1.05.2016 13:45, Sakari Ailus wrote: Hi Ivaylo, On Mon, Apr 25, 2016 at 12:08:04AM +0300, Ivaylo Dimitrov wrote: From: Sakari Ailus Required added multiplier (and divisor) calculation did not take into account the existing divisor when checking the values

[PATCH v2] media: fix use-after-free in cdev_put() when app exits after driver unbind

2016-05-03 Thread Shuah Khan
When driver unbinds while media_ioctl is in progress, cdev_put() fails with when app exits after driver unbinds. Add devnode struct device kobj as the cdev parent kobject. cdev_add() holds a reference to it and releases the reference in cdev_del() ensuring that the media_devnode is not

Re: [PATCH] media: fix use-after-free in cdev_put() when app exits after driver unbind

2016-05-03 Thread Lars-Peter Clausen
On 05/03/2016 05:06 PM, Shuah Khan wrote: > On 05/02/2016 04:16 AM, Lars-Peter Clausen wrote: >> On 04/30/2016 12:37 AM, Shuah Khan wrote: >> [...] >>> diff --git a/include/media/media-devnode.h b/include/media/media-devnode.h >>> index 5bb3b0e..ce9b051 100644 >>> ---

Re: Kernel docs: muddying the waters a bit

2016-05-03 Thread Keith Packard
Daniel Vetter writes: > So sphinx/rst y/n? Jon, is that ok with you from the doc maintainer > pov? I think the right answer for today is to use sphinx to generate docs From inline comments, to encourage outline docs to give it a try but to allow doc writers to use

g_webcam driver not working properly on newer kernel

2016-05-03 Thread Marco Madrigal
Hi all, I am working with the g_webcam driver on Linux 3.14 and 4.3 with an embedded system. It works pretty well for most of Linux hosts but when trying to register the webcam driver on Windows 7/8 I got "This device cannot start (Code 10)" message. I decided to try out the old version of

Re: [PATCH] media: fix use-after-free in cdev_put() when app exits after driver unbind

2016-05-03 Thread Shuah Khan
On 05/02/2016 04:16 AM, Lars-Peter Clausen wrote: > On 04/30/2016 12:37 AM, Shuah Khan wrote: > [...] >> diff --git a/include/media/media-devnode.h b/include/media/media-devnode.h >> index 5bb3b0e..ce9b051 100644 >> --- a/include/media/media-devnode.h >> +++ b/include/media/media-devnode.h >> @@

[PATCH] Delete Eurobird1-28.5E tuning file (satellite moved)

2016-05-03 Thread Nick Morrott
Remove tuning file for Eurobird1 at 28.5E. The satellite was moved to 33.0E (July 2015) and is now known as Eutelsat 33C using different transponders. Signed-off-by: Nick Morrott --- dvb-s/Eurobird1-28.5E | 26 -- 1 file changed, 26

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

2016-05-03 Thread Ivaylo Dimitrov
The sensor is found in Nokia N900 main camera Signed-off-by: Ivaylo Dimitrov --- .../bindings/media/i2c/toshiba,et8ek8.txt | 53 + drivers/media/i2c/Kconfig |1 + drivers/media/i2c/Makefile |1 +

Re: Kernel docs: muddying the waters a bit

2016-05-03 Thread Daniel Vetter
Hi all, So sounds like moving ahead with rst/sphinx is the option that should allow us to address everyone's concerns eventually? Of course the first one won't have it all (media seems really tricky), but I'd like to get something awesome in this area closer to mainline. I'm stalling on typing

Re: [PATCH v2] s5p-mfc: Don't try to put pm->clock if lookup failed

2016-05-03 Thread Javier Martinez Canillas
Hello Krzysztof, On 05/03/2016 06:15 AM, Krzysztof Kozlowski wrote: > On Mon, May 02, 2016 at 03:14:22PM -0400, Javier Martinez Canillas wrote: >> Failing to get the struct s5p_mfc_pm .clock is a non-fatal error so the >> clock field can have a errno pointer value. But s5p_mfc_final_pm() only >>

[PATCH v10 4/8] dt-bindings: Add a binding for Mediatek Video Encoder

2016-05-03 Thread Tiffany Lin
Add a DT binding documentation of Video Encoder for the MT8173 SoC from Mediatek. Signed-off-by: Tiffany Lin Acked-by: Rob Herring --- .../devicetree/bindings/media/mediatek-vcodec.txt | 59 1 file changed, 59 insertions(+)

[PATCH v10 7/8] [media] vcodec: mediatek: Add Mediatek H264 Video Encoder Driver

2016-05-03 Thread Tiffany Lin
Add h264 encoder driver for MT8173 Signed-off-by: PoChun Lin Signed-off-by: Tiffany Lin --- drivers/media/platform/mtk-vcodec/Makefile |1 + .../media/platform/mtk-vcodec/venc/venc_h264_if.c | 679

[PATCH v10 2/8] [media] VPU: mediatek: support Mediatek VPU

2016-05-03 Thread Tiffany Lin
From: Andrew-CT Chen The VPU driver for hw video codec embedded in Mediatek's MT8173 SOCs. It is able to handle video decoding/encoding of in a range of formats. The driver provides with VPU firmware download, memory management and the communication interface between

[PATCH v10 1/8] dt-bindings: Add a binding for Mediatek Video Processor

2016-05-03 Thread Tiffany Lin
From: Andrew-CT Chen Add a DT binding documentation of Video Processor Unit for the MT8173 SoC from Mediatek. Signed-off-by: Andrew-CT Chen Signed-off-by: Tiffany Lin Acked-by: Rob Herring

[PATCH v10 6/8] [media] vcodec: mediatek: Add Mediatek VP8 Video Encoder Driver

2016-05-03 Thread Tiffany Lin
Add vp8 encoder driver for MT8173 Signed-off-by: PoChun Lin Signed-off-by: Tiffany Lin --- drivers/media/platform/mtk-vcodec/Makefile |6 +- .../media/platform/mtk-vcodec/venc/venc_vp8_if.c | 481

[PATCH v10 0/8] Add MT8173 Video Encoder Driver and VPU Driver

2016-05-03 Thread Tiffany Lin
== Introduction == The purpose of this series is to add the driver for video codec hw embedded in the Mediatek's MT8173 SoCs. Mediatek Video Codec is able to handle video encoding of in a range of formats. This patch series also include VPU driver. Mediatek Video Codec

[PATCH v10 3/8] arm64: dts: mediatek: Add node for Mediatek Video Processor Unit

2016-05-03 Thread Tiffany Lin
From: Andrew-CT Chen Add VPU drivers for MT8173 Signed-off-by: Andrew-CT Chen Signed-off-by: Tiffany Lin --- arch/arm64/boot/dts/mediatek/mt8173.dtsi | 23 +++ 1 file changed, 23

[PATCH v10 8/8] arm64: dts: mediatek: Add Video Encoder for MT8173

2016-05-03 Thread Tiffany Lin
Add video encoder node for MT8173 Signed-off-by: Tiffany Lin --- arch/arm64/boot/dts/mediatek/mt8173.dtsi | 39 ++ 1 file changed, 39 insertions(+) diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi

Re: [PATCH v2] s5p-mfc: Don't try to put pm->clock if lookup failed

2016-05-03 Thread Krzysztof Kozlowski
On Mon, May 02, 2016 at 03:14:22PM -0400, Javier Martinez Canillas wrote: > Failing to get the struct s5p_mfc_pm .clock is a non-fatal error so the > clock field can have a errno pointer value. But s5p_mfc_final_pm() only > checks if .clock is not NULL before attempting to unprepare and put it. >

[PATCH v10 5/8] [media] vcodec: mediatek: Add Mediatek V4L2 Video Encoder Driver

2016-05-03 Thread Tiffany Lin
Add v4l2 layer encoder driver for MT8173 Signed-off-by: Tiffany Lin --- drivers/media/platform/Kconfig | 18 + drivers/media/platform/Makefile|2 + drivers/media/platform/mtk-vcodec/Makefile | 14 +

Re: cron job: media_tree daily build: ERRORS

2016-05-03 Thread Hans Verkuil
On 05/03/2016 05:05 AM, Hans Verkuil wrote: > This message is generated daily by a cron job that builds media_tree for > the kernels and architectures in the list below. > > Results of the daily build of media_tree: > > date: Tue May 3 04:00:18 CEST 2016 > git branch: test > git hash: