Re: [PATCH] pxa_camera: merge soc_mediabus.c into pxa_camera.c

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

 



Hans Verkuil <hverkuil@xxxxxxxxx> writes:

>> Hi Hans,
>> 
>> I wonder why you chose to copy-paste this code instead of adding in the Kconfig
>> a "depends on !SOC_CAMERA". Any specific reason ? As this will have to be dealt
>> with later anyway as you pointed out earlier, this format translation I mean, I
>> was wondering if this was the best approach.
>
> I thought about that, but that would make it impossible to COMPILE_TEST both the
> pxa and the soc_camera driver. In addition, the pxa and soc_camera are the only
> drivers that use this, and I prefer to just merge that code into pxa so that it can
> be modified independently from soc_camera.
>
> I really want to remove all dependencies to soc_camera. That will also make it easier
> to refactor soc_camera once I get the atmel-isi driver out of soc_camera.
Ok, fair enough.

I have tested that for at least YUV422, YUYV, YVYU and RGB565 formats :
Tested-by: Robert Jarzmik <robert.jarzmik@xxxxxxx>

Cheers.

-- 
Robert
--
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