Re: [PATCH v6 1/3] [media] v4l: add parsed MPEG-2 support

2017-07-07 Thread ayaka
On 07/08/2017 02:33 AM, Nicolas Dufresne wrote: Le samedi 08 juillet 2017 à 01:29 +0800, ayaka a écrit : On 07/04/2017 05:29 PM, Hugues FRUCHET wrote: Hi Randy, Thanks for review, and sorry for late reply, answers inline. BR, Hugues. On 06/11/2017 01:41 PM, ayaka wrote: On 04/28/2017 09:25

Re: [PATCH 1/2] staging: media: atomisp2: css2400: Replace kfree()/vfree() with kvfree()

2017-07-07 Thread Joe Perches
On Fri, 2017-07-07 at 20:40 -0400, Amitoj Kaur Chawla wrote: > Conditionally calling kfree()/vfree() can be replaced by a call to > kvfree() which handles both kmalloced memory and vmalloced memory. [] > diff --git a/drivers/staging/media/atomisp/pci/atomisp2/css2400/sh_css.c > b/drivers/staging/

[PATCH 2/2] staging: media: atomisp2: Replace kfree()/vfree() with kvfree()

2017-07-07 Thread Amitoj Kaur Chawla
Conditionally calling kfree()/vfree() can be replaced by a call to kvfree() which handles both kmalloced memory and vmalloced memory. Consequently removed an unnecessary comment. The Coccinelle semantic patch used to make the change is as follows: // @@ expression a; @@ - if(...) { vfree(a); }

[PATCH 1/2] staging: media: atomisp2: css2400: Replace kfree()/vfree() with kvfree()

2017-07-07 Thread Amitoj Kaur Chawla
Conditionally calling kfree()/vfree() can be replaced by a call to kvfree() which handles both kmalloced memory and vmalloced memory. The Coccinelle semantic patch used to make the change is as follows: // @@ expression a; @@ - if(...) { vfree(a); } - else { kfree(a); } + kvfree(a); @@ expression

Re: [PATCH v2 00/10] STV0910/STV6111 drivers, ddbridge CineS2 V7 support

2017-07-07 Thread Jasmin J.
Hello! > This series adds drivers for the ST STV0910 DVB-S/S2 demodulator ICs and > the ST STV6111 DVB-S/S2 tuners, and utilises them to enable ddbridge to > support the current line of Digital Devices DVB-S/S2 hardware (e.g. Cine > S2 V7/V7A adapters, DuoFlex S2 V4 addon modules and maybe more, w

RE: [PATCH v4 1/1] i2c: Add Omnivision OV5670 5M sensor support

2017-07-07 Thread Mani, Rajmohan
Hi Chiran, > + > +/* Supported link frequencies */ > +#define OV5670_LINK_FREQ_840MBPS 84000 > +#define OV5670_LINK_FREQ_840MBPS_INDEX 0 > +static const struct ov5670_link_freq_config link_freq_configs[] = { > + { > + .pixel_rate = 33600, > + .reg_list

Re: [PATCH v4 1/1] i2c: Add Omnivision OV5670 5M sensor support

2017-07-07 Thread Sakari Ailus
Hi Chiranjeevi, A few relatively minor comments below. On Sat, Jul 01, 2017 at 01:49:06PM -0700, chiranjeevi.rap...@intel.com wrote: > From: Chiranjeevi Rapolu > > Provides single source pad with up to 2592x1944 pixels at 10-bit raw > bayer format over MIPI CSI2 two lanes at 840Mbps/lane. > The

Re: [PATCH] rcar_fdp1: constify vb2_ops structure

2017-07-07 Thread Gustavo A. R. Silva
Hi Kieran, Quoting Kieran Bingham : Hi Gustavo, Thank you for the patch. Absolutely, glad to help. :) On 06/07/17 21:25, Gustavo A. R. Silva wrote: Check for vb2_ops structures that are only stored in the ops field of a vb2_queue structure. That field is declared const, so vb2_ops struct

Re: [PATCH v6 1/3] [media] v4l: add parsed MPEG-2 support

2017-07-07 Thread Nicolas Dufresne
Le samedi 08 juillet 2017 à 01:29 +0800, ayaka a écrit : > > On 07/04/2017 05:29 PM, Hugues FRUCHET wrote: > > Hi Randy, > > Thanks for review, and sorry for late reply, answers inline. > > BR, > > Hugues. > > > > On 06/11/2017 01:41 PM, ayaka wrote: > > > > > > On 04/28/2017 09:25 PM, Hugues Fr

Re: [PATCH 09/12] [media] vivid: mark vivid queues as ordered

2017-07-07 Thread Shuah Khan
On 06/16/2017 01:39 AM, Gustavo Padovan wrote: > From: Gustavo Padovan > > To enable vivid to be used with explicit synchronization we need > to mark its queues as ordered. > > Signed-off-by: Gustavo Padovan > --- > drivers/media/platform/vivid/vivid-core.c | 5 + > 1 file changed, 5 inser

Re: [PATCH v6 1/3] [media] v4l: add parsed MPEG-2 support

2017-07-07 Thread ayaka
On 07/04/2017 05:29 PM, Hugues FRUCHET wrote: Hi Randy, Thanks for review, and sorry for late reply, answers inline. BR, Hugues. On 06/11/2017 01:41 PM, ayaka wrote: On 04/28/2017 09:25 PM, Hugues Fruchet wrote: Add "parsed MPEG-2" pixel format & related controls needed by stateless video d

RE:

2017-07-07 Thread Mrs Alice Walton
-- my name is Mrs. Alice Walton, a business woman an America Citizen and the heiress to the fortune of Walmart stores, born October 7, 1949. I have a mission for you worth $100,000,000.00(Hundred Million United State Dollars) which I intend using for CHARITY PROJECT to help the less privil

Re: [PATCH 05/12] [media] vivid: assign the specific device to the vb2_queue->dev

2017-07-07 Thread Shuah Khan
On 06/16/2017 01:39 AM, Gustavo Padovan wrote: > 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. > > Sign

Re: [PATCH v2 4/6] [media] rc: pwm-ir-tx: add new driver

2017-07-07 Thread Pavel Machek
On Fri 2017-07-07 10:52:02, Sean Young wrote: > This is new driver which uses pwm, so it is more power-efficient > than the bit banging gpio-ir-tx driver. > > Signed-off-by: Sean Young > --- > MAINTAINERS | 6 ++ > drivers/media/rc/Kconfig | 12 > drivers/media/rc/Ma

Re: [PATCH 1/2] dt-bindings: media: Add Cadence MIPI-CSI2RX Device Tree bindings

2017-07-07 Thread Rob Herring
On Mon, Jul 03, 2017 at 02:40:22PM +0200, Maxime Ripard wrote: > The Cadence MIPI-CSI2 RX controller is a CSI2RX bridge that supports up to > 4 CSI-2 lanes, and can route the frames to up to 4 streams, depending on > the hardware implementation. Streams and lanes are separate, right? Do you need t

Re: [PATCH 4/4] [media] rc: pwm-ir-tx: add new driver

2017-07-07 Thread Pavel Machek
On Fri 2017-07-07 08:59:28, Rob Herring wrote: > On Sun, Jul 02, 2017 at 12:06:13PM +0100, Sean Young wrote: > > This is new driver which uses pwm, so it is more power-efficient > > than the bit banging gpio-ir-tx driver. > > > > Signed-off-by: Sean Young > > --- > > .../devicetree/bindings/leds

Re: [Patch v5 12/12] Documention: v4l: Documentation for HEVC CIDs

2017-07-07 Thread Stanimir Varbanov
Hi, On 06/19/2017 08:10 AM, Smitha T Murthy wrote: > Added V4l2 controls for HEVC encoder > > Signed-off-by: Smitha T Murthy > --- > Documentation/media/uapi/v4l/extended-controls.rst | 364 > + > 1 file changed, 364 insertions(+) > > diff --git a/Documentation/media/uapi/

Re: [Patch v5 05/12] [media] videodev2.h: Add v4l2 definition for HEVC

2017-07-07 Thread Stanimir Varbanov
Hi Smitha, On 06/19/2017 08:10 AM, Smitha T Murthy wrote: > Add V4L2 definition for HEVC compressed format > > Signed-off-by: Smitha T Murthy > Reviewed-by: Andrzej Hajda > --- > include/uapi/linux/videodev2.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/uapi/linux/videode

[PATCH v2] staging: atomisp: use kstrdup to replace kmalloc and memcpy

2017-07-07 Thread Hari Prasath
kstrdup kernel primitive can be used to replace kmalloc followed by string copy. This was reported by coccinelle tool Signed-off-by: Hari Prasath --- .../media/atomisp/pci/atomisp2/css2400/sh_css_firmware.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/dri

Re: [PATCH 04/12] [media] uvc: enable subscriptions to other events

2017-07-07 Thread Shuah Khan
On 06/16/2017 01:39 AM, Gustavo Padovan wrote: > From: Gustavo Padovan > > Call v4l2_ctrl_subscribe_event to subscribe to more events supported by > v4l. > > Signed-off-by: Gustavo Padovan > --- > drivers/media/usb/uvc/uvc_v4l2.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > dif

Re: [PATCH] stm32-dcmi: constify vb2_ops structure

2017-07-07 Thread Hugues FRUCHET
Acked-by: Hugues Fruchet On 07/06/2017 10:05 PM, Gustavo A. R. Silva wrote: > Check for vb2_ops structures that are only stored in the ops field of a > vb2_queue structure. That field is declared const, so vb2_ops structures > that have this property can be declared as const also. > > This issue

Re: [PATCH] st-delta: constify vb2_ops structures

2017-07-07 Thread Hugues FRUCHET
Acked-by: Hugues Fruchet On 07/06/2017 10:14 PM, Gustavo A. R. Silva wrote: > Check for vb2_ops structures that are only stored in the ops field of a > vb2_queue structure. That field is declared const, so vb2_ops structures > that have this property can be declared as const also. > > This issue

Re: [PATCH 1/1] docs-rst: v4l: Fix sink compose selection target documentation

2017-07-07 Thread Hans Verkuil
On 07/07/17 16:20, Sakari Ailus wrote: > The rectangle which the sink compose rectangle is related to is documented > to be the source compose bounds rectangle. This is in obvious conflict with > the ground rule of the format propagation (from sink to source). The reason > behind this is that this

Re: coda 2040000.vpu: firmware request failed

2017-07-07 Thread Jagan Teki
On Fri, Jul 7, 2017 at 7:09 PM, Philipp Zabel wrote: > Hi Jagan, > > On Fri, 2017-07-07 at 18:15 +0530, Jagan Teki wrote: >> Hi, >> >> I'm observing firmware request failure with i.MX6Q board, This is with >> latest linux-next (4.12) with firmware from, [1] and converted >> v4l-coda960-imx6q.bin u

[PATCH 1/1] docs-rst: v4l: Fix sink compose selection target documentation

2017-07-07 Thread Sakari Ailus
The rectangle which the sink compose rectangle is related to is documented to be the source compose bounds rectangle. This is in obvious conflict with the ground rule of the format propagation (from sink to source). The reason behind this is that this was always supposed to be the sink compose boun

Re: coda 2040000.vpu: firmware request failed

2017-07-07 Thread Fabio Estevam
Hi Jagan, On Fri, Jul 7, 2017 at 9:45 AM, Jagan Teki wrote: > Hi, > > I'm observing firmware request failure with i.MX6Q board, This is with > latest linux-next (4.12) with firmware from, [1] and converted > v4l-coda960-imx6q.bin using [2]. There is no need to do the conversion with current code

Re: [PATCH 4/4] [media] rc: pwm-ir-tx: add new driver

2017-07-07 Thread Rob Herring
On Sun, Jul 02, 2017 at 12:06:13PM +0100, Sean Young wrote: > This is new driver which uses pwm, so it is more power-efficient > than the bit banging gpio-ir-tx driver. > > Signed-off-by: Sean Young > --- > .../devicetree/bindings/leds/irled/pwm-ir-tx.txt | 13 ++ Please make this a separate

Re: [PATCH 3/4] [media] rc: gpio-ir-tx: add new driver

2017-07-07 Thread Rob Herring
On Sun, Jul 02, 2017 at 12:06:11PM +0100, Sean Young wrote: > This is a simple bit-banging GPIO IR TX driver. > > Signed-off-by: Sean Young > --- > .../devicetree/bindings/leds/irled/gpio-ir-tx.txt | 11 ++ Please make this a separate patch. > drivers/media/rc/Kconfig

[PATCH v4] uvcvideo: add a metadata device node

2017-07-07 Thread Guennadi Liakhovetski
Some UVC video cameras contain metadata in their payload headers. This patch extracts that data, adding more clock synchronisation information, on both bulk and isochronous endpoints and makes it available to the user space on a separate video node, using the V4L2_CAP_META_CAPTURE capability and th

Re: coda 2040000.vpu: firmware request failed

2017-07-07 Thread Philipp Zabel
Hi Jagan, On Fri, 2017-07-07 at 18:15 +0530, Jagan Teki wrote: > Hi, > > I'm observing firmware request failure with i.MX6Q board, This is with > latest linux-next (4.12) with firmware from, [1] and converted > v4l-coda960-imx6q.bin using [2]. > > Log: > -- > coda 204.vpu: Direct firmwar

Re: [PATCH] staging: atomisp: replace kmalloc & memcpy with kmemdup

2017-07-07 Thread hari prasath
On 07-Jul-2017 5:25 PM, "Alan Cox" wrote: On Fri, 2017-07-07 at 17:20 +0530, Hari Prasath wrote: > kmemdup can be used to replace kmalloc followed by a memcpy.This was > pointed out by the coccinelle tool. And kstrdup could do the job even better I think ? > Yes & thanks for pointing me that. I

Re: [PATCH 7/8] omap3isp: Check for valid port in endpoints

2017-07-07 Thread Sakari Ailus
On Thu, Jul 06, 2017 at 01:11:49PM +0200, Sebastian Reichel wrote: > Hi, > > On Thu, Jul 06, 2017 at 02:00:18AM +0300, Sakari Ailus wrote: > > Check that we do have a valid port in an endpoint, return an error if not. > > > > Signed-off-by: Sakari Ailus > > Reviewed-by: Sebastian Reichel Than

Re: [PATCH 2/3] media: ti-vpe: cal: use of_graph_get_remote_endpoint()

2017-07-07 Thread Benoit Parrot
Acked-by: Benoit Parrot Kuninori Morimoto wrote on Wed [2017-Jun-28 00:33:00 +]: > > From: Kuninori Morimoto > > Now, we can use of_graph_get_remote_endpoint(). Let's use it. > > Signed-off-by: Kuninori Morimoto > --- > based on 4c9c3d595f1bad021cc126d20879df4016801736 > ("of_graph: a

coda 2040000.vpu: firmware request failed

2017-07-07 Thread Jagan Teki
Hi, I'm observing firmware request failure with i.MX6Q board, This is with latest linux-next (4.12) with firmware from, [1] and converted v4l-coda960-imx6q.bin using [2]. Log: -- coda 204.vpu: Direct firmware load for vpu_fw_imx6q.bin failed with error -2 coda 204.vpu: Direct firmware

Re: [PATCH] staging: atomisp: replace kmalloc & memcpy with kmemdup

2017-07-07 Thread Alan Cox
On Fri, 2017-07-07 at 17:20 +0530, Hari Prasath wrote: > kmemdup can be used to replace kmalloc followed by a memcpy.This was > pointed out by the coccinelle tool. And kstrdup could do the job even better I think ? Alan

[PATCH] staging: atomisp: replace kmalloc & memcpy with kmemdup

2017-07-07 Thread Hari Prasath
kmemdup can be used to replace kmalloc followed by a memcpy.This was pointed out by the coccinelle tool. Signed-off-by: Hari Prasath --- drivers/staging/media/atomisp/pci/atomisp2/css2400/sh_css_firmware.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/me

[PATCH v7.1 1/5] [media] cec.h: Add stub function for cec_register_cec_notifier()

2017-07-07 Thread Jose Abreu
Add a new stub function for cec_register_cec_notifier() so that we can still call this function when CONFIG_CEC_NOTIFIER and CONFIG_CEC_CORE are not set. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Hans Verkuil --- include/media/cec.h | 8 1 file changed, 8 insertions(+) diff -

Re: [PATCH v7 2/6] [media] cec-notifier.h: Prevent build warnings using forward declaration

2017-07-07 Thread Jose Abreu
Hi Hans, On 07-07-2017 12:16, Hans Verkuil wrote: > On 07/07/17 13:08, Jose Abreu wrote: >> When CONFIC_CEC_NOTIFIER is not set and we only include cec-notifier.h >> we can get build warnings like these ones: >> >> "warning: ‘struct cec_notifier’ declared inside parameter list will >> not be visi

Re: [PATCH v7 2/6] [media] cec-notifier.h: Prevent build warnings using forward declaration

2017-07-07 Thread Hans Verkuil
On 07/07/17 13:08, Jose Abreu wrote: > When CONFIC_CEC_NOTIFIER is not set and we only include cec-notifier.h > we can get build warnings like these ones: > > "warning: ‘struct cec_notifier’ declared inside parameter list will > not be visible outside of this definition or declaration" > > Preven

[PATCH v7 4/6] MAINTAINERS: Add entry for Synopsys Designware HDMI drivers

2017-07-07 Thread Jose Abreu
Add a entry for Synopsys Designware HDMI Receivers drivers and phys. Signed-off-by: Jose Abreu Cc: Carlos Palminha --- MAINTAINERS | 7 +++ 1 file changed, 7 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index c4be6d4..7ebc6dd 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -11354,6 +

[PATCH v7 3/6] dt-bindings: media: Document Synopsys Designware HDMI RX

2017-07-07 Thread Jose Abreu
Document the bindings for the Synopsys Designware HDMI RX. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Rob Herring Cc: Mark Rutland Cc: Mauro Carvalho Chehab Cc: Hans Verkuil Cc: Sylwester Nawrocki Cc: devicet...@vger.kernel.org Changes from v6: - Document which properties ar

[PATCH v7 0/6] Synopsys Designware HDMI Video Capture Controller + PHY

2017-07-07 Thread Jose Abreu
The Synopsys Designware HDMI RX controller is an HDMI receiver controller that is responsible to process digital data that comes from a phy. The final result is a stream of raw video data that can then be connected to a video DMA, for example, and transfered into RAM so that it can be displayed. T

[PATCH v7 5/6] [media] platform: Add Synopsys Designware HDMI RX PHY e405 Driver

2017-07-07 Thread Jose Abreu
This adds support for the Synopsys Designware HDMI RX PHY e405. This phy receives and decodes HDMI video that is delivered to a controller. Main features included in this driver are: - Equalizer algorithm that chooses the phy best settings according to the detected HDMI cable chara

[PATCH v7 2/6] [media] cec-notifier.h: Prevent build warnings using forward declaration

2017-07-07 Thread Jose Abreu
When CONFIC_CEC_NOTIFIER is not set and we only include cec-notifier.h we can get build warnings like these ones: "warning: ‘struct cec_notifier’ declared inside parameter list will not be visible outside of this definition or declaration" Prevent these warnings by using forward declaration of no

[PATCH v7 6/6] [media] platform: Add Synopsys Designware HDMI RX Controller Driver

2017-07-07 Thread Jose Abreu
This is an initial submission for the Synopsys Designware HDMI RX Controller Driver. This driver interacts with a phy driver so that a communication between them is created and a video pipeline is configured. The controller + phy pipeline can then be integrated into a fully featured system that ca

[PATCH v7 1/6] [media] cec.h: Add stub function for cec_register_cec_notifier()

2017-07-07 Thread Jose Abreu
Add a new stub function for cec_register_cec_notifier() so that we can still call this function when CONFIG_CEC_NOTIFIER and CONFIG_CEC_CORE are not set. Signed-off-by: Jose Abreu Cc: Carlos Palminha Cc: Hans Verkuil --- include/media/cec.h | 7 +++ 1 file changed, 7 insertions(+) diff --

Re: [PATCH] staging: atomisp: gc0310: constify acpi_device_id.

2017-07-07 Thread Alan Cox
On Thu, 2017-07-06 at 21:50 +0530, Arvind Yadav wrote: > acpi_device_id are not supposed to change at runtime. All functions > working with acpi_device_id provided by work with > const acpi_device_id. So mark the non-const structs as const. > > File size before: >    text    data bss   

Re: RFC: Selecting which sensor discrete frame size to use

2017-07-07 Thread Hans Verkuil
On 07/07/17 11:03, Hans Verkuil wrote: > Hi all, > > Hugues wants to add cropping support to the stm32-dcmi driver. The problem he > encountered is that the sensor driver has a list of discrete framesizes and > that causes problems with the API. > > Currently S_FMT is used to select which framesi

[PATCH 5/5] [media] coda: mark CODA960 firmware versions 2.3.10 and 3.1.1 as supported

2017-07-07 Thread Philipp Zabel
Firmware versions 2.3.10 revision 40778 and 3.1.1 revision 46072 are contained in the i.MX firmware download archives provided by NXP at http://www.nxp.com/lgfiles/NMG/MAD/YOCTO/firmware-imx-3.5.7-1.0.0.bin and http://www.nxp.com/lgfiles/NMG/MAD/YOCTO/firmware-imx-5.4.bin, respectively. Signed-off

[PATCH 3/5] [media] coda: align internal mpeg4 framebuffers to 16x16 macroblocks

2017-07-07 Thread Philipp Zabel
This fixes visual artifacts in the first macroblock row of encoded MPEG-4 video output caused by 8 additional lines of luma data leaking into the chroma planes of the internal reference framebuffers: the buffer size is rounded up to a multiple of 16x16 macroblock size, same as for the h.264 encoder

[PATCH 2/5] [media] coda: set field of destination buffers

2017-07-07 Thread Philipp Zabel
Set the field of destination buffers properly. Signed-off-by: Philipp Zabel --- drivers/media/platform/coda/coda-bit.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/media/platform/coda/coda-bit.c b/drivers/media/platform/coda/coda-bit.c index 50eed636830f8..a4abeabfa5377 100644

[PATCH 4/5] [media] coda: set MPEG-4 encoder class register

2017-07-07 Thread Philipp Zabel
Explicitly set MPEG-4 encoder class register instead of relying on the default value of 0. Signed-off-by: Philipp Zabel --- drivers/media/platform/coda/coda-bit.c | 4 drivers/media/platform/coda/coda_regs.h | 1 + 2 files changed, 5 insertions(+) diff --git a/drivers/media/platform/coda/

[PATCH 1/5] [media] coda: extend GOP size range

2017-07-07 Thread Philipp Zabel
CodaDx6 only accepts GOP sizes up to 60 frames, but CODA960 can handle up to 99 frames. If we disable automatic I frame generation altogether by setting GOP size to 0, we can let an application produce arbitrarily large I frame intervals using the V4L2_CID_MPEG_VIDEO_FORCE_KEY_FRAME control. Signe

[PATCH v2 5/6] [media] dt-bindings: gpio-ir-tx: add support for GPIO IR Transmitter

2017-07-07 Thread Sean Young
Document the device tree bindings for the GPIO Bit Banging IR Transmitter. Signed-off-by: Sean Young --- Documentation/devicetree/bindings/leds/irled/gpio-ir-tx.txt | 11 +++ 1 file changed, 11 insertions(+) create mode 100644 Documentation/devicetree/bindings/leds/irled/gpio-ir-tx.txt

[PATCH v2 6/6] [media] dt-bindings: pwm-ir-tx: Add support for PWM IR Transmitter

2017-07-07 Thread Sean Young
Document the device tree bindings for the PWM IR Transmitter. Signed-off-by: Sean Young --- Documentation/devicetree/bindings/leds/irled/pwm-ir-tx.txt | 13 + 1 file changed, 13 insertions(+) create mode 100644 Documentation/devicetree/bindings/leds/irled/pwm-ir-tx.txt diff --git a

Re: [PATCH v6 2/4] [media] platform: Add Synopsys Designware HDMI RX Controller Driver

2017-07-07 Thread Hans Verkuil
On 07/07/17 11:34, Jose Abreu wrote: > On 06-07-2017 21:52, kbuild test robot wrote: >>drivers/media//platform/dwc/dw-hdmi-rx.c: In function >> 'dw_hdmi_registered': drivers/media//platform/dwc/dw-hdmi-rx.c:1452:9: error: implicit declaration of function 'v4l2_async_subnotifier_regi

[PATCH v2 3/6] [media] rc: gpio-ir-tx: add new driver

2017-07-07 Thread Sean Young
This is a simple bit-banging GPIO IR TX driver. Signed-off-by: Sean Young --- MAINTAINERS | 6 ++ drivers/media/rc/Kconfig | 11 +++ drivers/media/rc/Makefile | 1 + drivers/media/rc/gpio-ir-tx.c | 189 ++ 4 files changed, 2

[PATCH v2 4/6] [media] rc: pwm-ir-tx: add new driver

2017-07-07 Thread Sean Young
This is new driver which uses pwm, so it is more power-efficient than the bit banging gpio-ir-tx driver. Signed-off-by: Sean Young --- MAINTAINERS | 6 ++ drivers/media/rc/Kconfig | 12 drivers/media/rc/Makefile| 1 + drivers/media/rc/pwm-ir-tx.c | 138

[PATCH v2 2/6] [media] rc: mce kbd decoder not needed for IR TX drivers

2017-07-07 Thread Sean Young
Without this patch, an input device is created which is not necessary. Signed-off-by: Sean Young --- drivers/media/rc/ir-mce_kbd-decoder.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/media/rc/ir-mce_kbd-decoder.c b/drivers/media/rc/ir-mce_kbd-decoder.c index 6a4d58b..0e07442

[PATCH v2 1/6] [media] rc-core: rename input_name to device_name

2017-07-07 Thread Sean Young
When an ir-spi is registered, you get this message. rc rc0: Unspecified device as /devices/platform/soc/3f215080.spi/spi_master/spi32766/spi32766.128/rc/rc0 "Unspecified device" refers to input_name, which makes no sense for IR TX only devices. So, rename to device_name. Also make driver_name c

[PATCH v2 0/6] Generic Raspberry Pi IR transmitters

2017-07-07 Thread Sean Young
These drivers are generic, but they've been tested on Raspberry Pi. Note that the gpio-ir-recv is a separate driver, so you will end up with two /dev/lircN devices if you want a receiver and a transmitter, whilst with the lirc_rpi driver you have one /dev/lircN device. Also note that it might be

Re: [PATCH v6 2/4] [media] platform: Add Synopsys Designware HDMI RX Controller Driver

2017-07-07 Thread Jose Abreu
On 06-07-2017 21:52, kbuild test robot wrote: >drivers/media//platform/dwc/dw-hdmi-rx.c: In function 'dw_hdmi_registered': >>> drivers/media//platform/dwc/dw-hdmi-rx.c:1452:9: error: implicit >>> declaration of function 'v4l2_async_subnotifier_register' >>> [-Werror=implicit-function-declarat

Re: [PATCH v6 4/4] dt-bindings: media: Document Synopsys Designware HDMI RX

2017-07-07 Thread Jose Abreu
On 06-07-2017 21:30, Sylwester Nawrocki wrote: > On 07/06/2017 12:24 PM, Jose Abreu wrote: +- edid-phandle: phandle to the EDID handler block. >>> Could you make this property optional and when it is missing assume that >>> device >>> corresponding to the parent node of this node handles EDID

RFC: Selecting which sensor discrete frame size to use

2017-07-07 Thread Hans Verkuil
Hi all, Hugues wants to add cropping support to the stm32-dcmi driver. The problem he encountered is that the sensor driver has a list of discrete framesizes and that causes problems with the API. Currently S_FMT is used to select which framesize to use. Which works fine as long as there is no c

[PATCH v2] [media] v4l2-mediabus: Add helper functions

2017-07-07 Thread Todor Tomov
Add helper functions for mbus to/from mplane pixel format conversion. Signed-off-by: Todor Tomov --- include/media/v4l2-mediabus.h | 26 ++ 1 file changed, 26 insertions(+) diff --git a/include/media/v4l2-mediabus.h b/include/media/v4l2-mediabus.h index 34cc99e..125afcd

Re: [PATCH] [media] v4l2-mediabus: Add helper functions

2017-07-07 Thread Todor Tomov
On 07/07/2017 11:23 AM, Hans Verkuil wrote: > On 07/07/2017 10:18 AM, Todor Tomov wrote: >> Add helper functions for mbus to/from mplane pixel format conversion. >> >> Signed-off-by: Todor Tomov >> --- >> include/media/v4l2-mediabus.h | 26 ++ >> 1 file changed, 26 inser

[PATCH] [media] rc: constify attribute_group structures.

2017-07-07 Thread Arvind Yadav
attribute_groups are not supposed to change at runtime. All functions working with attribute_groups provided by work with const attribute_group. So mark the non-const structs as const. File size before: textdata bss dec hex filename 11605 880 20 1250530d9 drive

Re: [PATCH] [media] v4l2-mediabus: Add helper functions

2017-07-07 Thread Hans Verkuil
On 07/07/2017 10:18 AM, Todor Tomov wrote: Add helper functions for mbus to/from mplane pixel format conversion. Signed-off-by: Todor Tomov --- include/media/v4l2-mediabus.h | 26 ++ 1 file changed, 26 insertions(+) diff --git a/include/media/v4l2-mediabus.h b/includ

[PATCH] [media] v4l2-mediabus: Add helper functions

2017-07-07 Thread Todor Tomov
Add helper functions for mbus to/from mplane pixel format conversion. Signed-off-by: Todor Tomov --- include/media/v4l2-mediabus.h | 26 ++ 1 file changed, 26 insertions(+) diff --git a/include/media/v4l2-mediabus.h b/include/media/v4l2-mediabus.h index 34cc99e..f97fd4a

[PATCH] [media] imon: constify attribute_group structures.

2017-07-07 Thread Arvind Yadav
attribute_groups are not supposed to change at runtime. All functions working with attribute_groups provided by work with const attribute_group. So mark the non-const structs as const. File size before: textdata bss dec hex filename 185512256 77 208845194 drive

Re: [PATCH 03/12] [media] vb2: add in-fence support to QBUF

2017-07-07 Thread Hans Verkuil
On 07/07/2017 03:53 AM, Gustavo Padovan wrote: help If you want to use Webcams, Video grabber devices and/or TV devices enable this option and other options below. diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c index

Re: [PATCH 03/12] [media] vb2: add in-fence support to QBUF

2017-07-07 Thread Hans Verkuil
On 07/07/2017 04:00 AM, Gustavo Padovan wrote: 2017-07-06 Hans Verkuil : On 06/16/17 09:39, Gustavo Padovan wrote: 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 th