Hi Sebastian,
Sebastian Reichel wrote:
Hi Sakari,
On Tue, May 02, 2017 at 01:25:47PM +0300, Sakari Ailus wrote:
Camera flash drivers (and LEDs) are separate from the sensor devices in
DT. In order to make an association between the two, provide the
association information to the software.
Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>
---
Documentation/devicetree/bindings/media/video-interfaces.txt | 11 +++++++++++
1 file changed, 11 insertions(+)
diff --git a/Documentation/devicetree/bindings/media/video-interfaces.txt b/Documentation/devicetree/bindings/media/video-interfaces.txt
index 9cd2a36..d6c62bc 100644
--- a/Documentation/devicetree/bindings/media/video-interfaces.txt
+++ b/Documentation/devicetree/bindings/media/video-interfaces.txt
@@ -67,6 +67,17 @@ are required in a relevant parent node:
identifier, should be 1.
- #size-cells : should be zero.
+
+Optional properties
+-------------------
+
+- flash: An array of phandles that refer to the flash light sources
+ related to an image sensor. These could be e.g. LEDs. In case the LED
+ driver drives more than a single LED, then the phandles here refer to
+ the child nodes of the LED driver describing individual LEDs. Only
+ valid for device nodes that are related to an image sensor.
s/driver/controller/g - DT describes HW. Otherwise
Driver is hardware in this case. :-) The chip that acts as a current
sink or source for the LED is the driver. E.g. the adp1653 documentation
describes the chip as "Compact, High Efficiency, High Power, Flash/Torch
LED Driver with Dual Interface".
It might be still possible to improve the wording. Software oriented
folks are more likely to misunderstand the meaning of driver here, but
controller might seem ambiguous for hardware oriented people.
How about:
- flash: An array of phandles that refer to the flash light sources
related to an image sensor. These could be e.g. LEDs. In case the LED
driver (current sink or source chip for the LED(s)) drives more than a
single LED, then the phandles here refer to the child nodes of the LED
driver describing individual LEDs. Only valid for device nodes that are
related to an image sensor.
--
Regards,
Sakari Ailus
sakari.ailus@xxxxxxxxxxxxxxx
