Re: [PATCH 1/4] media: mx2_camera: Remove i.mx25 support.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Guennadi, Fabio,

On 30 October 2012 13:29, Guennadi Liakhovetski <g.liakhovetski@xxxxxx> wrote:
> On Tue, 30 Oct 2012, Fabio Estevam wrote:
>
>> Javier,
>>
>> On Tue, Oct 30, 2012 at 10:16 AM, Javier Martin
>> <javier.martin@xxxxxxxxxxxxxxxxx> wrote:
>> > i.MX25 support has been broken for several releases
>> > now and nobody seems to care about it.
>>
>> I will work on fixing camera support for mx25. Please do not remove its support.
>
> This is good to hear, thanks for doing this! But we also don't want to
> slow down Javier's work, if he works on features, only available on i.MX27
> or that he can only test there. How about separating parts of code,
> specific to each platform more cleanly? Maybe add an mx27_camera.c file to
> build the final driver from both files and mx27 and only from one on mx25?
> Or something similar? Would this be difficult or make sense at all?
>

It's pretty good that you want to provide proper support for video
capture on mx25 but I am still in favour of applying this patch for
several reasons:

1. i.mx25 "support" is so broken now that it would be better to start
from scratch IMHO.
2. AFAIK mx25 is not provided with an eMMa-PrP module. The current
mx2_camera driver relies on this module to perform DMA transfers. If
we added support for i.MX25 in this file, we'd have to use generic
DMAs again, which is something we already removed in the past.
3. CSI provided in i.mx25 has more features than the one in the
i.MX27, so the code they possibly share is even more reduced.

By the way, removal of all i.mx25 traces in this file was  announced
several times in the past:

9 Jul 2012
[PATCH] [v3] i.MX27: Fix emma-prp clocks in mx2_camera.c
26 Jul 2012
[PATCH 2/4] media: mx2_camera: Mark i.MX25 support as BROKEN.
[PATCH 3/4] Schedule removal of i.MX25 support in mx2_camera.c

In my opinion. i.mx25 video capture support should be added in a
separate file later. Though some CSI features are common, the lack of
eMMa-PrP in i.mx25 will make the driver be very different.

Please, expect a v2 of this patch soon. I've just remembered that I
missed removing i.MX25 traces from the Kconfig file too.

Regards.
-- 
Javier Martin
Vista Silicon S.L.
CDTUC - FASE C - Oficina S-345
Avda de los Castros s/n
39005- Santander. Cantabria. Spain
+34 942 25 32 60
www.vista-silicon.com
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux