Re: [PATCH 0/2] media: soc_camera: rcar_vin: Add r8a7794, r8a7793 device support

2014-10-16 Thread Yoshihiro Kaneko
2014-10-17 15:02 GMT+09:00 Simon Horman : > Hi Guennadi, > > On Thu, Oct 16, 2014 at 10:27:13PM +0200, Guennadi Liakhovetski wrote: >> Hello, >> >> Thanks for the patches. Could you please fold these two into one - they >> really don't deserve to be separated. > > Thanks. Kaneko-san could you squas

Re: [PATCH 0/2] media: soc_camera: rcar_vin: Add r8a7794, r8a7793 device support

2014-10-16 Thread Simon Horman
Hi Guennadi, On Thu, Oct 16, 2014 at 10:27:13PM +0200, Guennadi Liakhovetski wrote: > Hello, > > Thanks for the patches. Could you please fold these two into one - they > really don't deserve to be separated. Thanks. Kaneko-san could you squash these patches and repost? > As for your other ser

cron job: media_tree daily build: WARNINGS

2014-10-16 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 Oct 17 04:00:18 CEST 2014 git branch: test git hash: cf3167cf1e969b17671a4d3d956d22718a8ceb85 gcc versi

Re: GrabBee-HD

2014-10-16 Thread Steven Toth
Ok, no nobody jumped in the first time around. my turn I guess... :) Comments below. On Thu, Oct 16, 2014 at 5:18 PM, Dave Kimble wrote: > I have just bought an HDMI to USB-2.0 grabber called "GrabBee-HD". > http://www.greada.com/grabbeex-hd.html > Motherboard photo: http://www.davekimble.or

GrabBee-HD

2014-10-16 Thread Dave Kimble
I have just bought an HDMI to USB-2.0 grabber called "GrabBee-HD". http://www.greada.com/grabbeex-hd.html Motherboard photo: http://www.davekimble.org.au/computers/GrabBee-HD.jpg Inside it has chips labelled "Sigma PL330B-CPE3" and "iTE IT6604E". Note that it compresses the video with H.264 . I k

Re: [PATCH resend] [media] rc-core: fix protocol_change regression in ir_raw_event_register

2014-10-16 Thread David Härdeman
On Thu, Oct 09, 2014 at 09:30:36PM +0300, Tomas Melin wrote: >IR reciever using nuvoton-cir and lirc was not working anymore after >upgrade from kernel 3.16 to 3.17-rcX. >Bisected regression to commit da6e162d6a4607362f8478c715c797d84d449f8b >("[media] rc-core: simplify sysfs code"). > >The regress

Re: [PATCH 0/2] media: soc_camera: rcar_vin: Add r8a7794, r8a7793 device support

2014-10-16 Thread Guennadi Liakhovetski
Hello, Thanks for the patches. Could you please fold these two into one - they really don't deserve to be separated. As for your other series - patches there look mostly good - from just a formal review. If you don't mind, I'll adjust a couple of cosmetic issues like missing curly braces in

Re: [PATCH 03/12] [media] V4L2: mx3_camer: use dmaengine_pause() API

2014-10-16 Thread Guennadi Liakhovetski
On Sat, 11 Oct 2014, Vinod Koul wrote: > The drivers should use dmaengine_pause() API instead of > accessing the device_control which will be deprecated soon > > Signed-off-by: Vinod Koul Acked-by: Guennadi Liakhovetski Thanks Guennadi > --- > drivers/media/platform/soc_camera/mx3_camera.c

Re: [alsa-devel] [PATCH v2 5/6] sound/usb: pcm changes to use media token api

2014-10-16 Thread Shuah Khan
On 10/16/2014 08:48 AM, Takashi Iwai wrote: > At Thu, 16 Oct 2014 08:39:14 -0600, > Shuah Khan wrote: >> >> On 10/16/2014 08:16 AM, Takashi Iwai wrote: >>> At Thu, 16 Oct 2014 08:10:52 -0600, >>> Shuah Khan wrote: On 10/16/2014 08:01 AM, Takashi Iwai wrote: > At Thu, 16 Oct 2014 07:10

Re: [alsa-devel] [PATCH v2 5/6] sound/usb: pcm changes to use media token api

2014-10-16 Thread Takashi Iwai
At Thu, 16 Oct 2014 08:39:14 -0600, Shuah Khan wrote: > > On 10/16/2014 08:16 AM, Takashi Iwai wrote: > > At Thu, 16 Oct 2014 08:10:52 -0600, > > Shuah Khan wrote: > >> > >> On 10/16/2014 08:01 AM, Takashi Iwai wrote: > >>> At Thu, 16 Oct 2014 07:10:37 -0600, > >>> Shuah Khan wrote: > >

Re: [PATCH v2 1/6] media: add media token device resource framework

2014-10-16 Thread Shuah Khan
On 10/16/2014 08:00 AM, Takashi Iwai wrote: > At Wed, 15 Oct 2014 18:53:28 -0600, > Shuah Khan wrote: >> >> On 10/15/2014 11:05 AM, Takashi Iwai wrote: >> +#if defined(CONFIG_MEDIA_SUPPORT) +extern int media_tknres_create(struct device *dev); +extern int media_tknres_destroy(struct d

Re: [alsa-devel] [PATCH v2 5/6] sound/usb: pcm changes to use media token api

2014-10-16 Thread Shuah Khan
On 10/16/2014 08:16 AM, Takashi Iwai wrote: > At Thu, 16 Oct 2014 08:10:52 -0600, > Shuah Khan wrote: >> >> On 10/16/2014 08:01 AM, Takashi Iwai wrote: >>> At Thu, 16 Oct 2014 07:10:37 -0600, >>> Shuah Khan wrote: On 10/16/2014 06:00 AM, Lars-Peter Clausen wrote: > On 10/14/2014 04:58

Re: [alsa-devel] [PATCH v2 5/6] sound/usb: pcm changes to use media token api

2014-10-16 Thread Takashi Iwai
At Thu, 16 Oct 2014 08:10:52 -0600, Shuah Khan wrote: > > On 10/16/2014 08:01 AM, Takashi Iwai wrote: > > At Thu, 16 Oct 2014 07:10:37 -0600, > > Shuah Khan wrote: > >> > >> On 10/16/2014 06:00 AM, Lars-Peter Clausen wrote: > >>> On 10/14/2014 04:58 PM, Shuah Khan wrote: > >>> [...] > s

Re: [alsa-devel] [PATCH v2 5/6] sound/usb: pcm changes to use media token api

2014-10-16 Thread Shuah Khan
On 10/16/2014 08:01 AM, Takashi Iwai wrote: > At Thu, 16 Oct 2014 07:10:37 -0600, > Shuah Khan wrote: >> >> On 10/16/2014 06:00 AM, Lars-Peter Clausen wrote: >>> On 10/14/2014 04:58 PM, Shuah Khan wrote: >>> [...] switch (cmd) { case SNDRV_PCM_TRIGGER_START: +err

Re: [PATCH v2 0/6] media token resource framework

2014-10-16 Thread Takashi Iwai
At Wed, 15 Oct 2014 14:21:34 -0600, Shuah Khan wrote: > > On 10/15/2014 10:48 AM, Takashi Iwai wrote: > > At Tue, 14 Oct 2014 08:58:36 -0600, > > Shuah Khan wrote: > >> > >> Add media token device resource framework to allow sharing > >> resources such as tuner, dma, audio etc. across media driver

Re: [alsa-devel] [PATCH v2 5/6] sound/usb: pcm changes to use media token api

2014-10-16 Thread Takashi Iwai
At Thu, 16 Oct 2014 07:10:37 -0600, Shuah Khan wrote: > > On 10/16/2014 06:00 AM, Lars-Peter Clausen wrote: > > On 10/14/2014 04:58 PM, Shuah Khan wrote: > > [...] > >> switch (cmd) { > >> case SNDRV_PCM_TRIGGER_START: > >> +err = media_get_audio_tkn(&subs->dev->dev); > >> +

Re: [PATCH v2 1/6] media: add media token device resource framework

2014-10-16 Thread Takashi Iwai
At Wed, 15 Oct 2014 18:53:28 -0600, Shuah Khan wrote: > > On 10/15/2014 11:05 AM, Takashi Iwai wrote: > > >> +#if defined(CONFIG_MEDIA_SUPPORT) > >> +extern int media_tknres_create(struct device *dev); > >> +extern int media_tknres_destroy(struct device *dev); > >> + > >> +extern int media_get_tu

re: V4L/DVB (7535): saa717x: add new audio/video decoder i2c driver

2014-10-16 Thread Dan Carpenter
Hello Hans Verkuil, The patch fb7b37cf913c: "V4L/DVB (7535): saa717x: add new audio/video decoder i2c driver" from Apr 9, 2008, leads to the following static checker warning: drivers/media/i2c/saa717x.c:155 saa717x_read() warn: right shifting more than type allows drivers/media/i

Re: [alsa-devel] [PATCH v2 5/6] sound/usb: pcm changes to use media token api

2014-10-16 Thread Shuah Khan
On 10/16/2014 06:00 AM, Lars-Peter Clausen wrote: > On 10/14/2014 04:58 PM, Shuah Khan wrote: > [...] >> switch (cmd) { >> case SNDRV_PCM_TRIGGER_START: >> +err = media_get_audio_tkn(&subs->dev->dev); >> +if (err == -EBUSY) { >> +dev_info(&subs->dev->dev, "%s

Re: [alsa-devel] [PATCH v2 5/6] sound/usb: pcm changes to use media token api

2014-10-16 Thread Lars-Peter Clausen
On 10/14/2014 04:58 PM, Shuah Khan wrote: [...] switch (cmd) { case SNDRV_PCM_TRIGGER_START: + err = media_get_audio_tkn(&subs->dev->dev); + if (err == -EBUSY) { + dev_info(&subs->dev->dev, "%s device is busy\n", +

Re: [PATCH 1/2] [media] uvcvideo: add a new quirk UVC_QUIRK_SINGLE_ISO

2014-10-16 Thread John Crispin
Hi Laurent, On 11/10/2014 14:03, Laurent Pinchart wrote: > Hi John, > > On Friday 10 October 2014 20:41:12 John Crispin wrote: >> The following patch adds the usb ids for the iPassion chip. This chip is >> found on D-Link DIR-930 IP cameras. For them to work this patch needs to be >> applied. I am

Re: [patch] [media] usbvision-video: two use after frees

2014-10-16 Thread Laurent Pinchart
On Thursday 16 October 2014 10:23:05 Hans Verkuil wrote: > On 10/16/2014 10:09 AM, Laurent Pinchart wrote: > > On Thursday 16 October 2014 10:57:21 Dan Carpenter wrote: > >> The lock has been freed in usbvision_release() so there is no need to > >> call mutex_unlock() here. > > > > Yuck :-/ > > >

Re: [patch] [media] usbvision-video: two use after frees

2014-10-16 Thread Hans Verkuil
On 10/16/2014 10:09 AM, Laurent Pinchart wrote: > On Thursday 16 October 2014 10:57:21 Dan Carpenter wrote: >> The lock has been freed in usbvision_release() so there is no need to >> call mutex_unlock() here. > > Yuck :-/ > > The driver should really be converted to use video_device::release.

Re: [patch] [media] usbvision-video: two use after frees

2014-10-16 Thread Laurent Pinchart
On Thursday 16 October 2014 10:57:21 Dan Carpenter wrote: > The lock has been freed in usbvision_release() so there is no need to > call mutex_unlock() here. Yuck :-/ The driver should really be converted to use video_device::release. That might be out of scope for this fix though. Is usbvision

[patch] [media] usbvision-video: two use after frees

2014-10-16 Thread Dan Carpenter
The lock has been freed in usbvision_release() so there is no need to call mutex_unlock() here. Signed-off-by: Dan Carpenter diff --git a/drivers/media/usb/usbvision/usbvision-video.c b/drivers/media/usb/usbvision/usbvision-video.c index 68bc961..9bfa041 100644 --- a/drivers/media/usb/usbvision

Kernel 3.17.0 broke xc4000-based DTV1800h

2014-10-16 Thread Rodney Baker
Since installing kernel 3.17.0-1.gc467423-desktop (on openSuSE 13.1) my xc4000/zl10353/cx88 based DTV card has failed to initialise on boot. The following messages are from dmesg; [ 78.468221] xc4000: I2C read failed [ 80.074604] xc4000: I2C read failed [ 80.074605] Unable to read tuner r