Nordic NRF SDK v1.6: Issue with AT_CMD_PARSER

RE: Nordic NRF SDK v1.6: Issue with AT_CMD_PARSER

mglettig — Wed, 30 Jun 2021 15:13:57 GMT

Hi Håkon

[quote userid="104198" url="~/f/nordic-q-a/76717/nordic-nrf-sdk-v1-6-issue-with-at_cmd_parser/317162#317162"]

If I don't get an answer I will create a new feature request. In that case I will keep you posted.

[/quote]

As I didn't hear anything in the old PR thread I created a new Issue in Zephyr: https://github.com/zephyrproject-rtos/zephyr/issues/36645

I hope somebody will add the functions to minimal libc.

Best regards,

Michael

RE: Nordic NRF SDK v1.6: Issue with AT_CMD_PARSER

mglettig — Fri, 25 Jun 2021 09:19:44 GMT

Hi Håkon

Thanks for the update. I agree with you. It would be good to have the support for strtoll/strtoull in minimal libc and then ged rid of the dependeny to new libc. I posted in this thread here: https://github.com/zephyrproject-rtos/zephyr/pull/16511

If I don't get an answer I will create a new feature request. In that case I will keep you posted.

Kind regards,

Michael

RE: Nordic NRF SDK v1.6: Issue with AT_CMD_PARSER

Håkon Alseth — Fri, 25 Jun 2021 07:32:36 GMT

Hi Michael,

I reproduced this by compiling zephyr/samples/net/civetweb/http_server/ and setting newlibc.

Using civetweb with newlib looks to be a hard incompatibility, as per this issue:

https://github.com/zephyrproject-rtos/zephyr/issues/16683

It looks like the simplest path is to get strtoll/strtoull support for minimal libc, then remove this dep:

https://github.com/nrfconnect/sdk-nrf/blob/master/lib/at_cmd_parser/Kconfig#L9

Unfortunately, both needs to be addressed in the zephyr project. Have you tried creating an issue on their github?

https://github.com/zephyrproject-rtos/zephyr/issues

Kind regards,

Håkon

RE: Nordic NRF SDK v1.6: Issue with AT_CMD_PARSER

mglettig — Thu, 24 Jun 2021 12:51:45 GMT

Hi Håkon

Thanks for pointing out the reason for using NEW_LIBC. If I try to compile with NEW_LIBC I get errors about multiple definitions of functions and variables related to POSIX threads.

Please have a look at the following log-file to see the details:

devzone.nordicsemi.com/.../buildErrorNewLib.log

We need CONFIG_POSIX_API=y and CONFIG_PTHREAD_IPC=y as well as CONFIG_POSIX_MQUEUE=y in our application because we are using Civetweb (webserver).

Do you have any ideas on how to go ahead with NEW_LIBC?

Kind regards,

Michael

RE: Nordic NRF SDK v1.6: Issue with AT_CMD_PARSER

Håkon Alseth — Thu, 24 Jun 2021 11:43:24 GMT

Hi,

The modem can return values > 32 bit, so strtoll() is required, which minimal libc lacks support for at this time.

There has been work on this in the upstream zephyr, but it seems to be stale:

https://github.com/zephyrproject-rtos/zephyr/pull/16511

[quote user=""]The compile issues have to do with multiple definitions of POSIX functions.[/quote]

Is there any specific functions that clashes?

Kind regards,

Håkon