On 7/2/24 22:14, Andrew Davis wrote:
Add DT overlay for the Grove Sunlight Sensor[0].
[0] https://wiki.seeedstudio.com/Grove-Sunlight_Sensor/
Signed-off-by: Andrew Davis <afd@xxxxxx>
---
arch/arm64/boot/dts/ti/Makefile | 3 ++
.../boot/dts/ti/grove-sunlight-sensor.dtso | 31 +++++++++++++++++++
2 files changed, 34 insertions(+)
create mode 100644 arch/arm64/boot/dts/ti/grove-sunlight-sensor.dtso
diff --git a/arch/arm64/boot/dts/ti/Makefile b/arch/arm64/boot/dts/ti/Makefile
index a859629a6072c..7d1ce7a5d97bc 100644
--- a/arch/arm64/boot/dts/ti/Makefile
+++ b/arch/arm64/boot/dts/ti/Makefile
@@ -8,6 +8,9 @@
# Entries are grouped as per SoC present on the board. Groups are sorted
# alphabetically.
+# This needs a better directory location
+dtb-$(CONFIG_OF_OVERLAY) += grove-sunlight-sensor.dtbo
+
# Boards with AM62x SoC
dtb-$(CONFIG_ARCH_K3) += k3-am625-beagleplay.dtb
dtb-$(CONFIG_ARCH_K3) += k3-am625-beagleplay-csi2-ov5640.dtbo
diff --git a/arch/arm64/boot/dts/ti/grove-sunlight-sensor.dtso b/arch/arm64/boot/dts/ti/grove-sunlight-sensor.dtso
new file mode 100644
index 0000000000000..ab2f102e1f8ab
--- /dev/null
+++ b/arch/arm64/boot/dts/ti/grove-sunlight-sensor.dtso
@@ -0,0 +1,31 @@
+// SPDX-License-Identifier: GPL-2.0-or-later OR MIT
+/**
+ * Grove - Sunlight Sensor v1.0
+ *
+ * https://wiki.seeedstudio.com/Grove-Sunlight_Sensor/
+ *
+ * Copyright (C) 2024 Texas Instruments Incorporated - http://www.ti.com/
+ */
+
+/dts-v1/;
+/plugin/;
+
+&GROVE_CONNECTOR {
+ status = "okay";
+ pinctrl-names = "default";
+ pinctrl-0 = <&GROVE_PIN1_MUX_I2C_SCL>,
+ <&GROVE_PIN2_MUX_I2C_SDA>;
+};
On setting pinctrl in the mikrobus connector, I seem to encounter
problem with the SPI driver trying to use the device before the pins are
ready. So I think, the pinctrl should probably be defined in the
respective i2c, spi, etc nodes instead of connector.
+
+&GROVE_PIN1_I2C {
+ status = "okay";
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ clock-frequency = <100000>;
+
+ si1145@60 {
+ compatible = "si,si1145";
+ reg = <0x60>;
+ };
+};
I also have question regarding how to define reg property in SPI
(chipselect). Ideally, we want to define it relative to the connector
pins, but since the SPI device(s) is a child of SPI controller, I am not
sure how I can do remapping.
Ayush Singh
