Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-24 Thread Philipp Zabel
Hi Steve, On Mon, 2017-01-09 at 20:43 +0100, Philipp Zabel wrote: > On the other hand, there is currently no way to communicate to userspace > that the IC can't downscale bilinearly, but only to 1/2 or 1/4 of the > input resolution, and then scale up bilinearly for there. This is completely

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-16 Thread Philipp Zabel
Hi Steve, On Sat, 2017-01-14 at 12:26 -0800, Steve Longerbeam wrote: > > On 01/13/2017 03:05 AM, Philipp Zabel wrote: > > Hi Steve, > > > > Am Donnerstag, den 12.01.2017, 15:22 -0800 schrieb Steve Longerbeam: [...] > >> I would > >> imagine it will need two additional inputs and another output

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-14 Thread Steve Longerbeam
On 01/13/2017 03:05 AM, Philipp Zabel wrote: Hi Steve, Am Donnerstag, den 12.01.2017, 15:22 -0800 schrieb Steve Longerbeam: Hi Philipp, JM, First, let me say that you both have convinced me that we need a VDIC entity. I've implemented that in the branch imx-media-staging-md-vdic. At this

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-13 Thread Philipp Zabel
Am Donnerstag, den 12.01.2017, 18:22 -0800 schrieb Steve Longerbeam: > > On 01/12/2017 03:43 PM, Steve Longerbeam wrote: > > > > > > On 01/12/2017 03:22 PM, Steve Longerbeam wrote: > >> > >> > and since my PRPVF entity roles > up the VDIC internally, it is actually receiving from the

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-13 Thread Philipp Zabel
Am Donnerstag, den 12.01.2017, 15:22 -0800 schrieb Steve Longerbeam: > Hi Philipp, JM, > > First, let me say that you both have convinced me that we need a VDIC > entity. I've implemented that in the branch imx-media-staging-md-vdic. > At this point it only implements the M/C de-interlacing

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-13 Thread Philipp Zabel
Hi Steve, Am Donnerstag, den 12.01.2017, 15:22 -0800 schrieb Steve Longerbeam: > Hi Philipp, JM, > > First, let me say that you both have convinced me that we need a VDIC > entity. I've implemented that in the branch imx-media-staging-md-vdic. > At this point it only implements the M/C

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-12 Thread Steve Longerbeam
On 01/12/2017 03:43 PM, Steve Longerbeam wrote: On 01/12/2017 03:22 PM, Steve Longerbeam wrote: and since my PRPVF entity roles up the VDIC internally, it is actually receiving from the VDIC channel. So unless you think we should have a distinct VDIC entity, I would like to keep this

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-12 Thread Steve Longerbeam
On 01/12/2017 03:22 PM, Steve Longerbeam wrote: and since my PRPVF entity roles up the VDIC internally, it is actually receiving from the VDIC channel. So unless you think we should have a distinct VDIC entity, I would like to keep this the way it is. Yes, I think VDIC should be separated

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-12 Thread Steve Longerbeam
Hi Philipp, JM, First, let me say that you both have convinced me that we need a VDIC entity. I've implemented that in the branch imx-media-staging-md-vdic. At this point it only implements the M/C de-interlacing function, not the plane Combiner. So it has only one input and one output pad. I

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-11 Thread Philipp Zabel
Am Montag, den 09.01.2017, 16:15 -0800 schrieb Steve Longerbeam: [...] > Since the VDIC sends its motion compensated frames to the PRP VF task, > I've created the PRPVF entity solely for motion compensated de-interlace > support. I don't really see any other use for the PRPVF task except for >

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-11 Thread Philipp Zabel
Hi Steve, Am Dienstag, den 10.01.2017, 15:52 -0800 schrieb Steve Longerbeam: > > On 01/09/2017 04:15 PM, Steve Longerbeam wrote: > > Hi Philipp, > > > > > > On 01/09/2017 11:43 AM, Philipp Zabel wrote: > > > > > > > >> One is the amount and organization of subdevices/media entities visible > >>

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-11 Thread Jean-Michel Hautbois
Hi Steve and Philipp, 2017-01-11 0:52 GMT+01:00 Steve Longerbeam : > > > On 01/09/2017 04:15 PM, Steve Longerbeam wrote: >> >> Hi Philipp, >> >> >> On 01/09/2017 11:43 AM, Philipp Zabel wrote: >> >> >> >>> >>> One is the amount and organization of subdevices/media

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-10 Thread Steve Longerbeam
On 01/09/2017 04:15 PM, Steve Longerbeam wrote: Hi Philipp, On 01/09/2017 11:43 AM, Philipp Zabel wrote: One is the amount and organization of subdevices/media entities visible to userspace. The SMFCs should not be user controllable subdevices, but can be implicitly enabled when a CSI is

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-09 Thread Steve Longerbeam
Hi Philipp, On 01/09/2017 11:43 AM, Philipp Zabel wrote: Hi Steve, Am Freitag, den 30.12.2016, 12:26 -0800 schrieb Steve Longerbeam: On 12/30/2016 11:06 AM, Marek Vasut wrote: On 12/29/2016 09:51 PM, Robert Schwebel wrote: Hi Jean-Michel, Hi, On Thu, Dec 29, 2016 at 04:08:33PM +0100,

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-09 Thread Philipp Zabel
Hi Steve, Am Freitag, den 30.12.2016, 12:26 -0800 schrieb Steve Longerbeam: > > On 12/30/2016 11:06 AM, Marek Vasut wrote: > > On 12/29/2016 09:51 PM, Robert Schwebel wrote: > >> Hi Jean-Michel, > > Hi, > > > >> On Thu, Dec 29, 2016 at 04:08:33PM +0100, Jean-Michel Hautbois wrote: > >>> What is

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-02 Thread Steve Longerbeam
Hi JM, On 01/02/2017 06:59 AM, Jean-Michel Hautbois wrote: Steve: which branch is the correct one on your github ? branch imx-media-staging-md-v4 on g...@github.com:slongerbeam/mediatree.git Steve -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-02 Thread Fabio Estevam
Hi Jean-Michel, On Mon, Jan 2, 2017 at 12:59 PM, Jean-Michel Hautbois wrote: > Steve: which branch is the correct one on your github ? I have tested imx-media-staging-md-v2 (based on 4.9-rc) and also imx-media-staging-md-v4 branch, which is based on 4.10-rc1.

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-02 Thread Steve Longerbeam
Hi Hans, On 01/02/2017 06:45 AM, Hans Verkuil wrote: On 01/02/17 14:51, Jean-Michel Hautbois wrote: Hi, 2016-12-30 21:26 GMT+01:00 Steve Longerbeam : On 12/30/2016 11:06 AM, Marek Vasut wrote: On 12/29/2016 09:51 PM, Robert Schwebel wrote: Hi Jean-Michel,

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-02 Thread Fabio Estevam
On Mon, Jan 2, 2017 at 12:45 PM, Hans Verkuil wrote: > Steve's series is definitely preferred from my point of view. The feature > set is clearly superior to Philipp's driver. > > I plan on reviewing Steve's series soonish but a quick scan didn't see > anything > suspicious.

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-02 Thread Jean-Michel Hautbois
2017-01-02 15:45 GMT+01:00 Hans Verkuil : > On 01/02/17 14:51, Jean-Michel Hautbois wrote: >> >> Hi, >> >> 2016-12-30 21:26 GMT+01:00 Steve Longerbeam : >>> >>> >>> >>> On 12/30/2016 11:06 AM, Marek Vasut wrote: On 12/29/2016 09:51

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-02 Thread Hans Verkuil
On 01/02/17 14:51, Jean-Michel Hautbois wrote: Hi, 2016-12-30 21:26 GMT+01:00 Steve Longerbeam : On 12/30/2016 11:06 AM, Marek Vasut wrote: On 12/29/2016 09:51 PM, Robert Schwebel wrote: Hi Jean-Michel, Hi, On Thu, Dec 29, 2016 at 04:08:33PM +0100,

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2017-01-02 Thread Jean-Michel Hautbois
Hi, 2016-12-30 21:26 GMT+01:00 Steve Longerbeam : > > > On 12/30/2016 11:06 AM, Marek Vasut wrote: >> >> On 12/29/2016 09:51 PM, Robert Schwebel wrote: >>> >>> Hi Jean-Michel, >> >> Hi, >> >>> On Thu, Dec 29, 2016 at 04:08:33PM +0100, Jean-Michel Hautbois wrote:

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2016-12-30 Thread Steve Longerbeam
On 12/30/2016 11:06 AM, Marek Vasut wrote: On 12/29/2016 09:51 PM, Robert Schwebel wrote: Hi Jean-Michel, Hi, On Thu, Dec 29, 2016 at 04:08:33PM +0100, Jean-Michel Hautbois wrote: What is the status of this work? Philipp's patches have been reworked with the review feedback from the last

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2016-12-30 Thread Marek Vasut
On 12/29/2016 09:51 PM, Robert Schwebel wrote: > Hi Jean-Michel, Hi, > On Thu, Dec 29, 2016 at 04:08:33PM +0100, Jean-Michel Hautbois wrote: >> What is the status of this work? > > Philipp's patches have been reworked with the review feedback from the > last round and a new version will be

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2016-12-29 Thread Robert Schwebel
Hi Jean-Michel, On Thu, Dec 29, 2016 at 04:08:33PM +0100, Jean-Michel Hautbois wrote: > What is the status of this work? Philipp's patches have been reworked with the review feedback from the last round and a new version will be posted when he is back from holidays. rsc -- Pengutronix e.K.

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2016-12-29 Thread Jean-Michel Hautbois
I am so sorry... This mail wasn't send to the mailing list as this ** gmail switched back to HTML... 2016-12-29 16:08 GMT+01:00 Jean-Michel Hautbois : > Hi Philipp and al., > > > 2016-10-19 23:30 GMT+02:00 Sakari Ailus : >> >> On Fri,

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2016-10-19 Thread Sakari Ailus
On Fri, Oct 14, 2016 at 07:34:20PM +0200, Philipp Zabel wrote: > Hi, > > the second round removes the prepare_stream callback and instead lets the > intermediate subdevices propagate s_stream calls to their sources rather > than individually calling s_stream on each subdevice from the bridge

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2016-10-17 Thread Philipp Zabel
Hi Gary, Am Montag, den 17.10.2016, 12:18 +0200 schrieb Gary Bisson: [...] > For the whole series: > Tested-by: Gary Bisson > > Tested on Nitrogen6x + BD_HDMI_MIPI daughter board on linux-next > 20161016. > > This required using your v4l2-ctl patch to set the

Re: [PATCH v2 00/21] Basic i.MX IPUv3 capture support

2016-10-17 Thread Gary Bisson
Hi Philipp, On Fri, Oct 14, 2016 at 07:34:20PM +0200, Philipp Zabel wrote: > Hi, > > the second round removes the prepare_stream callback and instead lets the > intermediate subdevices propagate s_stream calls to their sources rather > than individually calling s_stream on each subdevice from