From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Sebastian Reichel <sre@debian.org>
Cc: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>,
Sakari Ailus <sakari.ailus@iki.fi>,
linux-media@vger.kernel.org, devicetree@vger.kernel.org,
Rob Herring <rob.herring@calxeda.com>,
Pawel Moll <pawel.moll@arm.com>,
Mark Rutland <mark.rutland@arm.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>
Subject: Re: [RFCv2] Device Tree bindings for OMAP3 Camera System
Date: Thu, 23 Jan 2014 12:44:46 +0100 [thread overview]
Message-ID: <1550284.gaVFBYq9I9@avalon> (raw)
In-Reply-To: <20140123001128.GA12425@earth.universe>
[-- Attachment #1: Type: text/plain, Size: 1310 bytes --]
Hi Sebastian,
On Thursday 23 January 2014 01:11:29 Sebastian Reichel wrote:
> On Wed, Jan 22, 2014 at 11:57:45PM +0100, Laurent Pinchart wrote:
> > [...]
> >
> >> camera-switch {
> >>
> >> /*
> >>
> >> * TODO:
> >> * - check if the switching code is generic enough to use a
> >> * more generic name like "gpio-camera-switch".
> >
> > I think you can use a more generic name. You could probably get some
> > inspiration from the i2c-mux-gpio DT bindings.
>
> My main concern is, that the gpio used for switching is also connected to
> the reset pin of one of the cameras. Maybe that fact can just be neglected,
> though?
I'm not the only one to wish we could change that, but alas! we'll have to
live with that stupid hardware design decision :-)
What we want to ensure here is that the two sensors won't be accessed at the
same time, as that would lead to errors. This was previously handled by
callback function to board code, but board code is now going away. The
challenge is to find a way to express the constraints in DT. I'm not sure
whether that's doable in a generic way, and this might be one of the rare
cases where board code is still needed.
Sakari, have you given this a thought ?
--
Regards,
Laurent Pinchart
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 490 bytes --]
next prev parent reply other threads:[~2014-01-23 11:44 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-03 22:03 [early RFC] Device Tree bindings for OMAP3 Camera Subsystem Sebastian Reichel
2013-11-04 19:49 ` jean-philippe francois
2013-11-06 0:48 ` Laurent Pinchart
2013-11-15 17:18 ` Mark Rutland
2014-01-15 19:41 ` [RFCv2] Device Tree bindings for OMAP3 Camera System Sebastian Reichel
2014-01-20 4:19 ` Sakari Ailus
2014-01-20 22:16 ` Sylwester Nawrocki
2014-01-20 23:27 ` Sebastian Reichel
2014-01-22 22:27 ` Sylwester Nawrocki
2014-01-22 22:57 ` Laurent Pinchart
2014-01-23 0:11 ` Sebastian Reichel
2014-01-23 11:44 ` Laurent Pinchart [this message]
2014-02-01 9:39 ` Sakari Ailus
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=1550284.gaVFBYq9I9@avalon \
--to=laurent.pinchart@ideasonboard.com \
--cc=devicetree@vger.kernel.org \
--cc=ijc+devicetree@hellion.org.uk \
--cc=linux-media@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=pawel.moll@arm.com \
--cc=rob.herring@calxeda.com \
--cc=sakari.ailus@iki.fi \
--cc=sre@debian.org \
--cc=sylvester.nawrocki@gmail.com \
/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