- Add skeleton for RV-8263-C8 RTC

- Rename display driver to match it with the vendor_type scheme of the other drivers - Remove Funkytown song from Buzzer driver - Disable selecting of the step counter and battery arc in digital watchface - Reorganize Kconfig and move the application configuration into the "Applications" menu
jakkra · May 27, 2024 · b469e66 · b469e66
1 parent 8152f4b
commit b469e66
Show file tree

Hide file tree

Showing 27 changed files with 839 additions and 83 deletions.
diff --git a/app/Kconfig b/app/Kconfig
@@ -70,6 +70,31 @@ menu "ZSWatch"
             bool
         prompt "Activate the application 'PPT remote'"
         default n
+
+        menu "Configuration"
+            menu "Compass"
+                depends on APPLICATIONS_USE_COMPASS
+
+                config APPLICATIONS_CONFIGURATION_COMPASS_REFRESH_INTERVAL_MS
+                    int
+                prompt "Refresh interval in milliseconds"
+                default 50
+
+                config APPLICATIONS_CONFIGURATION_COMPASS_CALIBRATION_TIME_S
+                    int
+                prompt "Calibration time in seconds"
+                default 30
+            endmenu
+
+            menu "Sensors Summary"
+                depends on APPLICATIONS_USE_SENSORS_SUMMARY
+
+                config APPLICATIONS_CONFIGURATION_SENSORS_SUMMARY_REFRESH_INTERVAL_MS
+                    int
+                prompt "Refresh interval in milliseconds"
+                default 100
+            endmenu
+        endmenu
     endmenu
 
     menu "Watchface"
@@ -111,29 +136,6 @@ menu "ZSWatch"
                 default 85
         endmenu
 
-        menu "Sensors Summary"
-            depends on APPLICATIONS_USE_SENSORS_SUMMARY
-
-            config DEFAULT_CONFIGURATION_SENSORS_SUMMARY_REFRESH_INTERVAL_MS
-                int
-            prompt "Refresh interval in milliseconds"
-            default 100
-        endmenu
-
-        menu "Compass"
-            depends on APPLICATIONS_USE_COMPASS
-
-            config DEFAULT_CONFIGURATION_COMPASS_REFRESH_INTERVAL_MS
-                int
-            prompt "Refresh interval in milliseconds"
-            default 50
-
-            config DEFAULT_CONFIGURATION_COMPASS_CALIBRATION_TIME_S
-                int
-            prompt "Calibration time in seconds"
-            default 30
-        endmenu
-
         menu "Power Management"
             config POWER_MANAGEMENT_IDLE_TIMEOUT_SECONDS
                 int

diff --git a/app/boards/arm/zswatch_nrf5340/zswatch_nrf5340_cpuapp_5.conf b/app/boards/arm/zswatch_nrf5340/zswatch_nrf5340_cpuapp_5.conf
@@ -1,9 +1,7 @@
 CONFIG_ZSWATCH_PCB_REV=5
 CONFIG_BT_DIS_HW_REV_STR="5"
 
-#---------------
 # FLASH
-#-----------------
 CONFIG_FLASH=y
 CONFIG_FLASH_PAGE_LAYOUT=y
 CONFIG_STREAM_FLASH=y
@@ -49,3 +47,5 @@ CONFIG_NVS=y
 
 CONFIG_DEBUG_COREDUMP_BACKEND_OTHER=y
 
+CONFIG_RTC=y
+CONFIG_RTC_ALARM=y
diff --git a/app/boards/arm/zswatch_nrf5340/zswatch_nrf5340_cpuapp_5.overlay b/app/boards/arm/zswatch_nrf5340/zswatch_nrf5340_cpuapp_5.overlay
@@ -77,6 +77,7 @@
 				<NRF_PSEL(QSPI_IO3, 0, 16)>;
 			low-power-enable;
 		};
+
 		group2 {
 			psels = <NRF_PSEL(QSPI_CSN, 0, 18)>;
 			low-power-enable;
@@ -125,21 +126,25 @@
 
 	buttons: buttons {
         compatible = "gpio-keys";
+
         button1: button_1 {
             label = "top-right";
             gpios = <&gpio0 27 (GPIO_PULL_UP | GPIO_ACTIVE_LOW)>;
             zephyr,code = <INPUT_KEY_1>;
         };
+
         button2: button_2 {
             label = "bottom-left";
             gpios = <&gpio1 10 (GPIO_PULL_UP | GPIO_ACTIVE_LOW)>;
             zephyr,code = <INPUT_KEY_2>;
         };
+
         button3: button_3 {
             label = "bottom-right";
             gpios = <&gpio0 31 (GPIO_PULL_UP | GPIO_ACTIVE_LOW)>;
             zephyr,code = <INPUT_KEY_KP0>;
         };
+
         button4: button_4 {
             label = "top-left";
             gpios = <&gpio0 26 (GPIO_PULL_UP | GPIO_ACTIVE_LOW)>;
@@ -167,12 +172,15 @@
 
 	pwmleds {
         compatible = "pwm-leds";
+
         display_blk: pwm_led_0 {
             pwms = <&pwm0 0 PWM_USEC(15) PWM_POLARITY_INVERTED>;
         };
+
         vibrator_pwm: pwm_led_1 {
             pwms = <&pwm1 0 PWM_MSEC(20) 0>;
         };
+
 		buzzer_pwm: pwm_led_2 {
             pwms = <&pwm2 0 PWM_HZ(880) PWM_POLARITY_NORMAL>;
         };
@@ -181,6 +189,13 @@
 };
 
 &i2c1 {
+	rv_8263_c8: rv-8263-c8@A2 {
+		compatible = "microcrystal,rv-8263-c8";
+		reg = <0xA2>;
+		status = "okay";
+		clkout = <7>;
+	};
+
     bmi270: bmi270@68 {
         compatible = "bosch,bmi270-plus";
         reg = <0x68>;
@@ -200,6 +215,7 @@
         resolution = <0>;
         frequency = <0>;
     };
+
 	npm1300_ek_pmic: pmic@6b {
 		compatible = "nordic,npm1300";
 		reg = <0x6b>;
@@ -253,6 +269,7 @@
 			thermistor-beta = <3380>;
 			charging-enable;
 		};
+
 		npm1300_ek_buttons: buttons {
 			compatible = "gpio-keys";
 			status = "disabled";
@@ -286,6 +303,7 @@
 	pinctrl-0 = <&qspi_default>;
 	pinctrl-1 = <&qspi_sleep>;
 	pinctrl-names = "default", "sleep";
+
 	gd25lq128d: gd25lq128d@0 {
 		compatible = "nordic,qspi-nor";
 		reg = <0>;
@@ -309,6 +327,7 @@
 / {
 	fstab {
 		compatible = "zephyr,fstab";
+
 		lvgl_lfs: lvgl_lfs {
 			compatible = "zephyr,fstab,littlefs";
 			mount-point = "/lvgl_lfs";
@@ -336,13 +355,15 @@
 			label = "littlefs_storage";
 			reg = <0x00000000 0x00200000>;
 		};
+
 		lvgl_raw_partition: partition@200000 {
 			label = "lvgl_raw_partition";
 			reg = <0x00200000 0x00200000>;
 		};
+
 		storage_partition: partition@400000 {
 			label = "storage_partition";
 			reg = <0x400000 0x100000 >;
 		};
 	};
-};
+};
diff --git a/app/drivers/CMakeLists.txt b/app/drivers/CMakeLists.txt
@@ -1,4 +1,5 @@
 add_subdirectory(display)
 add_subdirectory(input)
+add_subdirectory(rtc)
 
 add_subdirectory_ifdef(CONFIG_SENSOR sensor)
diff --git a/app/drivers/Kconfig b/app/drivers/Kconfig
@@ -1,6 +1,4 @@
 rsource "display/Kconfig"
 rsource "input/Kconfig"
-
-menu "Drivers"
-    rsource "sensor/Kconfig"
-endmenu
+rsource "rtc/Kconfig"
+rsource "sensor/Kconfig"
diff --git a/app/drivers/display/gc9a01/CMakeLists.txt b/app/drivers/display/gc9a01/CMakeLists.txt
@@ -1 +1 @@
-zephyr_sources(gc9a01.c)
+zephyr_sources(buydisplay_gc9a01.c)
diff --git a/app/drivers/display/gc9a01/Kconfig b/app/drivers/display/gc9a01/Kconfig
@@ -1,5 +1,5 @@
 menuconfig GC9A01
-    bool "GC9A01 compatible display controller driver"
+    bool "GC9A01 Display"
     default n
     select SPI
     help

diff --git a/app/drivers/display/gc9a01/gc9a01.c → ...rivers/display/gc9a01/buydisplay_gc9a01.c b/app/drivers/display/gc9a01/gc9a01.c → ...rivers/display/gc9a01/buydisplay_gc9a01.c
diff --git a/app/drivers/input/cst816s/Kconfig b/app/drivers/input/cst816s/Kconfig
@@ -3,7 +3,7 @@
 # SPDX-License-Identifier: Apache-2.0
 
 menuconfig INPUT_MODIFIED_CST816S
-	bool "Use modified out of tree CST816S capacitive touch panel driver"
+	bool "CST816S Capacitive touch panel driver"
 	default y
 	depends on DT_HAS_HYNITRON_CST816S_ENABLED
 	select I2C
@@ -26,4 +26,4 @@ config INPUT_CST816S_INTERRUPT
 	help
 	  Enable interrupt support (requires GPIO).
 
-endif # INPUT_CST816S
+endif
diff --git a/app/drivers/rtc/CMakeLists.txt b/app/drivers/rtc/CMakeLists.txt
@@ -0,0 +1,6 @@
+# Copyright (c) 2024, Daniel Kampert
+#
+# SPDX-License-Identifier: Apache-2.0
+#
+
+add_subdirectory_ifdef(CONFIG_RV_8263_C8 rv8263c8)
diff --git a/app/drivers/rtc/Kconfig b/app/drivers/rtc/Kconfig
@@ -0,0 +1,3 @@
+if RTC
+    rsource "rv8263c8/Kconfig"
+endif
diff --git a/app/drivers/rtc/rv8263c8/CMakeLists.txt b/app/drivers/rtc/rv8263c8/CMakeLists.txt
@@ -0,0 +1,7 @@
+# Copyright (c) 2024, Daniel Kampert
+#
+# SPDX-License-Identifier: Apache-2.0
+#
+
+zephyr_sources(microcrystal_rv8263c8.c)
+zephyr_sources(interrupt/microcrystal_rv8263c8_interrupt.c)
diff --git a/app/drivers/rtc/rv8263c8/Kconfig b/app/drivers/rtc/rv8263c8/Kconfig
@@ -0,0 +1,58 @@
+# RV-8263-C8 RTC configuration options.
+
+# Copyright (c) 240, Daniel Kampert
+#
+# SPDX-License-Identifier: Apache-2.0
+
+menuconfig RV_8263_C8
+    bool "RV-8263-C8 RTC"
+    depends on DT_HAS_MICROCRYSTAL_RV_8263_C8_ENABLED
+    default y
+    select I2C
+    help
+        Enable the driver for the RV-8263-C8 RTC.
+
+if RV_8263_C8
+
+module = MICROCRYSTAL_RV_8263_C8
+module-str = MICROCRYSTAL_RV_8263_C8
+source "subsys/logging/Kconfig.template.log_config"
+
+    config RV_8263_C8_THREAD_STACK_SIZE
+        int "RTC delayed work thread stack size"
+        depends on RV_8263_C8_INTERRUPT_OWN_THREAD
+        default 1024
+
+    config RV_8263_C8_THREAD_PRIORITY
+        int "RV-8263-C8 thread priority"
+        depends on RV_8263_C8_INTERRUPT_OWN_THREAD
+        default 10
+
+    choice
+        prompt "RV-8263-C8 interrupt mode"
+        default RV_8263_C8_INTERRUPT_NONE
+
+        config RV_8263_C8_INTERRUPT_NONE
+            bool "No interrupt"
+            help
+                Disable the INTERRUPT interface.
+
+        config RV_8263_C8_INTERRUPT_GLOBAL_THREAD
+            depends on GPIO
+            select RV_8263_C8_INTERRUPT
+            bool "Use workqueue"
+            help
+                Use the global workqueue to process the interrupts from the RV-8263-C8.
+
+        config RV_8263_C8_INTERRUPT_OWN_THREAD
+            depends on GPIO
+            select RV_8263_C8_INTERRUPT
+            bool "Use driver thread"
+            help
+                Use a separate thread to process the interrupts from the RV-8263-C8.
+    endchoice
+
+    config RV_8263_C8_INTERRUPT
+        bool
+
+endif
diff --git a/app/drivers/rtc/rv8263c8/interrupt/microcrystal_rv8263c8_interrupt.c b/app/drivers/rtc/rv8263c8/interrupt/microcrystal_rv8263c8_interrupt.c
@@ -0,0 +1,70 @@
+/* microcrystal_rv8263c8_interrupt.c - Driver for Micro Crystal RV-8263-C8 RTC. */
+
+/*
+ * Copyright (c) 2024, Daniel Kampert
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+#include <zephyr/logging/log.h>
+#include <zephyr/drivers/sensor.h>
+
+#include "../microcrystal_rv8263c8.h"
+#include "../private/microcrystal_rv8263c8_types.h"
+
+LOG_MODULE_REGISTER(microcrystal_rv8263c8_interrupt, CONFIG_MICROCRYSTAL_RV_8263_C8_LOG_LEVEL);
+
+#ifdef CONFIG_RV_8263_C8_TRIGGER_OWN_THREAD
+static K_KERNEL_STACK_DEFINE(rv8263c8_thread_stack, CONFIG_RV_8263_C8_THREAD_STACK_SIZE);
+static struct k_thread rv8263c8_thread;
+#endif
+
+/** @brief
+ *  @param p_dev
+ *  @param p_cb
+ *  @param pins
+*/
+static void rv8263c8_gpio_on_interrupt_callback(const struct device *p_dev, struct gpio_callback *p_cb, uint32_t pins)
+{
+    ARG_UNUSED(pins);
+    ARG_UNUSED(p_dev);
+}
+
+int rv8263c8_init_interrupt(const struct device *p_dev)
+{
+    struct rv8263c8_data *data = p_dev->data;
+    const struct rv8263c8_config *config = p_dev->config;
+
+    LOG_DBG("Initialize interrupts...");
+
+    if (!gpio_is_ready_dt(&config->int_gpio)) {
+        LOG_ERR("INT GPIO device not ready!");
+        return -ENODEV;
+    }
+
+    if (gpio_pin_configure_dt(&config->int_gpio, GPIO_INPUT)) {
+        return -EFAULT;
+    }
+    /*
+        data->dev = p_dev;
+
+    #ifdef CONFIG_RV_8263_C8_TRIGGER_OWN_THREAD
+        k_sem_init(&data->sem, 0, K_SEM_MAX_LIMIT);
+
+        k_thread_create(&bmi2_thread, bmi2_thread_stack, CONFIG_RV_8263_C8_THREAD_STACK_SIZE,
+                        bmi2_worker_thread, data, NULL, NULL, K_PRIO_COOP(CONFIG_RV_8263_C8_THREAD_PRIORITY),
+                        0, K_NO_WAIT);
+    #else
+        data->work.handler = bmi2_worker;
+    #endif
+
+        gpio_init_callback(&data->gpio_handler, rv8263c8_gpio_on_interrupt_callback, BIT(config->int_gpio.pin));
+
+        if (gpio_add_callback(config->int_gpio.port, &data->gpio_handler)) {
+            return -EFAULT;
+        }
+
+        LOG_DBG("Interrupts ready!");
+    */
+    return 0;
+}