[PATCH v3] [media] au8522: Avoid memory leak for device config data

2015-12-21 Thread Mauro Carvalho Chehab
As reported by kmemleak: unreferenced object 0x880321e1da40 (size 32): comm "modprobe", pid 3309, jiffies 4295019569 (age 2359.636s) hex dump (first 32 bytes): 47 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 G... 00 00 00 00 00

Raw ATSC Stream Capture

2015-12-21 Thread Thomás Inskip
Hi. I do not know whether this is the appropriate forum for my question, but as I cannot find a more suitable one, here it goes: I have successfully built and installed the drivers using the media_build distribution, as I have a slightly older kernel. The driver modules are loading correctly,

Re: Raw ATSC Stream Capture

2015-12-21 Thread Steven Toth
> on how to do this online. Could someone please send me some info on > how to do this, or at least point me in the right direction? > > Oh, I'm using a Hauppauge HDR-955Q tuner. v4l2-ctl isn't used for ATSC transmissions, its generally used for analog TV only. Take a look at the 'azap' and

Re: Raw ATSC Stream Capture

2015-12-21 Thread Thomás Inskip
Thank you. scan + azap did exactly what I needed. On Mon, Dec 21, 2015 at 10:31 AM, Steven Toth wrote: >> on how to do this online. Could someone please send me some info on >> how to do this, or at least point me in the right direction? >> >> Oh, I'm using a Hauppauge

[PATCH] media: i2c: ov2659: speedup probe if no device connected

2015-12-21 Thread Grygorii Strashko
The ov2659 driver performs device detection and initialization in the following way: - send reset command REG_SOFTWARE_RESET - load array of predefined register's setting (~150 values) - read device version REG_SC_CHIP_ID_H/REG_SC_CHIP_ID_L - check version and exit if invalid. As result, for

Re: Sabrent (stk1160) / Easycap driver problem

2015-12-21 Thread Ezequiel Garcia
Hi Philippe, On 20 December 2015 at 20:32, Philippe Desrochers wrote: > Hello Ezequiel, > > I tested with saa7115.c and the problem is in the "saa711x_detect_chip" > function. > In fact, the CJC7113 chip seems to returns all '1' when reading register 0. >

Re: [PATCH 1/3] rtl2832: add support for slave ts pid filter

2015-12-21 Thread Benjamin Larsson
On 12/21/2015 03:42 AM, Antti Palosaari wrote: Patch looks acceptable, but it is broken in a mean it does not apply :( I'll rebase on git://linuxtv.org/anttip/media_tree.git rtl2832u_pull. MvH Benjamin Larsson -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the

[PATCH] [media] uvcvideo: Only register media dev if MEDIA_CONTROLLER is defined

2015-12-21 Thread Javier Martinez Canillas
Commit 1590ad7b5271 ("[media] media-device: split media initialization and registration") split the media dev initialization and registration but introduced a build error since media_device_register() was called unconditionally even when the MEDIA_CONTROLLER config was not enabled: from

Re: [PATCH 4/5] [media] au0828-core: fix compilation when !CONFIG_MEDIA_CONTROLLER

2015-12-21 Thread Javier Martinez Canillas
Hello Mauro, On 12/16/2015 02:11 PM, Mauro Carvalho Chehab wrote: > commit 1590ad7b52714 ("[media] media-device: split media initialization > and registration") moved the media controller register to a > separate function. That caused the following compilation issue, > if

Re: [PATCH v3 00/23] Unrestricted media entity ID range support

2015-12-21 Thread Javier Martinez Canillas
On 12/16/2015 02:39 PM, Mauro Carvalho Chehab wrote: > Em Wed, 16 Dec 2015 16:03:01 +0200 > Sakari Ailus escreveu: > >> Hi Javier, >> >> On Wed, Dec 16, 2015 at 03:32:15PM +0200, Sakari Ailus wrote: >>> This is the third version of the unrestricted media entity ID range >>>

Re: [PATCH v2] [media] media-device: handle errors at media_device_init()

2015-12-21 Thread Javier Martinez Canillas
Hello Mauro, On 12/15/2015 09:22 AM, Mauro Carvalho Chehab wrote: > Changeset 43ac4401dca9 ("[media] media-device: split media > initialization and registration") broke media device register > into two separate functions, but introduced a BUG_ON() and > made media_device_init() void. It also

Re: [PATCH 2/2] [media] media-device: split media initialization and registration

2015-12-21 Thread Javier Martinez Canillas
Hello Mauro, On 12/15/2015 08:13 AM, Mauro Carvalho Chehab wrote: [snip] >>> >>> /** >>> - * media_device_register - register a media device >>> + * media_device_init() - initialize a media device >>> * @mdev: The media device >>> * >>> * The caller is responsible for initializing the

cron job: media_tree daily build: OK

2015-12-21 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: Tue Dec 22 04:00:13 CET 2015 git branch: test git hash: 0aff8a894a2be4c22e6414db33061153a4b35bc9 gcc

[linux-next:master 6315/7948] drivers/media/usb/cx231xx/cx231xx-cards.c:1741:36: error: 'struct cx231xx' has no member named 'media_dev'

2015-12-21 Thread kbuild test robot
Hi Javier, FYI, the error/warning still remains. tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: 6fadd3a85f4381c065ca2accbedaa622f83697ad commit: 1590ad7b52714fddc958189103c95541b49b1dae [6315/7948] [media] media-device: split media initialization and

Re: [PATCH 2/3] mn88472: add work around for failing firmware loading

2015-12-21 Thread Benjamin Larsson
On 12/21/2015 03:52 AM, Antti Palosaari wrote: Hello I am not sure if problem is I2C adapter/bus or that demodulator I2C slave. If it is demod, then that workaround is correct place, but if it is not, then that is wrong and I2C adapter repeating logic should be used. I did some testing again...

Re: [PATCH v5 0/3] [media] Fix race between graph enumeration and entities registration

2015-12-21 Thread Javier Martinez Canillas
Hello Mauro, On 12/12/2015 10:50 AM, Mauro Carvalho Chehab wrote: > Em Fri, 11 Dec 2015 19:57:06 -0300 > Javier Martinez Canillas escreveu: > >> Hello, >> >> This series fixes the issue of media device nodes being registered before >> all the media entities and pads

[PATCH] [media] rc: sunxi-cir: Initialize the spinlock properly

2015-12-21 Thread Chen-Yu Tsai
The driver allocates the spinlock but fails to initialize it correctly. The kernel reports a BUG indicating bad spinlock magic when spinlock debugging is enabled. Call spin_lock_init() on it to initialize it correctly. Fixes: b4e3e59fb59c ("[media] rc: add sunxi-ir driver") Signed-off-by:

[PATCH] au8522: Avoid memory leak in case of errors

2015-12-21 Thread Mauro Carvalho Chehab
As reported by kmemleak: unreferenced object 0x880321e1da40 (size 32): comm "modprobe", pid 3309, jiffies 4295019569 (age 2359.636s) hex dump (first 32 bytes): 47 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 G... 00 00 00 00 00

Re: [PATCH 2/2] [media] media-device: better lock media_device_unregister()

2015-12-21 Thread Javier Martinez Canillas
Hello Mauro, On 12/15/2015 07:43 AM, Mauro Carvalho Chehab wrote: > If media_device_unregister() is called by two different > drivers, a race condition may happen, as the check if the > device is not registered is not protected. > > Move the spin_lock() to happen earlier in the function, in

Re: [PATCH v5 1/3] [media] media-device: check before unregister if mdev was registered

2015-12-21 Thread Javier Martinez Canillas
Hello Mauro, On 12/15/2015 07:41 AM, Mauro Carvalho Chehab wrote: > Em Fri, 11 Dec 2015 16:36:27 -0700 > Shuah Khan escreveu: > >> On 12/11/2015 03:57 PM, Javier Martinez Canillas wrote: >>> Most media functions that unregister, check if the corresponding register >>>

Re: [PATCH 1/2] [media] media-device: move media entity register/unregister functions

2015-12-21 Thread Javier Martinez Canillas
Hello Mauro, On 12/15/2015 07:43 AM, Mauro Carvalho Chehab wrote: > media entity register and unregister functions are called by media > device register/unregister. Move them to occur earlier, as we'll need > an unlocked version of media_device_entity_unregister() and we don't > want to add a

Re: [v4l-utils PATCH 1/1] Allow building static binaries

2015-12-21 Thread Sakari Ailus
Hi Gregor, On Wed, Dec 16, 2015 at 09:21:16PM +0100, Gregor Jasny wrote: > Hello, > > On 10/12/15 14:21, Sakari Ailus wrote: > > I discussed with Hans and he thought you'd be the best person to take a look > > at this. > > > > The case is that I'd like to build static binaries and that doesn't

[PATCH v2] [media] au8522: Avoid memory leak for device config data

2015-12-21 Thread Mauro Carvalho Chehab
As reported by kmemleak: unreferenced object 0x880321e1da40 (size 32): comm "modprobe", pid 3309, jiffies 4295019569 (age 2359.636s) hex dump (first 32 bytes): 47 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 G... 00 00 00 00 00

[PATCH] [media] cx231xx: fix compilation when !CONFIG_MEDIA_CONTROLLER

2015-12-21 Thread Javier Martinez Canillas
Commit 1590ad7b5271 ("[media] media-device: split media initialization and registration") split the media dev initialization and registration but introduced a build error since media_device_register() was called unconditionally even when the MEDIA_CONTROLLER config was not enabled:

[PATCH] [media] ir-lirc-codec.c: don't leak lirc->drv-rbuf

2015-12-21 Thread Mauro Carvalho Chehab
As reported by kmemleak: unreferenced object 0x8802adae0ba0 (size 192): comm "modprobe", pid 3024, jiffies 4296503588 (age 324.368s) hex dump (first 32 bytes): 00 00 00 00 ad 4e ad de ff ff ff ff 00 00 00 00 .N.. ff ff ff ff ff

Re: [PATCH] usbtv: discard redundant video fields

2015-12-21 Thread Lubomir Rintel
On Sun, 2015-12-20 at 12:57 +0100, Nikola Forró wrote: > There are many dropped fields with some sources, leading to many > redundant fields without counterparts. When this redundant field > is odd, a new frame is pushed containing this odd field interleaved > with whatever was left in the buffer,

Re: [PATCH v2] [media] au8522: Avoid memory leak for device config data

2015-12-21 Thread kbuild test robot
Hi Mauro, [auto build test WARNING on linuxtv-media/master] [also build test WARNING on v4.4-rc6 next-20151221] url: https://github.com/0day-ci/linux/commits/Mauro-Carvalho-Chehab/au8522-Avoid-memory-leak-for-device-config-data/20151222-010649 base: git://linuxtv.org/media_tree.git master