drivers: dac: make output buffer for STM32 DAC configurable

If an operational amplifier is used on the DAC output it is preferrable to disable the DAC output buffer. Signed-off-by: Benedikt Schmidt <[email protected]>
zephyrproject-rtos · May 10, 2023 · 33d7951 · 33d7951
1 parent 3e6bf03
commit 33d7951
Show file tree

Hide file tree

Showing 2 changed files with 36 additions and 23 deletions.
diff --git a/drivers/dac/dac_stm32.c b/drivers/dac/dac_stm32.c
@@ -48,6 +48,8 @@ struct dac_stm32_cfg {
 	struct stm32_pclken pclken;
 	/* pinctrl configurations. */
 	const struct pinctrl_dev_config *pcfg;
+	/* disable output buffer */
+	bool output_buffer_disable;
 };
 
 /* Runtime driver data */
@@ -84,6 +86,7 @@ static int dac_stm32_channel_setup(const struct device *dev,
 {
 	struct dac_stm32_data *data = dev->data;
 	const struct dac_stm32_cfg *cfg = dev->config;
+	uint32_t output_buffer;
 
 	if ((channel_cfg->channel_id - STM32_FIRST_CHANNEL >=
 			data->channel_count) ||
@@ -100,10 +103,15 @@ static int dac_stm32_channel_setup(const struct device *dev,
 		return -ENOTSUP;
 	}
 
-	/* enable output buffer by default */
+	if (cfg->output_buffer_disable) {
+		output_buffer = LL_DAC_OUTPUT_BUFFER_DISABLE;
+	} else {
+		output_buffer = LL_DAC_OUTPUT_BUFFER_ENABLE;
+	}
+
 	LL_DAC_SetOutputBuffer(cfg->base,
 		table_channels[channel_cfg->channel_id - STM32_FIRST_CHANNEL],
-		LL_DAC_OUTPUT_BUFFER_ENABLE);
+		output_buffer);
 
 	LL_DAC_Enable(cfg->base,
 		table_channels[channel_cfg->channel_id - STM32_FIRST_CHANNEL]);
@@ -147,27 +155,28 @@ static const struct dac_driver_api api_stm32_driver_api = {
 };
 
 
-#define STM32_DAC_INIT(index)						\
-									\
-PINCTRL_DT_INST_DEFINE(index);						\
-									\
-static const struct dac_stm32_cfg dac_stm32_cfg_##index = {		\
-	.base = (DAC_TypeDef *)DT_INST_REG_ADDR(index),			\
-	.pclken = {							\
-		.enr = DT_INST_CLOCKS_CELL(index, bits),		\
-		.bus = DT_INST_CLOCKS_CELL(index, bus),			\
-	},								\
-	.pcfg = PINCTRL_DT_INST_DEV_CONFIG_GET(index),			\
-};									\
-									\
-static struct dac_stm32_data dac_stm32_data_##index = {			\
-	.channel_count = STM32_CHANNEL_COUNT				\
-};									\
-									\
-DEVICE_DT_INST_DEFINE(index, &dac_stm32_init, NULL,			\
-		    &dac_stm32_data_##index,				\
-		    &dac_stm32_cfg_##index, POST_KERNEL,		\
-		    CONFIG_DAC_INIT_PRIORITY,				\
+#define STM32_DAC_INIT(index)						     \
+									     \
+PINCTRL_DT_INST_DEFINE(index);						     \
+									     \
+static const struct dac_stm32_cfg dac_stm32_cfg_##index = {		     \
+	.base = (DAC_TypeDef *)DT_INST_REG_ADDR(index),			     \
+	.pclken = {							     \
+		.enr = DT_INST_CLOCKS_CELL(index, bits),		     \
+		.bus = DT_INST_CLOCKS_CELL(index, bus),			     \
+	},								     \
+	.pcfg = PINCTRL_DT_INST_DEV_CONFIG_GET(index),			     \
+	.output_buffer_disable = DT_INST_PROP(index, output_buffer_disable), \
+};									     \
+									     \
+static struct dac_stm32_data dac_stm32_data_##index = {			     \
+	.channel_count = STM32_CHANNEL_COUNT				     \
+};									     \
+									     \
+DEVICE_DT_INST_DEFINE(index, &dac_stm32_init, NULL,			     \
+		    &dac_stm32_data_##index,				     \
+		    &dac_stm32_cfg_##index, POST_KERNEL,		     \
+		    CONFIG_DAC_INIT_PRIORITY,				     \
 		    &api_stm32_driver_api);
 
 DT_INST_FOREACH_STATUS_OKAY(STM32_DAC_INIT)
diff --git a/dts/bindings/dac/st,stm32-dac.yaml b/dts/bindings/dac/st,stm32-dac.yaml
@@ -23,5 +23,9 @@ properties:
   pinctrl-names:
     required: true
 
+  output-buffer-disable:
+    type: boolean
+    description: disable output buffer
+
 io-channel-cells:
   - output