Re: Shawn Guo: your attetion is needed here Re: [PATCH v8 00/34] i.MX Media Driver

2017-06-20 Thread Pavel Machek
On Tue 2017-06-20 08:05:05, Fabio Estevam wrote: > On Tue, Jun 20, 2017 at 5:29 AM, Pavel Machek wrote: > > > Hmm. I changed the subject to grab Shawn's attetion. > > > > But his acks should not be needed for forward progress. Yes, it would > > be good, but he does not react -- so

Re: Shawn Guo: your attetion is needed here Re: [PATCH v8 00/34] i.MX Media Driver

2017-06-20 Thread Fabio Estevam
On Tue, Jun 20, 2017 at 5:29 AM, Pavel Machek wrote: > Hmm. I changed the subject to grab Shawn's attetion. > > But his acks should not be needed for forward progress. Yes, it would > be good, but he does not react -- so just reorder the series so that > dts changes come last, then

Shawn Guo: your attetion is needed here Re: [PATCH v8 00/34] i.MX Media Driver

2017-06-20 Thread Pavel Machek
Hi! > >> But as Pavel pointed out, in fact we are missing many > >> Acks still, for all of the dts source changes (patches > >> 4-14), as well as really everything else (imx-media staging > >> driver patches). > > > > No Acks needed for the staging part. It's staging, so not held > > to the same

Re: [PATCH v8 00/34] i.MX Media Driver

2017-06-12 Thread Steve Longerbeam
On 06/11/2017 01:05 PM, Vladimir Zapolskiy wrote: On 06/10/2017 02:26 AM, Hans Verkuil wrote: On 10/06/17 01:16, Steve Longerbeam wrote: On 06/07/2017 12:02 PM, Hans Verkuil wrote: We're still waiting for an Ack for patch 02/34, right? Hi Hans, Rub has provided an Ack for patch 2.

Re: [PATCH v8 00/34] i.MX Media Driver

2017-06-11 Thread Vladimir Zapolskiy
On 06/10/2017 02:26 AM, Hans Verkuil wrote: > On 10/06/17 01:16, Steve Longerbeam wrote: >> >> >> On 06/07/2017 12:02 PM, Hans Verkuil wrote: >>> We're still waiting for an Ack for patch 02/34, right? >>> >> >> Hi Hans, Rub has provided an Ack for patch 2. >> >>> Other than that everything is

Re: [PATCH v8 00/34] i.MX Media Driver

2017-06-11 Thread Hans Verkuil
On 06/11/2017 10:28 AM, Hans Verkuil wrote: Hi Steve, Philipp, While preparing the pull request I noticed that the MAINTAINERS file wasn't updated. Steve, can you post a patch adding entries for the imx and ov5640 driver? Philipp, can you do the same for the video mux? I assume you're the

Re: [PATCH v8 00/34] i.MX Media Driver

2017-06-11 Thread Hans Verkuil
Hi Steve, Philipp, While preparing the pull request I noticed that the MAINTAINERS file wasn't updated. Steve, can you post a patch adding entries for the imx and ov5640 driver? Philipp, can you do the same for the video mux? I assume you're the maintainer for this? Thanks! I also made it

Re: [PATCH v8 00/34] i.MX Media Driver

2017-06-10 Thread Pavel Machek
Hi! > >> Other than that everything is ready AFAICT. > >> > > > > But as Pavel pointed out, in fact we are missing many > > Acks still, for all of the dts source changes (patches > > 4-14), as well as really everything else (imx-media staging > > driver patches). > > No Acks needed for the

Re: [PATCH v8 00/34] i.MX Media Driver

2017-06-09 Thread Hans Verkuil
On 10/06/17 01:16, Steve Longerbeam wrote: > > > On 06/07/2017 12:02 PM, Hans Verkuil wrote: >> We're still waiting for an Ack for patch 02/34, right? >> > > Hi Hans, Rub has provided an Ack for patch 2. > >> Other than that everything is ready AFAICT. >> > > But as Pavel pointed out, in fact

Re: [PATCH v8 00/34] i.MX Media Driver

2017-06-09 Thread Steve Longerbeam
On 06/09/2017 04:16 PM, Steve Longerbeam wrote: On 06/07/2017 12:02 PM, Hans Verkuil wrote: We're still waiting for an Ack for patch 02/34, right? Hi Hans, Rub damn, I really need to proof-read before hitting send. "Rob" (sorry Rob!). Steve

Re: [PATCH v8 00/34] i.MX Media Driver

2017-06-09 Thread Steve Longerbeam
On 06/07/2017 12:02 PM, Hans Verkuil wrote: We're still waiting for an Ack for patch 02/34, right? Hi Hans, Rub has provided an Ack for patch 2. Other than that everything is ready AFAICT. But as Pavel pointed out, in fact we are missing many Acks still, for all of the dts source

Re: [PATCH v8 00/34] i.MX Media Driver

2017-06-07 Thread Steve Longerbeam
On 06/07/2017 12:02 PM, Hans Verkuil wrote: We're still waiting for an Ack for patch 02/34, right? Hi Hans, Yes still waiting for an ack for the imx-media bindings. Other than that everything is ready AFAICT. Agreed. Steve Regards, Hans On 07/06/17 20:33, Steve Longerbeam

Re: [PATCH v8 00/34] i.MX Media Driver

2017-06-07 Thread Hans Verkuil
We're still waiting for an Ack for patch 02/34, right? Other than that everything is ready AFAICT. Regards, Hans On 07/06/17 20:33, Steve Longerbeam wrote: > In version 8: > > - Switched to v4l2_fwnode APIs. > > - Always pass a valid CSI id to ipu_set_ic_src_mux() in imx-ic-prp, even

[PATCH v8 00/34] i.MX Media Driver

2017-06-07 Thread Steve Longerbeam
In version 8: - Switched to v4l2_fwnode APIs. - Always pass a valid CSI id to ipu_set_ic_src_mux() in imx-ic-prp, even if the IC is receiving from the VDIC. The reason is due to a bug in the i.MX6 reference manual: from experiment it is determined that the CSI id select bit in IPU_CONF