Re: [Mesa-dev] [PATCH 4/4] st/omx/dec: make decoder video buffer progressive

2016-07-20 Thread Emil Velikov
On 7 July 2016 at 18:16, Leo Liu wrote: > Hi Emil, > > Have a look again, I think the logic is nothing wrong. > I did not mention that it's doing something "wrong" :-) > reason In lines > > > On 07/07/2016 11:39 AM, Emil Velikov wrote: >> >> On 6 July 2016 at 19:03, Leo Liu wrote: >>> >>> The id

Re: [Mesa-dev] [PATCH 4/4] st/omx/dec: make decoder video buffer progressive

2016-07-07 Thread Leo Liu
Hi Emil, Have a look again, I think the logic is nothing wrong. reason In lines On 07/07/2016 11:39 AM, Emil Velikov wrote: On 6 July 2016 at 19:03, Leo Liu wrote: The idea of encode tunneling is to use video buffer directly for encoder, but currently the encoder doesn’t support interlaced

Re: [Mesa-dev] [PATCH 4/4] st/omx/dec: make decoder video buffer progressive

2016-07-07 Thread Leo Liu
On 07/07/2016 11:39 AM, Emil Velikov wrote: On 6 July 2016 at 19:03, Leo Liu wrote: The idea of encode tunneling is to use video buffer directly for encoder, but currently the encoder doesn’t support interlaced surface, the OMX decoder set progressive surface before on that purpose. Since no

Re: [Mesa-dev] [PATCH 4/4] st/omx/dec: make decoder video buffer progressive

2016-07-07 Thread Leo Liu
On 07/07/2016 11:28 AM, Julien Isorce wrote: No encoder with nouveau driver so I cannot do a tunneling test but at least this does not break the decoder part so the 4 patches are: Tested-by: Julien Isorce > Thx! Thanks for that. Leo On 6 July 2016 at 19:03, L

Re: [Mesa-dev] [PATCH 4/4] st/omx/dec: make decoder video buffer progressive

2016-07-07 Thread Emil Velikov
On 6 July 2016 at 19:03, Leo Liu wrote: > The idea of encode tunneling is to use video buffer directly for encoder, > but currently the encoder doesn’t support interlaced surface, the OMX > decoder set progressive surface before on that purpose. > > Since now we are polling the driver for interlac

Re: [Mesa-dev] [PATCH 4/4] st/omx/dec: make decoder video buffer progressive

2016-07-07 Thread Julien Isorce
No encoder with nouveau driver so I cannot do a tunneling test but at least this does not break the decoder part so the 4 patches are: Tested-by: Julien Isorce Thx! On 6 July 2016 at 19:03, Leo Liu wrote: > The idea of encode tunneling is to use video buffer directly for encoder, > but current

[Mesa-dev] [PATCH 4/4] st/omx/dec: make decoder video buffer progressive

2016-07-06 Thread Leo Liu
The idea of encode tunneling is to use video buffer directly for encoder, but currently the encoder doesn’t support interlaced surface, the OMX decoder set progressive surface before on that purpose. Since now we are polling the driver for interlacing information for decoder, we got the interlaced