zara/zephyr - zephyr - Gitea: Git with a cup of tea

Author	SHA1	Message	Date
Carles Cufi	8c748fd005	kernel: Modify the signature of k_mem_slab_free() Modify the signature of the k_mem_slab_free() function with a new one, replacing the old void *mem with void mem as a parameter. The following function: void k_mem_slab_free(struct k_mem_slab slab, void mem); has the wrong signature. mem is only used as a regular pointer, so there is no need to use a double-pointer. The correct signature should be: void k_mem_slab_free(struct k_mem_slab slab, void *mem); The issue with the current signature, although functional, is that it is extremely confusing. I myself, a veteran Zephyr developer, was confused by this parameter when looking at it recently. All in-tree uses of the function have been adapted. Fixes #61888. Signed-off-by: Carles Cufi <carles.cufi@nordicsemi.no>	2 years ago
Jordan Yates	ec362d4a48	modem: modem_iface_uart_async: handle RX errors Re-enable reception on the UART instance when `UART_RX_DISABLED` triggers, which can happen when errors occur on the line. This stops the driver permanently moving into an unresponsive state. Signed-off-by: Jordan Yates <jordan.yates@data61.csiro.au>	2 years ago
Bjarki Arge Andreasen	a4afcf8c93	drivers/modem/modem_iface_uart: Update API The UART IFACE API currently exposes the context struct modem_iface_uart_data, expecting the user to fill in some of the fields, with no documentation specifying which fields and what they mean. This API update moves all user configurable values in the context out into a config struct, modem_iface_uart_config, within which members are documented. This prevents the user from interacting directly with the context making use of the library safer and more readable. The config structure helps make code readable by using "named args" in a const struct instead of a long, nameless, parameter list passed to the modem_iface_uart_init function. The new API function modem_iface_uart_rx_wait is added to prevent the user from having to interact with the rx sem in the context directly. The context can now be safely interated with without direct access to any of its members. Pointers to the ring buffer params in the context have been moved to config struct, as these are only useful during initialization of the context. Signed-off-by: Bjarki Arge Andreasen <baa@trackunit.com>	2 years ago
Jordan Yates	95d8deb572	modem: modem_iface_uart_async: added Adds a communications backend based on the asynchronous UART API, instead of the interrupt-driven UART API. The primary advantage of this backend is an improved robustness to dropping bytes under high interrupt or critical section loads. Under all loads system efficiency is improved by: * Reducing the time spent writing out individual bytes. * Reducing the number of UART interrupts fired. * Waking up the RX thread much less often. When utilising this backend over `nordic,nrf-uarte` on a nRF52840, the baudrate of an esp-at modem could be pushed to at least 921600 without dropping bytes, compared to a maximum of 230400 with the interrupt API. Signed-off-by: Jordan Yates <jordan.yates@data61.csiro.au>	3 years ago