Skip to content

Commit

Permalink
shields: Enable SR co-existence only for nRF52 and nRF53 series SoC's
Browse files Browse the repository at this point in the history
For nRF91 series SR co-existence isn't applicable as there is no SR.
This saves 4 GPIOs that can be used for other purposes.

This can be done in many ways, but using board specific overlays to
include a common co-ex overlay was chosen because:

* Easier to handle DTS/Kconfig ordering
* In future if each SoC has different pins, easier to use a custom
overlay
* adding a new board it forces to consciously enable the SR co-ex

Fixes SHEL-2708.

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
  • Loading branch information
krish2718 authored and rlubos committed Apr 26, 2024
1 parent d7a4ef4 commit 9d4161f
Show file tree
Hide file tree
Showing 16 changed files with 45 additions and 4 deletions.
5 changes: 5 additions & 0 deletions boards/shields/nrf7002eb/boards/nrf52840dk_nrf52840.overlay
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
/* Copyright (c) 2024 Nordic Semiconductor ASA
*
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
#include "../nrf7002eb_coex.overlay"
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
/* Copyright (c) 2024 Nordic Semiconductor ASA
*
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
#include "../nrf7002eb_coex.overlay"
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/

#include "../nrf7002eb_coex.overlay"

/* The below overlays might not be explicitly required but still
* kept here to warn of the conflicting pins that could hamper
* functionality later
Expand Down
1 change: 0 additions & 1 deletion boards/shields/nrf7002eb/nrf7002eb.overlay
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
#include <freq.h>
#include "nrf7002eb_coex.overlay"

/ {
nordic_wlan0: nordic_wlan0 {
Expand Down
5 changes: 5 additions & 0 deletions boards/shields/nrf7002ek/boards/nrf52840dk_nrf52840.overlay
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
/* Copyright (c) 2024 Nordic Semiconductor ASA
*
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
#include "../nrf7002ek_coex.overlay"
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
*
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
#include "../nrf7002ek_coex.overlay"

/* This node by default forwards the UART1 pins to CPUNET, but as UART1 uses
* same pins as bucken and iovdd-ctrl, we need these pins to be controlled by
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
/* Copyright (c) 2024 Nordic Semiconductor ASA
*
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
#include "../nrf7002ek_coex.overlay"
1 change: 0 additions & 1 deletion boards/shields/nrf7002ek/nrf7002ek.overlay
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
#include <freq.h>
#include "nrf7002ek_coex.overlay"

/ {
nordic_wlan0: nordic_wlan0 {
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
/* Copyright (c) 2024 Nordic Semiconductor ASA
*
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
#include "../nrf7002ek_nrf7000_coex.overlay"
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
*
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
#include "../nrf7002ek_nrf7000_coex.overlay"

/* This node by default forwards the UART1 pins to CPUNET, but as UART1 uses
* same pins as bucken and iovdd-ctrl, we need these pins to be controlled by
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
/* Copyright (c) 2024 Nordic Semiconductor ASA
*
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
#include "../nrf7002ek_nrf7000_coex.overlay"
1 change: 0 additions & 1 deletion boards/shields/nrf7002ek_nrf7000/nrf7002ek_nrf7000.overlay
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
#include <freq.h>
#include "nrf7002ek_nrf7000_coex.overlay"

/ {
nordic_wlan0: nordic_wlan0 {
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
/* Copyright (c) 2024 Nordic Semiconductor ASA
*
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
#include "../nrf7002ek_nrf7001_coex.overlay"
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
*
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
#include "../nrf7002ek_nrf7001_coex.overlay"

/* This node by default forwards the UART1 pins to CPUNET, but as UART1 uses
* same pins as bucken and iovdd-ctrl, we need these pins to be controlled by
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
/* Copyright (c) 2024 Nordic Semiconductor ASA
*
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
#include "../nrf7002ek_nrf7001_coex.overlay"
1 change: 0 additions & 1 deletion boards/shields/nrf7002ek_nrf7001/nrf7002ek_nrf7001.overlay
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
* SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
*/
#include <freq.h>
#include "nrf7002ek_nrf7001_coex.overlay"

/ {
nordic_wlan0: nordic_wlan0 {
Expand Down

0 comments on commit 9d4161f

Please sign in to comment.