Thread (27 messages) 27 messages, 7 authors, 2017-12-22

Re: [PATCH] devicetree: Add video bus switch

From: Rob Herring <robh@kernel.org>
Date: 2017-02-09 23:16:14
Also in: linux-media, lkml

On Thu, Feb 9, 2017 at 5:02 PM, Rob Herring [off-list ref] wrote:
On Wed, Feb 8, 2017 at 4:30 PM, Pavel Machek [off-list ref] wrote:
quoted
On Wed 2017-02-08 15:36:09, Rob Herring wrote:
quoted
On Fri, Feb 03, 2017 at 01:35:08PM +0100, Pavel Machek wrote:
quoted
N900 contains front and back camera, with a switch between the
two. This adds support for the switch component, and it is now
possible to select between front and back cameras during runtime.

This adds documentation for the devicetree binding.

Signed-off-by: Sebastian Reichel <sre@kernel.org>
Signed-off-by: Ivaylo Dimitrov <redacted>
Signed-off-by: Pavel Machek <redacted>

diff --git a/Documentation/devicetree/bindings/media/video-bus-switch.txt b/Documentation/devicetree/bindings/media/video-bus-switch.txt
new file mode 100644
index 0000000..1b9f8e0
--- /dev/null
+++ b/Documentation/devicetree/bindings/media/video-bus-switch.txt
@@ -0,0 +1,63 @@
+Video Bus Switch Binding
+========================
I'd call it a mux rather than switch.
It is a switch, not a multiplexor (
https://en.wikipedia.org/wiki/Multiplexing ). Only one camera can
operate at a time.
It's no different than an i2c mux. It's one at a time.
quoted
quoted
BTW, there's a new mux-controller binding under review you might look
at. It would only be needed here if the mux ctrl also controls other
things.
Do you have a pointer?
Let me Google that for you:
https://lwn.net/Articles/713971/
quoted
quoted
quoted
+Required Port nodes
+===================
+
+More documentation on these bindings is available in
+video-interfaces.txt in the same directory.
+
+reg                : The interface:
+             0 - port for image signal processor
+             1 - port for first camera sensor
+             2 - port for second camera sensor
This could be used for display side as well. So describe these just as
inputs and outputs.
I'd prefer not to confuse people. I guess that would be 0 -- output
port, 1, 2 -- input ports... But this is media data, are you sure it
is good idea to change this?
And I'd prefer something that can be reused by others.
quoted
                                                                        Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help