Re: [PATCH v4 0/2] media: ov7670: Implement mbus configuration

2018-01-24 Thread Sakari Ailus
Hi Jacopo, On Wed, Jan 24, 2018 at 10:30:48AM +0100, Jacopo Mondi wrote: > Hello, >4th round for this series, now based on Hans' 'parm' branch from > git://linuxtv.org/hverkuil/media_tree.git > > I addressed Sakari's comments on bindings documentation and driver error path, > and I hope to

[PATCH-next] media: s2255drv: Remove unneeded if else blocks

2018-01-24 Thread Christopher Díaz Riveros
Given the following definitions from s2255drv.c #define LINE_SZ_4CIFS_NTSC 640 #define LINE_SZ_2CIFS_NTSC 640 #define LINE_SZ_1CIFS_NTSC 320 and #define LINE_SZ_4CIFS_PAL 704 #define LINE_SZ_2CIFS_PAL 704 #define LINE_SZ_1CIFS_PAL 352 f->fmt.pix.width

Re: [PATCH] staging: imx-media-vdic: fix inconsistent IS_ERR and PTR_ERR

2018-01-24 Thread Steve Longerbeam
Acked-by: Steve Longerbeam On 01/23/2018 04:43 PM, Gustavo A. R. Silva wrote: Fix inconsistent IS_ERR and PTR_ERR in vdic_get_ipu_resources. The proper pointer to be passed as argument is ch. This issue was detected with the help of Coccinelle. Fixes:

Re: [PATCH] media: cxusb, dib0700: ignore XC2028_I2C_FLUSH

2018-01-24 Thread Enrico Mioso
Thank you very very much for the fix. You did really make my everyday experience better. On Wed, 24 Jan 2018, Mauro Carvalho Chehab wrote: Date: Wed, 24 Jan 2018 12:05:24 From: Mauro Carvalho Chehab To: Enrico Mioso , Linux Media Mailing

Re: [PATCH] dib700: stop flooding system ring buffer

2018-01-24 Thread Enrico Mioso
Hello Honza, thank you very very much for your help and hints. These things make a community what it is, and I apreciate it extremely. You're perfectly right, I didn't think about this well and/or enough. Apreciated your hints. On Wed, 24 Jan 2018, Honza Petrouš wrote: Date: Wed, 24 Jan

[PATCH] media: leds: as3645a: Add CONFIG_OF support

2018-01-24 Thread Akash Gajjar
From: Akash Gajjar Witth this changes, the driver builds with CONFIG_OF support Signed-off-by: Akash Gajjar --- drivers/media/i2c/as3645a.c | 9 + 1 file changed, 9 insertions(+) diff --git a/drivers/media/i2c/as3645a.c

Your quick response will be highly appreciated.

2018-01-24 Thread Mrs Ann Willi
-- Dear Beneficiary This is to inform you that your Long Time Awaiting Funds amounting in the tone of USD$10,400,000.00 {TEN MILLION, FOUR HUNDRED THOUSAND UNITED STATES DOLLARS) Which is presently in the custody of a finance house in Germany has been approved for immediate delivery to you in

Re: [PATCH v1 0/2] Remove duplicate driver for MyGica T230C

2018-01-24 Thread Stefan Brüns
On Wednesday, 10 January 2018 00:33:37 CET Stefan Brüns wrote: > In 2017-02, two drivers for the T230C where submitted, but until now > only the one based on the older dvb-usb/cxusb.c driver has been part > of the mainline kernel. As a dvb-usb-v2 driver is preferable, remove > the other driver. >

cron job: media_tree daily build: ERRORS

2018-01-24 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: Thu Jan 25 05:00:17 CET 2018 media-tree git hash:4852fdca8818972d0ea5b5ce7114da628f9954a4 media_build

[PATCH] v4l2-dev.h: fix symbol collision in media_entity_to_video_device()

2018-01-24 Thread Niklas Söderlund
A recent change to the media_entity_to_video_device() macro breaks some use-cases for the macro due to a symbol collision. Before the change this worked: vdev = media_entity_to_video_device(link->sink->entity); While after the change it results in a compiler error "error: 'struct

[PATCH v1] media: ov13858: Avoid possible null first frame

2018-01-24 Thread Chiranjeevi Rapolu
Previously, the sensor, with default settings, was outputting SOF without data. This results in frame sync error on the receiver side. Now, configure the sensor to output SOF with MIPI data for all frames. This avoids possible null first frame on the bus. Signed-off-by: Chiranjeevi Rapolu

Re: [Patch v7 10/12] [media] v4l2: Add v4l2 control IDs for HEVC encoder

2018-01-24 Thread Hans Verkuil
On 24/01/18 11:59, Smitha T Murthy wrote: > Add v4l2 controls for HEVC encoder > > Signed-off-by: Smitha T Murthy > Reviewed-by: Andrzej Hajda Acked-by: Hans Verkuil Thanks! Hans > --- >

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

2018-01-24 Thread Hans Verkuil
On 24/01/18 11:59, Smitha T Murthy wrote: > Added V4l2 controls for HEVC encoder > > Signed-off-by: Smitha T Murthy > --- > Documentation/media/uapi/v4l/extended-controls.rst | 400 > + > 1 file changed, 400 insertions(+) > > diff --git

RE: [PATCH v2 1/4] dt-bindings: media: rcar_vin: Reverse SoC part number list

2018-01-24 Thread Fabrizio Castro
Hello guys, I am sorry to bother you, just wondering if this patch has any chance to end up in v4.16? Thanks, Fabrizio > Subject: [PATCH v2 1/4] dt-bindings: media: rcar_vin: Reverse SoC part number > list > > Change the sorting of the part numbers from descending to ascending to > match

RE: [PATCH v2 2/4] dt-bindings: media: rcar_vin: add device tree support for r8a774[35]

2018-01-24 Thread Fabrizio Castro
Hello guys, I am sorry to bother you, just wondering if this patch has any chance to end up in v4.16? Thanks, Fabrizio > Subject: [PATCH v2 2/4] dt-bindings: media: rcar_vin: add device tree support > for r8a774[35] > > Add compatible strings for r8a7743 and r8a7745. No driver change > is

Re: [PATCH] [media] s3c-camif: array underflow in __camif_subdev_try_format()

2018-01-24 Thread Dan Carpenter
On Mon, Jan 22, 2018 at 09:50:04PM +0100, Sylwester Nawrocki wrote: > On 01/22/2018 11:37 AM, Dan Carpenter wrote: > > --- a/drivers/media/platform/s3c-camif/camif-capture.c > > +++ b/drivers/media/platform/s3c-camif/camif-capture.c > > @@ -1261,11 +1261,11 @@ static void

Re: [PATCH] [media] s3c-camif: array underflow in __camif_subdev_try_format()

2018-01-24 Thread Arnd Bergmann
On Wed, Jan 24, 2018 at 9:13 AM, Dan Carpenter wrote: > On Mon, Jan 22, 2018 at 09:50:04PM +0100, Sylwester Nawrocki wrote: >> On 01/22/2018 11:37 AM, Dan Carpenter wrote: > I happened to be looking at the same bugs but using Smatch. Did you get > these two bugs as

Re: [PATCH] staging: media: remove unused VIDEO_ATOMISP_OV8858 kconfig

2018-01-24 Thread Dan Carpenter
On Tue, Jan 23, 2018 at 07:31:27PM +0200, Andy Shevchenko wrote: > On Tue, Jan 23, 2018 at 4:37 PM, Corentin Labbe wrote: > > Nothing in kernel use VIDEO_ATOMISP_OV8858 since commit 3a81c7660f80 > > ("media: staging: atomisp: Remove IMX sensor support") > > Lets remove this

[PATCH v4 0/2] media: ov7670: Implement mbus configuration

2018-01-24 Thread Jacopo Mondi
Hello, 4th round for this series, now based on Hans' 'parm' branch from git://linuxtv.org/hverkuil/media_tree.git I addressed Sakari's comments on bindings documentation and driver error path, and I hope to get both driver and bindings acked to have this included in next merge window. Thanks

[PATCH v4 2/2] v4l2: i2c: ov7670: Implement OF mbus configuration

2018-01-24 Thread Jacopo Mondi
ov7670 driver supports two optional properties supplied through platform data, but currently does not support any standard video interface property. Add support through OF parsing for 2 generic properties (vsync and hsync polarities) and for one custom property already supported through platform

[PATCH v4 1/2] media: dt-bindings: Add OF properties to ov7670

2018-01-24 Thread Jacopo Mondi
Describe newly introduced OF properties for ov7670 image sensor. The driver supports two standard properties to configure synchronism signals polarities and one custom property already supported as platform data options to suppress pixel clock during horizontal blankings. Re-phrase child nodes

Re: [PATCH v2] media: ov5640: add JPEG support

2018-01-24 Thread Sakari Ailus
Hi Hugues, On Tue, Jan 23, 2018 at 02:23:14PM +0100, Hugues Fruchet wrote: > +static int ov5640_get_frame_desc(struct v4l2_subdev *sd, unsigned int pad, > + struct v4l2_mbus_frame_desc *fd) > +{ > + struct ov5640_dev *sensor = to_ov5640_dev(sd); > + > + if

[Patch v7 04/12] [media] s5p-mfc: Support MFCv10.10 buffer requirements

2018-01-24 Thread Smitha T Murthy
Aligning the luma_dpb_size, chroma_dpb_size, mv_size and me_buffer_size for MFCv10.10. Signed-off-by: Smitha T Murthy Reviewed-by: Andrzej Hajda Acked-by: Kamil Debski Acked-by: Hans Verkuil ---

[Patch v7 03/12] [media] s5p-mfc: Use min scratch buffer size as provided by F/W

2018-01-24 Thread Smitha T Murthy
After MFC v8.0, mfc f/w lets the driver know how much scratch buffer size is required for decoder. If mfc f/w has the functionality, E_MIN_SCRATCH_BUFFER_SIZE, driver can know how much scratch buffer size is required for encoder too. Signed-off-by: Smitha T Murthy

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

2018-01-24 Thread Smitha T Murthy
Add V4L2 definition for HEVC compressed format Signed-off-by: Smitha T Murthy Reviewed-by: Andrzej Hajda Reviewed-by: Stanimir Varbanov Acked-by: Hans Verkuil --- include/uapi/linux/videodev2.h |

[Patch v7 02/12] [media] s5p-mfc: Adding initial support for MFC v10.10

2018-01-24 Thread Smitha T Murthy
Adding the support for MFC v10.10, with new register file and necessary hw control, decoder, encoder and structural changes. CC: Rob Herring CC: devicet...@vger.kernel.org Signed-off-by: Smitha T Murthy Reviewed-by: Andrzej Hajda

[Patch v7 00/12] Add MFC v10.10 support

2018-01-24 Thread Smitha T Murthy
This patch series adds MFC v10.10 support. MFC v10.10 is used in some of Exynos7 variants. This adds support for following: * Add support for HEVC encoder and decoder * Add support for VP9 decoder * Update Documentation for control id definitions * Update computation of min scratch buffer size

[Patch v7 01/12] [media] s5p-mfc: Rename IS_MFCV8 macro

2018-01-24 Thread Smitha T Murthy
This patch renames macro IS_MFCV8 to IS_MFCV8_PLUS so that the MFCv8 code can be resued for MFCv10.10 support. Since the MFCv8 specific code holds good for MFC v10.10 also. Signed-off-by: Smitha T Murthy Acked-by: Andrzej Hajda Acked-by: Hans Verkuil

Re: [PATCH] dib700: stop flooding system ring buffer

2018-01-24 Thread Honza Petrouš
Hi Enrico, I'm not maintener, so treat next hints as hints only :) 2018-01-24 8:40 GMT+01:00 Enrico Mioso : > Stop flooding system ring buffer with messages like: > dib0700: stk7700ph_xc3028_callback: unknown command 2, arg 0 > while tuning an Asus My Cinema-U3000Hybrid dvb

[Patch v7 10/12] [media] v4l2: Add v4l2 control IDs for HEVC encoder

2018-01-24 Thread Smitha T Murthy
Add v4l2 controls for HEVC encoder Signed-off-by: Smitha T Murthy Reviewed-by: Andrzej Hajda --- drivers/media/v4l2-core/v4l2-ctrls.c | 119 +++ include/uapi/linux/v4l2-controls.h | 93 ++- 2

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

2018-01-24 Thread Smitha T Murthy
Added V4l2 controls for HEVC encoder Signed-off-by: Smitha T Murthy --- Documentation/media/uapi/v4l/extended-controls.rst | 400 + 1 file changed, 400 insertions(+) diff --git a/Documentation/media/uapi/v4l/extended-controls.rst

[Patch v7 09/12] [media] s5p-mfc: Add VP9 decoder support

2018-01-24 Thread Smitha T Murthy
Add support for codec definition and corresponding buffer requirements for VP9 decoder. Signed-off-by: Smitha T Murthy Reviewed-by: Andrzej Hajda Acked-by: Hans Verkuil --- drivers/media/platform/s5p-mfc/regs-mfc-v10.h | 6

[Patch v7 11/12] [media] s5p-mfc: Add support for HEVC encoder

2018-01-24 Thread Smitha T Murthy
Add HEVC encoder support and necessary registers, V4L2 CIDs, and hevc encoder parameters Signed-off-by: Smitha T Murthy Acked-by: Hans Verkuil --- drivers/media/platform/s5p-mfc/regs-mfc-v10.h | 28 +- drivers/media/platform/s5p-mfc/s5p_mfc.c

[Patch v7 08/12] [media] s5p-mfc: Add support for HEVC decoder

2018-01-24 Thread Smitha T Murthy
Add support for codec definition and corresponding buffer requirements for HEVC decoder. Signed-off-by: Smitha T Murthy Reviewed-by: Andrzej Hajda Acked-by: Hans Verkuil --- drivers/media/platform/s5p-mfc/regs-mfc-v10.h | 1

[PATCH] media: cxusb, dib0700: ignore XC2028_I2C_FLUSH

2018-01-24 Thread Mauro Carvalho Chehab
The XC2028_I2C_FLUSH only needs to be implemented on a few devices. Others can safely ignore it. That prevents filling the dmesg with lots of messages like: dib0700: stk7700ph_xc3028_callback: unknown command 2, arg 0 Reported-by: Enrico Mioso Signed-off-by: Mauro

[Patch v7 07/12] Documentation: v4l: Documentation for HEVC v4l2 definition

2018-01-24 Thread Smitha T Murthy
Add V4L2 definition for HEVC compressed format which is also known as H.265. Signed-off-by: Smitha T Murthy Reviewed-by: Andrzej Hajda Reviewed-by: Stanimir Varbanov Acked-by: Hans Verkuil ---

[Patch v7 06/12] [media] v4l2-ioctl: add HEVC format description

2018-01-24 Thread Smitha T Murthy
HEVC is a video coding format Signed-off-by: Smitha T Murthy Reviewed-by: Stanimir Varbanov Acked-by: Hans Verkuil --- drivers/media/v4l2-core/v4l2-ioctl.c | 1 + 1 file changed, 1 insertion(+) diff --git