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

[nrf fromtree] manifest: Update nrf hw models to latest including UBSAN fixes #1484

Conversation

ivaniushkov
Copy link
Contributor

@ivaniushkov ivaniushkov commented Feb 8, 2024

  • Update the HW models module to 52d0b4b7b7431d8da6222cc3b17a8afdcb099baf
    Including the following:
    • 52d0b4b UART: FIFO backend: Do not error out if other side disconnects Rx
    • 3582b68 UART: FIFO backend: Avoid possible race
    • 414f160 AAR: Fix UBSAN warnings
    • 24f5d3d PPI: Fix UBSAN warning
    • d17c9d7 UART: Bugfix: Support STARTRX while Rx is stopping
    • d982e76 UART: Minor bugfix: Fix 2 warning messages
    • eda7af9 UART: Bugfix: Handle task STOPTX while stopping
    • b8ea63b UART: BugFix: Support STARTTx even if Tx is not Off
    • c0d28cc nrf_common: Add replacement for nrf_dma_accessible_check()
    • feb91bb nrfx_common replacement: nrfx_get_irq_number() Add missing peri
    • c95d9af Makefile: By default lets build both 52833 and 5340 targets
    • cf41110 UART: FIFO backend: Check for failure of fcntl
    • 6b6ae36 UART: Minor optimization
    • afe84c1 fake timer: Minor speed optimization for nrf52
    • a5a4dfd RTC: Fix counter value when CLEAR after STOP
    • 6b6ae36 UART: Minor optimization
    • afe84c1 fake timer: Minor speed optimization for nrf52
    • a5a4dfd RTC: Fix counter value when CLEAR after STOP
    • a08acc7 UART(E): Duty cycle warnings due to Rx while Rx Off
    • aa1f38d UART(E): FIFO backend Minor bugfix
    • 8b0819f UART(E): Add Loopback backend
    • bc648f2 UART(E): Update RTS pin level as soon as the conf is changed
    • b39e448 UART(E): Detect receiver opening mid frame
    • docs: UART: several minor fixes
    • UART(E): Add peripherals and connect to (D)PPI

@ivaniushkov ivaniushkov force-pushed the cherry_pick_manifest_bsim_nrf_hw_models_update branch from 027c2ab to 5b776d7 Compare February 8, 2024 16:20
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Feb 8, 2024

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
nrf_hw_models zephyrproject-rtos/nrf_hw_models@9b985ea zephyrproject-rtos/nrf_hw_models@52d0b4b (main) zephyrproject-rtos/nrf_hw_models@9b985ea6..52d0b4b7

Note: This message is automatically posted and updated by the Manifest GitHub Action.

Copy link
Contributor

@aescolar aescolar left a comment

Choose a reason for hiding this comment

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

Please do a cherry-pick from Zephyr with -x (so the "cherry-picked from" line is there)
And pick all commits, do not squash them together.

85d3ad1
992c04c
33d4b6d
c9d489d
189d4d4

@ivaniushkov ivaniushkov force-pushed the cherry_pick_manifest_bsim_nrf_hw_models_update branch from a03f584 to b191c0f Compare February 9, 2024 09:46
@aescolar aescolar self-requested a review February 9, 2024 09:51
@ivaniushkov ivaniushkov force-pushed the cherry_pick_manifest_bsim_nrf_hw_models_update branch from b191c0f to 822ec3a Compare February 9, 2024 09:52
@ivaniushkov
Copy link
Contributor Author

CI in manifest PR is green:
nrfconnect/sdk-nrf#13934

Copy link
Contributor

@ilhanates ilhanates left a comment

Choose a reason for hiding this comment

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

Can u point me a test-ble build to verify changes?
I could not see it here https://jenkins-ncs.nordicsemi.no/job/latest/job/sdk-nrf/job/PR-13934/

* Update the HW models module to
c744f2c762aad79e59bd7e99002f2fcab0a2f288

Including the following:
* docs: UART: several minor fixes
* UART(E): Add peripherals and connect to (D)PPI

Signed-off-by: Alberto Escolar Piedras <alberto.escolar.piedras@nordicsemi.no>
(cherry picked from commit 85d3ad1)
Signed-off-by: Ivan Iushkov <ivan.iushkov@nordicsemi.no>
* Update the HW models module to
a08acc7d3a853f890df2bf82167c02ab2153ffe7

Including the following:
* a08acc7 UART(E): Duty cycle warnings due to Rx while Rx Off
* aa1f38d UART(E): FIFO backend Minor bugfix
* 8b0819f UART(E): Add Loopback backend
* bc648f2 UART(E): Update RTS pin level as soon as the conf is changed
* b39e448 UART(E): Detect receiver opening mid frame

Signed-off-by: Alberto Escolar Piedras <alberto.escolar.piedras@nordicsemi.no>
(cherry picked from commit 992c04c)
Signed-off-by: Ivan Iushkov <ivan.iushkov@nordicsemi.no>
* Update the HW models module to
6b6ae3652c92c95edd945dce6ad9ef9892aab89b

Including the following:
* 6b6ae36 UART: Minor optimization
* afe84c1 fake timer: Minor speed optimization for nrf52
* a5a4dfd RTC: Fix counter value when CLEAR after STOP

Signed-off-by: Alberto Escolar Piedras <alberto.escolar.piedras@nordicsemi.no>
(cherry picked from commit 33d4b6d)
Signed-off-by: Ivan Iushkov <ivan.iushkov@nordicsemi.no>
* Update the HW models module to
d17c9d749c817d2522468fa3c0eee3705feb8951

Including the following:
* d17c9d7 UART: Bugfix: Support STARTRX while Rx is stopping
* d982e76 UART: Minor bugfix: Fix 2 warning messages
* eda7af9 UART: Bugfix: Handle task STOPTX while stopping
* b8ea63b UART: BugFix: Support STARTTx even if Tx is not Off
* c0d28cc nrf_common: Add replacement for nrf_dma_accessible_check()
* feb91bb nrfx_common replacement: nrfx_get_irq_number() Add missing peri
* c95d9af Makefile: By default lets build both 52833 and 5340 targets
* cf41110 UART: FIFO backend: Check for failure of fcntl

Signed-off-by: Alberto Escolar Piedras <alberto.escolar.piedras@nordicsemi.no>
(cherry picked from commit c9d489d)
Signed-off-by: Ivan Iushkov <ivan.iushkov@nordicsemi.no>
* Update the HW models module to
52d0b4b7b7431d8da6222cc3b17a8afdcb099baf

Including the following:
* 52d0b4b UART: FIFO backend: Do not error out if other side disconnects Rx
* 3582b68 UART: FIFO backend: Avoid possible race
* 414f160 AAR: Fix UBSAN warnings
* 24f5d3d PPI: Fix UBSAN warning

Signed-off-by: Alberto Escolar Piedras <alberto.escolar.piedras@nordicsemi.no>
(cherry picked from commit 189d4d4)
Signed-off-by: Ivan Iushkov <ivan.iushkov@nordicsemi.no>
@rlubos rlubos merged commit 7e22d68 into nrfconnect:main Feb 12, 2024
15 checks passed
@ivaniushkov ivaniushkov deleted the cherry_pick_manifest_bsim_nrf_hw_models_update branch February 12, 2024 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants