[RFC PATCH V0 0/4] media: support Mediatek sensor interface driver

Hello, This is the first version of the RFC patch series adding Sensor Inferface(seninf) driver on Mediatek mt8183 SoC, which will be used in camera features on CrOS application. It belongs to the first Mediatek's camera driver series based on V4L2 and media controller framework. I posted the ma

[RFC PATCH V0 2/4] media: platform: Add Mediatek sensor interface driver KConfig

This patch adds KConfig for sensor interface driver. Sensor interface driver is a MIPI-CSI2 host driver, namely, a HW camera interface controller. It support a widely adopted, simple, high-speed protocol primarily intended for point-to-point image and video transmission between cameras and host dev

[RFC PATCH V0 3/4] dt-bindings: mt8183: Added sensor interface dt-bindings

This patch adds the DT binding documentation for the sensor interface module in Mediatek SoCs. Signed-off-by: Louis Kuo --- .../devicetree/bindings/media/mediatek-seninf.txt | 52 ++ 1 file changed, 52 insertions(+) create mode 100644 Documentation/devicetree/bindings/media

[RFC PATCH V0 1/4] media: platform: mtk-isp: Add Mediatek sensor interface driver

This patch adds Mediatek's sensor interface driver. Sensor interface driver is a MIPI-CSI2 host driver, namely, a HW camera interface controller. It support a widely adopted, simple, high-speed protocol primarily intended for point-to-point image and video transmission between cameras and host devi

[RFC PATCH V0 4/4] dts: arm64: mt8183: Add sensor interface nodes

Add nodes for Mediatek's sensor interface device. Sensor interface module embedded in Mediatek SOCs, works as a HW camera interface controller intended for image and data transmission between cameras and host devices. Signed-off-by: Louis Kuo --- arch/arm64/boot/dts/mediatek/mt8183.dtsi | 34 +++

cron job: media_tree daily build: OK

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 Feb 21 05:00:17 CET 2019 media-tree git hash:9fabe1d108ca4755a880de43f751f1c054f8894d media_build git

Re: [RESEND PATCH 3/7] mm/gup: Change GUP fast to use flags rather than a write 'bool'

Hi Ira, On Wed, Feb 20, 2019 at 11:01 AM wrote: > > From: Ira Weiny > > To facilitate additional options to get_user_pages_fast() change the > singular write parameter to be gup_flags. > > This patch does not change any functionality. New functionality will > follow in subsequent patches. > > S

Re: [PATCH v4 0/9] mm: Use vm_map_pages() and vm_map_pages_zero() API

On Fri, Feb 15, 2019 at 8:06 AM Souptick Joarder wrote: > > Previouly drivers have their own way of mapping range of > kernel pages/memory into user vma and this was done by > invoking vm_insert_page() within a loop. > > As this pattern is common across different drivers, it can > be generalized b

[PATCH -next] media: rc: remove unused including

Remove including that don't need it. Signed-off-by: YueHaibing --- drivers/media/rc/ir-rcmm-decoder.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/rc/ir-rcmm-decoder.c b/drivers/media/rc/ir-rcmm-decoder.c index f1096ac1e5c5..64fb65a9a19f 100644 --- a/drivers/media/rc/ir-rcm

[PATCH v2 1/4] media: imx: csi: Allow unknown nearest upstream entities

On i.MX6, the nearest upstream entity to the CSI can only be the CSI video muxes or the Synopsys DW MIPI CSI-2 receiver. However the i.MX53 has no CSI video muxes or a MIPI CSI-2 receiver. So allow for the nearest upstream entity to the CSI to be something other than those. Fixes: bf3cfaa712e5c (

[PATCH v2 2/4] media: imx: Clear fwnode link struct for each endpoint iteration

In imx_media_create_csi_of_links(), the 'struct v4l2_fwnode_link' must be cleared for each endpoint iteration, otherwise if the remote port has no "reg" property, link.remote_port will not be reset to zero. This was discovered on the i.MX53 SMD board, since the OV5642 connects directly to ipu1_csi0

[PATCH v2 3/4] media: imx: Rename functions that add IPU-internal subdevs

For the functions that add and remove the internal IPU subdevice descriptors, rename them to make clear they are the subdevs internal to the IPU. Also rename the platform data structure for the internal IPU subdevices. No functional changes. Signed-off-by: Steve Longerbeam Acked-by: Philipp Zabel

[PATCH v2 4/4] media: imx: Don't register IPU subdevs/links if CSI port missing

The second IPU internal sub-devices were being registered and links to them created even when the second IPU is not present. This is wrong for i.MX6 S/DL and i.MX53 which have only a single IPU. Fixes: e130291212df5 ("[media] media: Add i.MX media core driver") Signed-off-by: Steve Longerbeam Re

[PATCH v2 0/4] media: imx: Various fixes for i.MX5 support

Some fixes and improvements to support video capture on i.MX5. History: v2: - Rebased with merge of imx7 capture patches. Steve Longerbeam (4): media: imx: csi: Allow unknown nearest upstream entities media: imx: Clear fwnode link struct for each endpoint iteration media: imx: Rename functi

[PATCH V2 2/2] Add suport for Avermedia TD310

This patch add support for Avermedia TD310 usb stik. Signed-off-by: Jose Alberto Reguero diff -upr linux/drivers/media/usb/dvb-usb-v2/af9035.c linux.new/drivers/media/usb/dvb-usb-v2/af9035.c --- linux/drivers/media/usb/dvb-usb-v2/af9035.c 2019-02-20 16:54:24.121847069 +0100 +++ linux.new/dri

[PATCH V2 1/2] af9035: init i2c already in it930x_frontend_attach

This patch init i2c in it930x_frontend_attach instead of it930x_tuner_attach. From: Andreas Kemnade Signed-off-by: Jose Alberto Reguero diff -upr linux/drivers/media/usb/dvb-usb-v2/af9035.c linux.new/drivers/media/usb/dvb-usb-v2/af9035.c --- linux/drivers/media/usb/dvb-usb-v2/af9035.c 2018-09

[PATCH V2 0/2] Add suport for Avermedia TD310

This serie add support for Avermedia TD310 usb stik. Jose Alberto Reguero -- Enviado desde mi dispositivo Android con K-9 Mail. Por favor, disculpa mi brevedad.

Re: [PATCH yavta 0/3] Fixes for compound control support

On Wed, Feb 20, 2019 at 05:19:49PM +0200, Laurent Pinchart wrote: > Hello, > > This small series fixes issues in yavta reported during the review of > the compound control support patches. > > Laurent Pinchart (3): > Fix emulation of old API for string controls > Print numerical control type

Re: [PATCH yavta 0/7] Compound controls and controls reset support

Hi Laurent, On Wed, Feb 20, 2019 at 02:51:16PM +0200, Laurent Pinchart wrote: > Hello, > > This patch series implements support for compound controls in yavta, > including the ability to reset controls to their default value. Only > array compound controls are supported for now, other types may b

Re: [PATCH yavta 6/7] Support setting control from values stored in a file

Hi Laurent, On Wed, Feb 20, 2019 at 02:51:22PM +0200, Laurent Pinchart wrote: > Signed-off-by: Laurent Pinchart > --- > yavta.c | 25 + > 1 file changed, 25 insertions(+) > > diff --git a/yavta.c b/yavta.c > index 1490878c6f7e..2d49131a4271 100644 > --- a/yavta.c > +++ b

Re: [PATCH yavta 4/7] Implement compound control set support

Hi Laurent, On Wed, Feb 20, 2019 at 02:51:20PM +0200, Laurent Pinchart wrote: > Only arrays of integer types are supported. > > Signed-off-by: Laurent Pinchart > --- > yavta.c | 228 ++-- > 1 file changed, 172 insertions(+), 56 deletions(-) >

Re: [PATCH yavta 3/7] Implement compound control get support

Hi Laurent, On Wed, Feb 20, 2019 at 04:55:06PM +0200, Laurent Pinchart wrote: > Hi Sakari, > > On Wed, Feb 20, 2019 at 04:06:43PM +0200, Sakari Ailus wrote: > > On Wed, Feb 20, 2019 at 02:51:19PM +0200, Laurent Pinchart wrote: > > > Signed-off-by: Laurent Pinchart > > > --- > > > yavta.c | 154

Re: [RESEND PATCH 3/7] mm/gup: Change GUP fast to use flags rather than a write 'bool'

Hi Ira Martin and I looked at your patch and agree that it doesn't change functionality for Orangefs. Reviewed-by: Mike Marshall On Wed, Feb 20, 2019 at 12:32 AM wrote: > > From: Ira Weiny > > To facilitate additional options to get_user_pages_fast() change the > singular write parameter to

Re: [RESEND PATCH 0/7] Add FOLL_LONGTERM to GUP fast and use it

On Wed, Feb 20, 2019 at 07:19:30AM -0800, Christoph Hellwig wrote: > On Tue, Feb 19, 2019 at 09:30:33PM -0800, ira.we...@intel.com wrote: > > From: Ira Weiny > > > > Resending these as I had only 1 minor comment which I believe we have > > covered > > in this series. I was anticipating these go

Re: [PATCH v4 2/2] media: cedrus: Add H264 decoding support

Hi! I really wanted to do another review on previous series but got distracted by analyzing one particulary troublesome H264 sample. It still doesn't work correctly, so I would ask you if you can test it with your stack (it might be userspace issue): http://jernej.libreelec.tv/videos/problemat

Re: [RESEND PATCH 0/7] Add FOLL_LONGTERM to GUP fast and use it

On Tue, Feb 19, 2019 at 09:30:33PM -0800, ira.we...@intel.com wrote: > From: Ira Weiny > > Resending these as I had only 1 minor comment which I believe we have covered > in this series. I was anticipating these going through the mm tree as they > depend on a cleanup patch there and the IB chang

[PATCH yavta 1/3] Fix emulation of old API for string controls

For kernel versions that didn't define the V4L2_CTRL_FLAG_HAS_PAYLOAD flag, commits 2f146567186f ("Implement compound control get support") and 4480b561404f ("Implement compound control set support") broke string control support. Fix this by emulating the V4L2_CTRL_FLAG_HAS_PAYLOAD and compound con

[PATCH yavta 3/3] Fix control array parsing

Commit 4480b561404f ("Implement compound control set support") didn't properly parse control array values. Fix it. Fixes: 4480b561404f ("Implement compound control set support") Signed-off-by: Laurent Pinchart --- yavta.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/y

[PATCH yavta 2/3] Print numerical control type for unsupported types

Help diagnosing problems by reporting the type of unsupported control types instead of just printing "unsupported". Signed-off-by: Laurent Pinchart --- yavta.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/yavta.c b/yavta.c index 7d9c40c9f9be..86447f3f057d 100644 --- a/yavt

[PATCH yavta 0/3] Fixes for compound control support

Hello, This small series fixes issues in yavta reported during the review of the compound control support patches. Laurent Pinchart (3): Fix emulation of old API for string controls Print numerical control type for unsupported types Fix control array parsing yavta.c | 13 ++--- 1

Re: [PATCH yavta 3/7] Implement compound control get support

Hi Sakari, On Wed, Feb 20, 2019 at 04:06:43PM +0200, Sakari Ailus wrote: > On Wed, Feb 20, 2019 at 02:51:19PM +0200, Laurent Pinchart wrote: > > Signed-off-by: Laurent Pinchart > > --- > > yavta.c | 154 ++-- > > 1 file changed, 115 insertions(

[PATCH v4 1/2] media: uapi: Add H264 low-level decoder API compound controls.

From: Pawel Osciak Stateless video codecs will require both the H264 metadata and slices in order to be able to decode frames. This introduces the definitions for a new pixel format for H264 slices that have been parsed, as well as the structures used to pass the metadata from the userspace to t

[PATCH v4 2/2] media: cedrus: Add H264 decoding support

Introduce some basic H264 decoding support in cedrus. So far, only the baseline profile videos have been tested, and some more advanced features used in higher profiles are not even implemented. Signed-off-by: Maxime Ripard --- drivers/staging/media/sunxi/cedrus/Makefile | 3 +- drivers/

[PATCH v4 0/2] media: cedrus: Add H264 decoding support

Hi, Here is a new version of the H264 decoding support in the cedrus driver. As you might already know, the cedrus driver relies on the Request API, and is a reverse engineered driver for the video decoding engine found on the Allwinner SoCs. This work has been possible thanks to the work done b

Re: [PATCH yavta 1/7] yavta: Refactor video_list_controls()

Hi Laurent, On Wed, Feb 20, 2019 at 04:07:53PM +0200, Laurent Pinchart wrote: > Hi Sakari, > > On Wed, Feb 20, 2019 at 03:21:57PM +0200, Sakari Ailus wrote: > > On Wed, Feb 20, 2019 at 02:51:17PM +0200, Laurent Pinchart wrote: > > > Separate iteration over controls from printing, in order to reus

Re: [PATCH yavta 1/7] yavta: Refactor video_list_controls()

Hi Sakari, On Wed, Feb 20, 2019 at 03:21:57PM +0200, Sakari Ailus wrote: > On Wed, Feb 20, 2019 at 02:51:17PM +0200, Laurent Pinchart wrote: > > Separate iteration over controls from printing, in order to reuse the > > iteration to implement control reset. > > > > Signed-off-by: Laurent Pinchart

Re: [PATCH yavta 3/7] Implement compound control get support

Hi Laurent, On Wed, Feb 20, 2019 at 02:51:19PM +0200, Laurent Pinchart wrote: > Signed-off-by: Laurent Pinchart > --- > yavta.c | 154 ++-- > 1 file changed, 115 insertions(+), 39 deletions(-) > > diff --git a/yavta.c b/yavta.c > index eb50d59

Re: [RFCv2 PATCH] videodev2.h: introduce VIDIOC_DQ_EXT_EVENT

Hi Hans, On Tue, Feb 05, 2019 at 02:49:45PM +0100, Hans Verkuil wrote: > This patch adds an extended version of VIDIOC_DQEVENT that: > > 1) is Y2038 safe by using a __u64 for the timestamp > 2) needs no compat32 conversion code > 3) is able to handle control events from 64-bit control types >

Re: [PATCH yavta 1/7] yavta: Refactor video_list_controls()

Hi Laurent, Thanks for the patchset. On Wed, Feb 20, 2019 at 02:51:17PM +0200, Laurent Pinchart wrote: > Separate iteration over controls from printing, in order to reuse the > iteration to implement control reset. > > Signed-off-by: Laurent Pinchart > --- > yavta.c | 133 +

Re: [PATCH] Add suport for Avermedia TD310

On Wed, Feb 20, 2019 at 01:49:28PM +0100, Jose Alberto Reguero wrote: > El 20 de febrero de 2019 10:37:15 CET, Sean Young escribió: > >On Tue, Feb 19, 2019 at 11:51:25PM +0100, Jose Alberto Reguero wrote: > >> El 19 de febrero de 2019 22:57:31 CET, Sean Young > >escribió: > >> >On Mon, Feb 11, 20

[PATCH yavta 3/7] Implement compound control get support

Signed-off-by: Laurent Pinchart --- yavta.c | 154 ++-- 1 file changed, 115 insertions(+), 39 deletions(-) diff --git a/yavta.c b/yavta.c index eb50d592736f..6428c22f88d7 100644 --- a/yavta.c +++ b/yavta.c @@ -529,6 +529,7 @@ static int get_con

[PATCH yavta 2/7] Implement VIDIOC_QUERY_EXT_CTRL support

Use the new extended control query ioctl when available with an automatic fall back to VIDIOC_QUERYCTRL. Signed-off-by: Laurent Pinchart --- yavta.c | 63 +++-- 1 file changed, 48 insertions(+), 15 deletions(-) diff --git a/yavta.c b/yavta.c i

[PATCH yavta 4/7] Implement compound control set support

Only arrays of integer types are supported. Signed-off-by: Laurent Pinchart --- yavta.c | 228 ++-- 1 file changed, 172 insertions(+), 56 deletions(-) diff --git a/yavta.c b/yavta.c index 6428c22f88d7..d1bfd380c03b 100644 --- a/yavta.c +++ b/y

[PATCH yavta 1/7] yavta: Refactor video_list_controls()

Separate iteration over controls from printing, in order to reuse the iteration to implement control reset. Signed-off-by: Laurent Pinchart --- yavta.c | 133 ++-- 1 file changed, 82 insertions(+), 51 deletions(-) diff --git a/yavta.c b/yavta.

[PATCH yavta 5/7] Add support to reset device controls

From: Kieran Bingham Provide a new option '--reset-controls' which will enumerate the available controls on a device or sub-device, and re-initialise them to defaults. Signed-off-by: Kieran Bingham Signed-off-by: Laurent Pinchart --- yavta.c | 62 ++

[PATCH yavta 0/7] Compound controls and controls reset support

Hello, This patch series implements support for compound controls in yavta, including the ability to reset controls to their default value. Only array compound controls are supported for now, other types may be added later. The patches have lived out of the master branch for long enough, it's tim

[PATCH yavta 6/7] Support setting control from values stored in a file

Signed-off-by: Laurent Pinchart --- yavta.c | 25 + 1 file changed, 25 insertions(+) diff --git a/yavta.c b/yavta.c index 1490878c6f7e..2d49131a4271 100644 --- a/yavta.c +++ b/yavta.c @@ -1334,6 +1334,31 @@ static int video_parse_control_array(const struct v4l2_query_ext

[PATCH yavta 7/7] Remove unneeded conditional compilation for old V4L2 API versions

As we include a copy of the V4L2 kernel headers, there's no need for conditional compilation to support old versions of the API. Signed-off-by: Laurent Pinchart --- yavta.c | 11 --- 1 file changed, 11 deletions(-) diff --git a/yavta.c b/yavta.c index 2d49131a4271..741df82a8db0 100644 -

Re: [PATCH] Add suport for Avermedia TD310

El 20 de febrero de 2019 10:37:15 CET, Sean Young escribió: >On Tue, Feb 19, 2019 at 11:51:25PM +0100, Jose Alberto Reguero wrote: >> El 19 de febrero de 2019 22:57:31 CET, Sean Young >escribió: >> >On Mon, Feb 11, 2019 at 11:13:25PM +0100, Jose Alberto Reguero >wrote: >> >> This patch add suppor

[ragnatech:media-tree] BUILD SUCCESS 9fabe1d108ca4755a880de43f751f1c054f8894d

tree/branch: git://git.ragnatech.se/linux media-tree branch HEAD: 9fabe1d108ca4755a880de43f751f1c054f8894d media: ipu3-mmu: fix some kernel-doc macros elapsed time: 383m configs tested: 140 The following configs have been built successfully. More configs may be tested in the coming days. par

Re: [PATCH v14 05/13] media: staging/imx7: add MIPI CSI-2 receiver subdev for i.MX7

Hi Hans, On Wed 20 Feb 2019 at 08:56, Hans Verkuil wrote: On 2/6/19 4:13 PM, Rui Miguel Silva wrote: Adds MIPI CSI-2 subdev for i.MX7 to connect with sensors with a MIPI CSI-2 interface. Signed-off-by: Rui Miguel Silva --- drivers/staging/media/imx/Makefile |1 + drivers/staging/

Re: [PATCH] imx7: fix smatch error

Hi Hans, Thanks for the patch. On Wed 20 Feb 2019 at 08:59, Hans Verkuil wrote: Fixes this smatch error: drivers/staging/media/imx/imx7-mipi-csis.c:716 mipi_csis_set_fmt() error: we previously assumed 'fmt' could beor null (see line 709) fmt is never NULL, so remove the 'fmt &&' condition.

Re: [PATCH 2/2] media: imx7-media-csi: get rid of unused var

Oi Mauro, On Tue 19 Feb 2019 at 13:24, Mauro Carvalho Chehab wrote: drivers/staging/media/imx/imx7-media-csi.c: In function 'imx7_csi_enum_mbus_code': drivers/staging/media/imx/imx7-media-csi.c:926:33: warning: variable 'in_cc' set but not used [-Wunused-but-set-variable] const str

Re: [PATCH 1/2] media: imx7-media-csi: don't store a floating pointer

Oi Mauro, On Tue 19 Feb 2019 at 13:24, Mauro Carvalho Chehab wrote: if imx7_csi_try_fmt() fails, outcc variable won't be initialized and csi->cc[IMX7_CSI_PAD_SRC] would be pointing to a random location. Signed-off-by: Mauro Carvalho Chehab Thanks for this, looks good. Reviewed-by: Rui Miguel

[PATCH 0/5] Fix most ImgU driver compiler / checker warnings

Hi Mauro, Hans, This set addresses most of the compiler / checker warnings from the ImgU driver. There are a few about stack usage from the compiler left but those were not trivial to address so I postponed that. There are also a few about memset but Hans suggested addressing that by increasing th

[PATCH 4/5] staging: imgu: Address compiler / checker warnings in MMU code

Address C compiler, sparse and smatch warnings and little style issues in the IMGU MMU code. Signed-off-by: Sakari Ailus --- drivers/staging/media/ipu3/ipu3-mmu.c | 39 +++ 1 file changed, 35 insertions(+), 4 deletions(-) diff --git a/drivers/staging/media/ipu3/i

[PATCH 5/5] Revert "media: ipu3: shut up warnings produced with W=1"

This reverts commit 0bdfc56c13c0ffe003f28395fcde2cd9b5ea0622. The original patch suppressed compiler and checker warnings and those warnings have been addressed. Do not attempt to suppress these warnings anymore. Signed-off-by: Sakari Ailus --- drivers/staging/media/ipu3/Makefile | 6 -- dr

[PATCH 3/5] staging: imgu: Remove redundant checks

Remove redundant checks for less than zero on unsigned variables. Signed-off-by: Sakari Ailus --- drivers/staging/media/ipu3/ipu3-css-fw.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/media/ipu3/ipu3-css-fw.c b/drivers/staging/media/ipu3/ipu3-css-fw.

[PATCH 1/5] staging: imgu: Switch to __aligned() from __attribute__((aligned()))

__aligned() is preferred. The patch has been generated using the following command in the drivers/staging/media/ipu3 directory: $ git grep -l 'aligned(32)' | \ xargs perl -i -pe \ 's/__attribute__\s*\(\(\s*aligned\s*\(([0-9]+)\s*\)\s*\)\)/__aligned($1)/g;' Signed-off-by: Sakari A

[PATCH 2/5] staging: imgu: Fix struct ipu3_uapi_awb_fr_config_s alignment

struct ipu3_uapi_awb_fr_config_s is labelled as to be aligned to 32 bytes but that's not the case in the ISP parameter struct of the driver, struct ipu3_uapi_acc_param which is packed. Also there is no need for the alignment as the struct is only handled by the driver. Remove the alignment from th

Re: [PATCH 1/3] [media] rc/keymaps: add keytable for Pine64 IR Remote Controller

On Mon, Feb 18, 2019 at 09:59:36PM +, Jonas Karlman wrote: > This RC map is based on remote key schema at [1], the mouse button key > did not have an obvious target and was mapped to KEY_CONTEXT_MENU. How about BTN_LEFT ? Thanks, Sean > > [1] http://files.pine64.org/doc/Pine%20A64%20Schema

hint needed : how to integrate sublvds to mipi csi-2 converter

Hello, I am working on a driver for sony sensors that output sublvds, but that must be connected to a processor expecting mipi csi-2. We use a Lattice lif-md6000 fpga for that purpose, but unfortunately the conversion is not transparent. The fpga must be instructed to know the bpp and line-lengt

Re: [PATCH] Add suport for Avermedia TD310

On Tue, Feb 19, 2019 at 11:51:25PM +0100, Jose Alberto Reguero wrote: > El 19 de febrero de 2019 22:57:31 CET, Sean Young escribió: > >On Mon, Feb 11, 2019 at 11:13:25PM +0100, Jose Alberto Reguero wrote: > >> This patch add support for Avermedia TD310 usb stik. > >> This patch must be aplied afte

[ragnatech:media-tree 306/317] drivers/staging/media/imx/imx7-media-csi.c:1086:11: error: passing argument 2 of 'imx_media_mbus_fmt_to_pix_fmt' from incompatible pointer type

tree: git://git.ragnatech.se/linux media-tree head: 9fabe1d108ca4755a880de43f751f1c054f8894d commit: 05f634040c0d05f59f2dcd39722157cb3b57c85b [306/317] media: staging/imx7: add imx7 CSI subdev driver config: sh-allmodconfig (attached as .config) compiler: sh4-linux-gnu-gcc (Debian 8.2.0-11) 8.

[PATCH] imx7: fix smatch error

Fixes this smatch error: drivers/staging/media/imx/imx7-mipi-csis.c:716 mipi_csis_set_fmt() error: we previously assumed 'fmt' could be null (see line 709) fmt is never NULL, so remove the 'fmt &&' condition. Signed-off-by: Hans Verkuil --- diff --git a/drivers/staging/media/imx/imx7-mipi-csis

Re: [PATCH v14 05/13] media: staging/imx7: add MIPI CSI-2 receiver subdev for i.MX7

On 2/6/19 4:13 PM, Rui Miguel Silva wrote: > Adds MIPI CSI-2 subdev for i.MX7 to connect with sensors with a MIPI > CSI-2 interface. > > Signed-off-by: Rui Miguel Silva > --- > drivers/staging/media/imx/Makefile |1 + > drivers/staging/media/imx/imx7-mipi-csis.c | 1186 ++

Re: [PATCH 01/10] media: Introduce helpers to fill pixel format structs

On 2/20/19 7:53 AM, Tomasz Figa wrote: > On Thu, Feb 7, 2019 at 1:36 AM Hans Verkuil wrote: >> >> On 2/6/19 5:22 PM, Ezequiel Garcia wrote: >>> On Wed, 2019-02-06 at 11:43 +0100, Hans Verkuil wrote: Hi Ezequiel, A quick review below. This looks really useful, BTW. On 2/5/1

Re: [linux-uvc-devel] metadata device file

Hi Moritz, On Wed, 20 Feb 2019, Moritz Dötterl wrote: > Hi Kieran > > > Thank you very much for your explanation. > > At least in the setup we are using every /dev/video device is a capture > device. > > Wouldn't it be good to have that configurable with a module parameter? I don't think it