[PATCH] sound/usb: fix to release stream resources from media_snd_device_delete()

2016-03-18 Thread Shuah Khan
Fix to release stream resources from media_snd_device_delete() before media device is unregistered. Without this change, stream resource free is attempted after the media device is unregistered which would result in use-after-free errors. Signed-off-by: Shuah Khan --- - Ran bind/unbind loop (100

Re: [PATCH v2] [media] media-device: use kref for media_device instance

2016-03-18 Thread Shuah Khan
On 03/18/2016 06:42 PM, Mauro Carvalho Chehab wrote: > Now that the media_device can be used by multiple drivers, > via devres, we need to be sure that it will be dropped only > when all drivers stop using it. > > Signed-off-by: Mauro Carvalho Chehab Looks good. Tested it doing bind/unbind, rmmo

Re: [PATCHv13 01/17] dts: exynos4*: add HDMI CEC pin definition to pinctrl

2016-03-18 Thread Krzysztof Kozlowski
On Fri, Mar 18, 2016 at 03:07:00PM +0100, Hans Verkuil wrote: > From: Kamil Debski > > Add pinctrl nodes for the HDMI CEC device to the Exynos4210 and > Exynos4x12 SoCs. These are required by the HDMI CEC device. > > Signed-off-by: Kamil Debski > Signed-off-by: Hans Verkuil > Acked-by: Krzyszt

Re: [PATCH] sound/usb: fix to release stream resources from media_snd_device_delete()

2016-03-18 Thread Mauro Carvalho Chehab
Em Fri, 18 Mar 2016 20:50:31 -0600 Shuah Khan escreveu: > Fix to release stream resources from media_snd_device_delete() before > media device is unregistered. Without this change, stream resource free > is attempted after the media device is unregistered which would result > in use-after-free er

[PATCH v2] [media] media-device: use kref for media_device instance

2016-03-18 Thread Mauro Carvalho Chehab
Now that the media_device can be used by multiple drivers, via devres, we need to be sure that it will be dropped only when all drivers stop using it. Signed-off-by: Mauro Carvalho Chehab --- v2: The kref is now used only when media_device is allocated via the media_device*_devress. This wa

media: rc: make raw event fifo size a module parameter

2016-03-18 Thread Heiner Kallweit
Currently the fifo size is 512 elements. After a recent patch the size of struct ir_raw_event is down to 8 bytes, so the fifo still consumes 4KB. In most cases a much smaller fifo is sufficient, e.g. nuvoton-cir triggers event processing after 24 events latest. However the needed fifo size may also

cron job: media_tree daily build: ERRORS

2016-03-18 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: Sat Mar 19 04:00:36 CET 2016 git branch: test git hash: c2f555fe3ce4515a8bf1393af5b9a48ce60ebbfe gcc versio

[PATCHv13 15/17] cobalt: add cec support

2016-03-18 Thread Hans Verkuil
Add CEC support to the cobalt driver. Signed-off-by: Hans Verkuil --- drivers/media/pci/cobalt/Kconfig | 1 + drivers/media/pci/cobalt/cobalt-driver.c | 115 +++- drivers/media/pci/cobalt/cobalt-driver.h | 2 + drivers/media/pci/cobalt/cobalt-irq.c| 3 +

Re: [PATCH 5/5] [media] media-device: make media_device_cleanup() static

2016-03-18 Thread Javier Martinez Canillas
Hello Mauro, The patch looks mostly good to me, I just have a question below: On Wed, Mar 16, 2016 at 9:04 AM, Mauro Carvalho Chehab wrote: [snip] > > -void media_device_cleanup(struct media_device *mdev) > +static void media_device_cleanup(struct media_device *mdev) > { > ida_destroy

Re: [PATCH 4/5] [media] media-device: use kref for media_device instance

2016-03-18 Thread Shuah Khan
On 03/16/2016 06:04 AM, Mauro Carvalho Chehab wrote: > Now that the media_device can be used by multiple drivers, > via devres, we need to be sure that it will be dropped only > when all drivers stop using it. > > Signed-off-by: Mauro Carvalho Chehab > --- > drivers/media/media-device.c | 48 >

[PATCH] [media] m88ds3103: fix undefined division

2016-03-18 Thread Peter Rosin
From: Peter Rosin s32tmp in the below code may be negative, and dev->mclk_khz is an unsigned type. s32tmp = 0x1 * (tuner_frequency - c->frequency); s32tmp = DIV_ROUND_CLOSEST(s32tmp, dev->mclk_khz); This is undefined, as DIV_ROUND_CLOSEST is undefined for negative dividends

[PATCH] az6027: Add support for Elgato EyeTV Sat v3

2016-03-18 Thread Olli Salonen
Another version of Elgato EyeTV Sat USB DVB-S2 adapter needs just a USB ID addition. Signed-off-by: Christian Knippel Reported-by: Olli Salonen --- drivers/media/dvb-core/dvb-usb-ids.h | 1 + drivers/media/usb/dvb-usb/az6027.c | 7 ++- 2 files changed, 7 insertions(+), 1 deletion(-) dif

[PATCHv13 04/17] input.h: add BUS_CEC type

2016-03-18 Thread Hans Verkuil
Inputs can come in over the HDMI CEC bus, so add a new type for this. Signed-off-by: Hans Verkuil Acked-by: Dmitry Torokhov --- include/uapi/linux/input.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/uapi/linux/input.h b/include/uapi/linux/input.h index 2758687..4f8c813 100644 --

Re: [PATCH] media: rc: reduce size of struct ir_raw_event

2016-03-18 Thread Heiner Kallweit
Am 16.03.2016 um 23:28 schrieb Sean Young: > On Wed, Mar 16, 2016 at 10:18:38PM +0100, Heiner Kallweit wrote: >> struct ir_raw_event currently has a size of 12 bytes on most (all?) >> architectures. This can be reduced to 8 bytes whilst maintaining >> full backwards compatibility. >> This saves 2KB

Re: [PATCH v4 0/2] media: soc_camera: rcar_vin: add fallback and r8a7792 bindings

2016-03-18 Thread Laurent Pinchart
Hi Simon, Thank you for the patches. For the whole series, Acked-by: Laurent Pinchart On Tuesday 15 March 2016 09:40:25 Simon Horman wrote: > Hi, > > this short series adds add fallback and r8a7792 bindings to rcar_vin. > > Based on media-tree/master > > Changes since v3: > * Add Acks > * C

[PATCH v2] media: rc: reduce size of struct ir_raw_event

2016-03-18 Thread Heiner Kallweit
struct ir_raw_event currently has a size of 12 bytes on most (all?) architectures. This can be reduced to 8 bytes whilst maintaining full backwards compatibility. This saves 2KB in size of struct ir_raw_event_ctrl (as element kfifo is reduced by 512 * 4 bytes) and it allows to copy the full struct