Skip to contentSkip to conversation

dt-bindings: iio: light: bh1750: Add device tree binding documentation

PATCH Superseded ryang 3 participants 4 messages Sep 22, 2018 – Sep 29, 2018

Subsystems (3)
Device tree bindings IIO ROHM BH1750 ambient light sensor driver

v3 of 4 Superseded Latest is v4 →

lore
Share & tools
lore

ryang

author
#0 lore raw

PATCH v3 1/2 dt-bindings: iio: light: bh1750: Add device tree binding documentation

Document device tree bindings for ROHM BH1750 ambient light sensor driver.

Signed-off-by ryang <[email protected]>
---
 .../devicetree/bindings/iio/light/bh1750.txt   | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/iio/light/bh1750.txt
Documentation/devicetree/bindings/iio/light/bh1750.txt new file +18 -0
new file mode 100644
index 000000000000..468350cc7af6
@@ -0,0 +1,18 @@
+ROHM BH1750 - ALS, Ambient light sensor
+
+Required properties:
+
+- compatible: Must be one of:
+    "rohm,bh1710"
+    "rohm,bh1715"
+    "rohm,bh1721"
+    "rohm,bh1750"
+    "rohm,bh1751"
+- reg: the I2C address of the sensor
+
+Example:
+
+bh1750@23 {
+	compatible = "rohm,bh1750";
+	reg = <0x23>;
+};
Signature · 1 line
2.17.1
author
#1 lore raw

PATCH v3 2/2 iio: light: bh1750: Add device tree support

Add device tree support for ROHM BH1750 series ambient light sensors.

Signed-off-by ryang <[email protected]>

---
Changes in v2:
- Export module device table
Changes in v3:
- Remove of_match_ptr so that bh1750_of_match is available without CONFIG_OF.

 drivers/iio/light/bh1750.c | 11 +++++++++++
 1 file changed, 11 insertions(+)
drivers/iio/light/bh1750.c +11 -0
index c3a481452b67..28347df78cff 100644
@@ -307,9 +307,20 @@ static const struct i2c_device_id bh1750_id[] = {
 };
 MODULE_DEVICE_TABLE(i2c, bh1750_id);
 
+static const struct of_device_id bh1750_of_match[] = {
+	{ .compatible = "rohm,bh1710", },
+	{ .compatible = "rohm,bh1715", },
+	{ .compatible = "rohm,bh1721", },
+	{ .compatible = "rohm,bh1750", },
+	{ .compatible = "rohm,bh1751", },
+	{ }
+};
+MODULE_DEVICE_TABLE(of, bh1750_of_match);
+
 static struct i2c_driver bh1750_driver = {
 	.driver = {
 		.name = "bh1750",
+		.of_match_table = bh1750_of_match,
 		.pm = &bh1750_pm_ops,
 	},
 	.probe = bh1750_probe,
Signature · 1 line
2.17.1
maintainer Reviewed-by on v3 1/2
#2 lore raw
In reply to ryang
Quoted 19 lines · ryang

Document device tree bindings for ROHM BH1750 ambient light sensor driver.

Signed-off-by ryang <[email protected]>
---
 .../devicetree/bindings/iio/light/bh1750.txt   | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/iio/light/bh1750.txt
Documentation/devicetree/bindings/iio/light/bh1750.txt new file +5 -0
new file mode 100644
index 000000000000..468350cc7af6
@@ -0,0 +1,18 @@
+ROHM BH1750 - ALS, Ambient light sensor
+
+Required properties:
+
+- compatible: Must be one of:

Documentation/devicetree/bindings/iio/light/bh1750.txt line 15

Quoted 10 lines
Documentation/devicetree/bindings/iio/light/bh1750.txt new file +10 -0
+    "rohm,bh1710"
+    "rohm,bh1715"
+    "rohm,bh1721"
+    "rohm,bh1750"
+    "rohm,bh1751"
+- reg: the I2C address of the sensor
+
+Example:
+
+bh1750@23 {

light-sensor@23

Otherwise,

Reviewed-by Rob Herring <[email protected]>
Quoted 6 lines
Documentation/devicetree/bindings/iio/light/bh1750.txt new file +3 -0
+	compatible = "rohm,bh1750";
+	reg = <0x23>;
+};
Signature · 1 line
2.17.1
maintainer
#3 lore raw
Quoted 33 lines · On Thu, 27 Sep 2018 14:31:37 -0500 Rob Herring <[email protected]> wrote:
Quoted 19 lines · ryang

Document device tree bindings for ROHM BH1750 ambient light sensor driver.

Signed-off-by ryang <[email protected]>
---
 .../devicetree/bindings/iio/light/bh1750.txt   | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/iio/light/bh1750.txt
Documentation/devicetree/bindings/iio/light/bh1750.txt new file +5 -0
new file mode 100644
index 000000000000..468350cc7af6
@@ -0,0 +1,18 @@
+ROHM BH1750 - ALS, Ambient light sensor
+
+Required properties:
+
+- compatible: Must be one of:

Documentation/devicetree/bindings/iio/light/bh1750.txt line 15

Quoted 10 lines
Documentation/devicetree/bindings/iio/light/bh1750.txt new file +10 -0
+    "rohm,bh1710"
+    "rohm,bh1715"
+    "rohm,bh1721"
+    "rohm,bh1750"
+    "rohm,bh1751"
+- reg: the I2C address of the sensor
+
+Example:
+
+bh1750@23 {

light-sensor@23

Quoted 3 lines

Otherwise,

Reviewed-by Rob Herring <[email protected]>

Fixed up. Thanks Rob.

Jonathan

Quoted 7 lines
Quoted 7 lines
Documentation/devicetree/bindings/iio/light/bh1750.txt new file +3 -0
+	compatible = "rohm,bh1750";
+	reg = <0x23>;
+};
Signature · 1 line
2.17.1