cron job: media_tree daily build: ERRORS

2016-07-14 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: Fri Jul 15 04:00:11 CEST 2016 git branch: test git hash: 5cac1f67ea0363d463a58ec2d9118268fe2ba5d6 gcc

[RFC 00/16] Make use of kref in media device, grab references as needed

2016-07-14 Thread Sakari Ailus
Hi folks, I've been working on this for some time now but only got the full patchset working some moments ago. The patchset properly, I believe, fixes the issue of removing a device whilst streaming. Media device is refcounted and its memory is only released once the last reference is gone:

[RFC 06/16] media: Dynamically allocate the media device

2016-07-14 Thread Sakari Ailus
From: Sakari Ailus Allow allocating the media device dynamically. As the struct media_device embeds struct media_devnode, the lifetime of that object is that same than that of the media_device. Signed-off-by: Sakari Ailus ---

[RFC 12/16] media: Shuffle functions around

2016-07-14 Thread Sakari Ailus
As the call paths of the functions in question will change, move them around in anticipation of that. No other changes. Signed-off-by: Sakari Ailus --- drivers/media/media-device.c | 90 ++-- 1 file changed, 45 insertions(+),

[RFC 10/16] media: Document media device allocation API

2016-07-14 Thread Sakari Ailus
Document the addition of the media_device_alloc() function to allocate a media device. Also, document how reference counting and releasing a media device works. Deprecate API elements which are no longer needed with dynamically allocated media devices. Signed-off-by: Sakari Ailus

[RFC 07/16] media-device: struct media_device requires struct device

2016-07-14 Thread Sakari Ailus
The media device always has a device around. Require one as an argument for media_device_alloc(). Signed-off-by: Sakari Ailus --- drivers/media/media-device.c | 13 +++-- include/media/media-device.h | 4 +++- 2 files changed, 14 insertions(+), 3

[RFC 16/16] omap3isp: Don't rely on devm for memory resource management

2016-07-14 Thread Sakari Ailus
devm functions are fine for managing resources that are directly related to the device at hand and that have no other dependencies. However, a process holding a file handle to a device created by a driver for a device may result in the file handle left behind after the device is long gone. This

[RFC 13/16] media-device: Postpone graph object removal until free

2016-07-14 Thread Sakari Ailus
The media device itself will be unregistered based on it being unbound and driver's remove callback being called. The graph objects themselves may still be in use; rely on the kref release callback to release them. Signed-off-by: Sakari Ailus ---

[RFC 03/16] Revert "[media] media-device: dynamically allocate struct media_devnode"

2016-07-14 Thread Sakari Ailus
This reverts commit a087ce704b80 ("[media] media-device: dynamically allocate struct media_devnode"). The commit was part of an original patchset to avoid crashes when an unregistering device is in use. Signed-off-by: Sakari Ailus --- drivers/media/media-device.c

[RFC 01/16] Revert "[media] media: fix media devnode ioctl/syscall and unregister race"

2016-07-14 Thread Sakari Ailus
This reverts commit 6f0dd24a084a ("[media] media: fix media devnode ioctl/syscall and unregister race"). The commit was part of an original patchset to avoid crashes when an unregistering device is in use. Signed-off-by: Sakari Ailus ---

[RFC 08/16] media: Provide a way to the driver to set a private pointer

2016-07-14 Thread Sakari Ailus
Now that the media device can be allocated dynamically, drivers have no longer a way to conveniently obtain the driver private data structure. Provide one again in the form of a private pointer passed to the media_device_alloc() function. Signed-off-by: Sakari Ailus

[RFC 11/16] v4l: Acquire a reference to the media device for every video device

2016-07-14 Thread Sakari Ailus
The video device depends on the existence of its media device --- if there is one. Acquire a reference to it. Signed-off-by: Sakari Ailus --- drivers/media/v4l2-core/v4l2-dev.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git

[RFC 14/16] omap3isp: Allocate the media device dynamically

2016-07-14 Thread Sakari Ailus
Signed-off-by: Sakari Ailus --- drivers/media/platform/omap3isp/isp.c | 23 --- drivers/media/platform/omap3isp/isp.h | 2 +- drivers/media/platform/omap3isp/ispvideo.c | 2 +- 3 files changed, 14 insertions(+), 13 deletions(-) diff

[RFC 15/16] omap3isp: Release the isp device struct by media device callback

2016-07-14 Thread Sakari Ailus
Use the media device release callback to release the isp device's data structure. This approach has the benefit of not releasing memory which may still be accessed through open file handles whilst the isp driver is being unbound. Signed-off-by: Sakari Ailus ---

[RFC 09/16] media: Add release callback for media device

2016-07-14 Thread Sakari Ailus
The release callback may be used by the driver to signal the release of the media device. This makes it possible to embed a driver private struct to the same memory allocation. Signed-off-by: Sakari Ailus --- drivers/media/media-device.c | 11 ++-

[RFC 04/16] media: Remove useless curly braces and parentheses

2016-07-14 Thread Sakari Ailus
Signed-off-by: Sakari Ailus --- drivers/media/media-device.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/media/media-device.c b/drivers/media/media-device.c index a1cd50f..8bdc316 100644 --- a/drivers/media/media-device.c +++

[RFC 02/16] Revert "[media] media: fix use-after-free in cdev_put() when app exits after driver unbind"

2016-07-14 Thread Sakari Ailus
This reverts commit 5b28dde51d0c ("[media] media: fix use-after-free in cdev_put() when app exits after driver unbind"). The commit was part of an original patchset to avoid crashes when an unregistering device is in use. Signed-off-by: Sakari Ailus ---

Re: [PATCH 2/4] cxd2841er: provide signal strength for DVB-C

2016-07-14 Thread Abylay Ospan
Hi Mauro, Here is a mistake. Instead of 'cxd2841er_read_agc_gain_t_t2' should 'cxd2841er_read_agc_gain_c' be used for SYS_DVBC_ANNEX_ case. and should we use all DVB-C variants ( case SYS_DVBC_ANNEX_A, case SYS_DVBC_ANNEX_B, case SYS_DVBC_ANNEX_C ) here ? 2016-07-01 10:03 GMT-04:00 Mauro

Re: [PATCH 4/4] cxd2841er: adjust the dB scale for DVB-C

2016-07-14 Thread Abylay Ospan
Hello Mauro, Acked-by: Abylay Ospan I have checked values with reference signal from my modulator for DVB-C. Formula is working fine. Thanks ! 2016-07-01 10:03 GMT-04:00 Mauro Carvalho Chehab : > Instead of using a relative frequency range, calibrate

[PATCH] media: s5p-jpeg add missing blank lines after declarations

2016-07-14 Thread Shuah Khan
Missing blank lines after declarations are making it hard to read the code. Fix them and also fix other checkpatch warnings at the same time. Signed-off-by: Shuah Khan --- drivers/media/platform/s5p-jpeg/jpeg-core.c | 13 ++--- 1 file changed, 10 insertions(+),

Re: uvcvideo

2016-07-14 Thread Torbjorn Jansson
On 2016-07-14 18:30, Charles Stegall wrote: > On Thu, Jul 14, 2016 at 05:10:04PM +0200, Torbjorn Jansson wrote: >> On 2016-07-14 16:16, Charles Stegall wrote: >>> >>> this happens ... >>> >>> modprobe uvcvideo >>> modprobe: ERROR: could not insert 'uvcvideo': Exec format error >>> >> did you get

DRM device memory writeback (Mali-DP)

2016-07-14 Thread Brian Starkey
Hi, The Mali-DP display processors have a memory-writeback engine which can write the result of the composition (CRTC output) to a memory buffer in a variety of formats. We're looking for feedback/suggestions on how to expose this in the mali-dp DRM kernel driver - possibly via V4L2. We've got

[PATCH] [media] vb2: include length in dmabuf qbuf debug message

2016-07-14 Thread Javier Martinez Canillas
If the the VIDIOC_QBUF ioctl fails due a wrong dmabuf length, it's useful to get the invalid length as a debug information. Before this patch: vb2-core: __qbuf_dmabuf: invalid dmabuf length for plane 1 After this patch: vb2-core: __qbuf_dmabuf: invalid dmabuf length 221248 for plane 1

Re: uvcvideo

2016-07-14 Thread Torbjorn Jansson
On 2016-07-14 16:16, Charles Stegall wrote: this happens ... modprobe uvcvideo modprobe: ERROR: could not insert 'uvcvideo': Exec format error did you get any interesting output in dmesg ? like problem loading modules or symbol errors? this sounds a bit like a problem i had where dmesg

Re: [PATCH v2] media: s5p-mfc remove unnecessary error messages

2016-07-14 Thread Javier Martinez Canillas
Hello Shuah, On 07/14/2016 10:40 AM, Shuah Khan wrote: > Removed unnecessary error message as appropriate error code is returned. > Changed error message into a debug. > > Signed-off-by: Shuah Khan > --- Reviewed-by: Javier Martinez Canillas

[PATCH v2] media: s5p-mfc remove unnecessary error messages

2016-07-14 Thread Shuah Khan
Removed unnecessary error message as appropriate error code is returned. Changed error message into a debug. Signed-off-by: Shuah Khan --- Changes since v1: - Changed EBUSY error message to a debug message. drivers/media/platform/s5p-mfc/s5p_mfc.c | 3 +-- 1 file

uvcvideo

2016-07-14 Thread Charles Stegall
this happens ... modprobe uvcvideo modprobe: ERROR: could not insert 'uvcvideo': Exec format error and /dev/video0 is not created I followed, very carefully, several times, the directions on https://linuxtv.org/wiki/index.php?title=TBS_driver_installation=info kernel source 4.6.4 the

Re: [PATCH] media: s5p-mfc remove unnecessary error messages

2016-07-14 Thread Javier Martinez Canillas
Hello Shuah, On 07/14/2016 09:35 AM, Shuah Khan wrote: > On 07/14/2016 06:46 AM, Javier Martinez Canillas wrote: >> Hello Shuah, >> >> On 07/12/2016 08:33 PM, Shuah Khan wrote: >>> Removing unnecessary error messages as appropriate error code is returned. >>> >>> Signed-off-by: Shuah Khan

Re: [PATCH] media: s5p-mfc remove unnecessary error messages

2016-07-14 Thread Shuah Khan
On 07/14/2016 06:46 AM, Javier Martinez Canillas wrote: > Hello Shuah, > > On 07/12/2016 08:33 PM, Shuah Khan wrote: >> Removing unnecessary error messages as appropriate error code is returned. >> >> Signed-off-by: Shuah Khan >> --- >>

Re: [PATCH] media: s5p-mfc remove unnecessary error messages

2016-07-14 Thread Javier Martinez Canillas
Hello Shuah, On 07/12/2016 08:33 PM, Shuah Khan wrote: > Removing unnecessary error messages as appropriate error code is returned. > > Signed-off-by: Shuah Khan > --- > drivers/media/platform/s5p-mfc/s5p_mfc.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git

[PATCH 4/4] arm64: dts: mediatek: Add MDP for MT8173

2016-07-14 Thread Minghsiu Tsai
Add MDP node for MT8173 Signed-off-by: Minghsiu Tsai --- arch/arm64/boot/dts/mediatek/mt8173.dtsi | 80 ++ 1 file changed, 80 insertions(+) diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi

[PATCH 1/4] VPU: mediatek: Add mdp support

2016-07-14 Thread Minghsiu Tsai
VPU driver add mdp support Signed-off-by: Minghsiu Tsai --- drivers/media/platform/mtk-vpu/mtk_vpu.h |5 + 1 file changed, 5 insertions(+) diff --git a/drivers/media/platform/mtk-vpu/mtk_vpu.h b/drivers/media/platform/mtk-vpu/mtk_vpu.h index

[PATCH 2/4] dt-bindings: Add a binding for Mediatek MDP

2016-07-14 Thread Minghsiu Tsai
Add a DT binding documentation of MDP for the MT8173 SoC from Mediatek Signed-off-by: Minghsiu Tsai --- .../devicetree/bindings/media/mediatek-mdp.txt | 92 1 file changed, 92 insertions(+) create mode 100644

[PATCH 3/4] media: Add Mediatek MDP Driver

2016-07-14 Thread Minghsiu Tsai
Add MDP driver for MT8173 Signed-off-by: Minghsiu Tsai --- drivers/media/platform/Kconfig| 16 + drivers/media/platform/Makefile |2 + drivers/media/platform/mtk-mdp/Makefile |9 +

[PATCH 0/4] Add MT8173 MDP Driver

2016-07-14 Thread Minghsiu Tsai
== Introduction == The purpose of this series is to add the driver for Media Data Path HW embedded in the Mediatek's MT8173 SoC. MDP is used for scaling and color space conversion. It could convert V4L2_PIX_FMT_MT21 to V4L2_PIX_FMT_NV12M or V4L2_PIX_FMT_YUV420M.

hi linux

2016-07-14 Thread JULIAN GARDNER
hi linux http://mgmedi.com/community/images-/bill.php?ahead=gq1s60uqwct04k JULIAN GARDNER -- 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

[patch] [media] blackfin: check devm_pinctrl_get() for errors

2016-07-14 Thread Dan Carpenter
devm_pinctrl_get() can fail so we should check for that. Fixes: 0a6824bc10de ('[media] v4l2: blackfin: select proper pinctrl state in ppi_set_params if CONFIG_PINCTRL is enabled') Signed-off-by: Dan Carpenter diff --git a/drivers/media/platform/blackfin/ppi.c

Re: [PATCH 2/2] [media] doc-rst: increase depth of the main index

2016-07-14 Thread Markus Heiser
Hi Mauro, Am 13.07.2016 um 15:52 schrieb Mauro Carvalho Chehab : > It is useful to have an index with all the book contents somewhere, > as it makes easier to seek for something. So, increase maxdepth > to 5 for the main index at the beginning of the book. > > While

[PATCH] vivid: fix typo causing incorrect CEC physical addresses

2016-07-14 Thread Hans Verkuil
Fix typo in vivid that caused all HDMI outputs to have the same physical address. Signed-off-by: Hans Verkuil diff --git a/drivers/media/platform/vivid/vivid-core.c b/drivers/media/platform/vivid/vivid-core.c index 9966828..7f93713 100644 ---

Re: [PATCH] [media] mtk-vcodec: fix type mismatches

2016-07-14 Thread tiffany lin
On Mon, 2016-07-11 at 23:37 +0200, Arnd Bergmann wrote: > The newly added mtk-vcodec driver produces a number of warnings in an ARM > allmodconfig build, mainly since it assumes that dma_addr_t is 32-bit wide: > > mtk-vcodec/venc/venc_vp8_if.c: In function 'vp8_enc_alloc_work_buf': >

Re: [PATCH v3 3/9] DocBook/v4l: Add compressed video formats used on MT8173 codec driver

2016-07-14 Thread tiffany lin
Hi Nicolas, On Wed, 2016-07-13 at 09:55 -0400, Nicolas Dufresne wrote: > Le mercredi 13 juillet 2016 à 10:00 +0800, tiffany lin a écrit : > > Hi Nicolas, > > > > On Tue, 2016-07-12 at 15:14 -0400, Nicolas Dufresne wrote: > > > Le mardi 12 juillet 2016 à 15:08 -0400, Nicolas Dufresne a écrit : >

Re: [PATCH] vcodec: mediatek: Add g/s_selection support for V4L2 Encoder

2016-07-14 Thread tiffany lin
Hi Hans, On Mon, 2016-07-11 at 06:32 +0200, Hans Verkuil wrote: > Hi Tiffany, > > My apologies for the delay, but here is my review at last: > > On 05/30/2016 09:52 AM, Tiffany Lin wrote: > > This patch add g/s_selection support for MT8173 > > > > Signed-off-by: Tiffany Lin