Re: [PATCH v9 00/13] media: staging/imx7: add i.MX7 media driver

2019-01-18 Thread Rui Miguel Silva
Oi Fabio, On Fri 18 Jan 2019 at 16:49, Fabio Estevam wrote: Hi Rui, On Fri, Dec 7, 2018 at 10:44 AM Hans Verkuil wrote: I got a few checkpatch warnings about coding style: CHECK: Alignment should match open parenthesis #953: FILE: drivers/staging/media/imx/imx7-media-csi.c:911: +static

Re: [PATCH v9 00/13] media: staging/imx7: add i.MX7 media driver

2019-01-18 Thread Fabio Estevam
Hi Rui, On Fri, Dec 7, 2018 at 10:44 AM Hans Verkuil wrote: > I got a few checkpatch warnings about coding style: > > CHECK: Alignment should match open parenthesis > #953: FILE: drivers/staging/media/imx/imx7-media-csi.c:911: > +static struct v4l2_mbus_framefmt *imx7_csi_get_format(struct

Re: [PATCH v9 00/13] media: staging/imx7: add i.MX7 media driver

2018-12-07 Thread Rui Miguel Silva
Hi Hans and Dan, On Fri 07 Dec 2018 at 13:38, Dan Carpenter wrote: On Fri, Dec 07, 2018 at 01:44:00PM +0100, Hans Verkuil wrote: CHECK: Alignment should match open parenthesis #936: FILE: drivers/staging/media/imx/imx7-mipi-csis.c:921: + ret = v4l2_async_register_fwnode_subdev(mipi_sd, +

Re: [PATCH v9 00/13] media: staging/imx7: add i.MX7 media driver

2018-12-07 Thread Dan Carpenter
On Fri, Dec 07, 2018 at 01:44:00PM +0100, Hans Verkuil wrote: > CHECK: Alignment should match open parenthesis > #936: FILE: drivers/staging/media/imx/imx7-mipi-csis.c:921: > + ret = v4l2_async_register_fwnode_subdev(mipi_sd, > + sizeof(struct

Re: [PATCH v9 00/13] media: staging/imx7: add i.MX7 media driver

2018-12-07 Thread Hans Verkuil
On 11/22/2018 04:18 PM, Rui Miguel Silva wrote: > Hi, > This series introduces the Media driver to work with the i.MX7 SoC. it uses > the > already existing imx media core drivers but since the i.MX7, contrary to > i.MX5/6, do not have an IPU and because of that some changes in the imx media >

[PATCH v9 00/13] media: staging/imx7: add i.MX7 media driver

2018-11-22 Thread Rui Miguel Silva
Hi, This series introduces the Media driver to work with the i.MX7 SoC. it uses the already existing imx media core drivers but since the i.MX7, contrary to i.MX5/6, do not have an IPU and because of that some changes in the imx media core are made along this series to make it support that case.