Re: [PATCH RFC v2 0/2] pxa_camera transition to v4l2 standalone device
From: Hans Verkuil
Date: Mon Jul 04 2016 - 05:16:49 EST
Hi Robert,
On 04/02/2016 04:26 PM, Robert Jarzmik wrote:
> Hi Hans and Guennadi,
>
> This is the second opus of this RFC. The goal is still to see how close our
> ports are to see if there are things we could either reuse of change.
>
> From RFCv1, the main change is cleaning up in function names and functions
> grouping, and fixes to make v4l2-compliance happy while live tests still show no
> regression.
>
> For the next steps, I'll have to :
> - split the second patch, which will be a headache task, into :
> - first functions grouping and renaming
> => this to ensure the "internal functions" are almost untouched
> - the the port itself
>
> I'm leaving soc_mediabus for now, that's another task.
>
> I'm not seeing a big review traction, especially on the vb2 conversion, so I'll
> leave this patchset in RFC form until vb2 patch is reviewed and merged, and then
> will come back to this work.
I'm going to review this today.
I have been trying on-and-off to convert the sh_mobile_ceu_camera to a regular
driver with basically no success. One major problem is that the sh driver doesn't
use the device tree, so I can't copy code from the new rcar-vin driver. The scaling
and cropping code is also tightly coupled to soc-camera.
It is of course possible to do given enough time, but I don't think it is worth it.
So instead I am going for plan B: convert all other soc-camera drivers to 'regular'
drivers so in the end soc-camera is only used by the sh driver. Then I can turn
soc-camera into an sh driver, making it impossible for other drivers to use the
framework.
In other words, it would be great if you can continue this work, because after
this driver is converted only the atmel-isi driver remains (besides the sh driver,
of course).
Regards,
Hans