uart: nrfx_uarte: babblesim address conversion error

[JordanYates]

Describe the bug

UART instances initialize the timer_regs variable either from a hardware address via a timer property reference or directly from a NRF_TIMER{N} macro, if a CONFIG_UART_0_NRF_HW_ASYNC_TIMER kconfig exists.

zephyr/drivers/serial/uart_nrfx_uarte.c

Lines 3173 to 3177 in 8fc67e6

	.timer_regs = COND_CODE_1(UARTE_HAS_PROP(idx, timer), \
	(UARTE_TIMER_REG(idx)), \
	(COND_CODE_1(CONFIG_UART_##idx##_NRF_HW_ASYNC, \
	(NRFX_CONCAT(NRF_TIMER, CONFIG_UART_##idx##_NRF_HW_ASYNC_TIMER)), \
	(NULL)))), \

For babblesim, these two addresses are not equivalent, NRF_TIMER{N} is a pointer to the simulated peripheral in RAM, the hardware address is some invalid memory on the simulated platform. The driver attempts to map the hardware address back to the simulated peripheral:

zephyr/drivers/serial/uart_nrfx_uarte.c

Lines 3116 to 3119 in 8fc67e6

	if (cfg->timer_regs != NULL) {
	((struct uarte_nrfx_config *)dev->config)->timer_regs =
	nhw_convert_periph_base_addr(cfg->timer_regs);
	}

But in the case where the pointer is already the simulated peripheral, this fails, instead setting cfg->timer_regs to NULL.

Regression

• This is a regression.

Steps to reproduce

No response

Relevant log output

d_03: @00:00:00.000000 ERROR: (modules/bsim_hw_models/nrf_hw_models/src/HW_models/NHW_misc.c:85): Could not find real peripheral addr 0x81c31d8

Impact

Annoyance – Minor irritation; no significant impact on usability or functionality.

Environment

• Zephyr v4.4.0

Additional Context

Introduced in 6e65fbe

[aescolar]

@JordanYates how to repro?

[JordanYates]

@JordanYates how to repro?

I had a serial loopback test for the nrf52 bsim target with the following configs:

CONFIG_SERIAL=y
CONFIG_UART_ASYNC_API=y
CONFIG_UART_0_INTERRUPT_DRIVEN=n
CONFIG_UART_0_ASYNC=y
CONFIG_UART_0_NRF_HW_ASYNC=y
CONFIG_UART_0_NRF_HW_ASYNC_TIMER=1

[aescolar]

cmake -GNinja -DBOARD=nrf52_bsim ../tests/drivers/uart/uart_async_api/ -DCONFIG_UART_0_INTERRUPT_DRIVEN=n -DCONFIG_UART_0_ASYNC=y -DCONFIG_UART_0_NRF_HW_ASYNC=y -DCONFIG_UART_0_NRF_HW_ASYNC_TIMER=1 
ninja
zephyr/zephyr.exe -uart0_loopback -uart1_loopback

[nordic-krch]

Created #108504 to use register value from DT instead of NRF_TIMERx.