linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Cc: Hans Verkuil <hverkuil@xs4all.nl>,
	Linux Media Mailing List <linux-media@vger.kernel.org>,
	Josh Wu <rainyfeeling@gmail.com>,
	Robert Jarzmik <robert.jarzmik@free.fr>,
	Fabio Estevam <fabio.estevam@freescale.com>,
	Javier Martin <javier.martin@vista-silicon.com>,
	Ludovic Desroches <ludovic.desroches@atmel.com>
Subject: Re: [RFC] Move some soc-camera drivers to staging in preparation for removal
Date: Mon, 22 Feb 2016 16:23:54 +0200	[thread overview]
Message-ID: <1864387.TRmC7Phqsl@avalon> (raw)
In-Reply-To: <Pine.LNX.4.64.1602221427510.10936@axis700.grange>

Hi Guennadi,

(CC'ing Ludovic Desroches)

On Monday 22 February 2016 14:39:08 Guennadi Liakhovetski wrote:
> Hi Laurent,
> 
> On Mon, 22 Feb 2016, Laurent Pinchart wrote:
> 
> [snip]
> 
> > As far as I know Renesas (or at least the kernel upstream team) doesn't
> > care. The driver is only used on five SH boards, I'd also say it can be
> > removed.
> [snip]
> 
> >>>> - atmel-isi: ATMEL Image Sensor Interface (ISI)
> >>>> 
> >>>>   I believe this is still actively maintained. Would someone be
> >>>>   willing to convert this? It doesn't look like a complex driver.
> > 
> > That would be nice, I would like to avoid dropping this one.
> 
> Thanks for clarifying the state of the CEU driver. I did say, that I am
> fine with dropping soc-camera gradually, and I stay with that. But I see
> now, that at least two drivers want to stay active: Atmel ISI and PXA270.
> One possibility is of course to make them independent drivers. If people
> are prepared to invest work into that - sure, would be great! If we
> however decide to keep soc-camera, I could propose the following: IIUC,
> the largest problem is sensor drivers, that cannot be reused for other
> non-soc-camera bridge drivers. The thing is, out of all the sensor drivers
> currently under drivers/media/i2c/soc_camera only a couple are in use on
> those active PXA270 and Atmel boards. I could propose the following:
> 
> 1. Remove all bridge drivers, that noone cares about.
> 2. If anyone ever needs to use any of soc-camera-associated sensor
>    drivers, take them out of soc-camera and _remove_ any soc-camera
>    dependencies
> 3. If any soc-camera boards will need that specific driver, which in
>    itself is already unlikely, we'll have to fix that by teaching
>    soc-camera to work with generic sensor drivers!

That sounds like a good plan.

Ludovic, any chance someone at Atmel could convert the ISI driver ?

-- 
Regards,

Laurent Pinchart


  reply	other threads:[~2016-02-22 14:23 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-19 13:24 [RFC] Move some soc-camera drivers to staging in preparation for removal Hans Verkuil
2016-02-19 16:24 ` Mauro Carvalho Chehab
2016-02-19 16:30   ` Hans Verkuil
2016-02-19 18:01     ` Robert Jarzmik
2016-02-19 18:12       ` Hans Verkuil
2016-02-20 21:33         ` Robert Jarzmik
2016-02-21 15:06           ` Hans Verkuil
2016-02-22  7:01 ` Guennadi Liakhovetski
2016-02-22  7:11   ` Guennadi Liakhovetski
2016-02-22 13:21     ` Laurent Pinchart
2016-02-22 13:39       ` Guennadi Liakhovetski
2016-02-22 14:23         ` Laurent Pinchart [this message]
2016-02-22 16:08           ` Ludovic Desroches
2016-02-23  7:06             ` Wu, Songjun
2016-02-23  7:26               ` Hans Verkuil

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1864387.TRmC7Phqsl@avalon \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=fabio.estevam@freescale.com \
    --cc=g.liakhovetski@gmx.de \
    --cc=hverkuil@xs4all.nl \
    --cc=javier.martin@vista-silicon.com \
    --cc=linux-media@vger.kernel.org \
    --cc=ludovic.desroches@atmel.com \
    --cc=rainyfeeling@gmail.com \
    --cc=robert.jarzmik@free.fr \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).