mpsl: fem: add support for nrf21540_gpio for nrf54l

[ankuns]

This PR adds support for the nRF21540 Front-End Module in GPIO mode for the nRF54L Series devices.

Note: This PR contains only code related configuration and documentation. The core of the support is provided by MPSL. The MPSL is prvided by a separate PR.

• Merge after MPSL contaning the support for nrf21540_gpio for nRF54L is deployed to sdk-nrfxlib:
  Update MPSL and SoftDevice Controller sdk-nrfxlib#1652
  manifest: sdk-nrfxlib: Update MPSL and SoftDevice Controller #20375

[NordicBuilder]

CI Information

^{To view the history of this post, clich the 'edited' button above}
Build number: 6

Inputs:

Sources:

sdk-nrf: PR head: df5396212ad429e6a139518bf499d78137e5a911

more details

sdk-nrf:

PR head: df5396212ad429e6a139518bf499d78137e5a911
merge base: 27c5b09a927c01982c21553e683cbabb422aee80
target head (main): b92e9353a9d6defe98061d7fd6f54d42f61e691c
Diff

Github labels

Enabled	Name	Description
	ci-disabled	Disable the ci execution
	ci-all-test	Run all of ci, no test spec filtering will be done
	ci-force-downstream	Force execution of downstream even if twister fails
	ci-run-twister	Force run twister
	ci-run-zephyr-twister	Force run zephyr twister

List of changed files detected by CI (5)

doc
│  ├── nrf
│  │  ├── app_dev
│  │  │  ├── device_guides
│  │  │  │  ├── fem
│  │  │  │  │  ├── fem_nrf21540_gpio.rst
│  │  │  │  │  │ index.rst
│  │  ├── releases_and_maturity
│  │  │  ├── releases
│  │  │  │  │ release-notes-changelog.rst
subsys
│  ├── mpsl
│  │  ├── fem
│  │  │  ├── Kconfig
│  │  │  ├── nrf21540_gpio
│  │  │  │  │ mpsl_fem_nrf21540_gpio.c

Outputs:

Toolchain

Version: 342151af73
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:342151af73_bbe5b33786

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

• ◻️ Toolchain - Skipped: existing toolchain is used
• ✅ Build twister
  • sdk-nrf test count: 885
• ✅ Integration tests
  • ✅ test_ble_nrf_config
  • ✅ test-fw-nrfconnect-ble_samples
  • ✅ test-fw-nrfconnect-chip
  • ✅ test-fw-nrfconnect-rs
  • ✅ test-fw-nrfconnect-fem
  • ✅ test-fw-nrfconnect-thread
  • ✅ test-fw-nrfconnect-zigbee
  • ✅ test-sdk-find-my

Disabled integration tests

• • desktop52_verification
  • doc-internal
  • test-fw-nrfconnect-apps
  • test-fw-nrfconnect-ble_mesh
  • test-fw-nrfconnect-boot
  • test-fw-nrfconnect-nfc
  • test-fw-nrfconnect-nrf-iot_lwm2m
  • test-fw-nrfconnect-nrf-iot_mosh
  • test-fw-nrfconnect-nrf-iot_positioning
  • test-fw-nrfconnect-nrf-iot_samples
  • test-fw-nrfconnect-nrf-iot_serial_lte_modem
  • test-fw-nrfconnect-nrf-iot_thingy91
  • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
  • test-fw-nrfconnect-nrf_crypto
  • test-fw-nrfconnect-proprietary_esb
  • test-fw-nrfconnect-ps
  • test-fw-nrfconnect-rpc
  • test-fw-nrfconnect-tfm
  • test-low-level
  • test-sdk-audio
  • test-sdk-dfu
  • test-sdk-mcuboot
  • test-sdk-pmic-samples
  • test-sdk-sidewalk
  • test-sdk-wifi
  • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

[NordicBuilder]

You can find the documentation preview for this PR at this link.

Note: This comment is automatically posted by the Documentation Publish GitHub Action.

[ankuns]

@umapraseeda this is ready for final review

[ankuns]

@peknis I think I fixed all nits you pointed out

[peknis]

@peknis I think I fixed all nits you pointed out

Actually, there would be one more nit. In the description of the process of using the FEM in GPIO mode, the last two steps should have an imperative verb (do something). Otherwise, they are not proper steps.

[ankuns]

I'm confused and don't understand what's wrong.
In the sentence about nRF53 the workd "must" is not imperative enough? The text has been there for a long time.
The step for nRF54L somehow complex, but it explains all the details that need to be taken into account.

[peknis]

I'm confused and don't understand what's wrong. In the sentence about nRF53 the workd "must" is not imperative enough? The text has been there for a long time. The step for nRF54L somehow complex, but it explains all the details that need to be taken into account.

There is a clear difference between "You must do this" and "Do this". I agree the previous step has been there a while, but now we have a chance to fix this.

[ankuns]

Okay, I get it. If changing from "You must do this" to "do this" is the only issue, then what about merging it as it is and fixing the what you pointed out as a separate PR. The reason is that this pattern with "you must do..." is used among all FEMs and I would like to fix them in one PR which will follow shortly after this one. I would like to avoid fixing unrelated stuff in a PR which introduces an awaited feature.

[ankuns]

It also seems that this piece of documentation has never appeared in our docs, as it is missing from app_dev/device_guides/fem/index.rst

https://ncsdoc.z6.web.core.windows.net/PR-20273/nrf/app_dev/device_guides/fem/index.html

[peknis]

My bad, I was looking in a wrong place. Ignore my comment. Let's do the wording improvements in a separate PR as you suggested.

[ankuns]

@martintv @nrfconnect/ncs-co-build-system @nrfconnect/ncs-doc-leads kindly asking for approval.