boards: nrf52840dk_nrf52840: Do not enable arduino_serial by default

This is a follow-up to commit 25d7a09.

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 <[email protected]>

Author: anangl

boards/arm/nrf21540dk_nrf52840/nrf21540dk_nrf52840.dts

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

boards/arm/nrf52833dk_nrf52833/nrf52833dk_nrf52833.dts

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

boards/arm/nrf52840dk_nrf52840/nrf52840dk_nrf52840.dts

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

tests/drivers/gpio/gpio_basic_api/boards/nrf52840dk_nrf52840.overlay

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

tests/drivers/i2s/i2s_api/boards/nrf52840dk_nrf52840.overlay

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

tests/drivers/i2s/i2s_speed/boards/nrf52840dk_nrf52840.overlay

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

tests/drivers/regulator/fixed/boards/nrf52840dk_nrf52840.overlay

@@ -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";
-};