tiacsys · rexut · Nov 10, 2023 · Nov 5, 2023 · Nov 6, 2023 · Nov 6, 2023
@@ -235,29 +235,18 @@ jobs:
         run: |
           #
           # Disabled in the meantime because the execution time is still
-          # too high:
+          # too high (up to about 45 min):
           #
+          # --> Grove Button Shields, ~640% more time (~550 more tests)
           # --testsuite-root bridle/tests/shields/grove_btn/dts_bindings \
+          #
+          # --> Grove LED Shields, ~880% more time (~890 more tests)
           # --testsuite-root bridle/tests/shields/grove_led/dts_bindings \
           #
-          west twister --verbose \
+          west twister --verbose --integration --cmake-only \
             --outdir twister-out --no-clean --inline-logs \
             --enable-size-report --platform-reports \
-            --platform arduino_zero \
-            --platform mimxrt1010_evk \
-            --platform mimxrt1060_evk \
-            --platform mimxrt1060_evkb \
-            --platform nucleo_f303re \
-            --platform nucleo_f401re \
-            --platform nucleo_f413zh \
-            --platform nucleo_f767zi \
-            --platform rpi_pico \
-            --platform seeed_xiao_samd21 \
-            --platform seeeduino_xiao \
-            --platform seeeduino_cm0 \
-            --platform seeeduino_lotus \
-            --platform waveshare_rp2040_plus \
-            --platform waveshare_rp2040_plus@16mb \
+            --quarantine-list bridle/tests/quarantine.yaml \
             --testsuite-root bridle/tests/shields/grove/dts_bindings \
             --testsuite-root bridle/tests/shields/x_grove_testbed/dts_bindings
 

diff --git a/boards/arm/arduino_zero/arduino_zero.yaml b/boards/arm/arduino_zero/arduino_zero.yaml
@@ -1,3 +1,4 @@
+vendor: arduino
 identifier: arduino_zero
 name: Arduino Zero
 type: mcu

diff --git a/boards/arm/seeed_xiao_samd21/seeed_xiao_connector.dtsi b/boards/arm/seeed_xiao_samd21/seeed_xiao_connector.dtsi
@@ -29,3 +29,4 @@
 xiao_spi: &sercom0 {};
 xiao_i2c: &sercom2 {};
 xiao_serial: &sercom4 {};
+xiao_dac: &dac0 {};
diff --git a/boards/arm/seeed_xiao_samd21/seeed_xiao_samd21.yaml b/boards/arm/seeed_xiao_samd21/seeed_xiao_samd21.yaml
@@ -1,3 +1,4 @@
+vendor: seeed
 identifier: seeed_xiao_samd21
 name: Seeed Studio XIAO SAMD21 (Seeeduino XIAO)
 type: mcu
@@ -13,6 +14,7 @@ supported:
   - xiao_i2c
   - xiao_spi
   - xiao_serial
+  - xiao_dac
   - dma
   - gpio
   - hwinfo

@@ -201,9 +201,9 @@ Following mappings are well known:
       +--------------------+-----------------------------+-------------------------+-------------------------+
       | Grove PWM Label    | phandle index to shield --> | **Signal** : *Meaning*  | ``seeeduino_cm0``       |
       +====================+=============================+=========================+=========================+
-      |                    | ``<&grove_pwms 0 …>``       | **D0:** *UART-RX*       |                         |
+      |                    | ``<&grove_pwms 0 …>``       | **D0:** *UART-RX*       | **not possible**        |
       +--------------------+-----------------------------+-------------------------+-------------------------+
-      |                    | ``<&grove_pwms 1 …>``       | **D1:** *UART-TX*       |                         |
+      |                    | ``<&grove_pwms 1 …>``       | **D1:** *UART-TX*       | **not possible**        |
       +--------------------+-----------------------------+-------------------------+-------------------------+
       |                    | ``<&grove_pwms 2 …>``       | **D2**                  | **not wired**           |
       +--------------------+-----------------------------+-------------------------+-------------------------+
@@ -237,9 +237,9 @@ Following mappings are well known:
       +--------------------+-----------------------------+-------------------------+-------------------------+
       |                    | ``<&grove_pwms 17 …>``      | **D17:** *A3*           | **not wired**           |
       +--------------------+-----------------------------+-------------------------+-------------------------+
-      |                    | ``<&grove_pwms 18 …>``      | **D18:** *I2C-SDA*      |                         |
+      | ``&grove_pwm_d18`` | ``<&grove_pwms 18 …>``      | **D18:** *I2C-SDA*      | ``<&tcc0 0 …>`` (WO4)   |
       +--------------------+-----------------------------+-------------------------+-------------------------+
-      |                    | ``<&grove_pwms 19 …>``      | **D19:** *I2C-SCL*      |                         |
+      | ``&grove_pwm_d19`` | ``<&grove_pwms 19 …>``      | **D19:** *I2C-SCL*      | ``<&tcc0 1 …>`` (WO5)   |
       +--------------------+-----------------------------+-------------------------+-------------------------+
       |                    | ``<&grove_pwms 20 …>``      | **D20:** *ADC6*         | **not wired**           |
       +--------------------+-----------------------------+-------------------------+-------------------------+

diff --git a/boards/arm/seeeduino_cm0/grove_connectors.dtsi b/boards/arm/seeeduino_cm0/grove_connectors.dtsi
@@ -40,26 +40,32 @@
 		#pwm-cells = <2>;
 		pwm-map-mask = <0xffffffff 0>;
 		pwm-map-pass-thru = <0 0xffffffff>;
-		pwm-map;
-			/*  0 - D0: not possible */
-			/*  1 - D1: not possible */
+		pwm-map =
+			/*  0 - D0:  not possible */
+			/*  1 - D1:  not possible */
 			/*  2 - D2:  not wired */
 			/*  3 - D3:  not wired */
 			/*  4 - D4:  not wired */
 			/*  5 - D5:  not wired */
 			/*  6 - D6:  not wired */
 			/*  7 - D7:  not wired */
-			/*  8 - D8:  not possible */
-			/*  9 - D9:  not possible */
-			/* 10 - D10: not possible */
-			/* 11 - D11: not possible */
-			/* 12 - D12: not possible */
-			/* 13 - D13: not possible */
-			/* 14 - D14: not possible */
-			/* 15 - D15: not possible */
-			/* 16 - D16: not possible */
-			/* 17 - D17: not possible */
+			/*  8 - D8:  not wired */
+			/*  9 - D9:  not wired */
+			/* 10 - D10: not wired */
+			/* 11 - D11: not wired */
+			/* 12 - D12: not wired */
+			/* 13 - D13: not wired */
+			/* 14 - D14: not wired */
+			/* 15 - D15: not wired */
+			/* 16 - D16: not wired */
+			/* 17 - D17: not wired */
+			<18 0 &tcc0 0 0>,		/* D18: TCC0CH0 (WO4) */
+			<19 0 &tcc0 1 0>;		/* D19: TCC0CH1 (WO5) */
 	};
 };
 
 grove_gpios: &grove_laced_if_cm0 {};
+grove_pwms: &grove_laced_if_cm0 {};
+
+grove_pwm_d18: &tcc0 {};
+grove_pwm_d19: &tcc0 {};
diff --git a/boards/arm/seeeduino_cm0/grove_i2c_connector.dtsi b/boards/arm/seeeduino_cm0/grove_i2c_connector.dtsi
@@ -6,6 +6,7 @@
 / {
 	grove_i2c_header: grove-i2c-connector {
 		compatible = "seeed,grove-connector";
+
 		#gpio-cells = <2>;
 		gpio-map-mask = <0xffffffff 0xffffffc0>;
 		gpio-map-pass-thru = <0 0x3f>;
@@ -15,3 +16,55 @@
 };
 
 grove_i2c: &sercom3 {};
+
+#undef GROVE_PWM_D18_FAST
+#define GROVE_PWM_D18_FAST				\
+	&tcc0 {						\
+		prescaler = <1>;			\
+	};
+
+#undef GROVE_PWM_D18_SLOW
+#define GROVE_PWM_D18_SLOW				\
+	&tcc0 {						\
+		prescaler = <1024>;			\
+	};
+
+#undef GROVE_PWM_D18_ENABLE
+#define GROVE_PWM_D18_ENABLE				\
+	&tcc0 {						\
+		status = "okay";			\
+	};
+
+#undef GROVE_PWM_D18_PINCTRL
+#define GROVE_PWM_D18_PINCTRL				\
+	&tcc0_pwm_default {				\
+		group4 {				\
+			pinmux = <PA22F_TCC0_WO4>;	\
+		};					\
+	};
+
+#undef GROVE_PWM_D19_FAST
+#define GROVE_PWM_D19_FAST				\
+	&tcc0 {						\
+		prescaler = <1>;			\
+	};
+
+#undef GROVE_PWM_D19_SLOW
+#define GROVE_PWM_D19_SLOW				\
+	&tcc0 {						\
+		prescaler = <1024>;			\
+	};
+
+#undef GROVE_PWM_D19_ENABLE
+#define GROVE_PWM_D19_ENABLE				\
+	&tcc0 {						\
+		status = "okay";			\
+	};
+
+#undef GROVE_PWM_D19_PINCTRL
+#define GROVE_PWM_D19_PINCTRL				\
+	&tcc0_pwm_default {				\
+		group5 {				\
+			pinmux = <PA23F_TCC0_WO5>;	\
+		};					\
+	};
diff --git a/boards/arm/seeeduino_cm0/seeeduino_cm0-pinctrl.dtsi b/boards/arm/seeeduino_cm0/seeeduino_cm0-pinctrl.dtsi
@@ -44,6 +44,8 @@
 		};
 	};
 
+	tcc0_pwm_default: tcc0_pwm_default {};
+
 	tcc2_pwm_default: tcc2_pwm_default {
 		group1 {
 			pinmux = <PA17E_TCC2_WO1>;

diff --git a/boards/arm/seeeduino_cm0/seeeduino_cm0.dts b/boards/arm/seeeduino_cm0/seeeduino_cm0.dts
@@ -109,6 +109,16 @@
 	pinctrl-names = "default";
 };
 
+&tcc0 {
+	status = "okay";
+	compatible = "atmel,sam0-tcc-pwm";
+	prescaler = <1>;
+	#pwm-cells = <2>;
+
+	pinctrl-0 = <&tcc0_pwm_default>;
+	pinctrl-names = "default";
+};
+
 &tcc2 {
 	status = "okay";
 	compatible = "atmel,sam0-tcc-pwm";

diff --git a/boards/arm/seeeduino_cm0/seeeduino_cm0.yaml b/boards/arm/seeeduino_cm0/seeeduino_cm0.yaml
@@ -1,3 +1,4 @@
+vendor: seeed
 identifier: seeeduino_cm0
 name: Seeeduino Cortex-M0+
 type: mcu
@@ -13,6 +14,8 @@ supported:
   - arduino_i2c
   - arduino_spi
   - arduino_serial
+  - grove_gpio
+  - grove_pwm
   - grove_i2c
   - grove_serial
   - hwinfo

@@ -219,9 +219,9 @@ Following mappings are well known:
       +--------------------+-----------------------------+-------------------------+-------------------------+
       | Grove PWM Label    | phandle index to shield --> | **Signal** : *Meaning*  | ``seeeduino_lotus``     |
       +====================+=============================+=========================+=========================+
-      |                    | ``<&grove_pwms 0 …>``       | **D0:** *UART-RX*       |                         |
+      |                    | ``<&grove_pwms 0 …>``       | **D0:** *UART-RX*       | **not possible**        |
       +--------------------+-----------------------------+-------------------------+-------------------------+
-      |                    | ``<&grove_pwms 1 …>``       | **D1:** *UART-TX*       |                         |
+      |                    | ``<&grove_pwms 1 …>``       | **D1:** *UART-TX*       | **not possible**        |
       +--------------------+-----------------------------+-------------------------+-------------------------+
       | ``&grove_pwm_d2``  | ``<&grove_pwms 2 …>``       | **D2**                  | ``<&tcc0 0 …>`` (WO4)   |
       +--------------------+-----------------------------+-------------------------+-------------------------+
@@ -235,7 +235,7 @@ Following mappings are well known:
       +--------------------+-----------------------------+-------------------------+-------------------------+
       | ``&grove_pwm_d7``  | ``<&grove_pwms 7 …>``       | **D7**                  | ``<&tcc0 3 …>`` (WO7)   |
       +--------------------+-----------------------------+-------------------------+-------------------------+
-      |                    | ``<&grove_pwms 8 …>``       | **D8**                  |                         |
+      | ``&grove_pwm_d8``  | ``<&grove_pwms 8 …>``       | **D8**                  | ``<&tcc1 0 …>`` (WO0)   |
       +--------------------+-----------------------------+-------------------------+-------------------------+
       |                    | ``<&grove_pwms 9 …>``       | **D9**                  | **not wired**           |
       +--------------------+-----------------------------+-------------------------+-------------------------+
@@ -247,17 +247,17 @@ Following mappings are well known:
       +--------------------+-----------------------------+-------------------------+-------------------------+
       |                    | ``<&grove_pwms 13 …>``      | **D13:** *SPI-CLK*      | **not wired**           |
       +--------------------+-----------------------------+-------------------------+-------------------------+
-      |                    | ``<&grove_pwms 14 …>``      | **D14:** *A0*           |                         |
+      |                    | ``<&grove_pwms 14 …>``      | **D14:** *A0*           | **not possible**        |
       +--------------------+-----------------------------+-------------------------+-------------------------+
-      |                    | ``<&grove_pwms 15 …>``      | **D15:** *A1*           |                         |
+      |                    | ``<&grove_pwms 15 …>``      | **D15:** *A1*           | **not possible**        |
       +--------------------+-----------------------------+-------------------------+-------------------------+
-      |                    | ``<&grove_pwms 16 …>``      | **D16:** *A2*           |                         |
+      |                    | ``<&grove_pwms 16 …>``      | **D16:** *A2*           | **not possible**        |
       +--------------------+-----------------------------+-------------------------+-------------------------+
-      |                    | ``<&grove_pwms 17 …>``      | **D17:** *A3*           |                         |
+      |                    | ``<&grove_pwms 17 …>``      | **D17:** *A3*           | **not possible**        |
       +--------------------+-----------------------------+-------------------------+-------------------------+
-      |                    | ``<&grove_pwms 18 …>``      | **D18:** *I2C-SDA*      |                         |
+      | ``&grove_pwm_d18`` | ``<&grove_pwms 18 …>``      | **D18:** *I2C-SDA*      | ``<&tcc0 0 …>`` (WO4)   |
       +--------------------+-----------------------------+-------------------------+-------------------------+
-      |                    | ``<&grove_pwms 19 …>``      | **D19:** *I2C-SCL*      |                         |
+      | ``&grove_pwm_d19`` | ``<&grove_pwms 19 …>``      | **D19:** *I2C-SCL*      | ``<&tcc0 1 …>`` (WO5)   |
       +--------------------+-----------------------------+-------------------------+-------------------------+
       |                    | ``<&grove_pwms 20 …>``      | **D20:** *ADC6*         | **not wired**           |
       +--------------------+-----------------------------+-------------------------+-------------------------+

diff --git a/boards/arm/seeeduino_lotus/grove_connectors.dtsi b/boards/arm/seeeduino_lotus/grove_connectors.dtsi
@@ -54,24 +54,36 @@
 		pwm-map =
 			/*  0 - D0: not possible */
 			/*  1 - D1: not possible */
-			<2 0 &grove_pwm_d2 0 0>,	/* D2: TCC0CH0 (WO4) */
-			<3 0 &grove_pwm_d3 1 0>,	/* D3: TCC0CH1 (WO1) */
-			<4 0 &grove_pwm_d4 0 0>,	/* D4: TCC0CH0 (WO0) */
-			<5 0 &grove_pwm_d5 1 0>,	/* D5: TCC0CH1 (WO5) */
-			<6 0 &grove_pwm_d6 2 0>,	/* D6: TCC0CH2 (WO6) */
-			<7 0 &grove_pwm_d7 3 0>;	/* D7: TCC0CH3 (WO7) */
-			/*  8 - D8:  not possible */
-			/*  9 - D9:  not possible */
-			/* 10 - D10: not possible */
-			/* 11 - D11: not possible */
-			/* 12 - D12: not possible */
-			/* 13 - D13: not possible */
+			<2 0 &tcc0 0 0>,	/* D2: TCC0CH0 (WO4) & D4/D18 */
+			<3 0 &tcc0 1 0>,	/* D3: TCC0CH1 (WO1) & D5/D19 */
+			<4 0 &tcc0 0 0>,	/* D4: TCC0CH0 (WO0) & D2/D18 */
+			<5 0 &tcc0 1 0>,	/* D5: TCC0CH1 (WO5) & D3/D19 */
+			<6 0 &tcc0 2 0>,	/* D6: TCC0CH2 (WO6) */
+			<7 0 &tcc0 3 0>,	/* D7: TCC0CH3 (WO7) */
+			<8 0 &tcc1 0 0>,	/* D8: TCC1CH0 (WO0) */
+			/*  9 - D9:  not wired */
+			/* 10 - D10: not wired */
+			/* 11 - D11: not wired */
+			/* 12 - D12: not wired */
+			/* 13 - D13: not wired */
 			/* 14 - D14: not possible */
 			/* 15 - D15: not possible */
 			/* 16 - D16: not possible */
 			/* 17 - D17: not possible */
+			<18 0 &tcc0 0 0>,	/* D18: TCC0CH0 (WO4) & D2/D4 */
+			<19 0 &tcc0 1 0>;	/* D19: TCC0CH1 (WO5) & D3/D5 */
 	};
 };
 
 grove_gpios: &grove_laced_if_lotus {};
 grove_pwms: &grove_laced_if_lotus {};
+
+grove_pwm_d2: &tcc0 {};
+grove_pwm_d3: &tcc0 {};
+grove_pwm_d4: &tcc0 {};
+grove_pwm_d5: &tcc0 {};
+grove_pwm_d6: &tcc0 {};
+grove_pwm_d7: &tcc0 {};
+grove_pwm_d8: &tcc1 {};
+grove_pwm_d18: &tcc0 {};
+grove_pwm_d19: &tcc0 {};
diff --git a/boards/arm/seeeduino_lotus/grove_d2_connector.dtsi b/boards/arm/seeeduino_lotus/grove_d2_connector.dtsi
@@ -39,5 +39,3 @@
 			pinmux = <PA14F_TCC0_WO4>;	\
 		};					\
 	};
-
-grove_pwm_d2: &tcc0 {};
diff --git a/boards/arm/seeeduino_lotus/grove_d3_connector.dtsi b/boards/arm/seeeduino_lotus/grove_d3_connector.dtsi
@@ -39,5 +39,3 @@
 			pinmux = <PA9E_TCC0_WO1>;	\
 		};					\
 	};
-
-grove_pwm_d3: &tcc0 {};
diff --git a/boards/arm/seeeduino_lotus/grove_d4_connector.dtsi b/boards/arm/seeeduino_lotus/grove_d4_connector.dtsi
@@ -39,5 +39,3 @@
 			pinmux = <PA8E_TCC0_WO0>;	\
 		};					\
 	};
-
-grove_pwm_d4: &tcc0 {};
diff --git a/boards/arm/seeeduino_lotus/grove_d5_connector.dtsi b/boards/arm/seeeduino_lotus/grove_d5_connector.dtsi
@@ -39,5 +39,3 @@
 			pinmux = <PA15F_TCC0_WO5>;	\
 		};					\
 	};
-
-grove_pwm_d5: &tcc0 {};
diff --git a/boards/arm/seeeduino_lotus/grove_d6_connector.dtsi b/boards/arm/seeeduino_lotus/grove_d6_connector.dtsi
@@ -39,5 +39,3 @@
 			pinmux = <PA20F_TCC0_WO6>;	\
 		};					\
 	};
-
-grove_pwm_d6: &tcc0 {};
diff --git a/boards/arm/seeeduino_lotus/grove_d7_connector.dtsi b/boards/arm/seeeduino_lotus/grove_d7_connector.dtsi
@@ -40,4 +40,28 @@
 		};					\
 	};
 
-grove_pwm_d7: &tcc0 {};
+#undef GROVE_PWM_D8_FAST
+#define GROVE_PWM_D8_FAST				\
+	&tcc1 {						\
+		prescaler = <1>;			\
+	};
+
+#undef GROVE_PWM_D8_SLOW
+#define GROVE_PWM_D8_SLOW				\
+	&tcc1 {						\
+		prescaler = <1024>;			\
+	};
+
+#undef GROVE_PWM_D8_ENABLE
+#define GROVE_PWM_D8_ENABLE				\
+	&tcc1 {						\
+		status = "okay";			\
+	};
+
+#undef GROVE_PWM_D8_PINCTRL
+#define GROVE_PWM_D8_PINCTRL				\
+	&tcc1_pwm_default {				\
+		group0 {				\
+			pinmux = <PA6E_TCC1_WO0>;	\
+		};					\
+	};
-Original file line number
+Diff line change
@@ Expand Up / @@ -39,5 +39,3 @@ @@
     			pinmux = <PA14F_TCC0_WO4>;	\
     		};					\
     	};
-    grove_pwm_d2: &tcc0 {};