[PATCH v3 6/6] ARM: dts: exynos: Remove unnecessary compatible

2017-09-07 Thread Hoegeun Kwon
Currently, the compatible('samsung,exynos5-gsc') is not used. Remove unnecessary compatible. Signed-off-by: Hoegeun Kwon --- arch/arm/boot/dts/exynos5250.dtsi | 8 arch/arm/boot/dts/exynos5420.dtsi | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/arch/arm/boot/dt

[PATCH v3 5/6] [media] exynos-gsc: Remove unnecessary compatible

2017-09-07 Thread Hoegeun Kwon
Currently, the compatible('samsung,exynos5-gsc') is not used. Remove unnecessary compatible. Signed-off-by: Hoegeun Kwon --- Documentation/devicetree/bindings/media/exynos5-gsc.txt | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Documentation/devicetree/bindings/media

[PATCH v3 4/6] [media] exynos-gsc: Add hardware rotation limits

2017-09-07 Thread Hoegeun Kwon
The hardware rotation limits of gsc depends on SOC (Exynos 5250/5420/5433). Distinguish them and add them to the driver data. Signed-off-by: Hoegeun Kwon --- drivers/media/platform/exynos-gsc/gsc-core.c | 96 1 file changed, 83 insertions(+), 13 deletions(-) diff --

[PATCH v3 3/6] drm/exynos/gsc: Add hardware rotation limits

2017-09-07 Thread Hoegeun Kwon
The gscaler has hardware rotation limits that need to be hardcoded into driver. Distinguish them and add them to the property list. The hardware rotation limits are related to the cropped source size. When swap occurs, use rot_max size instead of crop_max size. Also the scaling limits are related

[PATCH v3 2/6] ARM: dts: exynos: Add clean name of compatible.

2017-09-07 Thread Hoegeun Kwon
Exynos 5250 and 5420 have different hardware rotation limits. However, currently it uses only one compatible - "exynos5-gsc". Since we have to distinguish between these two, we add different compatible. Signed-off-by: Hoegeun Kwon --- arch/arm/boot/dts/exynos5250.dtsi | 8 arch/arm/boot

[PATCH v3 0/6] Exynos-gsc: Support the hardware rotation limits

2017-09-07 Thread Hoegeun Kwon
Hello all, The gscaler has hardware rotation limits. So this patch set support the rotate hardware limits of gsc. To avoid problems with bisectability, patches 1~4 must be merged and then merged 5 and 6. Changes for V3: - Fixed of_match_node() to of_device_get_match_data() in drm gsc driver. - A

[PATCH v3 1/6] [media] exynos-gsc: Add compatible for Exynos 5250 and 5420 specific version

2017-09-07 Thread Hoegeun Kwon
Exynos 5250 and 5420 have different hardware rotation limits. Since we have to distinguish between these two, we add different compatible(samsung,exynos5250-gsc and samsung,exynos5420-gsc). Signed-off-by: Hoegeun Kwon --- Documentation/devicetree/bindings/media/exynos5-gsc.txt | 1 + 1 file chan

cron job: media_tree daily build: WARNINGS

2017-09-07 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 Sep 8 05:00:22 CEST 2017 media-tree git hash:1efdf1776e2253b77413c997bed862410e4b6aaf media_build gi

Re: [PATCH v3 01/15] [media] v4l: Document explicit synchronization behaviour

2017-09-07 Thread Nicolas Dufresne
Le jeudi 07 septembre 2017 à 15:42 -0300, Gustavo Padovan a écrit : > From: Gustavo Padovan > > Add section to VIDIOC_QBUF about it > > v2: > - mention that fences are files (Hans) > - rework for the new API > > Signed-off-by: Gustavo Padovan > --- > Documentation/media/uapi/v4l/v

Re: [PATCH v2 00/10] media: rc: gpio-ir-recv: driver update

2017-09-07 Thread Andi Shyti
Hi Ladislav, > Serie was rebased on top of current linux.git, but something > happened there and my userspace decoder no longer works: driver > reports completely bogus timing such as (rc-5): > ^427, _1342, ^945, _183, ^1128, _671, ^1586, _91, ^1189, _1525, > ^1738, _1433, ^915, _1159, ^1464, _152

[PATCH v2 10/10] media: rc: gpio-ir-recv: use gpiolib API

2017-09-07 Thread Ladislav Michl
Gpiolib API is preferred way to access gpios. Signed-off-by: Ladislav Michl --- Changes: -v2: rebased to current linux.git drivers/media/rc/Kconfig| 1 + drivers/media/rc/gpio-ir-recv.c | 59 +++-- 2 files changed, 22 insertions(+), 38 deletions(-)

[PATCH v2 09/10] media: rc: gpio-ir-recv: remove gpio_ir_recv_platform_data

2017-09-07 Thread Ladislav Michl
gpio_ir_recv_platform_data are not used anywhere in kernel tree, so remove it. Signed-off-by: Ladislav Michl --- Changes: -v2: rebased to current linux.git drivers/media/rc/gpio-ir-recv.c | 98 +++- include/linux/platform_data/media/gpio-ir-recv.h | 23 ---

[PATCH v2 08/10] media: rc: gpio-ir-recv: use KBUILD_MODNAME

2017-09-07 Thread Ladislav Michl
There already is standard macro providing driver name, use it. Signed-off-by: Ladislav Michl --- Changes: -v2: rebased to current linux.git drivers/media/rc/gpio-ir-recv.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/media/rc/gpio-ir-recv.c b/drivers/media/

[PATCH v2 07/10] media: rc: gpio-ir-recv: use devm_request_irq

2017-09-07 Thread Ladislav Michl
Use of devm_request_irq simplifies error unwinding and as free_irq was the last user of driver remove function, remove it too. Signed-off-by: Ladislav Michl --- Changes: -v2: rebased to current linux.git drivers/media/rc/gpio-ir-recv.c | 22 +++--- 1 file changed, 3 insertions

[PATCH v2 06/10] media: rc: gpio-ir-recv: do not allow threaded interrupt handler

2017-09-07 Thread Ladislav Michl
Requesting any context irq is not actually great idea since threaded interrupt handler is run at too unpredictable time which turns timing information wrong. Fix it by requesting regular interrupt. Signed-off-by: Ladislav Michl --- Changes: -v2: rebased to current linux.git drivers/media/rc/g

[PATCH v2 01/10] media: rc: gpio-ir-recv: use helper vaiable to acess device info

2017-09-07 Thread Ladislav Michl
Using explicit struct device variable makes code a bit more readable. Signed-off-by: Ladislav Michl --- Changes: -v2: rebased to current linux.git drivers/media/rc/gpio-ir-recv.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/media/rc/gpio-ir-recv.c b

[PATCH v2 05/10] media: rc: gpio-ir-recv: use devm_rc_register_device

2017-09-07 Thread Ladislav Michl
Use of devm_rc_register_device simplifies error unwinding. Signed-off-by: Ladislav Michl --- Changes: -v2: rebased to current linux.git drivers/media/rc/gpio-ir-recv.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers/media/rc/gpio-ir-recv.c b/drivers/media/r

[PATCH v2 04/10] media: rc: gpio-ir-recv: use devm_gpio_request_one

2017-09-07 Thread Ladislav Michl
Use of devm_gpio_request_one simplifies error unwinding. Signed-off-by: Ladislav Michl --- Changes: -v2: rebased to current linux.git drivers/media/rc/gpio-ir-recv.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/media/rc/gpio-ir-recv.c b/drivers/media/rc

[PATCH v2 03/10] media: rc: gpio-ir-recv: use devm_rc_allocate_device

2017-09-07 Thread Ladislav Michl
Use of devm_rc_allocate_device simplifies error unwinding. Signed-off-by: Ladislav Michl --- Changes: -v2: rebased to current linux.git drivers/media/rc/gpio-ir-recv.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/media/rc/gpio-ir-recv.c b/drivers/media/rc/

[PATCH v2 02/10] media: rc: gpio-ir-recv: use devm_kzalloc

2017-09-07 Thread Ladislav Michl
Use of devm_kzalloc simplifies error unwinding. Signed-off-by: Ladislav Michl --- Changes: -v2: rebased to current linux.git drivers/media/rc/gpio-ir-recv.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/drivers/media/rc/gpio-ir-recv.c b/drivers/media/rc/gpio-

[PATCH v2 00/10] media: rc: gpio-ir-recv: driver update

2017-09-07 Thread Ladislav Michl
This patch serie brings driver closer to recently used APIs and removes no longer used gpio_ir_recv_platform_data support. It was done as an excercise before writing similar driver using FIQ and hw timers as this one gives too imprecise timing. Serie was rebased on top of current linux.git, but s

Re: [PATCH v3 14/15] fs/files: export close_fd() symbol

2017-09-07 Thread Gustavo Padovan
On Fri, 2017-09-08 at 00:09 +0200, Hans Verkuil wrote: > On 09/07/2017 08:42 PM, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > Rename __close_fd() to close_fd() and export it to be able close > > files > > in modules using file descriptors. > > > > The usecase that motivates this chan

Re: [PATCH v3 14/15] fs/files: export close_fd() symbol

2017-09-07 Thread Hans Verkuil
On 09/07/2017 08:42 PM, Gustavo Padovan wrote: > From: Gustavo Padovan > > Rename __close_fd() to close_fd() and export it to be able close files > in modules using file descriptors. > > The usecase that motivates this change happens in V4L2 where we send > events to userspace with a fd that has

Re: [PATCH v3 14/15] fs/files: export close_fd() symbol

2017-09-07 Thread Al Viro
On Thu, Sep 07, 2017 at 06:22:45PM -0300, Gustavo Padovan wrote: > Sorry for my lack of knowledge here and thank you for the explanation, > things are a lot clear to me. For some reasons I were trying to delay > the sharing of the fd to a event later. I can delay the install of it > but that my re

Re: [PATCH v3 14/15] fs/files: export close_fd() symbol

2017-09-07 Thread Gustavo Padovan
2017-09-07 Al Viro : > On Thu, Sep 07, 2017 at 03:42:25PM -0300, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > Rename __close_fd() to close_fd() and export it to be able close files > > in modules using file descriptors. > > > > The usecase that motivates this change happens in V4L2

Re: [PATCH v3 14/15] fs/files: export close_fd() symbol

2017-09-07 Thread Al Viro
On Thu, Sep 07, 2017 at 03:42:25PM -0300, Gustavo Padovan wrote: > From: Gustavo Padovan > > Rename __close_fd() to close_fd() and export it to be able close files > in modules using file descriptors. > > The usecase that motivates this change happens in V4L2 where we send > events to userspace

[PATCH] [media] blackfin: Delete an error message for a failed memory allocation in ppi_create_instance()

2017-09-07 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 7 Sep 2017 22:14:43 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/platform/blackfin/ppi.c | 1 - 1 file changed, 1 deletion

Re: [PATCH v3 2/3] leds: as3645a: Add LED flash class driver

2017-09-07 Thread Pavel Machek
On Thu 2017-09-07 17:52:05, Sakari Ailus wrote: > On Thu, Sep 07, 2017 at 02:50:27PM +0200, Pavel Machek wrote: > > Hi! > > > > > On Mon, Aug 28, 2017 at 01:04:51PM +0200, Pavel Machek wrote: > > > > On Wed 2017-08-23 11:10:59, Sakari Ailus wrote: > > > > > Add a LED flash class driver for the as3

Re: Donation

2017-09-07 Thread Mavis Wanczyk Foundation
Greetings To You, My Name is Mavis wanczyk , the winner of the Power ball jackpot of $ $758.7 million in the AUGUST 24, 2017, My jackpot was a gift from God to me hence my Entire family/foundation has AGREED to do this. My foundation is donating $500,000.00USD to you. please contac maviswanczy

Re: [PATCH v3 14/15] fs/files: export close_fd() symbol

2017-09-07 Thread Eric Biggers
On Thu, Sep 07, 2017 at 03:42:25PM -0300, Gustavo Padovan wrote: > From: Gustavo Padovan > > Rename __close_fd() to close_fd() and export it to be able close files > in modules using file descriptors. > > The usecase that motivates this change happens in V4L2 where we send > events to userspace

[PATCH v3 00/15] V4L2 Explicit Synchronization support

2017-09-07 Thread Gustavo Padovan
From: Gustavo Padovan Hi, Refer to the documentation on the first patch for the details. The previous iteration is here: https://www.mail-archive.com/linux-media@vger.kernel.org/msg118077.html The 2nd patch proposes an userspace API for fences, then on patch 3 we prepare to the addition of in-

[PATCH v3 02/15] [media] vb2: add explicit fence user API

2017-09-07 Thread Gustavo Padovan
From: Gustavo Padovan Turn the reserved2 field into fence_fd that we will use to send an in-fence to the kernel and return an out-fence from the kernel to userspace. Two new flags were added, V4L2_BUF_FLAG_IN_FENCE, that should be used when sending a fence to the kernel to be waited on, and V4L2

[PATCH v3 03/15] [media] vb2: check earlier if stream can be started

2017-09-07 Thread Gustavo Padovan
From: Gustavo Padovan To support explicit synchronization we need to run all operations that can fail before we queue the buffer to the driver. With fences the queueing will be delayed if the fence is not signaled yet and it will be better if such callback do not fail. For that we move the vb2_s

[PATCH v3 05/15] [media] uvc: enable subscriptions to other events

2017-09-07 Thread Gustavo Padovan
From: Gustavo Padovan Call v4l2_ctrl_subscribe_event to subscribe to the BUF_QUEUED event as well. Signed-off-by: Gustavo Padovan --- drivers/media/usb/uvc/uvc_v4l2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/usb/uvc/uvc_v4l2.c b/drivers/media/usb/uvc/uv

[PATCH v3 04/15] [media] vb2: add in-fence support to QBUF

2017-09-07 Thread Gustavo Padovan
From: Gustavo Padovan Receive in-fence from userspace and add support for waiting on them before queueing the buffer to the driver. Buffers are only queued to the driver once they are ready. A buffer is ready when its in-fence signals. v4: - Add a comment about dma_fence_add_callback() n

[PATCH v3 10/15] [media] vb2: add 'ordered' property to queues

2017-09-07 Thread Gustavo Padovan
From: Gustavo Padovan For explicit synchronization (and soon for HAL3/Request API) we need the v4l2-driver to guarantee the ordering in which the buffers were queued by userspace. This is already true for many drivers, but we never needed to say it. Signed-off-by: Gustavo Padovan --- include/m

[PATCH v3 07/15] [media] v4l: add V4L2_EVENT_BUF_QUEUED event

2017-09-07 Thread Gustavo Padovan
From: Gustavo Padovan Add a new event the userspace can subscribe to receive notifications when a buffer is queued onto the driver. The event provides the index of the queued buffer. v2: - Add missing Documentation (Mauro) Signed-off-by: Gustavo Padovan --- Documentation/media/uapi/v4l/vidioc

[PATCH v3 06/15] [media] vivid: assign the specific device to the vb2_queue->dev

2017-09-07 Thread Gustavo Padovan
From: Gustavo Padovan Instead of assigning the global v4l2 device, assign the specific device. This was causing trouble when using using V4L2 events with vivid devices. The device's queue should be the same we opened in userspace. This is needed for the upcoming V4L2_EVENT_BUF_QUEUED support. Th

[PATCH v3 09/15] [media] v4l: add support to BUF_QUEUED event

2017-09-07 Thread Gustavo Padovan
From: Gustavo Padovan Implement the needed pieces to let userspace subscribe for V4L2_EVENT_BUF_QUEUED events. Videobuf2 will queue the event for the DQEVENT ioctl. v3: - Do not call v4l2 event API from vb2 (Mauro) v2: - Use VIDEO_MAX_FRAME to allocate room for events at v4l2_ev

[PATCH v3 11/15] [media] vivid: mark vivid queues as ordered

2017-09-07 Thread Gustavo Padovan
From: Gustavo Padovan To enable vivid to be used with explicit synchronization we need to mark its queues as ordered. vivid queues are already ordered by default so we no changes are needed. Signed-off-by: Gustavo Padovan Acked-by: Hans Verkuil --- drivers/media/platform/vivid/vivid-core.c |

[PATCH v3 12/15] [media] vb2: add videobuf2 dma-buf fence helpers

2017-09-07 Thread Gustavo Padovan
From: Javier Martinez Canillas Add a videobuf2-fence.h header file that contains different helpers for DMA buffer sharing explicit fence support in videobuf2. Signed-off-by: Javier Martinez Canillas Signed-off-by: Gustavo Padovan --- include/media/videobuf2-fence.h | 49 ++

[PATCH v3 14/15] fs/files: export close_fd() symbol

2017-09-07 Thread Gustavo Padovan
From: Gustavo Padovan Rename __close_fd() to close_fd() and export it to be able close files in modules using file descriptors. The usecase that motivates this change happens in V4L2 where we send events to userspace with a fd that has file installed in it. But if for some reason we have to canc

[PATCH v3 13/15] [media] vb2: add infrastructure to support out-fences

2017-09-07 Thread Gustavo Padovan
From: Gustavo Padovan Add vb2_setup_out_fence() and the needed members to struct vb2_buffer. Signed-off-by: Gustavo Padovan --- drivers/media/v4l2-core/videobuf2-core.c | 55 include/media/videobuf2-core.h | 34 2 files changed, 8

[PATCH v3 15/15] [media] vb2: add out-fence support to QBUF

2017-09-07 Thread Gustavo Padovan
From: Gustavo Padovan If V4L2_BUF_FLAG_OUT_FENCE flag is present on the QBUF call we create an out_fence and sent to userspace on the V4L2_EVENT_BUF_QUEUED when the buffer is queued to the driver. The out fence fd returned references the next buffer to be queued to the driver and not the buffer

[PATCH v3 08/15] [media] vb2: add .buffer_queued() to notify queueing in the driver

2017-09-07 Thread Gustavo Padovan
From: Gustavo Padovan With the upcoming explicit synchronization support to V4L2 we need a way to notify userspace when buffers are queued to the driver - buffers with fences attached to it can only be queued once the fence signal, so the queueing to the driver might be deferred. Yet, userspace

[PATCH v3 01/15] [media] v4l: Document explicit synchronization behaviour

2017-09-07 Thread Gustavo Padovan
From: Gustavo Padovan Add section to VIDIOC_QBUF about it v2: - mention that fences are files (Hans) - rework for the new API Signed-off-by: Gustavo Padovan --- Documentation/media/uapi/v4l/vidioc-qbuf.rst | 31 1 file changed, 31 insertions(+) di

[PATCH] Staging: atomisp: fix alloc_cast.cocci warnings

2017-09-07 Thread Branislav Radocaj
Remove casting the values returned by memory allocation functions like kmalloc, kzalloc, kmem_cache_alloc, kmem_cache_zalloc etc. Semantic patch information: This makes an effort to find cases of casting of values returned by kmalloc, kzalloc, kcalloc, kmem_cache_alloc, kmem_cache_zalloc, kmem_cac

dw2102 kernel oops

2017-09-07 Thread Sebastian Schmachtel
Hi, I recently bought a Tevii S650 USB DVB-S2 device (dw2102 driver). But when i connect it, my Desktop seems to get some HID inputs (some random keys are typed in a terminal). Pretty strange, but I set disable_rc_polling=1 for dvb_usb to get rid of this problem as the pattern reminded me of a Rem

Re: [PATCH] em28xx: add support for Hauppauge WinTV-dualHD DVB tuner

2017-09-07 Thread Mauro Carvalho Chehab
Hi Brad, Em Wed, 31 May 2017 15:01:00 -0500 Brad Love escreveu: > Christian et al, > > I am an engineer at Hauppauge. This repo is the staging area for all the > patches I am testing, with the intention of getting them upstreamed. I > will be inaccessible for the next 18 days however, so I will

Re: [PATCH v3 2/3] leds: as3645a: Add LED flash class driver

2017-09-07 Thread Sakari Ailus
On Thu, Sep 07, 2017 at 02:50:27PM +0200, Pavel Machek wrote: > Hi! > > > On Mon, Aug 28, 2017 at 01:04:51PM +0200, Pavel Machek wrote: > > > On Wed 2017-08-23 11:10:59, Sakari Ailus wrote: > > > > Add a LED flash class driver for the as3654a flash controller. A V4L2 > > > > flash > > > > driver

[GIT FIXES FOR v4.14] a800 probe fix

2017-09-07 Thread Sean Young
Hi Mauro, This is a single fix for doing dma from the stack. After this, the a800 works again. Thanks, Sean -- The following changes since commit 1efdf1776e2253b77413c997bed862410e4b6aaf: media: leds: as3645a: add V4L2_FLASH_LED_CLASS dependency (2017-09-05 16:32:45 -0400) are available in

Re: [PATCH v3 2/3] leds: as3645a: Add LED flash class driver

2017-09-07 Thread Pavel Machek
Hi! > On Mon, Aug 28, 2017 at 01:04:51PM +0200, Pavel Machek wrote: > > On Wed 2017-08-23 11:10:59, Sakari Ailus wrote: > > > Add a LED flash class driver for the as3654a flash controller. A V4L2 > > > flash > > > driver for it already exists (drivers/media/i2c/as3645a.c), and this > > > driver

Re: [PATCH 1/1] media: Check for active and has_no_links overrun

2017-09-07 Thread Hans Verkuil
On 08/29/17 15:46, Sakari Ailus wrote: > From: Sakari Ailus > > The active and has_no_links arrays will overrun in > media_entity_pipeline_start() if there's an entity which has more than > MEDIA_ENTITY_MAX_PAD pads. Ensure in media_entity_init() that there are > fewer pads than that. > > Signed

Re: [PATCH v8 06/21] v4l: fwnode: Support generic parsing of graph endpoints in a device

2017-09-07 Thread Hans Verkuil
On 09/07/17 11:58, Sakari Ailus wrote: > Hi Hans, > > On Thu, Sep 07, 2017 at 10:51:21AM +0200, Hans Verkuil wrote: >> On 09/07/17 09:34, Sakari Ailus wrote: >>> Hi Hans, >>> >>> On Wed, Sep 06, 2017 at 09:41:40AM +0200, Hans Verkuil wrote: On 09/05/2017 03:05 PM, Sakari Ailus wrote: > Th

Re: [PATCH v8 14/21] v4l: async: Allow binding notifiers to sub-devices

2017-09-07 Thread Hans Verkuil
On 09/07/17 10:32, Sakari Ailus wrote: > Hi Hans, > > On Wed, Sep 06, 2017 at 10:46:31AM +0200, Hans Verkuil wrote: >> On 09/05/2017 03:05 PM, Sakari Ailus wrote: >>> diff --git a/include/media/v4l2-async.h b/include/media/v4l2-async.h >>> index 3bc8a7c0d83f..12739be44bd1 100644 >>> --- a/include

[GIT PULL for v4.14-rc1] media updates

2017-09-07 Thread Mauro Carvalho Chehab
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 Hi Linus Please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media tags/media/v4.14-1 For the Brazil's independence day's pull request :-) This is one of the biggest media pull requests, with 625 patches affecting almos

Re: [PATCH v3 2/3] leds: as3645a: Add LED flash class driver

2017-09-07 Thread Sakari Ailus
Hi Pavel, On Mon, Aug 28, 2017 at 01:04:51PM +0200, Pavel Machek wrote: > On Wed 2017-08-23 11:10:59, Sakari Ailus wrote: > > Add a LED flash class driver for the as3654a flash controller. A V4L2 flash > > driver for it already exists (drivers/media/i2c/as3645a.c), and this driver > > is based on

Re: [PATCH v3 05/14] [media] cxd2880: Add tuner part of the driver

2017-09-07 Thread Mauro Carvalho Chehab
Em Thu, 7 Sep 2017 19:12:57 +0900 "Takiguchi, Yasunari" escreveu: > Dear Mauro > > Thanks for your review and reply. > > We are going to discuss how to change our code with your comments internally. > > I reply for your 2 comments, > > >> [Change list] > >> Changes in V3 > >>drivers/medi

Re: [PATCH v3 05/14] [media] cxd2880: Add tuner part of the driver

2017-09-07 Thread Takiguchi, Yasunari
Dear Mauro Thanks for your review and reply. We are going to discuss how to change our code with your comments internally. I reply for your 2 comments, >> [Change list] >> Changes in V3 >>drivers/media/dvb-frontends/cxd2880/cxd2880_dtv.h >> -removed code relevant to ISDB-T > > Just

Re: [PATCH v8 06/21] v4l: fwnode: Support generic parsing of graph endpoints in a device

2017-09-07 Thread Sakari Ailus
Hi Hans, On Thu, Sep 07, 2017 at 10:51:21AM +0200, Hans Verkuil wrote: > On 09/07/17 09:34, Sakari Ailus wrote: > > Hi Hans, > > > > On Wed, Sep 06, 2017 at 09:41:40AM +0200, Hans Verkuil wrote: > >> On 09/05/2017 03:05 PM, Sakari Ailus wrote: > >>> The current practice is that drivers iterate ov

[PATCH] media: cx23885: make const array buf static, reduces object code size

2017-09-07 Thread Colin King
From: Colin Ian King Don't populate the array buf on the stack, instead make it static. Makes the object code smaller by over 240 bytes: Before: textdata bss dec hex filename 21689 22992 416 45097b029 cx23885-cards.o After: textdata bss dec he

Re: [PATCH v8 06/21] v4l: fwnode: Support generic parsing of graph endpoints in a device

2017-09-07 Thread Hans Verkuil
On 09/07/17 09:34, Sakari Ailus wrote: > Hi Hans, > > On Wed, Sep 06, 2017 at 09:41:40AM +0200, Hans Verkuil wrote: >> On 09/05/2017 03:05 PM, Sakari Ailus wrote: >>> The current practice is that drivers iterate over their endpoints and >>> parse each endpoint separately. This is very similar in a

Re: [PATCH v8 14/21] v4l: async: Allow binding notifiers to sub-devices

2017-09-07 Thread Sakari Ailus
Hi Hans, On Wed, Sep 06, 2017 at 10:46:31AM +0200, Hans Verkuil wrote: > On 09/05/2017 03:05 PM, Sakari Ailus wrote: > > Registering a notifier has required the knowledge of struct v4l2_device > > for the reason that sub-devices generally are registered to the > > v4l2_device (as well as the media

Re: [PATCH v8 08/21] rcar-vin: Use generic parser for parsing fwnode endpoints

2017-09-07 Thread Sakari Ailus
Hi Hans, On Wed, Sep 06, 2017 at 09:44:32AM +0200, Hans Verkuil wrote: > On 09/05/2017 03:05 PM, Sakari Ailus wrote: > > static int rvin_digital_graph_init(struct rvin_dev *vin) > > { > > - struct v4l2_async_subdev **subdevs = NULL; > > int ret; > > > > - ret = rvin_digital_graph_parse

Re: [PATCH v8 12/21] v4l: async: Introduce helpers for calling async ops callbacks

2017-09-07 Thread Sakari Ailus
On Wed, Sep 06, 2017 at 09:50:36AM +0200, Hans Verkuil wrote: > On 09/05/2017 03:05 PM, Sakari Ailus wrote: > > Add three helper functions to call async operations callbacks. Besides > > simplifying callbacks, this allows async notifiers to have no ops set, > > i.e. it can be left NULL. > > What i

Re: [PATCH v8 06/21] v4l: fwnode: Support generic parsing of graph endpoints in a device

2017-09-07 Thread Sakari Ailus
Hi Hans, On Wed, Sep 06, 2017 at 09:41:40AM +0200, Hans Verkuil wrote: > On 09/05/2017 03:05 PM, Sakari Ailus wrote: > > The current practice is that drivers iterate over their endpoints and > > parse each endpoint separately. This is very similar in a number of > > drivers, implement a generic fu

Re: BUGREPORT: IR keytable 1.12.3

2017-09-07 Thread Sean Young
Hi Oliver, On Wed, Sep 06, 2017 at 09:20:14PM +0200, "Oliver Müller" wrote: > BUG IR keytable 1.12.3 >   > OS: Distributor ID:    Debian > Description:    Debian GNU/Linux 9.1 (stretch) > Release:    9.1 > Codename:    stretch >   > Kernel: 4.9.0-3-amd64 #1 SMP Debian 4.9.30-2+deb9u3 (