cron job: media_tree daily build: ERRORS

2017-08-04 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: Sat Aug 5 05:00:16 CEST 2017 media-tree git hash:da48c948c263c9d87dfc64566b3373a858cc8aa2 media_build gi

[PATCH] imon: constify attribute_group structures

2017-08-04 Thread Amitoj Kaur Chawla
Functions working with attribute_groups provided by work with const attribute_group. These attribute_group structures do not change at runtime so mark them as const. File size before: text data bss dec hex filename 3698116776 960 54717d5bd drivers/media/rc/imon.o

[GIT PULL] linux-firmware: intel: Add Kabylake IPU3 firmware

2017-08-04 Thread Mani, Rajmohan
Hi, Please review this PULL request to add Kabylake IPU3 firmware to the linux-firmware repository. The following changes since commit 7d2c913dcd1be083350d97a8cb1eba24cfacbc8a: ath10k: update year in license (2017-06-22 12:06:02 -0700) are available in the git repository at: https://gith

[PATCH v7] media: platform: Renesas IMR driver

2017-08-04 Thread Sergei Shtylyov
The image renderer, or the distortion correction engine, is a drawing processor with a simple instruction system capable of referencing video capture data or data in an external memory as the 2D texture data and performing texture mapping and drawing with respect to any shape that is split into tri

Re: [PATCH v4 0/7] vsp1 partition algorithm improvements

2017-08-04 Thread Laurent Pinchart
Hi Kieran, On Friday 04 Aug 2017 17:32:37 Kieran Bingham wrote: > Hans, this series should be ready for integration now. Could you pick up > these patches please? Just a note for Hans: the series merges cleanly with Dave's drm-next branch (git://people.freedesktop.org/~airlied/linux) that contai

[PATCH v4 5/7] v4l: vsp1: Move partition rectangles to struct and operate directly

2017-08-04 Thread Kieran Bingham
As we develop the partition algorithm, we need to store more information per partition to describe the phase and other parameters. To keep this data together, further abstract the existing v4l2_rect into a partition specific structure. As partitions only have horizontal coordinates, store the left

[PATCH v4 7/7] v4l: vsp1: Allow entities to participate in the partition algorithm

2017-08-04 Thread Kieran Bingham
The configuration of the pipeline and entities directly affects the inputs required to each entity for the partition algorithm. Thus it makes sense to involve those entities in the decision making process. Extend the entity ops API to provide an optional .partition() operation. This allows entitie

[PATCH v4 6/7] v4l: vsp1: Provide UDS register updates

2017-08-04 Thread Kieran Bingham
Provide register definitions required for UDS phase and partition algorithm support. The registers and bits defined here are available on Gen3 hardware only. Signed-off-by: Kieran Bingham Reviewed-by: Laurent Pinchart --- drivers/media/platform/vsp1/vsp1_regs.h | 14 ++ 1 file chang

[PATCH v4 4/7] v4l: vsp1: Remove redundant context variables

2017-08-04 Thread Kieran Bingham
The vsp1_pipe object context variables for div_size and current_partition allowed state to be maintained through processing the partitions during processing. Now that the partition tables are calculated during stream on, there is no requirement to store these variables in the pipe object. Utilise

[PATCH v4 3/7] v4l: vsp1: Calculate partition sizes at stream start

2017-08-04 Thread Kieran Bingham
Previously the active window and partition sizes for each partition were calculated for each partition every frame. This data is constant and only needs to be calculated once at the start of the stream. Extend the vsp1_pipe object to dynamically store the number of partitions required and pre-calc

[PATCH v4 2/7] v4l: vsp1: Move vsp1_video_pipeline_setup_partitions() function

2017-08-04 Thread Kieran Bingham
Separate the code change from the function move so that code changes can be clearly identified. This commit has no functional change. The partition algorithm functions will be changed, and vsp1_video_pipeline_setup_partitions() will call vsp1_video_partition(). To prepare for that, move the functi

[PATCH v4 0/7] vsp1 partition algorithm improvements

2017-08-04 Thread Kieran Bingham
Hans, this series should be ready for integration now. Could you pick up these patches please? Some updates and initial improvements for the VSP1 partition algorithm that remove redundant processing and variables, reducing the processing done in interrupt context slightly. Patch 1, fixes up a bug

[PATCH v4 1/7] v4l: vsp1: Release buffers in start_streaming error path

2017-08-04 Thread Kieran Bingham
Presently any received buffers are only released back to vb2 if vsp1_video_stop_streaming() is called. If vsp1_video_start_streaming() encounters an error, we will be warned by the vb2 handlers that buffers have not been returned. Move the buffer cleanup code to its own function to prevent duplica

Re: [PATCH v3 1/7] v4l: vsp1: Release buffers in start_streaming error path

2017-08-04 Thread Kieran Bingham
Hi Laurent, On 04/08/17 17:13, Laurent Pinchart wrote: > On Friday 04 Aug 2017 17:08:34 Kieran Bingham wrote: >> On 04/08/17 17:03, Laurent Pinchart wrote: >>> On Friday 04 Aug 2017 16:57:05 Kieran Bingham wrote: Presently any received buffers are only released back to vb2 if vsp1_video_

Re: [PATCH v3 1/7] v4l: vsp1: Release buffers in start_streaming error path

2017-08-04 Thread Laurent Pinchart
Hi Kieran, On Friday 04 Aug 2017 17:08:34 Kieran Bingham wrote: > On 04/08/17 17:03, Laurent Pinchart wrote: > > On Friday 04 Aug 2017 16:57:05 Kieran Bingham wrote: > >> Presently any received buffers are only released back to vb2 if > >> vsp1_video_stop_streaming() is called. If vsp1_video_start

Re: [PATCH v3 1/7] v4l: vsp1: Release buffers in start_streaming error path

2017-08-04 Thread Kieran Bingham
Hi Laurent, Thankyou for the speedy review. On 04/08/17 17:03, Laurent Pinchart wrote: > Hi Kieran, > > Thank you for the patch. > > On Friday 04 Aug 2017 16:57:05 Kieran Bingham wrote: >> Presently any received buffers are only released back to vb2 if >> vsp1_video_stop_streaming() is called.

Re: [PATCH v3 1/7] v4l: vsp1: Release buffers in start_streaming error path

2017-08-04 Thread Laurent Pinchart
On Friday 04 Aug 2017 19:03:28 Laurent Pinchart wrote: > Hi Kieran, > > Thank you for the patch. > > On Friday 04 Aug 2017 16:57:05 Kieran Bingham wrote: > > Presently any received buffers are only released back to vb2 if > > vsp1_video_stop_streaming() is called. If vsp1_video_start_streaming()

Re: [PATCH v3 1/7] v4l: vsp1: Release buffers in start_streaming error path

2017-08-04 Thread Laurent Pinchart
Hi Kieran, Thank you for the patch. On Friday 04 Aug 2017 16:57:05 Kieran Bingham wrote: > Presently any received buffers are only released back to vb2 if > vsp1_video_stop_streaming() is called. If vsp1_video_start_streaming() > encounters an error, we will be warned by the vb2 handlers that buf

[PATCH v3 5/7] v4l: vsp1: Move partition rectangles to struct and operate directly

2017-08-04 Thread Kieran Bingham
As we develop the partition algorithm, we need to store more information per partition to describe the phase and other parameters. To keep this data together, further abstract the existing v4l2_rect into a partition specific structure. As partitions only have horizontal coordinates, store the left

[PATCH v3 1/7] v4l: vsp1: Release buffers in start_streaming error path

2017-08-04 Thread Kieran Bingham
Presently any received buffers are only released back to vb2 if vsp1_video_stop_streaming() is called. If vsp1_video_start_streaming() encounters an error, we will be warned by the vb2 handlers that buffers have not been returned. Move the buffer cleanup code to it's own function to prevent duplic

[PATCH v3 3/7] v4l: vsp1: Calculate partition sizes at stream start

2017-08-04 Thread Kieran Bingham
Previously the active window and partition sizes for each partition were calculated for each partition every frame. This data is constant and only needs to be calculated once at the start of the stream. Extend the vsp1_pipe object to dynamically store the number of partitions required and pre-calc

[PATCH v3 2/7] v4l: vsp1: Move vsp1_video_pipeline_setup_partitions() function

2017-08-04 Thread Kieran Bingham
Separate the code change from the function move so that code changes can be clearly identified. This commit has no functional change. The partition algorithm functions will be changed, and vsp1_video_pipeline_setup_partitions() will call vsp1_video_partition(). To prepare for that, move the functi

[PATCH v3 6/7] v4l: vsp1: Provide UDS register updates

2017-08-04 Thread Kieran Bingham
Provide register definitions required for UDS phase and partition algorithm support. The registers and bits defined here are available on Gen3 hardware only. Signed-off-by: Kieran Bingham Reviewed-by: Laurent Pinchart --- drivers/media/platform/vsp1/vsp1_regs.h | 14 ++ 1 file chang

[PATCH v3 7/7] v4l: vsp1: Allow entities to participate in the partition algorithm

2017-08-04 Thread Kieran Bingham
The configuration of the pipeline and entities directly affects the inputs required to each entity for the partition algorithm. Thus it makes sense to involve those entities in the decision making process. Extend the entity ops API to provide an optional .partition() operation. This allows entitie

[PATCH v3 4/7] v4l: vsp1: Remove redundant context variables

2017-08-04 Thread Kieran Bingham
The vsp1_pipe object context variables for div_size and current_partition allowed state to be maintained through processing the partitions during processing. Now that the partition tables are calculated during stream on, there is no requirement to store these variables in the pipe object. Utilise

[PATCH v3 0/7] vsp1 partition algorithm improvements

2017-08-04 Thread Kieran Bingham
Some updates and initial improvements for the VSP1 partition algorithm that remove redundant processing and variables, reducing the processing done in interrupt context slightly. Patch 1, fixes up a bug to release buffers back to vb2 if errors occur in vsp1_video_start_streaming() Patches 2, 3 an

[GIT PULL FOR v4.14] RC changes (part #1)

2017-08-04 Thread Sean Young
Hi Mauro, This is missing David Härdeman lirc cleanups, since they conflict with a revert for v4.13. What we do have is three new RC drivers: ZTE ZX family SoCs, and two GPIO drivers for IR TX (one using pwm and another bit banging). These two drivers are useful for the Raspberry Pi. Also RC cor

[GIT FIXES FOR v4.13] RC use-after-free fix

2017-08-04 Thread Sean Young
Hi Mauro, While testing with kasan I discovered a regression in v4.13. The change is a simple revert. Thanks Sean The following changes since commit da48c948c263c9d87dfc64566b3373a858cc8aa2: media: fix warning on v4l2_subdev_call() result interpreted as bool (2017-07-26 13:43:17 -0400) are

[PATCH] Staging: bcm2048: fix bare use of 'unsigned' in radio-bcm2048.c

2017-08-04 Thread Branislav Radocaj
This is a patch to the radio-bcm2048.c file that fixes up a warning found by the checkpatch.pl tool. Signed-off-by: Branislav Radocaj --- drivers/staging/media/bcm2048/radio-bcm2048.c | 50 +-- 1 file changed, 25 insertions(+), 25 deletions(-) diff --git a/drivers/stagin

[PATCH] Revert "[media] lirc_dev: remove superfluous get/put_device() calls"

2017-08-04 Thread Sean Young
This reverts commit 5be2b76a9ca4ea5fd3e221114d62eeb0d78267ca. Only when the lirc device is freed, should we drop our reference to rc_dev, else we the rc_dev is freed to early. If userspace has a file descriptor open during unplug, it goes bang.

[PATCH 1/5] [media] cx18: constify videobuf_queue_ops structures

2017-08-04 Thread Julia Lawall
These videobuf_queue_ops structures are only passed as the second argument to videobuf_queue_vmalloc_init, which is declared as const. Thus the videobuf_queue_ops structures themselves can be const. Done with the help of Coccinelle. // @r disable optional_qualifier@ identifier i; position p; @@

[PATCH 2/5] [media] atomisp: constify videobuf_queue_ops structures

2017-08-04 Thread Julia Lawall
These videobuf_queue_ops structures are only passed as the second argument to videobuf_queue_vmalloc_init, which is declared as const. Thus the videobuf_queue_ops structures themselves can be const. Done with the help of Coccinelle. // @r disable optional_qualifier@ identifier i; position p; @@

[PATCH 0/5] constify videobuf_queue_ops structures

2017-08-04 Thread Julia Lawall
These videobuf_queue_ops structures are only passed as the second argument to videobuf_queue_vmalloc_init, which is declared as const. Thus the videobuf_queue_ops structures themselves can be const. Done with the help of Coccinelle. --- drivers/media/pci/cx18/cx18-streams.c

[PATCH 3/5] [media] cx231xx: constify videobuf_queue_ops structures

2017-08-04 Thread Julia Lawall
These videobuf_queue_ops structures are only passed as the second argument to videobuf_queue_vmalloc_init, which is declared as const. Thus the videobuf_queue_ops structures themselves can be const. Done with the help of Coccinelle. // @r disable optional_qualifier@ identifier i; position p; @@

[PATCH 4/5] [media] tm6000: constify videobuf_queue_ops structures

2017-08-04 Thread Julia Lawall
These videobuf_queue_ops structures are only passed as the second argument to videobuf_queue_vmalloc_init, which is declared as const. Thus the videobuf_queue_ops structures themselves can be const. Done with the help of Coccinelle. // @r disable optional_qualifier@ identifier i; position p; @@

[PATCH 5/5] [media] zr364xx: constify videobuf_queue_ops structures

2017-08-04 Thread Julia Lawall
These videobuf_queue_ops structures are only passed as the second argument to videobuf_queue_vmalloc_init, which is declared as const. Thus the videobuf_queue_ops structures themselves can be const. Done with the help of Coccinelle. // @r disable optional_qualifier@ identifier i; position p; @@

Re: [PATCH v1 5/5] [media] stm32-dcmi: g_/s_selection crop support

2017-08-04 Thread Hans Verkuil
On 28/07/17 12:05, Hugues Fruchet wrote: > Implements g_/s_selection crop support by using DCMI crop > hardware feature. > User can first get the maximum supported resolution of the sensor > by calling g_selection(V4L2_SEL_TGT_CROP_BOUNDS). > Then user call to s_selection(V4L2_SEL_TGT_CROP) will re

Re: [PATCH v1 4/5] [media] stm32-dcmi: set default format at open()

2017-08-04 Thread Hans Verkuil
On 28/07/17 12:05, Hugues Fruchet wrote: > Ensure that we start with default pixel format and resolution > when opening a new instance. Why? The format is persistent in V4L2 and (re)opening the video device shouldn't change the format. Suppose you use v4l2-ctl to set up a format. E.g. v4l2-ctl -v

Re: [PATCH v3 04/23] dt-bindings: media: Binding document for Qualcomm Camera subsystem driver

2017-08-04 Thread Todor Tomov
Hi Sakari, Thank you for the review. On 20.07.2017 13:13, Sakari Ailus wrote: > Hi Todor, > > On Mon, Jul 17, 2017 at 01:33:30PM +0300, Todor Tomov wrote: >> Add DT binding document for Qualcomm Camera subsystem driver. >> >> CC: Rob Herring >> CC: devicet...@vger.kernel.org >> Signed-off-by: T

Re: [PATCH v1 3/5] [media] stm32-dcmi: cleanup variable/fields namings

2017-08-04 Thread Hans Verkuil
On 28/07/17 12:05, Hugues Fruchet wrote: > Uniformize "pixfmt" variables to "pix". > Change "current_fmt" & "dcmi_fmt" variables to variables > with "sd_" prefix to explicitly refer to subdev format. > > Signed-off-by: Hugues Fruchet > --- > drivers/media/platform/stm32/stm32-dcmi.c | 103 > +++

[GIT PULL FOR v4.14] Fixes, fixes, ever more fixes :-)

2017-08-04 Thread Hans Verkuil
Lots of constify patches and some random other fixes. Except for the solo patch which is an actual feature enhancement. Regards, Hans The following changes since commit da48c948c263c9d87dfc64566b3373a858cc8aa2: media: fix warning on v4l2_subdev_call() result interpreted as bool (201

[PATCH] v4l2-compat-ioctl32.c: add missing controls to, ctrl_is_pointer

2017-08-04 Thread Hans Verkuil
We need to find a better method for this. But for now just add the missing pointer controls to this list. Also properly mask the id. The high flag bits shouldn't be used with these ioctls, but it certainly doesn't hurt. Signed-off-by: Hans Verkuil --- drivers/media/v4l2-core/v4l2-compat-ioctl32

[PATCH] v4l2-compat-ioctl32.c: add capabilities field to, v4l2_input32

2017-08-04 Thread Hans Verkuil
The v4l2_input32 struct wasn't updated when this field was added. It didn't cause a failure in the compat code, but it is better to keep it in sync with v4l2_input to avoid confusion. Signed-off-by: Hans Verkuil --- drivers/media/v4l2-core/v4l2-compat-ioctl32.c | 3 ++- 1 file changed, 2 inserti

[PATCH 2/2] [media] winbond-cir: buffer overrun during transmit

2017-08-04 Thread Sean Young
We're reading beyond the buffer before checking its length. BUG: KASAN: slab-out-of-bounds in wbcir_irq_tx Signed-off-by: Sean Young --- drivers/media/rc/winbond-cir.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/rc/winbond-cir.c b/drivers/media/rc/winbond-c

[PATCH 1/2] [media] mceusb: do not read data parameters unless required

2017-08-04 Thread Sean Young
This causes out-of-bounds read on device probe. BUG: KASAN: slab-out-of-bounds in mceusb_dev_printdata+0xdc/0x830 [mceusb] Signed-off-by: Sean Young --- drivers/media/rc/mceusb.c | 36 +--- 1 file changed, 17 insertions(+), 19 deletions(-) diff --git a/drivers/m

[PATCH 5/5] stm32-cec: use CEC_CAP_DEFAULTS

2017-08-04 Thread Hans Verkuil
From: Hans Verkuil Use the new CEC_CAP_DEFAULTS define in this driver. Signed-off-by: Hans Verkuil --- drivers/media/platform/stm32/stm32-cec.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/media/platform/stm32/stm32-cec.c b/drivers/media/platform/stm32/stm32-

[PATCH 1/5] media/cec.h: add CEC_CAP_DEFAULTS

2017-08-04 Thread Hans Verkuil
From: Hans Verkuil The CEC_CAP_LOG_ADDRS, CEC_CAP_TRANSMIT, CEC_CAP_PASSTHROUGH and CEC_CAP_RC capabilities are normally always present. Add a CEC_CAP_DEFAULTS define that ORs these four caps to simplify drivers. Signed-off-by: Hans Verkuil --- include/media/cec.h | 3 +++ 1 file changed, 3 i

[PATCH 2/5] adv*/vivid/pulse8/rainshadow: cec: use CEC_CAP_DEFAULTS

2017-08-04 Thread Hans Verkuil
From: Hans Verkuil Use the new CEC_CAP_DEFAULTS define in the adv, vivid, pulse8 and rainshadow CEC drivers. Signed-off-by: Hans Verkuil --- drivers/media/i2c/adv7511.c | 3 +-- drivers/media/i2c/adv7604.c | 3 +-- drivers/media/i2c/adv7842.c

[PATCH 0/5] cec: add CEC_CAP_DEFAULTS define

2017-08-04 Thread Hans Verkuil
From: Hans Verkuil Four CEC capabilities are normally always used. So combine them in a single CEC_CAP_DEFAULTS define. This also avoids missing one of these caps as happened in the stih-cec driver. Regards, Hans Hans Verkuil (5): media/cec.h: add CEC_CAP_DEFAULTS adv*/vivid/pulse8

[PATCH 3/5] s5p-cec: use CEC_CAP_DEFAULTS

2017-08-04 Thread Hans Verkuil
From: Hans Verkuil Use the new CEC_CAP_DEFAULTS define in the s5p-cec driver. Signed-off-by: Hans Verkuil --- drivers/media/platform/s5p-cec/s5p_cec.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/media/platform/s5p-cec/s5p_cec.c b/drivers/media/platform/s

[PATCH 4/5] stih-cec: use CEC_CAP_DEFAULTS

2017-08-04 Thread Hans Verkuil
From: Hans Verkuil Use the new CEC_CAP_DEFAULTS define in this driver. This also adds the CEC_CAP_RC capability which was missing here (and this is also the reason for this new define, to avoid missing such capabilities). Signed-off-by: Hans Verkuil --- drivers/media/platform/sti/cec/stih-cec.

Koch

2017-08-04 Thread shmeal camara
Ishmeal Camara, presently working under the central bank of Togo, i have financier offer for both of us 2.5 M USD deal.

[PATCH] [media] adv7604: Prevent out of bounds access

2017-08-04 Thread Dan Carpenter
These can only be accessed with CAP_SYS_ADMIN so it's not a critical security issue. The problem is that "page" is controlled by the user in the ioctl(). The test to see if the bit is set in state->info->page_mask is not sufficient because "page" can be very high and shift wrap around to a bit wh