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

H20 batch 0 #2385

Merged
merged 31 commits into from
Dec 15, 2024
Merged

H20 batch 0 #2385

merged 31 commits into from
Dec 15, 2024

Conversation

nordic-krch and others added 28 commits December 14, 2024 15:05
…uency change

Update SystemCoreClock to match current core frequency. This MDK
variable is used by nrfx_coredep_delay_us which performs busy
waiting and is used by k_busy_wait.

Upstream PR #: 82923

Signed-off-by: Krzysztof Chruściński <[email protected]>
Add clock source for timer120 and timer121 nodes

Upstream PR #: 82103

Signed-off-by: Krzysztof Chruściński <[email protected]>
…FSLL

Add requesting for global HFSLL when fast instance TIMER (e.g.
TIMER120 in nrf54h20) is used.

Upstream PR #: 82103

Signed-off-by: Krzysztof Chruściński <[email protected]>
spi-max-frequency was missing in overlay

Now it's set to 8 MHz as per datasheet

Signed-off-by: Jerzy Kasenberg <[email protected]>
(cherry picked from commit 1d4b1ad)
Add support for GRTC clock output pins.

Upstream PR #: 82788

Signed-off-by: Adam Kondraciuk <[email protected]>
Add support for GRTC clock output pins.

Upstream PR #: 82788

Signed-off-by: Adam Kondraciuk <[email protected]>
Add support for GRTC clock output pins.

Upstream PR #: 82788

Signed-off-by: Adam Kondraciuk <[email protected]>
Fix failing tests for nRF54H20 targets.

Upstream PR #: 82252

Signed-off-by: Adam Kondraciuk <[email protected]>
…ing function

This change ensures that LR will not be implicitly used when calling
`pm_s2ram_mark_check_and_clear`.

Upstream PR #: 81853

Signed-off-by: Adam Kondraciuk <[email protected]>
…artitions

This is possible due to the memory map changes introduced in
26603ce

Signed-off-by: Håkon Amundsen <[email protected]>
(cherry picked from commit 8a82b07)
Adding support for reset reasons in the nRF54H20 SoC.

Upstream PR #: 81751

Signed-off-by: Karol Lasończyk <[email protected]>
IAD must be before the interfaces it associates and therefore there is
no need for the class to be in charge of updating the bFirstInterface.
Update IAD in common initialization code and remove the updates from
classes.

This fixes UAC2 instances where the IAD bFirstInterface is not 0, e.g.
when HID was used together with UAC2.

Signed-off-by: Tomasz Moń <[email protected]>
(cherry picked from commit 9006780)
Do not allow enqueuing buffers on endpoints that were not enabled. Doing
so can lead to division by zero later on because the max packet size can
be 0 in disabled endpoint configuration.

Signed-off-by: Tomasz Moń <[email protected]>
(cherry picked from commit d39d7e5)
…ronous request"

This reverts commit a8bb747.

Signed-off-by: Bjarki Arge Andreasen <[email protected]>
…in progres API

It may be required to get information if NRF LF clock control calibration
is in progress. Some time sensitive operations could benefit from this
information.

The commit adds simple function that provides the information.
The function is nRF platform specific.

Signed-off-by: Piotr Pryga <[email protected]>
(cherry picked from commit fefc285)
…equest

Add API for synchronous request for clock attributes.

Signed-off-by: Krzysztof Chruściński <[email protected]>
(cherry picked from commit fe0e2db)
b6d4542 Added support for pin retention but it was added
only to the case when device PM is used. There is another mode
in which UARTE is disabled when idle (low power mode) and in
that case pin retention must also be added.

Signed-off-by: Krzysztof Chruściński <[email protected]>
(cherry picked from commit 760bba6)
…upt driven API

TX part of interrupt driven API was not calling PM device runtime
API. Additionally, when txstopped occurred after poll out it was
not handled correctly.

Signed-off-by: Krzysztof Chruściński <[email protected]>
(cherry picked from commit 13ed9e4)
…rgets

Or more precisely, do not remove the workaround for them.
The current HW models are accurate enough in this area for the
workaround to work properly.

Signed-off-by: Alberto Escolar Piedras <[email protected]>
(cherry picked from commit fedda6d)
…M mode

When fast UARTE instance is used (e.g. UARTE120 in nrf54h20), PM actions
are not ISR safe because they include communication over IPC so they can
only be called from the thread context. Extend driver to support both
PM modes. When non ISR mode is used then uart_rx_enable() and uart_tx()
will return error if they are called from ISR and resume operation
would need to be called because device is suspended. On completion,
driver is calling pm_device_runtime_put_async which can be called from
the ISR context.

Additionally, suspending in the TXSTOPPED and RXTO events has been
moved after user callback. It allows to support the case where
uart_rx_enable() or uart_tx() are called from that callback context.
Since suspending is called after returning from the callback it will
not trigger suspend action because API called in the callback context
will increment the usage counter (when pm_device_runtime_get() is
called).

Signed-off-by: Krzysztof Chruściński <[email protected]>
(cherry picked from commit c8d50e4)
…ast instance

Request fast global domain to run at 320 MHz during fast UARTE
activity. As request is asynchronous it cannot be called from
an ISR. Due to complexity to handle that without device runtime
power management a requirement is added so that if fast UARTE
is used device runtime PM must be enabled. Clock is request
and released in PM resume and suspended actions which in case
of fast UARTE are only called from thread context.

Upstream PR #: 82103

Signed-off-by: Krzysztof Chruściński <[email protected]>
1. ensure the two dma buffers all aligned with 32 bits
2. clean the rx_data_idx at test begin

Signed-off-by: Hake Huang <[email protected]>
(cherry picked from commit 193bfab)
…yword

Multiple variable in the test were missing static keyword.

Signed-off-by: Krzysztof Chruściński <[email protected]>
(cherry picked from commit bc007fd)
…ance

Rework the test to be able to run on multiple instances.

Signed-off-by: Krzysztof Chruściński <[email protected]>
(cherry picked from commit d2b69a4)
…to nrf54h20dk

Add second instance to be tested on nrf54h20dk. uart120 is a fast UARTE
which works on fixed pin locations. It is not available for cpuppr core.

Signed-off-by: Krzysztof Chruściński <[email protected]>
(cherry picked from commit f235ddb)
…vice PM

Enable device runtime PM for nrf54h20dk/nrf54h20/cpuapp where
fast UARTE (uart120) is tested.

Upstream PR #: 82103

Signed-off-by: Krzysztof Chruściński <[email protected]>
…rror

The fix 25173f7 created
`struct pm_device_base` and moved `usage` to it.
This has been overlooked so that I fixed it.

Signed-off-by: TOKITA Hiroshi <[email protected]>
(cherry picked from commit a511f51)
…endency

Since "power_domain,gpio_monitor" depends on PM_DEVICE,
specify it in Kconfig.

Signed-off-by: TOKITA Hiroshi <[email protected]>
(cherry picked from commit 663c7bc)
jonathannilsen and others added 2 commits December 14, 2024 17:25
…for SDFW v8.0.0"

No longer needed with SDFW v9.0.0 and more recent nrfutil device versions.

This reverts commit f3e33c4.

Signed-off-by: Jonathan Nilsen <[email protected]>
…for getting clock frequency"

This reverts commit 91c801c.

Signed-off-by: Bjarki Arge Andreasen <[email protected]>
Copy link

@shanthanordic shanthanordic left a comment

Choose a reason for hiding this comment

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

approved for scope

Copy link
Contributor

@anangl anangl left a comment

Choose a reason for hiding this comment

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

Last commit (67da08b) is missing the "cherry picked from" line, hence the commit tags check fails. Otherwise, LGTM.

…ing clock frequency

Add NRF_PERIPH_GET_FREQUENCY() that returns the frequency of clock
that is used for the node.

Upstream PR #: 82133

Signed-off-by: Krzysztof Chruściński <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.