Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1424734

Re: [RFC] [PATCH 0/3] media: an attempt to refresh omap1_camera driver

Show all headers | View raw

Hi Janusz,

On 06/16/2016 07:21 PM, Janusz Krzysztofik wrote:
> As requested by media subsystem maintainers, here is an attempt to 
> convert the omap1_camera driver to the vb2 framework. Also, conversion 
> to the dmaengine framework, long awaited by ARM/OMAP maintainers, is 
> done.
> 
> Next, I'm going to approach removal of soc-camera dependency. Please 
> let me know how much time I have for that, i.e., when the soc-camera
> framework is going to be depreciated.

Well, it is already deprecated (i.e. new drivers cannot use it), but it won't
be removed any time soon. There are still drivers depending on it, and some
aren't easy to rewrite.

I have to say that it is totally unexpected to see that this omap1 driver is still
used. In fact, we've already merged a patch that removed it for the upcoming
4.8 kernel. Based on this new development I'll revert that for the omap1
driver.

Out of curiosity: is supporting the Amstrad Delta something you do as a hobby
or are there other reasons?

A final note: once you've managed to drop the soc-camera dependency you should
run the v4l2-compliance test over the video node (https://git.linuxtv.org/v4l-utils.git/).

If that passes without failures, then this driver is in good shape and can be
moved out of staging again.

Regards,

	Hans

> 
> Thanks,
> Janusz
> 
> 
> Janusz Krzysztofik (3):
>   staging: media: omap1: drop videobuf-dma-sg mode
>   staging: media: omap1: convert to videobuf2
>   staging: media: omap1: use dmaengine
> 
>  drivers/staging/media/omap1/Kconfig              |   5 +-
>  drivers/staging/media/omap1/omap1_camera.c       | 948 +++++------------------
>  include/linux/platform_data/media/omap1_camera.h |   9 -
>  3 files changed, 217 insertions(+), 745 deletions(-)
> 

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[RFC] [PATCH 0/3] media: an attempt to refresh omap1_camera driver Janusz Krzysztofik <jmkrzyszt@gmail.com> - 2016-06-16 19:30 +0200
  [RFC] [PATCH 3/3] staging: media: omap1: use dmaengine Janusz Krzysztofik <jmkrzyszt@gmail.com> - 2016-06-16 19:30 +0200
  [RFC] [PATCH 1/3] staging: media: omap1: drop videobuf-dma-sg mode Janusz Krzysztofik <jmkrzyszt@gmail.com> - 2016-06-16 19:30 +0200
  [RFC] [PATCH 2/3] staging: media: omap1: convert to videobuf2 Janusz Krzysztofik <jmkrzyszt@gmail.com> - 2016-06-16 19:30 +0200
  Re: [RFC] [PATCH 0/3] media: an attempt to refresh omap1_camera  driver Hans Verkuil <hverkuil@xs4all.nl> - 2016-06-17 09:10 +0200
    Re: [RFC] [PATCH 0/3] media: an attempt to refresh omap1_camera  driver Tony Lindgren <tony@atomide.com> - 2016-06-17 12:10 +0200

csiph-web