Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: nrf5340_audio: note about +20 dBm limitation #12507

Merged
merged 1 commit into from
Oct 5, 2023

Conversation

greg-fer
Copy link
Contributor

@greg-fer greg-fer commented Oct 3, 2023

Added a mention of the missing support for +20 dBm when using the controller with the nRF21540 FEM, because of a class restriction in the controller's QDID.
OCT-2735.

@greg-fer greg-fer added the CI-disable Disable CI for this PR label Oct 3, 2023
@greg-fer greg-fer added this to the 2.5.0 milestone Oct 3, 2023
@github-actions github-actions bot added changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. doc-required PR must not be merged without tech writer approval. labels Oct 3, 2023
@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

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

@@ -405,6 +405,10 @@ You can add support for the nRF21540 front-end module (FEM) to this application

To set the TX power output, use the :ref:`CONFIG_NRF_21540_MAIN_TX_POWER <config_nrf53_audio_app_options>` and :ref:`CONFIG_NRF_21540_PRI_ADV_TX_POWER <config_nrf53_audio_app_options>` Kconfig options.

.. note::
When you build the nRF5340 Audio application with the nRF21540 FEM support, the :ref:`lib_bt_ll_acs_nrf53_readme` does not support the +20 dBm setting.
This is because of a class restriction in the controller's QDID.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
This is because of a class restriction in the controller's QDID.
This is because of a power class restriction in the controller's QDID.

@@ -69,6 +69,9 @@ The controller is marked as :ref:`experimental <software_maturity>`.
This controller and the LE Audio Controller Subsystem for nRF5340 it includes has been tested and works in configurations used by the :ref:`nrf53_audio_app` application (for example, 2 concurrent CIS, or BIS).
No other configurations than the ones used in the referenced application have been tested or documented for this library.

When you :ref:`build the nRF5340 Audio application with the nRF21540 FEM support <nrf53_audio_app_adding_FEM_support>`, the LE Audio controller for nRF5340 does not support the +20 dBm setting.
This is because of a class restriction in the controller's QDID.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
This is because of a class restriction in the controller's QDID.
This is because of a power class restriction in the controller's QDID.

@greg-fer greg-fer force-pushed the doc_oct_2735_fem_pcft_limitation branch 2 times, most recently from 61bbc83 to bf0b448 Compare October 5, 2023 08:21
@github-actions github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 5, 2023
@greg-fer greg-fer removed the doc-required PR must not be merged without tech writer approval. label Oct 5, 2023
@greg-fer greg-fer force-pushed the doc_oct_2735_fem_pcft_limitation branch from bf0b448 to 7333f53 Compare October 5, 2023 08:58
@github-actions github-actions bot added the doc-required PR must not be merged without tech writer approval. label Oct 5, 2023
Added a mention of the missing support for +20 dBm when using the
controller with the nRF21540 FEM, because of a class restriction
in the controller's QDID.
OCT-2735.

Signed-off-by: Grzegorz Ferenc <[email protected]>
@rlubos rlubos merged commit 7d7f354 into nrfconnect:main Oct 5, 2023
12 checks passed
@greg-fer greg-fer deleted the doc_oct_2735_fem_pcft_limitation branch October 6, 2023 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-disable Disable CI for this PR doc-required PR must not be merged without tech writer approval.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants