michaelh/zephyr
1
0
Fork 0
Code Releases Activity

boards: nrf52840dk_nrf52840: Do not enable arduino_serial by default

Browse source 
This is a follow-up to commit 25d7a09aa5.

The arduino_serial/uart1 node should not be enabled by default because
even if an application does not use it, the default CONFIG_SERIAL=y
setting causes that it is anyway initialized, so the UARTE peripheral
acquires its assigned pins (and they cannot be used in other way) and
its enabled receiver causes increased current consumption (by ~500 uA)
when the CPU is sleeping. This affects e.g. the boards/nrf/system_off
sample.
Applications that actually need to use this UART or shields that use
the arduino_serial node should enable the node explicitly.

Keep this node disabled by default for the nrf52840dk_nrf52840 board
and also for boards whose definitions are mostly copies of the above:
nrf52833dk_nrf52833 and nrf21540dk_nrf52840.

Update also accordingly a few overlay files in tests/ that were
disabling this node because of the pins it undesirably acquired.

Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
This commit is contained in:
Andrzej Głąbek 2023-01-12 13:46:37 +01:00 • committed by Carles Cufí
commit b4fb5300da
7 changed files with 0 additions and 20 deletions
Download patch file Download diff file Expand all files Collapse all files

1
boards/arm/nrf21540dk_nrf52840/nrf21540dk_nrf52840.dts
View file

@ -167,7 +167,6 @@
};
arduino_serial: &uart1 {
status = "okay";
current-speed = <115200>;
pinctrl-0 = <&uart1_default>;
pinctrl-1 = <&uart1_sleep>;

1
boards/arm/nrf52833dk_nrf52833/nrf52833dk_nrf52833.dts
View file

@ -144,7 +144,6 @@
};
arduino_serial: &uart1 {
status = "okay";
current-speed = <115200>;
pinctrl-0 = <&uart1_default>;
pinctrl-1 = <&uart1_sleep>;

1
boards/arm/nrf52840dk_nrf52840/nrf52840dk_nrf52840.dts
View file

@ -156,7 +156,6 @@
};
arduino_serial: &uart1 {
status = "okay";
current-speed = <115200>;
pinctrl-0 = <&uart1_default>;
pinctrl-1 = <&uart1_sleep>;

4
tests/drivers/gpio/gpio_basic_api/boards/nrf52840dk_nrf52840.overlay
View file

@ -11,7 +11,3 @@
in-gpios = <&gpio1 2 0>; /* Arduino D1 */
};
};
&uart1 {
status = "disabled";
};

4
tests/drivers/i2s/i2s_api/boards/nrf52840dk_nrf52840.overlay
View file

@ -9,10 +9,6 @@
};
};
&uart1 {
status = "disabled";
};
&i2s0 {
status = "okay";
pinctrl-0 = <&i2s0_default_alt>;

4
tests/drivers/i2s/i2s_speed/boards/nrf52840dk_nrf52840.overlay
View file

@ -17,10 +17,6 @@
};
};
&uart1 {
status = "disabled";
};
&i2s0 {
status = "okay";
pinctrl-0 = <&i2s0_default_alt>;

5
tests/drivers/regulator/fixed/boards/nrf52840dk_nrf52840.overlay
View file

@ -15,8 +15,3 @@
check-gpios = <&gpio1 2 (GPIO_ACTIVE_HIGH | GPIO_PULL_DOWN)>;
};
};
&uart1 {
/* Default-enabled Arduino UART steals the pins we want. */
status = "disabled";
};