Re: [PATCH] Add interlace support to sh_mobile_ceu_camera.c

[Guennadi Liakhovetski <g.liakhovetski@gmx.de>]

Hello Morimoto-san

I've got a question to you, regarding your interlaced support 
implementation for the CEU: do I understand it right, that the kind of 
support you actually have implemented is, that if an interlaced format is 
now requested from the CEU, it will interpret incoming data as interlaced 
and deinterlace it internally? If this is really the case, then, I think, 
it is a wrong way to implement this functionality. If a user requests 
interlaced data, it means, (s)he wants it interlaced in memory. Whereas 
deinterlacing should happen transparently - if the user requested 
progressive data and your source provides interlaced, you can decide to 
deinterlace it internally. Or am I misunderstanding your implementation?

Regardless of theoretical correctness - does your patch still work? Have 
you been able back then to get CEU to deinterlace data, and when have you 
last tested it?

Thanks
Guennadi
---
Guennadi Liakhovetski