Skip to content

Conversation

peknis
Copy link
Contributor

@peknis peknis commented Apr 4, 2025

Added v3-0-0 to known issues.

@peknis peknis requested a review from a team as a code owner April 4, 2025 05:59
@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Apr 4, 2025
@peknis peknis added CI-disable Disable CI for this PR doc-required PR must not be merged without tech writer approval. and removed doc-required PR must not be merged without tech writer approval. changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Apr 4, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Apr 4, 2025

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 39

Inputs:

Sources:

more details

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 (0)

Outputs:

Toolchain

Version:
Build docker image:

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

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ Integration tests

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

Copy link

github-actions bot commented Apr 4, 2025

You can find the documentation preview for this PR here.

Preview links for modified nRF Connect SDK documents:

https://ncsdoc.z6.web.core.windows.net/PR-21388/nrf/app_dev/device_guides/nrf54l/index.html
https://ncsdoc.z6.web.core.windows.net/PR-21388/nrf/releases_and_maturity/known_issues.html

@@ -3680,7 +3680,7 @@ NCSDK-31300: [suit] Leaving the recovery mode using the default DFU candidate is

**Workaround:** In such cases, preparation of a custom DFU candidate is needed.

.. rst-class:: v2-9-0-nRF54H20-1
.. rst-class:: v3-0-0 v2-9-0-nRF54H20-1
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
.. rst-class:: v3-0-0 v2-9-0-nRF54H20-1
.. rst-class:: v2-9-0-nRF54H20-1

@peknis peknis force-pushed the known-issues-300 branch from 0225dc0 to 66d0452 Compare April 4, 2025 06:26
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Apr 4, 2025
@peknis
Copy link
Contributor Author

peknis commented Apr 4, 2025

@peknis peknis force-pushed the known-issues-300 branch from 66d0452 to 90eeb90 Compare April 4, 2025 07:21
Copy link
Contributor

@nvlsianpu nvlsianpu left a comment

Choose a reason for hiding this comment

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

Please add to MCUboot section

.. rst-class:: v3-0-0

Encryption: Build error for default configuration on nrf52840dk/nrf52840 (ECDSA_P256)
Reason is inconsistency of configuration of the signature check and the encryption key extraction – both must use the same base encryption algorithm.
Workaround: for instances use can switch signature algorithm to RSA or switch crypto-library to tinycrypt.

Affected platform: nRF52840

@peknis peknis requested a review from nvlsianpu April 23, 2025 12:28
@peknis peknis force-pushed the known-issues-300 branch 2 times, most recently from 268bce9 to 83bff38 Compare April 23, 2025 12:47
@peknis peknis force-pushed the known-issues-300 branch from 83bff38 to 1df1ee1 Compare April 23, 2025 13:21
@peknis peknis added the DNM label Apr 23, 2025
@@ -1292,6 +1317,15 @@ Wi-Fi

The issues in this section are related to the :ref:`ug_wifi` protocol.

.. rst-class:: v3-0-0

SHEL-3596: For nRF5340DK + nRF7002EK, XiP is not working after Wi-Fi interface down/up
Copy link
Contributor

Choose a reason for hiding this comment

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

This issue has been fixed now. Please remove this from the known issues.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fix not yet backported. Keeping the issue.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Waiting for @bama-nordic to confirm whether we have a fix.

Copy link
Contributor

Choose a reason for hiding this comment

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

@peknis , too late now and not sure this fix will be merged to 3.0. I suggest we keep this issue for 3.0 and move ahead.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks Bansi!

@peknis peknis requested a review from bama-nordic April 24, 2025 06:01
@peknis peknis force-pushed the known-issues-300 branch from 1df1ee1 to cfcf7ac Compare April 24, 2025 06:43
@peknis peknis removed DNM changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Apr 24, 2025
@maje-emb
Copy link
Contributor

Please add:

Peripheral samples
==================

.. rst-class:: v3-0-0

NCSDK-31981: Incorrect characters transmitted or received when using 11110000 pattern on 4 Mbit PHY by :ref:`radio_test` sample.
 
  **Affected platforms:** nrf54L15
Bluetooth samples
=================

.. rst-class:: v3-0-0

NCSDK-33040: For the :ref:`direct_test_mode` sample, output power tests running with the Anritsu tester fail.

  **Affected platforms:** nRF5340, nRF21540

@peknis peknis force-pushed the known-issues-300 branch from cfcf7ac to b3dce88 Compare April 24, 2025 08:05
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Apr 24, 2025
Copy link
Contributor

@koffes koffes left a comment

Choose a reason for hiding this comment

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

I need to add one more issue. When is the definitive deadline?
Need to gather some information.

Comment on lines 2653 to 2655
Bluetooth samples
=================

Copy link
Contributor

@maje-emb maje-emb Apr 24, 2025

Choose a reason for hiding this comment

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

Suggested change
Bluetooth samples
=================

This seems to be duplicated

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Whoops, thanks for catching this. Fixed.

@@ -2225,17 +2284,17 @@ nRF5340 Audio

Copy link
Contributor

Choose a reason for hiding this comment

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

Please add:
OCT-3368: For 16 and 24 kHz: The application may repeatedly print: "audio_sync_timer: Unable to get new CC value" which may cause degraded audio quality.

.. rst-class:: v3-0-0

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

Added v3-0-0 to known issues.

Signed-off-by: Pekka Niskanen <[email protected]>
@peknis peknis force-pushed the known-issues-300 branch from b3dce88 to b17ecac Compare April 24, 2025 09:07
@peknis peknis requested a review from maje-emb April 24, 2025 09:08
@carlescufi carlescufi merged commit 5a29694 into nrfconnect:main Apr 24, 2025
14 checks passed
@peknis peknis deleted the known-issues-300 branch April 24, 2025 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport v3.0-branch changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. 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.