UART with openThread

RE: UART with openThread

Vidar Berg — Wed, 20 Oct 2021 08:56:48 GMT

Hi,

The issue is the pre-processor check in serial nrf_serial_init() - it should check if the legacy driver is actually enabled, not just if the legacy UART peripheral is present on the device.

The fix is to use the same check as in the nrf_drv_uart_config_t struct:

diff --git a/components/libraries/serial/nrf_serial.c b/components/libraries/serial/nrf_serial.c
index 730f4e74..c2e6f6bb 100644
--- a/components/libraries/serial/nrf_serial.c
+++ b/components/libraries/serial/nrf_serial.c
@@ -207,7 +207,7 @@ ret_code_t nrf_serial_init(nrf_serial_t const * p_serial,
     nrf_drv_uart_config_t drv_config;
     memcpy(&drv_config, p_drv_uart_config, sizeof(nrf_drv_uart_config_t));
     drv_config.p_context = (void *)p_serial;
-#if defined(UARTE_PRESENT) && defined(UART_PRESENT)
+#if defined(NRF_DRV_UART_WITH_UARTE) && defined(NRF_DRV_UART_WITH_UART)
     drv_config.use_easy_dma = (p_config->mode == NRF_SERIAL_MODE_DMA);
 #endif
     ret = nrf_drv_uart_init(&p_serial->instance,

I tested it with the following configuration:

// <q> UART_EASY_DMA_SUPPORT  - Driver supporting EasyDMA
 

#ifndef UART_EASY_DMA_SUPPORT
#define UART_EASY_DMA_SUPPORT 1
#endif

// <q> UART_LEGACY_SUPPORT  - Driver supporting Legacy mode
 

#ifndef UART_LEGACY_SUPPORT
#define UART_LEGACY_SUPPORT 0
#endif

// <e> UART0_ENABLED - Enable UART0 instance
//==========================================================
#ifndef UART0_ENABLED
#define UART0_ENABLED 0
#endif
// <q> UART0_CONFIG_USE_EASY_DMA  - Default setting for using EasyDMA
 

#ifndef UART0_CONFIG_USE_EASY_DMA
#define UART0_CONFIG_USE_EASY_DMA 0
#endif

// </e>

// <e> UART1_ENABLED - Enable UART1 instance
//==========================================================
#ifndef UART1_ENABLED
#define UART1_ENABLED 1
#endif
// </e>

// </e>

// </h>

Regards,

Vidar

RE: UART with openThread

ashish_shukla — Wed, 20 Oct 2021 04:32:35 GMT

Hi,

Do you have any update on this issue?

Regards,

Ashish Shukla

RE: UART with openThread

Sigurd Hellesvik — Thu, 14 Oct 2021 14:06:06 GMT

Aha, now I have reproduced this issue, thank you for clarifying.

I have not been able to fix it yet, but it is related to this answer.

To find a solution, I will talk to some of my colleagues, and return tomorrow.

Regards,
Sigurd Hellesvik

RE: UART with openThread

ashish_shukla — Thu, 14 Oct 2021 13:06:42 GMT

yes - the one shared by you.

#ifndef UART0_ENABLED
#define UART0_ENABLED 0
#endif

#ifndef UART1_ENABLED
#define UART1_ENABLED 1
#endif

#ifndef APP_UART_ENABLED
#define APP_UART_ENABLED 1
#endif

#ifndef APP_UART_DRIVER_INSTANCE
#define APP_UART_DRIVER_INSTANCE 1
#endif

RE: UART with openThread

Sigurd Hellesvik — Thu, 14 Oct 2021 13:02:12 GMT

[quote user="ashish_shukla"]To reproduce the issue, you can apply above settings to serial_uartes example.[/quote]

Can you post the settings you refer to, here?

Regards,
Sigurd Hellesvik

RE: UART with openThread

ashish_shukla — Thu, 14 Oct 2021 12:54:04 GMT

Hi,

I can't share the project, it's a project that we've deployed in the field. We're adding some new features into it.

Transition to nRF connect SDK will a big effort for us, we are considering that meanwhile.

To reproduce the issue, you can apply above settings to serial_uartes example. It's the same compilation issue that I'm facing in my project.

RE: UART with openThread

Sigurd Hellesvik — Thu, 14 Oct 2021 11:57:23 GMT

Could you upload a zip file with the project that fails?
This would make it easier to debug the problem.

If you are still early in your project, we recommend new OpenThread projects to use the nRF Connect SDK.
This is because new Thread features will be added to the nRF Connect SDK, and not the nRF5 SDK.

Regards,
Sigurd Hellesvik

RE: UART with openThread

Sigurd Hellesvik — Wed, 13 Oct 2021 13:50:18 GMT

I will add openthread to the "serial_uartes" example to try and recreate your issue.

This will take some time, so I will return here with my results tomorrow.

Regards,
Sigurd Hellesvik

RE: UART with openThread

ashish_shukla — Wed, 13 Oct 2021 13:35:47 GMT

Hi,

I'm familiar with serial_uartes example, built an application in past based on this.

With above settings nrf_serial.c gives compilation error -

nrf_serial.c:211:15

error: 'nrf_drv_uart_config_t {aka struct <anonymous>}' has no member named 'use_easy_dma'
drv_config.use_easy_dma = (p_config->mode == NRF_SERIAL_MODE_DMA);

Can you please try to compile this file at your end with open-thread ?

RE: UART with openThread

Sigurd Hellesvik — Wed, 13 Oct 2021 12:48:39 GMT

When I set the following in sdk_config.h of one of the thread examples, the example builds fine.
I found these in the userial_uartes example.

#ifndef UART_ENABLED
#define UART_ENABLED 1
#endif

#ifndef UART1_ENABLED
#define UART1_ENABLED 1
#endif

#ifndef NRFX_UARTE_ENABLED
#define NRFX_UARTE_ENABLED 1
#endif

#ifndef NRFX_UARTE1_ENABLED
#define NRFX_UARTE1_ENABLED 1
#endif

I suggest first testing the serial_uartes example, so you know how to use it. Then add relevant code and configuration to the thread example.

Regards,
Sigurd Hellesvik

RE: UART with openThread

ashish_shukla — Wed, 13 Oct 2021 12:32:09 GMT

I'm using UARTE peripheral.

Regards,

Ashish

RE: UART with openThread

Sigurd Hellesvik — Wed, 13 Oct 2021 12:14:52 GMT

Just to make sure: Are you using the UART peripheral or UARTE peripheral?

Regards,
Sigurd Hellesvik

RE: UART with openThread

ashish_shukla — Wed, 13 Oct 2021 12:00:27 GMT

Hi,

Thanks for your quick reply.

I'm using serial_uartes example within peripheral examples. Can you please suggest settings when UART is controlled by nrf_serial library ?

RE: UART with openThread

Sigurd Hellesvik — Wed, 13 Oct 2021 11:26:19 GMT

In this case, the easiest alternative could just be to use UART1 instead of UART0.

This depends on how you control the UART. But if you use the same method as in the UART sample, it should be the following settings:

#ifndef UART0_ENABLED
#define UART0_ENABLED 0
#endif

#ifndef UART1_ENABLED
#define UART1_ENABLED 1
#endif

#ifndef APP_UART_ENABLED
#define APP_UART_ENABLED 1
#endif

#ifndef APP_UART_DRIVER_INSTANCE
#define APP_UART_DRIVER_INSTANCE 1
#endif

Does this fix your issue?

Regards,
Sigurd Hellesvik