Re: [PATCH v2] Adding NV{12,21} and Y{U,V}12 pixel formats support.

2015-02-24 Thread Miguel Casas-Sanchez
On 24 February 2015 at 09:41, Hans Verkuil wrote: > On 02/24/2015 06:00 PM, Miguel Casas-Sanchez wrote: >> Hi Hans, >> >> go for it if you feel it's the best approach. Are you planning to add >> multiplanar formats? Particularly, I'm interested in YUV420M and its >> twin evil brother YVU420M. > >

Re: [PATCH v2] Adding NV{12,21} and Y{U,V}12 pixel formats support.

2015-02-24 Thread Hans Verkuil
On 02/24/2015 06:00 PM, Miguel Casas-Sanchez wrote: > Hi Hans, > > go for it if you feel it's the best approach. Are you planning to add > multiplanar formats? Particularly, I'm interested in YUV420M and its > twin evil brother YVU420M. Certainly. I'm planning to add all those YUV420 variants. On

Re: [PATCH v2] Adding NV{12,21} and Y{U,V}12 pixel formats support.

2015-02-24 Thread Miguel Casas-Sanchez
Hi Hans, go for it if you feel it's the best approach. Are you planning to add multiplanar formats? Particularly, I'm interested in YUV420M and its twin evil brother YVU420M. I would recommend adding support for a less-common format such as YUV410 (or variation thereof). Since this format is so d

Re: [PATCH v2] Adding NV{12,21} and Y{U,V}12 pixel formats support.

2015-02-24 Thread Hans Verkuil
Hi Miguel, Thanks for the patch. However, after reviewing it and testing it I decided to implement my own version. Partially because several features were still failing (crop/compose/scale), partially because I didn't like the way the tpg was changed: too much change basically. Yesterday I added

[PATCH v2] Adding NV{12,21} and Y{U,V}12 pixel formats support.

2015-02-18 Thread Miguel Casas-Sanchez
This is the second attempt at creating a patch doing that while respecting the pattern movements, crops, and other artifacts that can be added to the generated frames. Hope it addresses Hans' comments on the first patch. It should create properly moving patterns, border, square and noise. SAV/EA