commit 260fc89643fd64b74f9983065867d8b368f81a0d
author Roman Studenikin <srv@meta.com> Wed Jan 24 16:06:47 2024 +0000
committer Fabio Baltieri <fabio.baltieri@gmail.com> Tue Jan 30 00:26:58 2024 +0000
tree c907692755e513caed4bb06540b9a6ebd85865b6
parent 1799a2bf0c3d8b3b643a30db25b5ceda343344cc

drivers: use DT_INST_PROP over DT_INST_PROP_OR if possible

It might happens that DT(_INST)_PROP_OR is used with boolean properties.
For instance:

	.single_wire = DT_INST_PROP_OR(index, single_wire, false),	\
	.tx_rx_swap = DT_INST_PROP_OR(index, tx_rx_swap, false),	\

This is not required as boolean properties are generated with false
value when not present, so the _OR macro extension is superflous
and the above code can be replaced by:

	.single_wire = DT_INST_PROP(index, single_wire),		\
	.tx_rx_swap = DT_INST_PROP(index, tx_rx_swap),			\

Signed-off-by: Roman Studenikin <srv@meta.com>

drivers/serial/uart_stm32.c

diff --git a/drivers/serial/uart_stm32.c b/drivers/serial/uart_stm32.c
index 107dd88..834012c 100644
--- a/drivers/serial/uart_stm32.c
+++ b/drivers/serial/uart_stm32.c
@@ -2345,8 +2345,8 @@
 	.pclken = pclken_##index,					\
 	.pclk_len = DT_INST_NUM_CLOCKS(index),				\
 	.pcfg = PINCTRL_DT_INST_DEV_CONFIG_GET(index),			\
-	.single_wire = DT_INST_PROP_OR(index, single_wire, false),	\
-	.tx_rx_swap = DT_INST_PROP_OR(index, tx_rx_swap, false),	\
+	.single_wire = DT_INST_PROP(index, single_wire),		\
+	.tx_rx_swap = DT_INST_PROP(index, tx_rx_swap),			\
 	.rx_invert = DT_INST_PROP(index, rx_invert),			\
 	.tx_invert = DT_INST_PROP(index, tx_invert),			\
 	.de_enable = DT_INST_PROP(index, de_enable),			\