- Driver is functional

- Has some open topics (see TODO)
jakkra · Oct 8, 2023 · 04dafe8 · 04dafe8
1 parent be7f385
commit 04dafe8
Show file tree

Hide file tree

Showing 4 changed files with 415 additions and 36 deletions.
diff --git a/app/drivers/sensor/CMakeLists.txt b/app/drivers/sensor/CMakeLists.txt
@@ -3,4 +3,12 @@
 # SPDX-License-Identifier: Apache-2.0
 #
 
+# TODO:
+# Weird compilation issue when not using
+# zephyr_library()
+# zephyr_library_sources(apds9306.c)
+#
+# -> ..add_library cannot create target "..__app__drivers__sensor" because another
+# -> target with the same name already exists.  The existing target is a static...
+#
 add_subdirectory_ifdef(CONFIG_APDS9306 apds9306)
diff --git a/app/drivers/sensor/apds9306/Kconfig b/app/drivers/sensor/apds9306/Kconfig
@@ -4,9 +4,18 @@
 #
 # SPDX-License-Identifier: Apache-2.0
 
-config APDS9306
-	bool "APDS9306 light sensor"
+menuconfig APDS9306
+	bool "APDS9306 Sensor"
 	default y
+	# TODO
+	#depends on DT_HAS_AVAGO_APDS9306_ENABLED
 	select I2C
 	help
-	  Enable the driver for the APDS9306 digital light sensor.
+	  	Enable the driver for the APDS9306 digital light sensor.
+
+if APDS9306
+	config APDS9306_IS_APDS9306_065
+		bool "Enable this option if you are using the APDS-9306-065"
+		default n
+
+endif