drivers: add assert to check for max children in timer

check for max number of children in timer.

Signed-off-by: Richard Wheatley <richard.wheatley@ambiq.com>

Author: RichardSWheatley

drivers/counter/counter_ambiq_timer.c

@@ -438,12 +438,14 @@ static void counter_ambiq_isr(void *arg)
 #endif
 
 #define AMBIQ_COUNTER_INIT(idx)                                                                    \
+	BUILD_ASSERT(DT_CHILD_NUM_STATUS_OKAY(DT_INST_PARENT(idx)) == 1,                           \
+		     "Too many children for Timer!");                                              \
 	static void counter_irq_config_func_##idx(void);                                           \
 	static struct counter_ambiq_data counter_data_##idx;                                       \
 	static const struct counter_ambiq_config counter_config_##idx = {                          \
 		.instance = (DT_REG_ADDR(DT_INST_PARENT(idx)) - SOC_TIMER_BASE) /                  \
 			    DT_REG_SIZE(DT_INST_PARENT(idx)),                                      \
-		.clk_src = DT_ENUM_IDX(DT_INST_PARENT(idx), clk_source),                               \
+		.clk_src = DT_ENUM_IDX(DT_INST_PARENT(idx), clk_source),                           \
 		.counter_info = {.max_top_value = UINT32_MAX,                                      \
 				 .flags = COUNTER_CONFIG_INFO_COUNT_UP,                            \
 				 .channels = 1},                                                   \

drivers/pwm/pwm_ambiq_ctimer.c

@@ -127,8 +127,8 @@ static void start_clock(uint32_t clock_sel)
 }
 
 static int ambiq_ctimer_pwm_set_cycles(const struct device *dev, uint32_t channel,
-				      uint32_t period_cycles, uint32_t pulse_cycles,
-				      pwm_flags_t flags)
+				       uint32_t period_cycles, uint32_t pulse_cycles,
+				       pwm_flags_t flags)
 {
 	const struct pwm_ambiq_ctimer_config *config = dev->config;
 
@@ -172,7 +172,7 @@ static int ambiq_ctimer_pwm_set_cycles(const struct device *dev, uint32_t channe
 }
 
 static int ambiq_ctimer_pwm_get_cycles_per_sec(const struct device *dev, uint32_t channel,
-					      uint64_t *cycles)
+					       uint64_t *cycles)
 {
 	struct pwm_ambiq_ctimer_data *data = dev->data;
 	int err = 0;
@@ -230,21 +230,23 @@ static const struct DEVICE_API(pwm, pwm_ambiq_ctimer_driver_api) = {
 
 #define TEST_CHILDREN	DT_PATH(test, test_children)
 
-#define PWM_AMBIQ_CTIMER_DEVICE_INIT(n)                                                             \
+#define PWM_AMBIQ_CTIMER_DEVICE_INIT(n)                                                            \
+	BUILD_ASSERT(DT_CHILD_NUM_STATUS_OKAY(DT_INST_PARENT(n)) == 1,                             \
+		     "Too many children for Timer!");                                              \
 	PINCTRL_DT_INST_DEFINE(n);                                                                 \
-	static struct pwm_ambiq_ctimer_data pwm_ambiq_ctimer_data_##n = {                            \
+	static struct pwm_ambiq_ctimer_data pwm_ambiq_ctimer_data_##n = {                          \
 		.cycles = 0,                                                                       \
 	};                                                                                         \
-	static const struct pwm_ambiq_ctimer_config pwm_ambiq_ctimer_config_##n = {                  \
+	static const struct pwm_ambiq_ctimer_config pwm_ambiq_ctimer_config_##n = {                \
 		.timer_num = (DT_REG_ADDR(DT_INST_PARENT(n)) - CTIMER_BASE) /                      \
 			     DT_REG_SIZE(DT_INST_PARENT(n)),                                       \
 		.timer_seg = DT_INST_ENUM_IDX(n, timer_segment),                                   \
-		.clock_sel = DT_ENUM_IDX(DT_INST_PARENT(n), clk_source),                    \
+		.clock_sel = DT_ENUM_IDX(DT_INST_PARENT(n), clk_source),                           \
 		.pincfg = PINCTRL_DT_INST_DEV_CONFIG_GET(n),                                       \
 		.pwm_type = DT_INST_ENUM_IDX(n, pwm_type)};                                        \
                                                                                                    \
-	DEVICE_DT_INST_DEFINE(n, ambiq_ctimer_pwm_init, NULL, &pwm_ambiq_ctimer_data_##n,            \
-			      &pwm_ambiq_ctimer_config_##n, POST_KERNEL, CONFIG_PWM_INIT_PRIORITY,  \
+	DEVICE_DT_INST_DEFINE(n, ambiq_ctimer_pwm_init, NULL, &pwm_ambiq_ctimer_data_##n,          \
+			      &pwm_ambiq_ctimer_config_##n, POST_KERNEL, CONFIG_PWM_INIT_PRIORITY, \
 			      &pwm_ambiq_ctimer_driver_api);
 
 DT_INST_FOREACH_STATUS_OKAY(PWM_AMBIQ_CTIMER_DEVICE_INIT)

drivers/pwm/pwm_ambiq_timer.c

@@ -182,14 +182,16 @@ static const struct DEVICE_API(pwm, pwm_ambiq_timer_driver_api) = {
 };
 
 #define PWM_AMBIQ_TIMER_DEVICE_INIT(n)                                                             \
+	BUILD_ASSERT(DT_CHILD_NUM_STATUS_OKAY(DT_INST_PARENT(n)) == 1,                             \
+		     "Too many children for Timer!");                                              \
 	PINCTRL_DT_INST_DEFINE(n);                                                                 \
 	static struct pwm_ambiq_timer_data pwm_ambiq_timer_data_##n = {                            \
 		.cycles = 0,                                                                       \
 	};                                                                                         \
 	static const struct pwm_ambiq_timer_config pwm_ambiq_timer_config_##n = {                  \
 		.timer_num = (DT_REG_ADDR(DT_INST_PARENT(n)) - TIMER_BASE) /                       \
 			     DT_REG_SIZE(DT_INST_PARENT(n)),                                       \
-		.clock_sel = DT_ENUM_IDX(DT_INST_PARENT(n), clk_source),                    \
+		.clock_sel = DT_ENUM_IDX(DT_INST_PARENT(n), clk_source),                           \
 		.pincfg = PINCTRL_DT_INST_DEV_CONFIG_GET(n)};                                      \
                                                                                                    \
 	DEVICE_DT_INST_DEFINE(n, ambiq_timer_pwm_init, NULL, &pwm_ambiq_timer_data_##n,            \