cron job: media_tree daily build: ERRORS

2017-09-01 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 Sep 2 05:00:17 CEST 2017 media-tree git hash:fce4b371fe5c99a9c05db8493d72f0d1a474ab26 media_build gi

Re: [PATCH v6 5/5] v4l: fwnode: Support generic parsing of graph endpoints in a single port

2017-09-01 Thread Sakari Ailus
Hi Hans, Thanks for the review. On Fri, Sep 01, 2017 at 01:28:40PM +0200, Hans Verkuil wrote: > > diff --git a/include/media/v4l2-fwnode.h b/include/media/v4l2-fwnode.h > > index d063ab4ff67b..dd13604178b4 100644 > > --- a/include/media/v4l2-fwnode.h > > +++ b/include/media/v4l2-fwnode.h > > @@ -

[PATCH v1] media: ov5670: Fix not streaming issue after resume.

2017-09-01 Thread Chiranjeevi Rapolu
Previously, the sensor was not streaming after resume from suspend, i.e. on S0->S3->S0 transition. Due to this, camera app preview appeared as stuck. Now, handle streaming state correctly in case of suspend-resume. Signed-off-by: Chiranjeevi Rapolu --- drivers/media/i2c/ov5670.c | 5 + 1 fi

Re: [PATCH v6.1 4/5] v4l: fwnode: Support generic parsing of graph endpoints in a device

2017-09-01 Thread Sakari Ailus
Hi Hans, Thanks for the review! On Fri, Sep 01, 2017 at 01:18:55PM +0200, Hans Verkuil wrote: > Hi Sakari, > > On 30/08/17 14:45, 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

[PATCH v4 1/2] media:imx274 device tree binding file

2017-09-01 Thread Leon Luo
The binding file for imx274 CMOS sensor V4l2 driver Signed-off-by: Leon Luo --- v4: - no changes v3: - remove redundant properties and references - document 'reg' property v2: - no changes --- .../devicetree/bindings/media/i2c/imx274.txt | 32 ++ 1 file changed, 32

[PATCH v4 2/2] media:imx274 V4l2 driver for Sony imx274 CMOS sensor

2017-09-01 Thread Leon Luo
The imx274 is a Sony CMOS image sensor that has 1/2.5 image size. It supports up to 3840x2160 (4K) 60fps, 1080p 120fps. The interface is 4-lane MIPI CSI-2 running at 1.44Gbps each. This driver has been tested on Xilinx ZCU102 platform with a Leopard LI-IMX274MIPI-FMC camera board. Support for the

[PATCH] media: net.h: add kernel-doc and use it at Documentation/

2017-09-01 Thread Mauro Carvalho Chehab
As we did with frontend.h, ca.h and dmx.h, move the struct definition to net.h. That should help to keep it updated, as more stuff gets added there. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/net-add-if.rst | 34 - Documentation/media/uapi/

[PATCH 4/4] [media] sp2: Adjust three null pointer checks in sp2_exit()

2017-09-01 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 1 Sep 2017 21:16:06 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written !… Thus fix the affected source code pla

[PATCH 3/4] [media] sp2: Adjust a jump target in sp2_probe()

2017-09-01 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 1 Sep 2017 21:08:38 +0200 * Adjust a jump target so that a null pointer will not be passed to a call of the function "kfree". * Move this function call into an if branch. Signed-off-by: Markus Elfring --- drivers/media/dvb-frontends/sp2.c | 13 +++--

[PATCH 2/4] [media] sp2: Improve a size determination in sp2_probe()

2017-09-01 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 1 Sep 2017 20:46:18 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was dete

[PATCH] [media] dvb-frontends/mxl5xx: declare LIST_HEAD(mxllist) static

2017-09-01 Thread Daniel Scheller
From: Daniel Scheller Fixes one sparse warning: mxl5xx.c:46:1: warning: symbol 'mxllist' was not declared. Should it be static? Signed-off-by: Daniel Scheller --- drivers/media/dvb-frontends/mxl5xx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/dvb-front

[PATCH 09/14] media: dvbapi.rst: add an entry to DVB revision history

2017-09-01 Thread Mauro Carvalho Chehab
There are several missing items at the API history. Yet, as we're doing a significant change there, add a new entry. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/dvbapi.rst | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/Documentation/media/uapi

[PATCH 1/4] [media] sp2: Delete an error message for a failed memory allocation in sp2_probe()

2017-09-01 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 1 Sep 2017 20:44:05 +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/dvb-frontends/sp2.c | 1 - 1 file changed, 1 deletion(-)

[PATCH 00/14] Another set of DVB documentation patches

2017-09-01 Thread Mauro Carvalho Chehab
This series should come after this series: [PATCH v2 00/26] Improve DVB documentation and reduce its gap (with actually has 27 patches :-p ) It adds documentation for a dmx.h ioctl (DMX_GET_PES_PIDS) and do a cleanup at the documentation, to make it more coherent with the other me

[PATCH 0/4] [media] SP2: Adjustments for two function implementations

2017-09-01 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 1 Sep 2017 21:31:23 +0200 A few update suggestions were taken into account from static source code analysis. Markus Elfring (4): Delete an error message for a failed memory allocation Improve a size determination Adjust a jump target Adjust three null poin

[PATCH 11/14] media: dmx-get-pes-pids.rst: document the ioctl

2017-09-01 Thread Mauro Carvalho Chehab
This ioctl is supported by the DVB core, but was never documented. Add a documentation for it. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/dmx-get-pes-pids.rst | 22 -- 1 file changed, 20 insertions(+), 2 deletions(-) diff --git a/Documentation/med

[PATCH 05/14] media: ca-fopen.rst: Fixes the device node name for CA

2017-09-01 Thread Mauro Carvalho Chehab
The device node name for CA is wrong since ever. I suspect that the name there was before DVBv3 (with was the first API introduced at the Kernel). Anyway, use the right name there. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/ca-fopen.rst | 4 ++-- 1 file changed, 2 ins

[PATCH 13/14] media: intro.rst: don't assume audio and video codecs to be MPEG2

2017-09-01 Thread Mauro Carvalho Chehab
Originally, when DVB was introduced, all codecs would be part of MPEG2 standard. That's not true anymore, as there are a large number of codec standards used on digital TV nowadays. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/intro.rst | 9 - 1 file changed, 4 i

[PATCH 06/14] media: dvb uAPI docs: Prefer use "Digital TV instead of "DVB"

2017-09-01 Thread Mauro Carvalho Chehab
The usage of the term "DVB" at the dvb API docs is confusing, as, right now, it can refer to either the European digital TV standard or to the subsystem. So, prefer calling it as "Digital TV" on most places, to avoid ambiguity. Signed-off-by: Mauro Carvalho Chehab --- .../media/uapi/dvb/audio-c

[PATCH 01/14] media: dvb uapi docs: better organize header files

2017-09-01 Thread Mauro Carvalho Chehab
Instead of having one chapter per file, place all of them at the same chapter. That better organize the chapters at the uAPI documentation. As a side effect, now all uAPI headers are at the same page, at the html output, with makes easier to use it as a reference index for the spec. Signed-off-by

[PATCH 04/14] media: dvb uAPI docs: adjust return value ioctl descriptions

2017-09-01 Thread Mauro Carvalho Chehab
There are several issues on the return value for ioctls: - Text is confusing; - Some error codes don't exist; - The non-generic error codes should come before the text that points to the generic error codes; - Tables don't contain column size hints; - Some references are not marked as such. Cor

[PATCH 03/14] media: gen-errors.rst: document ENXIO error code

2017-09-01 Thread Mauro Carvalho Chehab
This error can be produced at least at the DVB subsystem. As it is generic enough, document it. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/gen-errors.rst | 5 + 1 file changed, 5 insertions(+) diff --git a/Documentation/media/uapi/gen-errors.rst b/Documentation/medi

[PATCH 10/14] media: dvb uAPI docs: minor editorial changes

2017-09-01 Thread Mauro Carvalho Chehab
Do minor editorial changes to improve documentation readability: - mark literals as such; - add table markups to hint sizes; - define what PES means; - instead of hardcoding devnode numbers to zero (like adapter0/) use a question mark, to indicate that multiple devnodes may exist; - add cross-re

[PATCH 08/14] media: dvb-frontend-parameters.rst: fix the name of a struct

2017-09-01 Thread Mauro Carvalho Chehab
The struct that contains an union of DVB parameters is called dvb_frontend_parameters (and not FrontendParameters). Fix it. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/dvb-frontend-parameters.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Docum

[PATCH 12/14] media: dvbstb.svg: use dots for the optional parts of the hardware

2017-09-01 Thread Mauro Carvalho Chehab
The hardware description mentions that some parts are optional. Make it clearer at the drawing. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/dvbstb.svg | 31 +++ 1 file changed, 15 insertions(+), 16 deletions(-) diff --git a/Documentation/med

[PATCH 02/14] media: gen-errors.rst: remove row number comments

2017-09-01 Thread Mauro Carvalho Chehab
Those are introduced by the conversion scripts and don't really help. Get rid of them. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/gen-errors.rst | 44 + 1 file changed, 11 insertions(+), 33 deletions(-) diff --git a/Documentation/media/uapi

[PATCH 07/14] media: dmx-fread.rst: specify how DMX_CHECK_CRC works

2017-09-01 Thread Mauro Carvalho Chehab
In the past, the documentation used to say that, if a CRC error was found, a "-ECRC" error would be returned. That's not true: the DVB core will just silently ignore such errors. So, add an explicit note about that. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/dmx-fread

[PATCH 14/14] media: frontend.h: Avoid the term DVB when doesn't refer to a delivery system

2017-09-01 Thread Mauro Carvalho Chehab
The DVB term can either refer to the subsystem or to a delivery system. Avoid it in the first case at the kernel-doc markups. Signed-off-by: Mauro Carvalho Chehab --- include/uapi/linux/dvb/frontend.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/include/uapi/linux/dv

Re: [PATCH v2 14/14] [media] v4l: Document explicit synchronization behaviour

2017-09-01 Thread Gustavo Padovan
Hi Hans, 2017-09-01 Hans Verkuil : > Hi Gustavo, > > I think I concentrate on this last patch first. Once I fully understand this > I can review the code. Remember, it's been a while for me since I last looked > at fences, so forgive me if I ask stupid questions. On the other hand, others > with

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

2017-09-01 Thread SF Markus Elfring
From: Markus Elfring Date: Fri, 1 Sep 2017 19:50:45 +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/dvb-frontends/si2168.c | 1 - 1 file changed, 1 deletion(

v4l2_ioctl vidioc_s_fmt/vidioc_try_fmt pix <--> pix_mp conversion

2017-09-01 Thread Rafal
Could somebody explain to me some strange behavior of v4l2_ioctl call? When a device supports |V4L2_CAP_VIDEO_OUTPUT_MPLANE capability but does not support ||V4L2_CAP_VIDEO_OUTPUT, the |v4l2_ioctl function converts VIDIOC_S_FMT ioctl call: for example, when the specified buffer type is V4L2_BU

[PATCH] [media] rtl28xxu: make array rc_nec_tab static const

2017-09-01 Thread Colin King
From: Colin Ian King Don't populate the array rc_nec_tab on the stack, instead make it static const. Makes the object code smaller by over 620 bytes: Before: textdata bss dec hex filename 49511 17040 64 66615 10437 rtl28xxu.o After: textdata bss de

[PATCH v2 03/27] media: dvb/intro: use the term Digital TV to refer to the system

2017-09-01 Thread Mauro Carvalho Chehab
On several places at the introduction, a digital TV board and its kernel support is called as DVB. The reason is simple: by the time the document was written, there were no other digital TV standards :-) Modernize the specs by referring to them as Digital TV. Signed-off-by: Mauro Carvalho Chehab

[PATCH v2 27/27] media: dst_ca: remove CA_SET_DESCR boilerplate

2017-09-01 Thread Mauro Carvalho Chehab
This ioctl is not implemented at dst_ca driver. There's just a boilerplate code there. Remove it, as it is unlikely that anyone would implement it those days. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/pci/bt8xx/dst_ca.c | 17 - 1 file changed, 17 deletions(-) diff -

[PATCH v2 21/27] media: ca-reset.rst: add some description to this ioctl

2017-09-01 Thread Mauro Carvalho Chehab
While we don't have any documentation for it, based on what's there at Kaffeine and VDR, it seems that this command should be issued before start using CA. So, document it as such. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/ca-reset.rst | 3 ++- 1 file changed, 2 inser

[PATCH v2 15/27] media: dmx.h: get rid of GET_DMX_EVENT

2017-09-01 Thread Mauro Carvalho Chehab
This seems to be a pure fictional API :-) It only exists at the DVB book, with no code implemeting it. So, just get rid of it. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/dmx-get-event.rst | 60 -- Documentation/media/uapi/dvb/dmx_fcalls.rst

[PATCH v1 5/7] staging: atomisp: Move to upstream IOSF MBI API

2017-09-01 Thread Andy Shevchenko
There is a common for x86 IOSF MBI API. Move atomisp code to use it. Signed-off-by: Andy Shevchenko --- drivers/staging/media/atomisp/pci/Kconfig | 1 + .../media/atomisp/pci/atomisp2/atomisp_cmd.c | 20 +++- .../media/atomisp/pci/atomisp2/atomisp_v4l2.c | 38 +++

[PATCH v1 3/7] staging: atomisp: Remove dead code for MID (#2)

2017-09-01 Thread Andy Shevchenko
intel_mid_soc_stepping() is not used anywhere. Signed-off-by: Andy Shevchenko --- drivers/staging/media/atomisp/include/asm/intel_mid_pcihelpers.h | 1 - .../media/atomisp/platform/intel-mid/intel_mid_pcihelpers.c| 7 --- 2 files changed, 8 deletions(-) diff --git a/drivers/stagin

[PATCH v1 7/7] staging: atomisp: Remove unneeded intel-mid.h inclusion

2017-09-01 Thread Andy Shevchenko
In many files in the driver the intel-mid.h header inclusion is redundant. Signed-off-by: Andy Shevchenko --- drivers/staging/media/atomisp/i2c/imx/drv201.c| 1 - drivers/staging/media/atomisp/i2c/imx/dw9714.c| 1 - drivers/staging/media/atomisp/i2

[PATCH v1 6/7] staging: atomisp: Remove dead code for MID (#4)

2017-09-01 Thread Andy Shevchenko
Since we switched to upstream IOSF MBI API the custom code become not in use anymore. Signed-off-by: Andy Shevchenko --- .../atomisp/include/asm/intel_mid_pcihelpers.h | 22 - .../media/atomisp/pci/atomisp2/atomisp_internal.h | 1 - .../media/atomisp/platform/intel-mid/Makefile |

[PATCH v1 1/7] staging: atomisp: Remove dead code for MID (#1)

2017-09-01 Thread Andy Shevchenko
Remove dead code. If someone needs it the P-Unit semaphore is handled by I2C DesignWare driver (drivers/i2c/busses/i2c-designware-baytrail.c). Signed-off-by: Andy Shevchenko --- .../atomisp/include/asm/intel_mid_pcihelpers.h | 2 - .../platform/intel-mid/intel_mid_pcihelpers.c | 101 -

[PATCH v1 4/7] staging: atomisp: Remove dead code for MID (#3)

2017-09-01 Thread Andy Shevchenko
intel_mid_msgbus_*_raw*() are not used anywhere. Signed-off-by: Andy Shevchenko --- .../atomisp/include/asm/intel_mid_pcihelpers.h | 4 -- .../platform/intel-mid/intel_mid_pcihelpers.c | 58 -- 2 files changed, 62 deletions(-) diff --git a/drivers/staging/media/ato

[PATCH v1 2/7] staging: atomisp: Don't override D3 delay settings here

2017-09-01 Thread Andy Shevchenko
The d3_delay parameter is set by arch/x86/pci/intel_mid_pci.c and drivers/pci/quirks.c. No need to override that settings in unrelated driver. Signed-off-by: Andy Shevchenko --- .../atomisp/include/asm/intel_mid_pcihelpers.h | 8 -- .../platform/intel-mid/intel_mid_pcihelpers.c |

[PATCH v2 17/27] media: net.rst: Fix the level of a section of the net chapter

2017-09-01 Thread Mauro Carvalho Chehab
Due to a mistake, the DVB net chapter was actually broken into two different chapters. Fix it. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/net.rst | 1 - 1 file changed, 1 deletion(-) diff --git a/Documentation/media/uapi/dvb/net.rst b/Documentation/media/uapi/dvb/net

[PATCH v2 11/27] media: fe_property_parameters.rst: better document bandwidth

2017-09-01 Thread Mauro Carvalho Chehab
Use a table to document the supported bandwidths. That makes it clearer to readers. Signed-off-by: Mauro Carvalho Chehab --- .../media/uapi/dvb/fe_property_parameters.rst | 44 +- 1 file changed, 26 insertions(+), 18 deletions(-) diff --git a/Documentation/media/uapi/dv

[PATCH v2 02/27] media: dmx.h: split typedefs from structs

2017-09-01 Thread Mauro Carvalho Chehab
Using typedefs inside the Kernel is against CodingStyle, and there's no good usage here. Just like we did at frontend.h, at changeset 0df289a209e0 ("[media] dvb: Get rid of typedev usage for enums"), let's keep those typedefs only to provide userspace backward compatibility. No functional changes

[PATCH v2 10/27] media: fe_property_parameters.rst: better define properties usage

2017-09-01 Thread Mauro Carvalho Chehab
Several frontend properties are specific to a subset of the delivery systems. Make it clearer when describing each property. Signed-off-by: Mauro Carvalho Chehab --- .../media/uapi/dvb/fe_property_parameters.rst | 82 -- 1 file changed, 75 insertions(+), 7 deletions(-)

[PATCH v2 23/27] media: ca-get-slot-info.rst: document this ioctl

2017-09-01 Thread Mauro Carvalho Chehab
Instead of a generic boilerplate, fill it with relevant information about this ioctl. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/ca-get-slot-info.rst | 98 +++ 1 file changed, 11 insertions(+), 87 deletions(-) diff --git a/Documentation/media/uapi/

[PATCH v2 12/27] media: dmx.h: get rid of unused DMX_KERNEL_CLIENT

2017-09-01 Thread Mauro Carvalho Chehab
There's a flag defined for Digital TV demux that is not used anywhere, called DMX_KERNEL_CLIENT. Get rid of it. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/dmx.h.rst.exceptions | 1 - Documentation/media/uapi/dvb/dmx_types.rst | 1 - include/uapi/linux/dvb/dmx.h

[PATCH v2 18/27] media: ca.h: get rid of CA_SET_PID

2017-09-01 Thread Mauro Carvalho Chehab
This ioctl seems to be some attempt to support a feature at the bt8xx dst_ca driver. Yet, as said there, it "needs more work". Right now, the code there is just a boilerplate. At the end of the day, no driver uses this ioctl, nor it is documented anywhere (except for "needs more work"). So, get r

[PATCH v2 16/27] media: dmx.h: add kernel-doc markups and use it at Documentation/

2017-09-01 Thread Mauro Carvalho Chehab
The demux documentation is pretty poor nowadays: most of the structs and enums aren't documented at all. Add proper kernel-doc markups for them and use it. Now, the demux API is fully documented :-) Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/dmx.h.rst.exceptions | 5 + Do

[PATCH v2 09/27] media: dvb frontend docs: use kernel-doc documentation

2017-09-01 Thread Mauro Carvalho Chehab
Now that frontend.h contains most documentation for the frontend, remove the duplicated information from Documentation/ and use the kernel-doc auto-generated one instead. That should simplify maintainership of DVB frontend uAPI, as most of the documentation will stick with the header file. Signed

[PATCH v2 07/27] media: dvb/frontend.h: move out a private internal structure

2017-09-01 Thread Mauro Carvalho Chehab
struct dtv_cmds_h is just an ancillary struct used by the dvb_frontend.c to internally store frontend commands. It doesn't belong to the userspace header, nor it is used anywhere, except inside the DVB core. So, remove it from the header. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/d

[PATCH v2 26/27] media: dvb rst: identify the documentation gap at the API

2017-09-01 Thread Mauro Carvalho Chehab
Now that DVB spec is almost in sync, document what's missing. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/ca.rst | 5 + Documentation/media/uapi/dvb/legacy_dvb_apis.rst | 5 + 2 files changed, 10 insertions(+) diff --git a/Documentation/media/uapi/

[PATCH v2 20/27] media: dst_ca: return a proper error code from CA errors

2017-09-01 Thread Mauro Carvalho Chehab
Right now, on several places, the driver is returning a "-1" error to userspace, instead of a proper error code. Fix it. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/pci/bt8xx/dst_ca.c | 41 +--- 1 file changed, 22 insertions(+), 19 deletions(-) di

[PATCH v2 06/27] media: dvb/intro: adjust the notices about optional hardware

2017-09-01 Thread Mauro Carvalho Chehab
Both CA and decoders are optional. Also, the presence or absence has nothing to do on being a PCI card or not. Nowadays, most hardware leaves the decoders to either the GPU or to some ISP inside the SoC, instead of implementing it inside the Digital TV part of the device. So, change the wording t

[PATCH v2 13/27] media: dmx.h: get rid of DMX_GET_CAPS

2017-09-01 Thread Mauro Carvalho Chehab
There's no driver currently using it; it is also not documented about what it would be supposed to do. So, get rid of it. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/dmx.h.rst.exceptions | 1 - Documentation/media/uapi/dvb/dmx-get-caps.rst | 41 ---

[PATCH v2 04/27] media: dvb/intro: update references for TV standards

2017-09-01 Thread Mauro Carvalho Chehab
The references there are only for DVB. Add missing references for ATSC and ISDB standards. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/intro.rst | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/Documentation/media/uapi/dvb/intro.rst b/Docume

[PATCH v2 24/27] media: ca-get-descr-info.rst: document this ioctl

2017-09-01 Thread Mauro Carvalho Chehab
Instead of a generic boilerplate, fill it with relevant information about this ioctl. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/ca-get-descr-info.rst | 29 +++--- 1 file changed, 4 insertions(+), 25 deletions(-) diff --git a/Documentation/media/uapi/d

[PATCH v2 25/27] media: dvb CA docs: place undocumented data together with ioctls

2017-09-01 Thread Mauro Carvalho Chehab
Right now, the same undocumented structs are on two places: at ca_data_types.rst and together with their ioctls. Move them to just one place and use the standard way to represent them. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/ca-get-msg.rst| 38 ++---

[PATCH v2 01/27] media: ca.h: split typedefs from structs

2017-09-01 Thread Mauro Carvalho Chehab
Using typedefs inside the Kernel is against CodingStyle, and there's no good usage here. Just like we did at frontend.h, at changeset 0df289a209e0 ("[media] dvb: Get rid of typedev usage for enums"), let's keep those typedefs only to provide userspace backward compatibility. No functional changes

[PATCH v2 14/27] media: dmx.h: get rid of DMX_SET_SOURCE

2017-09-01 Thread Mauro Carvalho Chehab
No driver uses this ioctl, nor it is documented anywhere. So, get rid of it. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/dmx.h.rst.exceptions| 13 Documentation/media/uapi/dvb/dmx-set-source.rst | 44 - Documentation/media/uapi/dvb/dmx_f

[PATCH v2 22/27] media: ca-get-cap.rst: document this ioctl

2017-09-01 Thread Mauro Carvalho Chehab
Instead of a generic boilerplate, fill it with relevant information about this ioctl. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/ca-get-cap.rst | 36 - 1 file changed, 5 insertions(+), 31 deletions(-) diff --git a/Documentation/media/uapi/d

[PATCH v2 05/27] media: dvb/intro: update the history part of the document

2017-09-01 Thread Mauro Carvalho Chehab
Convergence doesn't exist anymore. The community itself maintains the spec. Update accordingly. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dvb/intro.rst | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/Documentation/media/uapi/dvb/intro

[PATCH v2 08/27] media: dvb/frontend.h: document the uAPI file

2017-09-01 Thread Mauro Carvalho Chehab
Most of the stuff at the Digital TV frontend header file are documented only at the Documentation. However, a few kernel-doc markups are there, several of them with parsing issues. Add the missing documentation, copying definitions from the Documentation when it applies, fixing some bugs. Please

[PATCH v2 19/27] media: ca.h: document most CA data types

2017-09-01 Thread Mauro Carvalho Chehab
For most of the stuff there, documenting is easy, as the header file contains information. Yet, I was unable to document two data structs: ca_msg and ca_descr As those two structs are used by a few drivers, keep them. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/dv

[PATCH v2 00/26] Improve DVB documentation and reduce its gap

2017-09-01 Thread Mauro Carvalho Chehab
The DVB documentation was negligected for a long time, with resulted on several gaps between the API description and its documentation. I'm doing a new reading at the documentation. As result of it, this series: - improves the introductory chapter, making it more generic; - Do some adjustments at

Re: [PATCH 12/15] media: dmx.h: get rid of DMX_SET_SOURCE

2017-09-01 Thread Mauro Carvalho Chehab
Em Fri, 1 Sep 2017 13:14:04 +0200 Honza Petrouš escreveu: > 2017-09-01 12:45 GMT+02:00 Mauro Carvalho Chehab : > > Em Fri, 1 Sep 2017 11:53:11 +0200 > > Honza Petrouš escreveu: > > > >> 2017-09-01 11:37 GMT+02:00 Mauro Carvalho Chehab > >> : > >> > Em Fri, 1 Sep 2017 08:28:20 +0200 > >> > H

Re: [PATCH v2 14/14] [media] v4l: Document explicit synchronization behaviour

2017-09-01 Thread Hans Verkuil
Hi Gustavo, I think I concentrate on this last patch first. Once I fully understand this I can review the code. Remember, it's been a while for me since I last looked at fences, so forgive me if I ask stupid questions. On the other hand, others with a similar lack of understanding of fences probab

Re: DRM Format Modifiers in v4l2

2017-09-01 Thread Rob Clark
On Fri, Sep 1, 2017 at 3:13 AM, Laurent Pinchart wrote: > Hi Nicolas, > > On Thursday, 31 August 2017 19:12:58 EEST Nicolas Dufresne wrote: >> Le jeudi 31 août 2017 à 17:28 +0300, Laurent Pinchart a écrit : >> >> e.g. if I have two devices which support MODIFIER_FOO, I could attempt >> >> to share

Re: [PATCH v6 5/5] v4l: fwnode: Support generic parsing of graph endpoints in a single port

2017-09-01 Thread Hans Verkuil
On 30/08/17 13:49, Sakari Ailus wrote: > This is the preferred way to parse the endpoints. > > Comment rcar-vin as an example. > > Signed-off-by: Sakari Ailus > --- > drivers/media/platform/rcar-vin/rcar-core.c | 111 > > drivers/media/platform/rcar-vin/rcar-dma.c

Re: [PATCH v6.1 4/5] v4l: fwnode: Support generic parsing of graph endpoints in a device

2017-09-01 Thread Hans Verkuil
Hi Sakari, On 30/08/17 14:45, 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 function for the job. Driver specific matters > can be taken into account i

Re: [PATCH 12/15] media: dmx.h: get rid of DMX_SET_SOURCE

2017-09-01 Thread Honza Petrouš
2017-09-01 12:45 GMT+02:00 Mauro Carvalho Chehab : > Em Fri, 1 Sep 2017 11:53:11 +0200 > Honza Petrouš escreveu: > >> 2017-09-01 11:37 GMT+02:00 Mauro Carvalho Chehab : >> > Em Fri, 1 Sep 2017 08:28:20 +0200 >> > Honza Petrouš escreveu: >> > >> >> 2017-09-01 1:46 GMT+02:00 Mauro Carvalho Chehab

Re: [PATCH v6 3/5] docs-rst: v4l: Include Qualcomm CAMSS in documentation build

2017-09-01 Thread Hans Verkuil
On 30/08/17 13:49, Sakari Ailus wrote: > Qualcomm CAMSS was left out from documentation build. Fix this. > > Signed-off-by: Sakari Ailus > Reviewed-by: Laurent Pinchart Acked-by: Hans Verkuil Regards, Hans > --- > Documentation/media/v4l-drivers/index.rst | 1 + > 1 file changed,

Re: [PATCH v6 2/5] v4l: async: Add V4L2 async documentation to the documentation build

2017-09-01 Thread Hans Verkuil
On 30/08/17 13:49, Sakari Ailus wrote: > The V4L2 async wasn't part of the documentation build. Fix this. > > Signed-off-by: Sakari Ailus > Reviewed-by: Niklas Söderlund Acked-by: Hans Verkuil Regards, Hans > --- > Documentation/media/kapi/v4l2-async.rst | 3 +++ > Documentation/

Re: [PATCH v6 1/5] v4l: fwnode: Move KernelDoc documentation to the header

2017-09-01 Thread Hans Verkuil
On 30/08/17 13:49, Sakari Ailus wrote: > In V4L2 the practice is to have the KernelDoc documentation in the header > and not in .c source code files. This consequientally makes the V4L2 > fwnode function documentation part of the Media documentation build. > > Also correct the link related functio

Re: [PATCH 12/15] media: dmx.h: get rid of DMX_SET_SOURCE

2017-09-01 Thread Mauro Carvalho Chehab
Em Fri, 1 Sep 2017 11:53:11 +0200 Honza Petrouš escreveu: > 2017-09-01 11:37 GMT+02:00 Mauro Carvalho Chehab : > > Em Fri, 1 Sep 2017 08:28:20 +0200 > > Honza Petrouš escreveu: > > > >> 2017-09-01 1:46 GMT+02:00 Mauro Carvalho Chehab > >> : > >> > No driver uses this ioctl, nor it is docume

Re: [PATCH v6 1/7] media: add glossary.rst with a glossary of terms used at V4L2 spec

2017-09-01 Thread Hans Verkuil
On 29/08/17 15:17, Mauro Carvalho Chehab wrote: > Add a glossary of terms for V4L2, as several concepts are complex > enough to cause misunderstandings. > > Signed-off-by: Mauro Carvalho Chehab > --- > Documentation/media/uapi/v4l/glossary.rst | 150 > ++ > Documenta

Re: [PATCH 12/15] media: dmx.h: get rid of DMX_SET_SOURCE

2017-09-01 Thread Honza Petrouš
2017-09-01 11:37 GMT+02:00 Mauro Carvalho Chehab : > Em Fri, 1 Sep 2017 08:28:20 +0200 > Honza Petrouš escreveu: > >> 2017-09-01 1:46 GMT+02:00 Mauro Carvalho Chehab : >> > No driver uses this ioctl, nor it is documented anywhere. >> > >> > So, get rid of it. >> > >> > Signed-off-by: Mauro Carvalh

[ANN] Call for topics for the media mini-summit on Friday Oct 27 in Prague

2017-09-01 Thread Hans Verkuil
Hi all, We are organizing a media mini-summit on Friday October 27 in Prague, co-located with the ELCE conference: http://events.linuxfoundation.org/events/embedded-linux-conference-europe This is a call for topics to discuss during that mini-summit. Also, if you plan to attend, please let me k

Re: [PATCH 12/15] media: dmx.h: get rid of DMX_SET_SOURCE

2017-09-01 Thread Mauro Carvalho Chehab
Em Fri, 1 Sep 2017 08:28:20 +0200 Honza Petrouš escreveu: > 2017-09-01 1:46 GMT+02:00 Mauro Carvalho Chehab : > > No driver uses this ioctl, nor it is documented anywhere. > > > > So, get rid of it. > > > > Signed-off-by: Mauro Carvalho Chehab > > --- > > Documentation/media/dmx.h.rst.exception

Re: [PATCH 00/15] Improve DVB documentation and reduce its gap

2017-09-01 Thread Mauro Carvalho Chehab
Em Fri, 1 Sep 2017 10:40:28 +0200 Honza Petrouš escreveu: > 2017-09-01 1:46 GMT+02:00 Mauro Carvalho Chehab : > > The DVB documentation was negligected for a long time, with > > resulted on several gaps between the API description and its > > documentation. > > > > I'm doing a new reading at the

Re: [PATCH v5 4/5] v4l: fwnode: Support generic parsing of graph endpoints in a device

2017-09-01 Thread Sakari Ailus
Hi Laurent, On Fri, Sep 01, 2017 at 11:55:43AM +0300, Laurent Pinchart wrote: ... > > >> +static int parse_endpoint( > > >> +struct device *dev, struct v4l2_async_notifier *notifier, > > >> +struct fwnode_handle *endpoint, unsigned int asd_struct_size, > > >> +int (*parse_s

Re: [PATCH v5 4/5] v4l: fwnode: Support generic parsing of graph endpoints in a device

2017-09-01 Thread Laurent Pinchart
Hi Sakari, On Tuesday, 29 August 2017 17:31:21 EEST Sakari Ailus wrote: > On Tue, Aug 29, 2017 at 05:02:54PM +0300, Laurent Pinchart wrote: > > On Tuesday, 29 August 2017 14:03:12 EEST Sakari Ailus wrote: > >> The current practice is that drivers iterate over their endpoints and > >> parse each en

Re: [PATCH 00/15] Improve DVB documentation and reduce its gap

2017-09-01 Thread Honza Petrouš
2017-09-01 1:46 GMT+02:00 Mauro Carvalho Chehab : > The DVB documentation was negligected for a long time, with > resulted on several gaps between the API description and its > documentation. > > I'm doing a new reading at the documentation. As result of it, > this series: > > - improves the introd

Re: [PATCH V2 0/1] build: gpio-ir-tx for 3.13

2017-09-01 Thread Hans Verkuil
On 27/08/17 12:35, Jasmin J. wrote: > From: Jasmin Jessich > > Changes to V1: > Moved IR_GPIO_TX from 4.10.0 to 3.13.0 instead of adding it. > > Kernel 3.17 introduces GPIOD_OUT_LOW/HIGH. gpio-ir-tx requires this > definitions. This patch adds the API calls prior to 3.17 to be used > by gpio-ir

[PATCH] cec-pin.c: use proper ktime accessor functions

2017-09-01 Thread Hans Verkuil
Use ktime_to_ns/ns_to_ktime. This makes it possible to work with older kernels and the media_build compatibility system. For the mainline kernel these functions are NOPs. Signed-off-by: Hans Verkuil --- drivers/media/cec/cec-pin.c | 37 ++--- 1 file changed, 22 i

Re: [PATCH] [media_build] update v4.7_dma_attrs.patch

2017-09-01 Thread Hans Verkuil
On 01/09/17 09:41, Jasmin J. wrote: > Hi! > > To get media-build working again this is needs to be merged also. I just applied this. I had to wait until the ktime accessor function fixes were merged in the master tree and that got delayed a bit. Thanks, Hans > > BR, >Jasmin > >

Re: [PATCH] [media_build] update v4.7_dma_attrs.patch

2017-09-01 Thread Jasmin J.
Hi! To get media-build working again this is needs to be merged also. BR, Jasmin On 08/28/2017 06:08 PM, Daniel Scheller wrote: > From: Daniel Scheller > > Fixes apply_patches wrt > > commit 5b6f9abe5a49 ("media: vb2: add bidirectional flag in vb2_queue") > > Signed-off-by: Daniel Sche

Re: DRM Format Modifiers in v4l2

2017-09-01 Thread Laurent Pinchart
Hi Nicolas, On Thursday, 31 August 2017 19:12:58 EEST Nicolas Dufresne wrote: > Le jeudi 31 août 2017 à 17:28 +0300, Laurent Pinchart a écrit : > >> e.g. if I have two devices which support MODIFIER_FOO, I could attempt > >> to share a buffer between them which uses MODIFIER_FOO without > >> neces