Re: Recent nuvoton-cir changes introduce a hang with nct6775

On 7 July 2016 at 00:25, Heiner Kallweit wrote: > Am 06.07.2016 um 08:51 schrieb Antti Seppälä: >> Hello. >> >> I recently updated my kernel to a newer version but couldn't boot it >> because it hangs. >> >> It turns out that your patch[1] to nuvoton-cir has really bad >>

Re: [PATCH 6/6] ARM: dts: imx6-sabreauto: add video capture devices and connections

Hi, [auto build test ERROR on v4.7-rc6] [cannot apply to shawnguo/for-next next-20160706] [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/Steve-Longerbeam/ARM-dts-imx6-sabre-add-video-capture

Hi linux

salutations linux http://vistacayrents.com/clearly.php?sense=1f6bd29b2qzrsw rjkft -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 2/6] ARM: dts: imx6-sabresd: add video capture devices and connections

Hi, [auto build test ERROR on v4.7-rc6] [cannot apply to shawnguo/for-next next-20160706] [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/Steve-Longerbeam/ARM-dts-imx6-sabre-add-video-capture

cron job: media_tree daily build: ERRORS

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: Thu Jul 7 04:00:17 CEST 2016 git branch: test git hash: d81295d1bed850335f9f4ccb6b1aa4f6a123d4f0 gcc

Re: [PATCH v5 4/4] rcar-vin: implement EDID control ioctls

Hi Ulrich, Thanks for your patch. On 2016-07-06 17:39:36 +0200, Ulrich Hecht wrote: > Adds G_EDID and S_EDID. > > Signed-off-by: Ulrich Hecht > --- > drivers/media/platform/rcar-vin/rcar-v4l2.c | 17 + > 1 file changed, 17 insertions(+) > >

[PATCH 4/6] ARM: dts: imx6-sabreauto: add reset-gpios property for max7310_b

The reset pin to the port expander chip (MAX7310) is controlled by a gpio, so define a reset-gpios property to control it. There are three MAX7310's on the SabreAuto CPU card (max7310_[abc]), but all use the same pin for their reset. Since all can't acquire the same pin, assign it to max7310_b,

[PATCH 2/6] ARM: dts: imx6-sabresd: add video capture devices and connections

Defines two host video capture devices and two camera sensors: parallel-bus OV5642 and MIPI CSI-2 OV5640. One host capture device connects to the OV5642 via the parallel-bus mux input on the ipu1_csi0_mux. The other host capture device connects to the OV5640 via the MIPI CSI-2 receiver (directly

[PATCH 1/6] ARM: dts: imx6-sabrelite: add video capture devices and connections

Defines the host video capture device and the OV5642 parallel-bus sensor. The host capture device connects to the OV5642 via the parallel-bus mux input on the ipu1_csi0_mux. Note there is a pin conflict with GPIO6. This pin functions as a power input pin to the OV5642, but ENET requires it to

[PATCH 3/6] ARM: dts: imx6-sabreauto: create i2cmux for i2c3

The sabreauto uses a steering pin to select between the SDA signal on i2c3 bus, and a data-in pin for an SPI NOR chip. Use i2cmux to control this steering pin. Idle state of the i2cmux selects SPI NOR. This is not a classic way to use i2cmux, since one side of the mux selects something other than

[PATCH 6/6] ARM: dts: imx6-sabreauto: add video capture devices and connections

Defines the host video capture device node and the ADV7180 decoder sensor. The host capture device connects to the ADV7180 via the parallel-bus mux input on the ipu1_csi0_mux. On the sabreauto, two analog video inputs are routed to the ADV7180, composite on Ain1, and composite on Ain3. Those

[PATCH 5/6] ARM: dts: imx6-sabreauto: add pinctrl for gpt input capture

Add pinctrl groups for both GPT input capture channels. Signed-off-by: Steve Longerbeam --- arch/arm/boot/dts/imx6qdl-sabreauto.dtsi | 12 1 file changed, 12 insertions(+) diff --git a/arch/arm/boot/dts/imx6qdl-sabreauto.dtsi

[PATCH 0/6] ARM: dts: imx6-sabre*: add video capture nodes

Steve Longerbeam (6): ARM: dts: imx6-sabrelite: add video capture devices and connections ARM: dts: imx6-sabresd: add video capture devices and connections ARM: dts: imx6-sabreauto: create i2cmux for i2c3 ARM: dts: imx6-sabreauto: add reset-gpios property for max7310_b ARM: dts:

[PATCH 06/11] media: adv7180: add bt.656-4 OF property

Add a device tree boolean property "bt656-4" to allow setting the ITU-R BT.656-4 compatible bit. Signed-off-by: Steve Longerbeam --- drivers/media/i2c/adv7180.c | 18 +- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git

[PATCH 09/11] v4l: Add signal lock status to source change events

Add a signal lock status change to the source changes bitmask. This indicates there was a signal lock or unlock event detected at the input of a video decoder. Signed-off-by: Steve Longerbeam --- Documentation/DocBook/media/v4l/vidioc-dqevent.xml | 12 ++--

[PATCH 03/11] media: adv7180: add power pin control

Some targets control the ADV7180 power pin via a gpio, so add support for "pwdn-gpio" pin control. Signed-off-by: Steve Longerbeam --- drivers/media/i2c/Kconfig | 2 +- drivers/media/i2c/adv7180.c | 37 + 2 files changed, 38

[PATCH 04/11] media: adv7180: implement g_parm

Implement g_parm to return the current standard's frame period. Signed-off-by: Steve Longerbeam --- drivers/media/i2c/adv7180.c | 22 ++ 1 file changed, 22 insertions(+) diff --git a/drivers/media/i2c/adv7180.c b/drivers/media/i2c/adv7180.c

[PATCH 01/11] media: adv7180: Fix broken interrupt register access

Access to the interrupt page registers has been broken since at least 3999e5d01da74f1a22afbb0b61b3992fea301478. That commit forgot to add the inerrupt page number to the register defines. Signed-off-by: Steve Longerbeam --- drivers/media/i2c/adv7180.c | 18

[PATCH 05/11] media: adv7180: init chip with AD recommended register settings

Define and load register tables that conform to Analog Device's recommended register settings. It loads the default single-ended CVBS on Ain1 configuration for both ADV7180 and ADV7182 chips. New register addresses have been defined for the tables. Those new defines are also used in existing

[PATCH 07/11] media: adv7180: change mbus format to UYVY

Change the media bus format from YUYV8_2X8 to UYVY8_2X8. Colors now look correct when capturing with the i.mx6 backend. The other option is to set the SWPC bit in register 0x27 to swap the Cr and Cb output samples. Signed-off-by: Steve Longerbeam ---

[PATCH 00/11] adv7180 subdev fixes

Steve Longerbeam (11): media: adv7180: Fix broken interrupt register access Revert "[media] adv7180: fix broken standards handling" media: adv7180: add power pin control media: adv7180: implement g_parm media: adv7180: init chip with AD recommended register settings media: adv7180: add

[PATCH 10/11] media: adv7180: enable lock/unlock interrupts

Enable the SD lock/unlock interrupts and send V4L2_EVENT_SRC_CH_LOCK_STATUS in the interrupt handler on a detected lock/unlock. Keep track of current input lock status with state->curr_status. Signed-off-by: Steve Longerbeam --- drivers/media/i2c/adv7180.c | 35

[PATCH 08/11] adv7180: send V4L2_EVENT_SOURCE_CHANGE on std change

Subscribe to the V4L2_EVENT_SOURCE_CHANGE event and send V4L2_EVENT_SRC_CH_RESOLUTION in the interrupt handler on a detected standard change. Signed-off-by: Steve Longerbeam --- drivers/media/i2c/adv7180.c | 33 ++--- 1 file changed, 30

[PATCH 02/11] Revert "[media] adv7180: fix broken standards handling"

Autodetect was likely broken only because access to the interrupt registers were broken, so there were no standard change interrupts. After fixing that, and reverting this, autodetect seems to work just fine on an i.mx6q SabreAuto. This reverts commit 937feeed3f0ae8a0389d5732f6db63dd912acd99. ---

[PATCH 11/11] media: adv7180: fix field type

The ADV7180 and ADV7182 transmit whole fields, bottom field followed by top (or vice-versa, depending on detected video standard). So for chips that do not have support for explicitly setting the field mode, set the field mode to SEQ_BT for PAL, and SEQ_TB for NTSC (there seems to be conflicting

[PATCH 28/28] ARM: imx_v6_v7_defconfig: Enable staging video4linux drivers

Enable imx v4l2 staging drivers. For video capture on the SabreAuto, the ADV7180 video decoder also requires the i2c-mux-gpio and the max7310 port expander. Signed-off-by: Steve Longerbeam --- arch/arm/configs/imx_v6_v7_defconfig | 2 ++ 1 file changed, 2

[PATCH 27/28] ARM: dts: imx6qdl: add mem2mem devices

Enables ipu-mem2mem devices on imx6qdl. Signed-off-by: Steve Longerbeam --- arch/arm/boot/dts/imx6q.dtsi | 7 +++ arch/arm/boot/dts/imx6qdl.dtsi | 6 ++ 2 files changed, 13 insertions(+) diff --git a/arch/arm/boot/dts/imx6q.dtsi

[PATCH 03/28] gpu: ipu-cpmem: Add ipu_cpmem_get_burstsize()

Adds ipu_cpmem_get_burstsize(). Signed-off-by: Steve Longerbeam --- drivers/gpu/ipu-v3/ipu-cpmem.c | 6 ++ include/video/imx-ipu-v3.h | 1 + 2 files changed, 7 insertions(+) diff --git a/drivers/gpu/ipu-v3/ipu-cpmem.c b/drivers/gpu/ipu-v3/ipu-cpmem.c index

[PATCH 18/28] clocksource/drivers/imx: add input capture support

This patch adds support for the input capture function in the i.MX GPT. Output compare and input capture functions are mixed in the same register block, so we need to modify the irq ack/enable/ disable primitives to not stomp on the other function. The input capture API is modelled after

[PATCH 15/28] gpu: ipu-ic: allow multiple handles to ic

The image converter kernel API supports conversion contexts and job queues, so we should allow more than one handle to the IC, so that multiple users can add jobs to the queue. Note however that users that control the IC manually (that do not use the image converter APIs but setup the IC task by

[PATCH 20/28] media: imx: Add MIPI CSI-2 Receiver driver

Adds MIPI CSI-2 Receiver subdev driver. This subdev is required for sensors with a MIPI CSI2 interface. Signed-off-by: Steve Longerbeam --- drivers/staging/media/imx/capture/Kconfig | 8 + drivers/staging/media/imx/capture/Makefile| 1 +

[PATCH 02/28] gpu: ipu-cpmem: Add ipu_cpmem_set_uv_offset()

Adds ipu_cpmem_set_uv_offset(), to set planar U/V offsets. Signed-off-by: Steve Longerbeam --- drivers/gpu/ipu-v3/ipu-cpmem.c | 7 +++ include/video/imx-ipu-v3.h | 1 + 2 files changed, 8 insertions(+) diff --git a/drivers/gpu/ipu-v3/ipu-cpmem.c

[PATCH 21/28] media: imx: Add video switch

From: Philipp Zabel This driver can handle SoC internal and extern video bus multiplexers, controlled either by register bit fields or by GPIO. Signed-off-by: Sascha Hauer Signed-off-by: Philipp Zabel Signed-off-by: Steve

[PATCH 24/28] media: Add i.MX5/6 mem2mem driver

Adds a V4L2 mem2mem driver for i.MX5/6 SoC. Uses the IPU IC image converter (post-processor task) to perform scaling, rotation, and color space conversion. Signed-off-by: Steve Longerbeam --- Documentation/devicetree/bindings/media/imx.txt | 25 +-

[PATCH 22/28] media: imx: Add support for MIPI CSI-2 OV5640

This driver is based on ov5640_mipi.c from Freescale imx_3.10.17_1.0.0_beta branch, modified heavily for code cleanup and converted from int-device to subdev. Signed-off-by: Steve Longerbeam --- drivers/staging/media/imx/capture/Kconfig |8 +

[PATCH 25/28] ARM: dts: imx6qdl: Flesh out MIPI CSI2 receiver node

Add to the MIPI CSI2 receiver node: compatible string, interrupt sources, clocks. Signed-off-by: Steve Longerbeam --- arch/arm/boot/dts/imx6qdl.dtsi | 7 +++ 1 file changed, 7 insertions(+) diff --git a/arch/arm/boot/dts/imx6qdl.dtsi

[PATCH 26/28] ARM: dts: imx6qdl: Add mipi_ipu1/2 video muxes, mipi_csi, and their connections

From: Philipp Zabel This patch adds the device tree graph connecting the input multiplexers to the IPU CSIs and the MIPI-CSI2 gasket on i.MX6. On i.MX6Q/D two two-input multiplexers in front of IPU1 CSI0 and IPU2 CSI1 allow to select between CSI0/1 parallel input pads

[PATCH 11/28] gpu: ipu-v3: Fix CSI data format for 16-bit media bus formats

The CSI data format was being programmed incorrectly for the 1x16 media bus formats. The CSI data format for 16-bit must be bayer/generic (CSI_SENS_CONF_DATA_FMT_BAYER). Suggested-by: Carsten Resch Signed-off-by: Steve Longerbeam ---

[PATCH 14/28] gpu: ipu-ic: Add complete image conversion support with tiling

This patch implements complete image conversion support to ipu-ic, with tiling to support scaling to and from images up to 4096x4096. Image rotation is also supported. The internal API is subsystem agnostic (no V4L2 dependency except for the use of V4L2 fourcc pixel formats). Callers prepare for

[PATCH 06/28] gpu: ipu-v3: Add ipu_set_vdi_src_mux()

Adds ipu_set_vdi_src_mux() that selects the VDIC input (from CSI or memory). Signed-off-by: Steve Longerbeam --- drivers/gpu/ipu-v3/ipu-common.c | 20 include/video/imx-ipu-v3.h | 1 + 2 files changed, 21 insertions(+) diff --git

[PATCH 07/28] gpu: ipu-v3: Add VDI input IDMAC channels

Adds the VDIC field input IDMAC channels. These channels transfer fields F(n-1), F(n), and F(N+1) from memory to the VDIC (channels 8, 9, 10 respectively). Signed-off-by: Steve Longerbeam --- include/video/imx-ipu-v3.h | 3 +++ 1 file changed, 3 insertions(+) diff

[PATCH 17/28] gpio: pca953x: Add optional reset gpio control

Add optional reset-gpios pin control. If present, de-assert the specified reset gpio pin to bring the chip out of reset. Signed-off-by: Steve Longerbeam --- drivers/gpio/gpio-pca953x.c | 18 ++ 1 file changed, 18 insertions(+) diff --git

[PATCH 16/28] gpu: ipu-v3: rename CSI client device

Rename the CSI client device in the client_reg[] table to "imx-ipuv3-csi". Signed-off-by: Steve Longerbeam --- drivers/gpu/ipu-v3/ipu-common.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/ipu-v3/ipu-common.c

[PATCH 13/28] gpu: ipu-v3: Fix IRT usage

There can be multiple IC tasks using the IRT, so the IRT needs a separate use counter. Create a private ipu_irt_enable() to enable the IRT module when any IC task requires rotation, and ipu_irt_disable() when a task no longer needs the IRT. Signed-off-by: Steve Longerbeam

[PATCH 12/28] gpu: ipu-v3: Fix CSI0 blur in NTSC format

From: Suresh Dhandapani This patch will change the register IPU_CSI0_CCIR_CODE_2 value from 0x40596 to 0x405A6. The change is related to the Start of field 1 first blanking line command bit[5-3] for NTSC format only. This change is dependent with ADV chip where

[PATCH 05/28] gpu: ipu-v3: Add IDMA channel linking support

Adds functions to link and unlink IDMAC source channels to sink channels. So far the following links are supported: IPUV3_CHANNEL_IC_PRP_ENC_MEM -> IPUV3_CHANNEL_MEM_ROT_ENC PUV3_CHANNEL_IC_PRP_VF_MEM -> IPUV3_CHANNEL_MEM_ROT_VF IPUV3_CHANNEL_IC_PP_MEM -> IPUV3_CHANNEL_MEM_ROT_PP More

[PATCH 08/28] gpu: ipu-v3: Add ipu_csi_set_src()

Adds ipu_csi_set_src() which is just a wrapper around ipu_set_csi_src_mux(). Signed-off-by: Steve Longerbeam --- drivers/gpu/ipu-v3/ipu-csi.c | 8 include/video/imx-ipu-v3.h | 1 + 2 files changed, 9 insertions(+) diff --git

[PATCH 09/28] gpu: ipu-v3: Add ipu_ic_set_src()

Adds ipu_ic_set_src() which is just aa wrapper around ipu_set_ic_src_mux(). Signed-off-by: Steve Longerbeam --- drivers/gpu/ipu-v3/ipu-ic.c | 10 ++ include/video/imx-ipu-v3.h | 1 + 2 files changed, 11 insertions(+) diff --git

[PATCH 04/28] gpu: ipu-v3: Add ipu_get_num()

Adds of-alias id to ipu_soc and retrieve with ipu_get_num(). Signed-off-by: Steve Longerbeam --- drivers/gpu/ipu-v3/ipu-common.c | 8 drivers/gpu/ipu-v3/ipu-prv.h| 1 + include/video/imx-ipu-v3.h | 1 + 3 files changed, 10 insertions(+) diff --git

[PATCH 10/28] gpu: ipu-v3: set correct full sensor frame for PAL/NTSC

Set the sensor full frame based on whether the passed in mbus_fmt is 720x480 (NTSC) or 720x576 (PAL). Signed-off-by: Steve Longerbeam --- drivers/gpu/ipu-v3/ipu-csi.c | 20 +--- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git

[PATCH 01/28] gpu: ipu-v3: Add Video Deinterlacer unit

Adds the Video Deinterlacer (VDIC) unit. Signed-off-by: Steve Longerbeam --- drivers/gpu/ipu-v3/Makefile | 2 +- drivers/gpu/ipu-v3/ipu-common.c | 11 ++ drivers/gpu/ipu-v3/ipu-prv.h| 6 + drivers/gpu/ipu-v3/ipu-vdi.c| 266

[PATCH 00/28] i.MX5/6 Video Capture, v2

Philipp Zabel (2): media: imx: Add video switch ARM: dts: imx6qdl: Add mipi_ipu1/2 video muxes, mipi_csi, and their connections Steve Longerbeam (25): gpu: ipu-v3: Add Video Deinterlacer unit gpu: ipu-cpmem: Add ipu_cpmem_set_uv_offset() gpu: ipu-cpmem: Add ipu_cpmem_get_burstsize()

Re: Recent nuvoton-cir changes introduce a hang with nct6775

Am 06.07.2016 um 08:51 schrieb Antti Seppälä: > Hello. > > I recently updated my kernel to a newer version but couldn't boot it > because it hangs. > > It turns out that your patch[1] to nuvoton-cir has really bad > side-effects when it interacts with nct6775 module. It could be that > one of

Re: [PATCH v2 1/3] sur40: properly report a single frame rate of 60 FPS

On 06.07.2016 10:40, Hans Verkuil wrote: > On 07/05/16 09:06, Hans Verkuil wrote: >> On 07/05/2016 08:56 AM, Florian Echtler wrote: >>> On 05.07.2016 08:41, Hans Verkuil wrote: Why is s_parm added when you can't change the framerate? >>> >>> Oh, I thought it's mandatory to always have

Re: [PATCH v5 4/4] rcar-vin: implement EDID control ioctls

Hi, [auto build test ERROR on next-20160706] [cannot apply to linuxtv-media/master renesas/next v4.7-rc6 v4.7-rc5 v4.7-rc4 v4.7-rc6] [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/Ulrich-Hecht

[PATCH v5 0/4] Lager/Koelsch board HDMI input support

Hi! Sorry for the delay. This revision drops all patches that have since been picked up. It amends the default input selection to fall back to input 0 if nothing else is specified, and it replaces the hard-coded EDID blob with an implementation of G_EDID and S_EDID in rcar-vin. CU Uli

[PATCH v5 1/4] media: adv7604: automatic "default-input" selection

Fall back to input 0 if "default-input" property is not present. Additionally, documentation in commit bf9c82278c34 ("[media] media: adv7604: ability to read default input port from DT") states that the "default-input" property should reside directly in the node for adv7612. Hence, also adjust

[PATCH v5 2/4] ARM: dts: lager: Add entries for VIN HDMI input support

From: William Towle Add DT entries for vin0, vin0_pins, and adv7612. Sets the 'default-input' property for ADV7612, enabling image and video capture without the need to have userspace specifying routing. Signed-off-by: William Towle

[PATCH v5 4/4] rcar-vin: implement EDID control ioctls

Adds G_EDID and S_EDID. Signed-off-by: Ulrich Hecht --- drivers/media/platform/rcar-vin/rcar-v4l2.c | 17 + 1 file changed, 17 insertions(+) diff --git a/drivers/media/platform/rcar-vin/rcar-v4l2.c b/drivers/media/platform/rcar-vin/rcar-v4l2.c

[PATCH v5 3/4] ARM: dts: koelsch: add HDMI input

From: Hans Verkuil Add support in the dts for the HDMI input. Based on the Lager dts patch from Ultich Hecht. Signed-off-by: Hans Verkuil [uli: removed "renesas," prefixes from pfc nodes] Signed-off-by: Ulrich Hecht

Re: [PATCH v3 0/9] Add MT8173 Video Decoder Driver

Hi Tiffany, I plan to review this patch series on Friday, but one obvious question is what the reason for these failures is: > Input/Output configuration ioctls: > test VIDIOC_ENUM/G/S/QUERY_STD: OK (Not Supported) > test VIDIOC_ENUM/G/S/QUERY_DV_TIMINGS: OK (Not Supported) >

Re: [PATCH 3/9] omap_vout: convert g/s_crop to g/s_selection.

Hi Hans, Thank you for the patch. On Monday 04 Jul 2016 10:32:16 Hans Verkuil wrote: > From: Hans Verkuil > > This is part of a final push to convert all drivers to g/s_selection. > > Signed-off-by: Hans Verkuil > Cc: Laurent Pinchart

[PATCH v3 10/15] [media] lirc_dev: remove compat_ioctl assignment

There is no need to check for CONFIG_COMPAT and consequently assign the compat_ioctl. Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/media/rc/lirc_dev.c b/drivers/media/rc/lirc_dev.c index

[PATCH v3 03/15] [media] lirc_dev: remove unnecessary debug prints

Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 25 - 1 file changed, 25 deletions(-) diff --git a/drivers/media/rc/lirc_dev.c b/drivers/media/rc/lirc_dev.c index 154e553..9f20f94 100644 --- a/drivers/media/rc/lirc_dev.c +++

[PATCH v3 01/15] [media] lirc_dev: place buffer allocation on separate function

During the driver registration, move the buffer allocation on a separate function. Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 57 +++-- 1 file changed, 34 insertions(+), 23 deletions(-) diff --git

[PATCH v3 15/15] [media] lirc_dev: use LIRC_CAN_REC() define to check if the device can receive

The LIRC_CAN_REC() returns a boolean "flag & LIRC_CAN_REC_MASK" to check whether the device can receive data. Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/rc/lirc_dev.c

[PATCH v3 02/15] [media] lirc_dev: allow bufferless driver registration

Transmitters don't necessarily need to have a FIFO managed buffer for their transfers. When registering the driver, before allocating the buffer, check whether the device is a transmitter or receiver. Allocate the buffer only for receivers. Signed-off-by: Andi Shyti ---

[PATCH v3 00/15] lirc_dev fixes and beautification

Hi, This is a collection of fixes, added functionality, coding rework and trivial coding style fixes. The first patch is preparatory to the second, which allows the user to create a lirc driver without receiver buffer, which is obvious for transmitters. The rest of the patches is a series of

[PATCH v3 08/15] [media] lirc_dev: remove double if ... else statement

There are two if ... else which check the same thing in different part of the code, they can be merged in a single check. Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git

[PATCH v3 04/15] [media] lirc_dev: replace printk with pr_* or dev_*

This patch mutes also all the checkpatch warnings related to printk. Reword all the printouts so that the string doesn't need to be split, which fixes the following checkpatch warning: WARNING: quoted string split across lines Signed-off-by: Andi Shyti ---

[PATCH v3 12/15] [media] lirc_dev: fix error return value

If ioctl is called, it cannot be a case of invalid system call number (ENOSYS), that is a ENOTTY case which means that the device doesn't support that specific ioctl command. Replace ENOSYS with EPERM. Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 8

[PATCH v3 14/15] [media] lirc_dev: fix potential segfault

When opening or closing a lirc character device, the framework provides to the user the possibility to keep track of opening or closing of the device by calling two functions: - set_use_inc() when opening the device - set_use_dec() when closing the device if those are not set by the lirc user,

[PATCH v3 11/15] [media] lirc_dev: fix variable constant comparisons

When comparing a variable with a constant, the comparison should start from the variable and not from the constant. It's also written in the human DNA. Swap the terms of comparisons whenever the constant comes first and fix the following checkpatch warning: WARNING: Comparisons should place

[PATCH v3 09/15] [media] lirc_dev: merge three if statements in only one

The three if statements check the same thing, merge them in only one statement. Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/drivers/media/rc/lirc_dev.c

[PATCH v3 05/15] [media] lirc_dev: simplify goto paths

The code can be rearranged so that some goto paths can be removed Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 34 -- 1 file changed, 12 insertions(+), 22 deletions(-) diff --git a/drivers/media/rc/lirc_dev.c

[PATCH v3 06/15] [media] lirc_dev: do not use goto to create loops

... use "do .. while" instead. Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/drivers/media/rc/lirc_dev.c b/drivers/media/rc/lirc_dev.c index b11d026..cfa6031 100644 ---

[PATCH v3 13/15] [media] lirc_dev: extremely trivial comment style fix

Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/media/rc/lirc_dev.c b/drivers/media/rc/lirc_dev.c index ac00433..c78fe2b 100644 --- a/drivers/media/rc/lirc_dev.c +++

[PATCH v3 07/15] [media] lirc_dev: simplify if statement in lirc_add_to_buf

The whole function is inside an 'if' statement ("if (ir->d.add_to_buf)"). Check the opposite of that statement at the beginning and exit, this way we can have one level less of indentation. Signed-off-by: Andi Shyti --- drivers/media/rc/lirc_dev.c | 31

Re: [PATCH] media: rc: nuvoton: decrease size of raw event fifo

On Tue, Jul 05, 2016 at 07:48:34PM +0200, Heiner Kallweit wrote: > Am 05.07.2016 um 11:17 schrieb Sean Young: > > On Tue, Jul 05, 2016 at 08:05:16AM +0200, Heiner Kallweit wrote: > >> Am 04.07.2016 um 23:06 schrieb Sean Young: > >>> On Mon, Jul 04, 2016 at 10:51:50PM +0200, Heiner Kallweit wrote:

[PATCH] cec-funcs.h: add length checks

Add msg->len sanity checks to fix static checker warning: include/linux/cec-funcs.h:1154 cec_ops_set_osd_string() warn: setting length 'msg->len - 3' to negative one Signed-off-by: Hans Verkuil Reported-by: Dan Carpenter diff

Re: [PATCH v2 1/3] sur40: properly report a single frame rate of 60 FPS

On 07/05/16 09:06, Hans Verkuil wrote: > On 07/05/2016 08:56 AM, Florian Echtler wrote: >> Hello Hans, >> >> On 05.07.2016 08:41, Hans Verkuil wrote: >>> On 05/31/2016 10:15 PM, Florian Echtler wrote: The device hardware is always running at 60 FPS, so report this both via PARM_IOCTL and

[PATCH] vivid: set V4L2_CAP_TIMEPERFRAME

The vivid driver didn't set the V4L2_CAP_TIMEPERFRAME flag in s_parm for the non-webcam inputs. This caused a v4l2-compliance fail. Signed-off-by: Hans Verkuil diff --git a/drivers/media/platform/vivid/vivid-vid-cap.c b/drivers/media/platform/vivid/vivid-vid-cap.c index