Re: [PATCH v5 1/6] [media] rc-main: assign driver type during allocation

2016-12-15 Thread kbuild test robot
Hi Andi, [auto build test ERROR on linuxtv-media/master] [cannot apply to v4.9 next-20161215] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Andi-Shyti/Add-support-for-IR-transmitters/20161216

[PATCH v5 4/6] [media] rc-ir-raw: do not generate any receiving thread for raw transmitters

2016-12-15 Thread Andi Shyti
Raw IR transmitters do not need any thread listening for occurring events. Check the driver type before running the thread. Signed-off-by: Andi Shyti Reviewed-by: Sean Young --- drivers/media/rc/rc-ir-raw.c | 17 - 1 file changed, 12

[PATCH v5 5/6] Documentation: bindings: add documentation for ir-spi device driver

2016-12-15 Thread Andi Shyti
Document the ir-spi driver's binding which is a IR led driven through the SPI line. Signed-off-by: Andi Shyti Reviewed-by: Sean Young --- .../devicetree/bindings/leds/irled/spi-ir-led.txt | 29 ++ 1 file changed, 29 insertions(+)

[PATCH v5 6/6] [media] rc: add support for IR LEDs driven through SPI

2016-12-15 Thread Andi Shyti
The ir-spi is a simple device driver which supports the connection between an IR LED and the MOSI line of an SPI device. The driver, indeed, uses the SPI framework to stream the raw data provided by userspace through an rc character device. The chardev is handled by the LIRC framework and its

[PATCH v5 2/6] [media] rc-main: split setup and unregister functions

2016-12-15 Thread Andi Shyti
Move the input device allocation, map and protocol handling to different functions. Signed-off-by: Andi Shyti Reviewed-by: Sean Young --- drivers/media/rc/rc-main.c | 143 + 1 file changed, 81 insertions(+), 62

[PATCH v5 0/6] Add support for IR transmitters

2016-12-15 Thread Andi Shyti
Hi, The main goal is to add support in the rc framework for IR transmitters, which currently is only supported by lirc but that is not the preferred way. The last patch adds support for an IR transmitter driven by the MOSI line of an SPI controller, it's the case of the Samsung TM2(e) board

[PATCH v5 1/6] [media] rc-main: assign driver type during allocation

2016-12-15 Thread Andi Shyti
The driver type can be assigned immediately when an RC device requests to the framework to allocate the device. This is an 'enum rc_driver_type' data type and specifies whether the device is a raw receiver or scancode receiver. The type will be given as parameter to the rc_allocate_device device.

[PATCH v5 3/6] [media] rc-core: add support for IR raw transmitters

2016-12-15 Thread Andi Shyti
IR raw transmitter driver type is specified in the enum rc_driver_type as RC_DRIVER_IR_RAW_TX which includes all those devices that transmit raw stream of bit to a receiver. The data are provided by userspace applications, therefore they don't need any input device allocation, but still they need

cron job: media_tree daily build: WARNINGS

2016-12-15 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 Dec 16 05:00:26 CET 2016 media-tree git hash:d183e4efcae8d88a2f252e546978658ca6d273cc media_build

[PATCH] vivid: Enable 4k resolution for webcam capture device

2016-12-15 Thread Soren Brinkmann
Add 3840x2160 as valid resolution for the webcam capture input and adjust the webcam intervals accordingly. Signed-off-by: Soren Brinkmann --- Hi, we'd like to use the vivid webcam capture device with a 4k resolution. This basically seems to do the trick, though, I'm

[RFC v2 11/11] vb2: dma-contig: Add WARN_ON_ONCE() to check for potential bugs

2016-12-15 Thread Laurent Pinchart
From: Sakari Ailus The scatterlist should always be present when the cache would need to be flushed. Each buffer type has its own means to provide that. Add WARN_ON_ONCE() to check the scatterist exists. Signed-off-by: Sakari Ailus

[RFC v2 08/11] vb2: dma-contig: Don't warn on failure in obtaining scatterlist

2016-12-15 Thread Laurent Pinchart
From: Sakari Ailus vb2_dc_get_base_sgt() which obtains the scatterlist already prints information on why the scatterlist could not be obtained. Also, remove the useless warning of a failed kmalloc(). Signed-off-by: Sakari Ailus

[RFC v2 04/11] v4l: Unify cache management hint buffer flags

2016-12-15 Thread Laurent Pinchart
From: Sakari Ailus The V4L2_BUF_FLAG_NO_CACHE_INVALIDATE and V4L2_BUF_FLAG_NO_CACHE_CLEAN buffer flags are currently not used by the kernel. Replace the definitions by a single V4L2_BUF_FLAG_NO_CACHE_SYNC flag to be used by further patches. Different cache

[RFC v2 10/11] vb2: dma-contig: Let drivers decide DMA attrs of MMAP and USERPTR bufs

2016-12-15 Thread Laurent Pinchart
From: Sakari Ailus The desirable DMA attributes are not generic for all devices using Videobuf2 contiguous DMA ops. Let the drivers decide. This change also results in MMAP buffers always having an sg_table (dma_sgt field). Also arrange the header files

[RFC v2 09/11] vb2: dma-contig: Move vb2_dc_get_base_sgt() up

2016-12-15 Thread Laurent Pinchart
From: Sakari Ailus Just move the function up. It'll be soon needed earlier than previously. Signed-off-by: Sakari Ailus Reviewed-by: Laurent Pinchart ---

[RFC v2 02/11] vb2: Move buffer cache synchronisation to prepare from queue

2016-12-15 Thread Laurent Pinchart
From: Sakari Ailus The buffer cache should be synchronised in buffer preparation, not when the buffer is queued to the device. Fix this. Mmap buffers do not need cache synchronisation since they are always coherent. Signed-off-by: Sakari Ailus

[RFC v2 05/11] v4l2-core: Don't sync cache for a buffer if so requested

2016-12-15 Thread Laurent Pinchart
From: Samu Onkalo The user may request to the driver (vb2) to skip the cache maintenance operations in case the buffer does not need cache synchronisation, e.g. in cases where the buffer is passed between hardware blocks without it being touched by the CPU. Also document

[RFC v2 07/11] vb2: dma-contig: Remove redundant sgt_base field

2016-12-15 Thread Laurent Pinchart
From: Sakari Ailus The struct vb2_dc_buf contains two struct sg_table fields: sgt_base and dma_sgt. The former is used by DMA-BUF buffers whereas the latter is used by USERPTR. Unify the two, leaving dma_sgt. MMAP buffers do not need cache flushing since they have

[RFC v2 06/11] vb2: Improve struct vb2_mem_ops documentation; alloc and put are for MMAP

2016-12-15 Thread Laurent Pinchart
From: Sakari Ailus The alloc() and put() ops are for MMAP buffers only. Document it. Signed-off-by: Sakari Ailus Acked-by: Hans Verkuil Reviewed-by: Laurent Pinchart ---

[RFC v2 01/11] vb2: Rename confusingly named internal buffer preparation functions

2016-12-15 Thread Laurent Pinchart
From: Sakari Ailus Rename __qbuf_*() functions which are specific to a buffer type as __prepare_*() which matches with what they do. The naming was there for historical reasons; the purpose of the functions was changed without renaming them. Signed-off-by: Sakari

[RFC v2 03/11] vb2: Move cache synchronisation from buffer done to dqbuf handler

2016-12-15 Thread Laurent Pinchart
From: Sakari Ailus The cache synchronisation may be a time consuming operation and thus not best performed in an interrupt which is a typical context for vb2_buffer_done() calls. This may consume up to tens of ms on some machines, depending on the buffer size.

[RFC v2 00/11] vb2: Handle user cache hints, allow drivers to choose cache coherency

2016-12-15 Thread Laurent Pinchart
Hello, This is a rebased version of the vb2 cache hints support patch series posted by Sakari more than a year ago. The patches have been modified as needed by the upstream changes and received the occasional small odd fix but are otherwise not modified. Please see the individual commit messages

Re: [RFC RESEND 03/11] vb2: Move cache synchronisation from buffer done to dqbuf handler

2016-12-15 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Tuesday 15 Sep 2015 10:51:10 Sakari Ailus wrote: > Hans Verkuil wrote: > > On 09/11/2015 01:50 PM, Sakari Ailus wrote: > >> The cache synchronisation may be a time consuming operation and thus not > >> best performed in an interrupt which is a typical

Re: [PATCH 2/2] media: omap3isp change to devm for resources

2016-12-15 Thread Laurent Pinchart
Hi Shuah, On Thursday 15 Dec 2016 15:51:41 Shuah Khan wrote: > On 12/15/2016 03:33 PM, Laurent Pinchart wrote: > > Hi Shuah, > > > > Thank you for the patch. > > > > Sakari has submitted a similar patch as part of his kref series. Please > > use it as a base point and rework it if you want to

Re: [PATCH 1/2] media: omap3isp fix media_entity_cleanup() after media device unregister

2016-12-15 Thread Shuah Khan
On 12/15/2016 03:32 PM, Laurent Pinchart wrote: > Hello Shuah, > > Thank you for the patch. > > On Thursday 15 Dec 2016 12:40:07 Shuah Khan wrote: >> During unbind isp_remove() media_entity_cleanup() after it unregisters the > > I assume you meant "During unbind isp_remove() calls

Re: [PATCH 2/2] media: omap3isp change to devm for resources

2016-12-15 Thread Shuah Khan
On 12/15/2016 03:33 PM, Laurent Pinchart wrote: > Hi Shuah, > > Thank you for the patch. > > Sakari has submitted a similar patch as part of his kref series. Please use > it > as a base point and rework it if you want to get it merged separately. I've > reviewed the patch and left quite a few

Re: [PATCH 2/2] media: omap3isp change to devm for resources

2016-12-15 Thread Laurent Pinchart
Hi Shuah, Thank you for the patch. Sakari has submitted a similar patch as part of his kref series. Please use it as a base point and rework it if you want to get it merged separately. I've reviewed the patch and left quite a few comments that need to be addressed. On Thursday 15 Dec 2016

[PATCH v4 3/3] stk1160: Wait for completion of transfers to and from AC97 codec.

2016-12-15 Thread Marcel Hasler
The STK1160 needs some time to transfer data to and from the AC97 codec. The transfer completion is indicated by command read/write bits in the chip's audio control register. The driver should poll these bits and wait until they have been cleared by hardware before trying to retrive the results

Re: [PATCH 1/2] media: omap3isp fix media_entity_cleanup() after media device unregister

2016-12-15 Thread Laurent Pinchart
Hello Shuah, Thank you for the patch. On Thursday 15 Dec 2016 12:40:07 Shuah Khan wrote: > During unbind isp_remove() media_entity_cleanup() after it unregisters the I assume you meant "During unbind isp_remove() calls media_entity_cleanup()" > media_device. Cleanup routine calls

Re: [PATCH v4 1/3] stk1160: Remove stk1160-mixer and setup internal AC97 codec

2016-12-15 Thread Marcel Hasler
Sorry, somehow the email header got messed up. Ignore this one. 2016-12-15 23:13 GMT+01:00 Marcel Hasler : > automatically. > Reply-To: > In-Reply-To: <20161215221146.GA9398@arch-desktop> > > Exposing all the channels of the device's internal AC97 codec to userspace is >

[PATCH v4 1/3] stk1160: Remove stk1160-mixer and setup internal AC97 codec automatically.

2016-12-15 Thread Marcel Hasler
Exposing all the channels of the device's internal AC97 codec to userspace is unnecessary and confusing. Instead the driver should setup the codec with proper values. This patch removes the mixer and sets up the codec using optimal values, i.e. the same values set by the Windows driver. This

[PATCH v4 2/3] stk1160: Check whether to use AC97 codec.

2016-12-15 Thread Marcel Hasler
Some STK1160-based devices use the chip's internal 8-bit ADC. This is configured through a strap pin. The value of this and other pins can be read through the POSVA register. If the internal ADC is used, or if audio is disabled altogether, there's no point trying to setup the AC97 codec.

[PATCH v4 1/3] stk1160: Remove stk1160-mixer and setup internal AC97 codec

2016-12-15 Thread Marcel Hasler
automatically. Reply-To: In-Reply-To: <20161215221146.GA9398@arch-desktop> Exposing all the channels of the device's internal AC97 codec to userspace is unnecessary and confusing. Instead the driver should setup the codec with proper values. This patch removes the mixer and sets up the codec

[PATCH v4 0/3] stk1160: Let the driver setup the device's internal AC97 codec

2016-12-15 Thread Marcel Hasler
This patchset is a result of my attempt to fix a bug (https://bugzilla.kernel.org/show_bug.cgi?id=180071) that eventually turned out to be caused by a missing quirk in snd-usb-audio. My idea was to remove the AC97 interface and setup the codec using the same values and in the same order as the

Re: [RFC RESEND 11/11] vb2: dma-contig: Add WARN_ON_ONCE() to check for potential bugs

2016-12-15 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 11 Sep 2015 14:50:34 Sakari Ailus wrote: > The scatterlist should always be present when the cache would need to be > flushed. Each buffer type has its own means to provide that. Add > WARN_ON_ONCE() to check the scatterist exists. Do you think such

[PATCH] [media] pctv452e: move buffer to heap, no mutex

2016-12-15 Thread Max Kellermann
Revert part of commit 73d5c5c864f4094, but move the buffer to the heap (DMA capable), not to the stack (may not be DMA capable). Instead of sharing one buffer which needs mutex protection, do a new heap allocation for each call. This fixes a NULL pointer dereference which occurs when

Re: [RFC RESEND 10/11] vb2: dma-contig: Let drivers decide DMA attrs of MMAP and USERPTR bufs

2016-12-15 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 11 Sep 2015 14:50:33 Sakari Ailus wrote: > The desirable DMA attributes are not generic for all devices using > Videobuf2 contiguous DMA ops. Let the drivers decide. > > This change also results in MMAP buffers always having an sg_table > (dma_sgt

Re: [RFC RESEND 09/11] vb2: dma-contig: Don't warn on failure in obtaining scatterlist

2016-12-15 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 11 Sep 2015 14:50:32 Sakari Ailus wrote: > vb2_dc_get_base_sgt() which obtains the scatterlist already prints > information on why the scatterlist could not be obtained. > > Also, remove the useless warning of a failed kmalloc(). > > Signed-off-by:

Re: [RFC RESEND 08/11] vb2: dma-contig: Move vb2_dc_get_base_sgt() up

2016-12-15 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 11 Sep 2015 14:50:31 Sakari Ailus wrote: > Just move the function up. It'll be soon needed earlier than previously. > > Signed-off-by: Sakari Ailus Reviewed-by: Laurent Pinchart I

Re: [RFC RESEND 07/11] vb2: dma-contig: Remove redundant sgt_base field

2016-12-15 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 11 Sep 2015 14:50:30 Sakari Ailus wrote: > The struct vb2_dc_buf contains two struct sg_table fields: sgt_base and > dma_sgt. The former is used by DMA-BUF buffers whereas the latter is used > by USERPTR. > > Unify the two, leaving dma_sgt. Looks

Re: [RFC RESEND 06/11] vb2: Improve struct vb2_mem_ops documentation; alloc and put are for MMAP

2016-12-15 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 11 Sep 2015 14:50:29 Sakari Ailus wrote: > The alloc() and put() ops are for MMAP buffers only. Document it. > > Signed-off-by: Sakari Ailus > --- > include/media/videobuf2-core.h | 20 ++-- > 1 file

Re: [RFC RESEND 05/11] v4l2-core: Don't sync cache for a buffer if so requested

2016-12-15 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 11 Sep 2015 14:50:28 Sakari Ailus wrote: > From: Samu Onkalo > > The user may request to the driver (vb2) to skip the cache maintenance > operations in case the buffer does not need cache synchronisation, e.g. in > cases

Re: [RFC RESEND 04/11] v4l: Unify cache management hint buffer flags

2016-12-15 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 11 Sep 2015 14:50:27 Sakari Ailus wrote: > The V4L2_BUF_FLAG_NO_CACHE_INVALIDATE and V4L2_BUF_FLAG_NO_CACHE_CLEAN > buffer flags are currently not used by the kernel. Replace the definitions > by a single V4L2_BUF_FLAG_NO_CACHE_SYNC flag to be used

[PATCH 0/2] omap3 devm usage removal

2016-12-15 Thread Shuah Khan
This patch series contains two patches. The first one removes calls to media_entity_cleanup() after media device has been unregistered. The second one removes devm usage. Shuah Khan (2): media: omap3isp fix media_entity_cleanup() after media device unregister media: omap3isp change to

[PATCH 2/2] media: omap3isp change to devm for resources

2016-12-15 Thread Shuah Khan
Using devm resources that have external dependencies such as a dev for a file handler could result in devm resources getting released durin unbind while an application has the file open holding pointer to the devm resource. This results in use-after-free errors when the application exits.

[PATCH 1/2] media: omap3isp fix media_entity_cleanup() after media device unregister

2016-12-15 Thread Shuah Khan
During unbind isp_remove() media_entity_cleanup() after it unregisters the media_device. Cleanup routine calls media_entity_cleanup() accessing subdev entities that have been removed. This will cause problems during unbind. Signed-off-by: Shuah Khan ---

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

2016-12-15 Thread Shuah Khan
Hi Skarai, On 12/15/2016 04:57 AM, Laurent Pinchart wrote: > On Thursday 15 Dec 2016 13:45:25 Sakari Ailus wrote: >> Hi Laurent, >> >> On 12/15/16 13:42, Laurent Pinchart wrote: >>> You can split that part out. The devm_* removal is independent and could >>> be moved to the beginning of the

Re: [RFC v3 00/21] Make use of kref in media device, grab references as needed

2016-12-15 Thread Shuah Khan
On 12/15/2016 10:25 AM, Mauro Carvalho Chehab wrote: > Em Thu, 15 Dec 2016 10:09:53 -0700 > Shuah Khan escreveu: > >> On 12/15/2016 09:28 AM, Hans Verkuil wrote: >>> On 15/12/16 17:06, Shuah Khan wrote: > >>> >>> I think this will work for interface entities, but for

Re: [PATCH RFC] [media] s5k6aa: set usleep_range greater 0

2016-12-15 Thread Sylwester Nawrocki
On 12/15/2016 02:14 AM, Nicholas Mc Guire wrote: > if its actually unused then it might be best to completely drop the code > raher than fixing up dead-code. Is the EXYNOS the only system that had > this device in use ? If it shold stay in then setting it to the above > proposed 3000, 4000 would

[PATCH v5 3/9] MAINTAINERS: Add Qualcomm Venus video accelerator driver

2016-12-15 Thread Stanimir Varbanov
Add an entry for Venus video encoder/decoder accelerator driver. Signed-off-by: Stanimir Varbanov --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 52cc0775a799..9528488f538e 100644 --- a/MAINTAINERS +++

[PATCH v5 2/9] doc: DT: venus: binding document for Qualcomm video driver

2016-12-15 Thread Stanimir Varbanov
Add binding document for Venus video encoder/decoder driver Cc: Rob Herring Cc: Mark Rutland Cc: devicet...@vger.kernel.org Signed-off-by: Stanimir Varbanov --- .../devicetree/bindings/media/qcom,venus.txt | 68

[PATCH v5 7/9] media: venus: hfi: add Host Firmware Interface (HFI)

2016-12-15 Thread Stanimir Varbanov
This is the implementation of HFI. It is charged with the responsibility to comunicate with the firmware through an interface commands and messages. - hfi.c has interface functions used by the core, decoder and encoder parts to comunicate with the firmware. For example there are functions for

Re: [RFC v3 00/21] Make use of kref in media device, grab references as needed

2016-12-15 Thread Mauro Carvalho Chehab
Em Thu, 15 Dec 2016 10:09:53 -0700 Shuah Khan escreveu: > On 12/15/2016 09:28 AM, Hans Verkuil wrote: > > On 15/12/16 17:06, Shuah Khan wrote: > > > > I think this will work for interface entities, but for subdev entities this > > certainly won't work. Unbinding

[PATCH v5 9/9] media: venus: enable building of Venus video driver

2016-12-15 Thread Stanimir Varbanov
This adds Venus driver Makefile and changes v4l2 platform Makefile/Kconfig in order to enable building of the driver. Signed-off-by: Stanimir Varbanov --- drivers/media/platform/Kconfig | 13 + drivers/media/platform/Makefile| 2

[PATCH v5 8/9] media: venus: hfi: add Venus HFI files

2016-12-15 Thread Stanimir Varbanov
Here is the implementation of Venus video accelerator low-level functionality. It contanins code which setup the registers and startup uthe processor, allocate and manipulates with the shared memory used for sending commands and receiving messages. Signed-off-by: Stanimir Varbanov

[PATCH v5 4/9] media: venus: adding core part and helper functions

2016-12-15 Thread Stanimir Varbanov
* core.c has implemented the platform dirver methods, file operations and v4l2 registration. * helpers.c has implemented common helper functions for: - buffer management - vb2_ops and functions for format propagation, - functions for allocating and freeing buffers for internal

[PATCH v5 5/9] media: venus: vdec: add video decoder files

2016-12-15 Thread Stanimir Varbanov
This consists of video decoder implementation plus decoder controls. Signed-off-by: Stanimir Varbanov --- drivers/media/platform/qcom/venus/vdec.c | 952 + drivers/media/platform/qcom/venus/vdec.h | 32 +

[PATCH v5 6/9] media: venus: venc: add video encoder files

2016-12-15 Thread Stanimir Varbanov
This adds encoder part of the driver plus encoder controls. Signed-off-by: Stanimir Varbanov --- drivers/media/platform/qcom/venus/venc.c | 1100 drivers/media/platform/qcom/venus/venc.h | 32 +

[PATCH v5 1/9] media: v4l2-mem2mem: extend m2m APIs for more accurate buffer management

2016-12-15 Thread Stanimir Varbanov
this add functions for: - remove buffers from src/dst queue by index - remove exact buffer from src/dst queue also extends m2m API to iterate over a list of src/dst buffers in safely and non-safely manner. Signed-off-by: Stanimir Varbanov ---

[PATCH v5 0/9] Qualcomm video decoder/encoder driver

2016-12-15 Thread Stanimir Varbanov
Hi all, Here is fifth version of the patchset. The changes since v4 are: * removed pointless vdec_s_selection. * renamed v4l2_m2m_buf_remove_exact to v4l2_m2m_buf_remove_by_buf and added kernel doc style for missing function arguments. * fixed pm_runtime_get_sync error path in

Re: [RFC v3 00/21] Make use of kref in media device, grab references as needed

2016-12-15 Thread Shuah Khan
On 12/15/2016 09:28 AM, Hans Verkuil wrote: > On 15/12/16 17:06, Shuah Khan wrote: >> On 12/15/2016 08:26 AM, Hans Verkuil wrote: >>> On 15/12/16 15:45, Shuah Khan wrote: On 12/15/2016 07:03 AM, Hans Verkuil wrote: > On 15/12/16 13:56, Laurent Pinchart wrote: >> Hi Sakari, >>

Re: [RFC v3 00/21] Make use of kref in media device, grab references as needed

2016-12-15 Thread Mauro Carvalho Chehab
Em Thu, 15 Dec 2016 16:26:19 +0100 Hans Verkuil escreveu: > > Should all the entities stick around until all references to media > > device are gone? If an application has /dev/media open, does that > > mean all entities should not be free'd until this app. exits? What > >

Re: [PATCH RFC] omap3isp: prevent releasing MC too early

2016-12-15 Thread Mauro Carvalho Chehab
Em Thu, 15 Dec 2016 04:31:12 -0800 Greg KH escreveu: > On Thu, Dec 15, 2016 at 02:13:42PM +0200, Laurent Pinchart wrote: > > Hi Mauro, > > > > (CC'ing Greg) > > > > On Wednesday 14 Dec 2016 13:14:06 Mauro Carvalho Chehab wrote: > > > Avoid calling streamoff without having the

Re: [RFC v3 00/21] Make use of kref in media device, grab references as needed

2016-12-15 Thread Hans Verkuil
On 15/12/16 17:06, Shuah Khan wrote: On 12/15/2016 08:26 AM, Hans Verkuil wrote: On 15/12/16 15:45, Shuah Khan wrote: On 12/15/2016 07:03 AM, Hans Verkuil wrote: On 15/12/16 13:56, Laurent Pinchart wrote: Hi Sakari, On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote: On Tue, Dec 13, 2016

Re: [PATCH v4 2/6] [media] rc-main: split setup and unregister functions

2016-12-15 Thread Andi Shyti
> > + /* rc_open will be called here */ > > + rc = input_register_device(dev->input_dev); > > + if (rc) > > + goto out_table; > > + > > + dev->input_dev->dev.parent = >dev; > > + memcpy(>input_dev->id, >input_id, sizeof(dev->input_id)); > > + dev->input_dev->phys =

Re: [RFC v3 00/21] Make use of kref in media device, grab references as needed

2016-12-15 Thread Hans Verkuil
On 15/12/16 16:45, Mauro Carvalho Chehab wrote: Em Thu, 15 Dec 2016 15:45:22 +0100 Hans Verkuil escreveu: On 15/12/16 15:32, Mauro Carvalho Chehab wrote: Em Thu, 15 Dec 2016 15:03:36 +0100 Hans Verkuil escreveu: On 15/12/16 13:56, Laurent Pinchart

Re: [RFC v3 00/21] Make use of kref in media device, grab references as needed

2016-12-15 Thread Shuah Khan
On 12/15/2016 08:26 AM, Hans Verkuil wrote: > On 15/12/16 15:45, Shuah Khan wrote: >> On 12/15/2016 07:03 AM, Hans Verkuil wrote: >>> On 15/12/16 13:56, Laurent Pinchart wrote: Hi Sakari, On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote: > On Tue, Dec 13, 2016 at 10:24:47AM

Re: [PATCH v4 2/6] [media] rc-main: split setup and unregister functions

2016-12-15 Thread Sean Young
Hi Andi, This patch breaks all rc devices, none of them have input devices any more (see below). On Wed, Dec 14, 2016 at 11:00:26PM +0900, Andi Shyti wrote: > Move the input device allocation, map and protocol handling to > different functions. > > Signed-off-by: Andi Shyti

Re: [RFC v3 00/21] Make use of kref in media device, grab references as needed

2016-12-15 Thread Mauro Carvalho Chehab
Em Thu, 15 Dec 2016 15:45:22 +0100 Hans Verkuil escreveu: > On 15/12/16 15:32, Mauro Carvalho Chehab wrote: > > Em Thu, 15 Dec 2016 15:03:36 +0100 > > Hans Verkuil escreveu: > > > >> On 15/12/16 13:56, Laurent Pinchart wrote: > >>> Hi Sakari, > >>> >

Re: [RFC RESEND 01/11] vb2: Rename confusingly named internal buffer preparation functions

2016-12-15 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday 11 Sep 2015 14:50:24 Sakari Ailus wrote: > Rename __qbuf_*() functions which are specific to a buffer type as > __prepare_*() which matches with what they do. The naming was there for > historical reasons; the purpose of the functions was changed

Re: [RFC v3 00/21] Make use of kref in media device, grab references as needed

2016-12-15 Thread Hans Verkuil
On 15/12/16 15:45, Shuah Khan wrote: On 12/15/2016 07:03 AM, Hans Verkuil wrote: On 15/12/16 13:56, Laurent Pinchart wrote: Hi Sakari, On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote: On Tue, Dec 13, 2016 at 10:24:47AM -0200, Mauro Carvalho Chehab wrote: Em Tue, 13 Dec 2016 12:53:05

Re: [PATCH RFC] omap3isp: prevent releasing MC too early

2016-12-15 Thread Laurent Pinchart
Hi Greg, On Thursday 15 Dec 2016 04:31:12 Greg KH wrote: > On Thu, Dec 15, 2016 at 02:13:42PM +0200, Laurent Pinchart wrote: > > Hi Mauro, > > > > (CC'ing Greg) > > > > On Wednesday 14 Dec 2016 13:14:06 Mauro Carvalho Chehab wrote: > > > Avoid calling streamoff without having the media structs

Re: [RFC v3 00/21] Make use of kref in media device, grab references as needed

2016-12-15 Thread Shuah Khan
On 12/15/2016 03:39 AM, Laurent Pinchart wrote: > Hello, > > On Tuesday 13 Dec 2016 15:23:53 Shuah Khan wrote: >> On 12/13/2016 05:24 AM, Mauro Carvalho Chehab wrote: >>> Em Tue, 13 Dec 2016 12:53:05 +0200 Sakari Ailus escreveu: On Tue, Nov 29, 2016 at 09:13:05AM -0200, Mauro Carvalho Chehab

Re: [RFC v3 00/21] Make use of kref in media device, grab references as needed

2016-12-15 Thread Shuah Khan
On 12/15/2016 07:03 AM, Hans Verkuil wrote: > On 15/12/16 13:56, Laurent Pinchart wrote: >> Hi Sakari, >> >> On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote: >>> On Tue, Dec 13, 2016 at 10:24:47AM -0200, Mauro Carvalho Chehab wrote: Em Tue, 13 Dec 2016 12:53:05 +0200 Sakari Ailus

Re: [RFC v3 00/21] Make use of kref in media device, grab references as needed

2016-12-15 Thread Hans Verkuil
On 15/12/16 15:32, Mauro Carvalho Chehab wrote: Em Thu, 15 Dec 2016 15:03:36 +0100 Hans Verkuil escreveu: On 15/12/16 13:56, Laurent Pinchart wrote: Hi Sakari, On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote: On Tue, Dec 13, 2016 at 10:24:47AM -0200, Mauro Carvalho

Re: [RFC v3 00/21] Make use of kref in media device, grab references as needed

2016-12-15 Thread Mauro Carvalho Chehab
Em Thu, 15 Dec 2016 15:03:36 +0100 Hans Verkuil escreveu: > On 15/12/16 13:56, Laurent Pinchart wrote: > > Hi Sakari, > > > > On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote: > >> On Tue, Dec 13, 2016 at 10:24:47AM -0200, Mauro Carvalho Chehab wrote: > >>> Em Tue, 13

Re: [PATCH RFC] omap3isp: prevent releasing MC too early

2016-12-15 Thread Mauro Carvalho Chehab
Greg, Em Thu, 15 Dec 2016 05:44:38 -0800 Greg KH escreveu: > On Thu, Dec 15, 2016 at 10:57:16AM -0200, Mauro Carvalho Chehab wrote: > > Em Thu, 15 Dec 2016 09:42:35 -0300 > > Javier Martinez Canillas escreveu: > > > > > Hello Mauro, > > > > > > On

[PATCH 1/2] media: s5p-cec: Remove unneeded linux/miscdevice.h include

2016-12-15 Thread Corentin Labbe
s5p-cec: does not use any miscdevice so this patch remove this unnecessary inclusion. Signed-off-by: Corentin Labbe --- drivers/staging/media/s5p-cec/exynos_hdmi_cec.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/media/s5p-cec/exynos_hdmi_cec.h

[PATCH 2/2] media: s5p-cec: Remove references to non-existent PLAT_S5P symbol

2016-12-15 Thread Corentin Labbe
Commit d78c16ccde96 ("ARM: SAMSUNG: Remove remaining legacy code") removed the Kconfig symbol PLAT_S5P. This patch remove the last occurrence of this symbol. Signed-off-by: Corentin Labbe --- drivers/staging/media/s5p-cec/Kconfig | 2 +- 1 file changed, 1

Re: [RFC v3 00/21] Make use of kref in media device, grab references as needed

2016-12-15 Thread Hans Verkuil
On 15/12/16 13:56, Laurent Pinchart wrote: Hi Sakari, On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote: On Tue, Dec 13, 2016 at 10:24:47AM -0200, Mauro Carvalho Chehab wrote: Em Tue, 13 Dec 2016 12:53:05 +0200 Sakari Ailus escreveu: On Tue, Nov 29, 2016 at 09:13:05AM -0200, Mauro Carvalho

Re: [PATCH RFC] omap3isp: prevent releasing MC too early

2016-12-15 Thread Laurent Pinchart
Hi Mauro, On Thursday 15 Dec 2016 10:57:16 Mauro Carvalho Chehab wrote: > Em Thu, 15 Dec 2016 09:42:35 -0300 Javier Martinez Canillas escreveu: > > On 12/15/2016 09:37 AM, Mauro Carvalho Chehab wrote: > > > > [snip] > > > >> What happens is that omap3isp driver calls media_device_unregister() >

Re: [PATCH RFC] omap3isp: prevent releasing MC too early

2016-12-15 Thread Greg KH
On Thu, Dec 15, 2016 at 10:57:16AM -0200, Mauro Carvalho Chehab wrote: > Em Thu, 15 Dec 2016 09:42:35 -0300 > Javier Martinez Canillas escreveu: > > > Hello Mauro, > > > > On 12/15/2016 09:37 AM, Mauro Carvalho Chehab wrote: > > > > [snip] > > > > > > > > What happens

Re: [PATCH RFC] omap3isp: prevent releasing MC too early

2016-12-15 Thread Laurent Pinchart
Hi Mauro, On Thursday 15 Dec 2016 10:37:34 Mauro Carvalho Chehab wrote: > Em Thu, 15 Dec 2016 14:13:42 +0200 Laurent Pinchart escreveu: > > On Wednesday 14 Dec 2016 13:14:06 Mauro Carvalho Chehab wrote: > >> Avoid calling streamoff without having the media structs allocated. > >> > >>

Re: [RFC v3 00/21] Make use of kref in media device, grab references as needed

2016-12-15 Thread Laurent Pinchart
Hi Sakari, On Thursday 15 Dec 2016 13:30:41 Sakari Ailus wrote: > On Tue, Dec 13, 2016 at 10:24:47AM -0200, Mauro Carvalho Chehab wrote: > > Em Tue, 13 Dec 2016 12:53:05 +0200 Sakari Ailus escreveu: > >> On Tue, Nov 29, 2016 at 09:13:05AM -0200, Mauro Carvalho Chehab wrote: > >>> Hi Sakari, > >>>

[PATCH 1/2] serio.h: add SERIO_RAINSHADOW_CEC ID

2016-12-15 Thread Hans Verkuil
From: Hans Verkuil Add a new serio ID for the RainShadow Tech USB HDMI CEC adapter. Signed-off-by: Hans Verkuil --- include/uapi/linux/serio.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/uapi/linux/serio.h

[PATCH 0/2] Add support for the RainShadow Tech HDMI CEC adapter

2016-12-15 Thread Hans Verkuil
From: Hans Verkuil This patch series adds support to the RainShadow Tech HDMI CEC adapter (http://rainshadowtech.com/HdmiCecUsb.html). The first patch adds the needed serio ID, the second adds the driver itself. Dmitry, will you take the first patch, or can we take it

[PATCH 2/2] rainshadow-cec: new RainShadow Tech HDMI CEC driver

2016-12-15 Thread Hans Verkuil
From: Hans Verkuil This driver supports the RainShadow Tech USB HDMI CEC adapter. See: http://rainshadowtech.com/HdmiCecUsb.html Signed-off-by: Hans Verkuil --- MAINTAINERS | 7 +

Re: [PATCH RFC] omap3isp: prevent releasing MC too early

2016-12-15 Thread Mauro Carvalho Chehab
Em Thu, 15 Dec 2016 09:42:35 -0300 Javier Martinez Canillas escreveu: > Hello Mauro, > > On 12/15/2016 09:37 AM, Mauro Carvalho Chehab wrote: > > [snip] > > > > > What happens is that omap3isp driver calls media_device_unregister() > > too early. Right now, it is

[PATCH v6 05/18] [media] rc: raw IR drivers cannot handle cec, unknown or other

2016-12-15 Thread Sean Young
unknown and other are for IR protocols for which we have no decoder, so the raw IR drivers have no chance of generating them. cec is not an IR protocol. Signed-off-by: Sean Young Cc: Jiri Kosina Cc: Benjamin Tissoires Cc: Bruno

[PATCH v6 00/18] IR encoders and use for IR wakeup

2016-12-15 Thread Sean Young
This series introduces IR encoders and also makes winbond-cir and nuvoton-cir use the sysfs filter wakeup interface for programmable IR wakeup. To program hauppauge rc-5 remote power button for wakeup: cd /sys/class/rc/rc0 echo rc-5 > wakeup_protocols echo 0x1e3d >

[PATCH v6 04/18] [media] winbond-cir: use sysfs wakeup filter

2016-12-15 Thread Sean Young
Now that we can select the exact variant of the protocol for wakeup filter, the winbond-cir can use the wakeup filter rather than module parameters. Signed-off-by: Sean Young Cc: David Härdeman --- drivers/media/rc/winbond-cir.c | 257

[PATCH v6 07/18] [media] rc: rc-ir-raw: Add Manchester encoder (phase encoder) helper

2016-12-15 Thread Sean Young
From: Antti Seppälä Adding a simple Manchester encoder to rc-core. Manchester coding is used by at least RC-5 and RC-6 protocols and their variants. Signed-off-by: Antti Seppälä Signed-off-by: James Hogan Signed-off-by: Sean Young

[PATCH v6 06/18] [media] rc: rc-ir-raw: Add scancode encoder callback

2016-12-15 Thread Sean Young
From: James Hogan Add a callback to raw ir handlers for encoding and modulating a scancode to a set of raw events. This could be used for transmit, or for converting a wakeup scancode to a form that is more suitable for raw hardware wake up filters. Signed-off-by: James

[PATCH v6 02/18] [media] rc: Add scancode validation

2016-12-15 Thread Sean Young
We need to valdiate that scancodes are valid for their protocol; an incorrect necx scancode could actually be a nec scancode, for example. Signed-off-by: Sean Young --- drivers/media/rc/rc-main.c | 71 -- 1 file changed, 68

[PATCH v6 03/18] [media] rc: unify nec32 protocol scancode format

2016-12-15 Thread Sean Young
There are two different encodings used for nec32: - The ir-nec-decoder.c decoder treats it as 32 bit msb first. - The img-ir decoder/encoder, winbond wakeup, dib0700, ir-ctl userspace, treat nec32 analogous to necx and nec: 4 bytes, each lsb first. So this format reverses the 4 bytes.

[PATCH v6 08/18] [media] rc: rc-ir-raw: Add pulse-distance modulation helper

2016-12-15 Thread Sean Young
From: James Hogan Add IR encoding helper for pulse-distance modulation as used by the NEC protocol. Signed-off-by: James Hogan Signed-off-by: Sean Young Cc: Mauro Carvalho Chehab Cc: Antti Seppälä

[PATCH v6 18/18] [media] rc: nuvoton-cir: Add support wakeup via sysfs filter callback

2016-12-15 Thread Sean Young
From: Antti Seppälä Nuvoton-cir utilizes the encoding capabilities of rc-core to convert scancodes from user space to pulse/space format understood by the underlying hardware. Converted samples are then written to the wakeup fifo along with other necessary configuration to

[PATCH v6 12/18] [media] rc: ir-jvc-decoder: Add encode capability

2016-12-15 Thread Sean Young
Add the capability to encode JVC scancodes as raw events. Signed-off-by: Sean Young --- drivers/media/rc/ir-jvc-decoder.c | 39 +++ 1 file changed, 39 insertions(+) diff --git a/drivers/media/rc/ir-jvc-decoder.c

[PATCH v6 10/18] [media] rc: ir-rc6-decoder: Add encode capability

2016-12-15 Thread Sean Young
From: Antti Seppälä Add the capability to encode RC-6 and RC-6A scancodes as raw events. The Manchester modulation helper is used several times with various timings so that RC-6 header preamble, the header, header trailing bit and the data itself can be modulated correctly.

[PATCH v6 16/18] [media] rc: rc-core: Add support for encode_wakeup drivers

2016-12-15 Thread Sean Young
From: James Hogan Add support in rc-core for drivers which implement the wakeup scancode filter by encoding the scancode using the raw IR encoders. This is by way of rc_dev::encode_wakeup which should be set to true and rc_dev::allowed_wakeup_protocols should be set to the

  1   2   >