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.

Sign up for GitHub

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

Jump to bottom

doc: update known issues for NCS v2.5.0 #12657

Merged
merged 1 commit into from
Oct 26, 2023
Merged

doc: update known issues for NCS v2.5.0 #12657

merged 1 commit into from
Oct 26, 2023

Conversation

divipillai
Copy link
Contributor

@divipillai divipillai commented Oct 11, 2023 •
edited by MarkusLassila
Loading

Added v2.5.0 tag to known issues.

@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 11, 2023
@divipillai divipillai removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 11, 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.

@SeppoTakalo
Copy link
Contributor

This same template seem to be copied again.
I'm not the correct person to check any Zephyr issues. I have once done that when there was LwM2M related issues that needed to be checked.

So I can check LwM2M related issues, but not Zephyr in general.

@simensrostad
Copy link
Contributor

@divipillai CIA-890 is resolved for this release.

@alexsven
Copy link
Contributor

For Audio: Following issues has been fixed since last tag: OCT-2558 and OCT-2587 and should not be tagged with v2-5-0

@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 Oct 12, 2023
@divipillai divipillai added CI-disable Disable CI for this PR and removed changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Oct 12, 2023
@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 Oct 12, 2023
@divipillai
Copy link
Contributor Author

This same template seem to be copied again. I'm not the correct person to check any Zephyr issues. I have once done that when there was LwM2M related issues that needed to be checked.

So I can check LwM2M related issues, but not Zephyr in general.

My bad, I copied the old template. Update the description and commit message. Thanks!

@alexsven
Copy link
Contributor

Sorry, I missed the Controller subsystem for nRF5340 Audio I thought it was in another document. The following are fixed:

  • OCX-138
  • OCX-152/OCX-146
  • OCX-156
  • OCX-157
  • OCX-178
  • OCX-183
  • OCX-184
  • OCX-188
  • OCX-189

@nordicjm
Copy link
Contributor

Zephyr issues checked by @mbolivar-nordic or @carlescufi

One of them does not work here any more and the other is on holiday and not the right person, this should be assigned to the person that did the last zephyr upmerge, @de-nordic

@divipillai divipillai removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 12, 2023
@MarkusLassila
Copy link
Contributor

@divipillai: We have two new known issues, should those be added separately, or should I create PR against this?

@divipillai
Copy link
Contributor Author

@divipillai: We have two new known issues, should those be added separately, or should I create PR against this?

You can let me know here. I can add those in this PR.

SebastianBoe
SebastianBoe suggested changes Oct 12, 2023
View reviewed changes
Copy link
Contributor

@SebastianBoe SebastianBoe left a comment

Choose a reason for hiding this comment

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

changes requested

@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 Oct 12, 2023
@divipillai divipillai removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 24, 2023
@glarsennordic glarsennordic self-requested a review October 24, 2023 20:18
@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 Oct 25, 2023
@divipillai divipillai removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 25, 2023
@cvinayak cvinayak mentioned this pull request Oct 25, 2023
Merged
nvlsianpu
nvlsianpu requested changes Oct 25, 2023
View reviewed changes
Copy link
Contributor

@nvlsianpu nvlsianpu left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Please add another Known issue:
Mcuboot: If Fault Injection hardening (FIH)is enabled the bug in boot_image_check_hook() manifesting so multicore application can't be booted. nRF5340 mcuboot builds witch simultaneous multimage update enabled. Workaround are: disable FIH or patch #12846.
tracking: NCSDK-24203

erikrobstad
erikrobstad suggested changes Oct 25, 2023
View reviewed changes
doc/nrf/releases_and_maturity/known_issues.rst
@@ -1648,6 +1861,27 @@ OCX-189: When inputting -40 dBm to HCI_OPCODE_VS_SET_CONN_TX_PWR (0x3F6), the ac
The TX power for a connection cannot be less than -20 dBm.
Controller still output -20 dBm if the setting is -40 dBm.

.. rst-class:: v2-5-0
Copy link
Contributor

@erikrobstad erikrobstad Oct 25, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

We discovered last minute that OCX-188 has re-occurred, so we need to add v2-5-0 to the rst-class on this one.
We have also created a new OCX ticket (OCX-227), since OCX-188 was fixed, but then it broke again.
Is it possible to write it like this?:
OCX-184/OCX-227: If 0 dBm TX power is selected, the FEM/PA TX/RX pins do not toggle correctly

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will add them.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated. Please check.

Copy link
Contributor

Choose a reason for hiding this comment

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

@divipillai Bah, I wrote my suggestion for OCX-184 instead of OCX-188, sorry
Meaning:

  • Nothing regarding OCX-184 should be changed
  • The text for OCX-188 should be: OCX-188/OCX-227: The controller reserves some pins (0.28 - 0.31), which may collide with FEM/PA features

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated. Please check.

@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 Oct 25, 2023
@divipillai divipillai removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 25, 2023
@divipillai
Copy link
Contributor Author

Please add another Known issue: Mcuboot: If Fault Injection hardening (FIH)is enabled the bug in boot_image_check_hook() manifesting so multicore application can't be booted. nRF5340 mcuboot builds witch simultaneous multimage update enabled. Workaround are: disable FIH or patch #12846. tracking: NCSDK-24203

@nvlsianpu : Could you check and approve the PR.

@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 Oct 26, 2023
@divipillai
doc: update known issues for NCS v2.5.0
0c00c06 
Added v2.5.0 tag to known issues.

----

- [ ] Asset Tracker v2 done by @simensrostad
- [ ] Serial LTE Modem issue checked by @MarkusLassila
- [ ]  LTE Sensor Gateway issue checked by @lemrey (NCSDK-20026)
- [ ] Low Power UART sample issue on nRF5340 DK
      by @nordic-krch and @anangl
- [ ] Thingy:53 issue checked by @MarekPieta
- [ ] Thread issues checked by @bpienk
- [ ] Zigbee issues checked by @sebastiandraus
- [ ] Matter issues checked by @kkasperczyk-no
- [ ] HomeKit issues checked by @annwoj and @mariuszpos
- [ ] nRF Desktop, CAF, nRF Machine Learning checked by @MarekPieta
- [ ] nRF Audio (including Controller) checked
      by @rick1082 and @alexsven
- [ ] CAF issues checked by @MarekPieta
- [ ] NCS modem libraries and Networking libraries issues
      checked by @simensrostad
- [ ] Sidewalk issues checked by @annwoj
- [ ] BLE issue checked by @grochu
- [ ] BT Mesh issue checked by @Balaklaka
- [ ] Build system issue checked by @osaether
- [ ] DFU and FOTA issues checked by @osaether
- [ ] NFC issues checked by @grochu
- [ ] SPM issues checked by @osaether
- [ ] Crypto issues checked by @frkv and @joerchan
- [ ] MPSL issues checked by @laxiLang
- [ ] Radio driver issues checked by @ankuns
- [ ] SD Controller issues checked by @laxiLang
- [ ] TF-M done by @SebastianBoe
- [ ] Radio test sample issue checked by @KAGA164 and @grochu
- [ ] nRFxlib Modem library issues checked by @lemrey
- [ ] Zephyr issues checked by @de-nordic
- [ ] Zephyr LwM2M issues checked by @SeppoTakalo

Signed-off-by: divya pillai <[email protected]>

doc: doc review

doc review

Signed-off-by: divya pillai <[email protected]>

doc: doc review

doc review

Signed-off-by: divya pillai <[email protected]>
@divipillai divipillai removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 26, 2023
@cvinayak cvinayak merged commit 6b31313 into nrfconnect:main Oct 26, 2023
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KAGA164 KAGA164 KAGA164 approved these changes

@MarekPieta MarekPieta MarekPieta approved these changes

@peknis peknis peknis approved these changes

@Vge0rge Vge0rge Vge0rge approved these changes

@annwoj annwoj annwoj approved these changes

@mia-ko mia-ko mia-ko approved these changes

@kapi-no kapi-no kapi-no approved these changes

@alstrzebonski alstrzebonski alstrzebonski approved these changes

@nordic-hani nordic-hani nordic-hani approved these changes

@nvlsianpu nvlsianpu nvlsianpu approved these changes

@osaether osaether osaether approved these changes

@greg-fer greg-fer greg-fer approved these changes

@mariuszpos mariuszpos mariuszpos approved these changes

@kkasperczyk-no kkasperczyk-no kkasperczyk-no approved these changes

@tomi-font tomi-font tomi-font approved these changes

@joerchan joerchan joerchan approved these changes

@MarkusLassila MarkusLassila MarkusLassila approved these changes

@ankuns ankuns ankuns approved these changes

@laxiLang laxiLang laxiLang approved these changes

@umapraseeda umapraseeda umapraseeda approved these changes

@bpienk bpienk bpienk approved these changes

@glarsennordic glarsennordic glarsennordic approved these changes

@erikrobstad erikrobstad erikrobstad approved these changes

@cvinayak cvinayak cvinayak approved these changes

@SeppoTakalo SeppoTakalo SeppoTakalo approved these changes

@sebastiandraus sebastiandraus sebastiandraus approved these changes

@carlescufi carlescufi Awaiting requested review from carlescufi

@SebastianBoe SebastianBoe Awaiting requested review from SebastianBoe

@lemrey lemrey Awaiting requested review from lemrey

@grochu grochu Awaiting requested review from grochu

@frkv frkv Awaiting requested review from frkv

@alexsven alexsven Awaiting requested review from alexsven

@anangl anangl Awaiting requested review from anangl

@b-gent b-gent Awaiting requested review from b-gent

@rick1082 rick1082 Awaiting requested review from rick1082

@Balaklaka Balaklaka Awaiting requested review from Balaklaka

Assignees
No one assigned
Labels
CI-disable Disable CI for this PR doc-required PR must not be merged without tech writer approval.
Projects
None yet
Milestone
2.5.0
Development

Successfully merging this pull request may close these issues.