RE: BASIC UART CLI.

WilliamMcCoy21 — Thu, 05 Oct 2023 14:47:58 GMT

Hi Jorgen,

Thanks i see what you mean, my problem with this is that once the end of the buffer is reached the offset and length wont match the command correctly. Im maybe just being picky and setting a large buffer length will sort that issue for me.

Regards,

William

RE: BASIC UART CLI.

Jørgen Holmefjord — Thu, 05 Oct 2023 14:32:47 GMT

Hi,

[quote user=""]How can I clear the buffer, length and offset? i had set to 0 and "" but on next callback the values are back.[/quote]

This is how how the event structs work. The struct will be filled with the correct information for the event each time the event is generated.

You should rather focus only on the part of the rx.buf where you expect the string to be (if you are able to receive the full string each time, it should be between the offset and len parameters). You should enable some more logging in your application to see what parameters you are sending, which part of the buffer you are using, etc.

Best regards,
Jørgen

RE: BASIC UART CLI.

WilliamMcCoy21 — Thu, 05 Oct 2023 09:21:40 GMT

This is the code so far for reference

* SPDX-License-Identifier: Apache-2.0

/* Controlling LEDs through UART. Press 1-3 on your keyboard to toggle LEDS 1-3 on your development kit */

#include <string.h>

#include <zephyr/kernel.h>

#include <zephyr/device.h>

#include <zephyr/devicetree.h>

#include <zephyr/drivers/gpio.h>

#include <zephyr/sys/printk.h>

/* STEP 3 - Include the header file of the UART driver in main.c */

#include <zephyr/drivers/uart.h>

/* 1000 msec = 1 sec */

#define SLEEP_TIME_MS 1000

/* STEP 10.1.1 - Define the size of the receive buffer */

#define RECEIVE_BUFF_SIZE 255

/* STEP 10.2 - Define the receiving timeout period */

#define RECEIVE_TIMEOUT 10000

/* STEP 5.1 - Get the device pointers of the LEDs through gpio_dt_spec */

/* The nRF7002dk has only 2 LEDs so this step uses a compile-time condition to reflect the DK you are building for */

#if defined (CONFIG_BOARD_NRF7002DK_NRF5340_CPUAPP)|| defined (CONFIG_BOARD_NRF7002DK_NRF5340_CPUAPP_NS)

static const struct gpio_dt_spec led0 = GPIO_DT_SPEC_GET(DT_ALIAS(led0), gpios);

static const struct gpio_dt_spec led1 = GPIO_DT_SPEC_GET(DT_ALIAS(led1), gpios);

#else

static const struct gpio_dt_spec led0 = GPIO_DT_SPEC_GET(DT_ALIAS(led0), gpios);

static const struct gpio_dt_spec led1 = GPIO_DT_SPEC_GET(DT_ALIAS(led1), gpios);

static const struct gpio_dt_spec led2 = GPIO_DT_SPEC_GET(DT_ALIAS(led2), gpios);

#endif

/* STEP 4.1 - Get the device pointer of the UART hardware */

const struct device *uart= DEVICE_DT_GET(DT_NODELABEL(uart0));

/* STEP 9.1 - Define the transmission buffer, which is a buffer to hold the data to be sent over UART */

static uint8_t tx_buf[] = {"nRF Connect SDK Fundamentals Course\n\r"

"Press 1-3 on your keyboard to toggle LEDS 1-3 on your development kit\n\r"};

/* STEP 10.1.2 - Define the receive buffer */

static uint8_t rx_buf[RECEIVE_BUFF_SIZE] = {0};

static uint8_t CLI_rx_buf[RECEIVE_BUFF_SIZE] = {0};

int find_data(const uint8_t *data, const char *target) {

int size = sizeof(data);

int target_size = strlen(target);

for (int i = 0; i < size - target_size + 1; i++) {

if (memcmp(data + i, target, target_size) == 0) {

return i; // Return the index where the target data was found

}

return -1; // Return -1 if the target data was not found in the array

}

/* STEP 7 - Define the callback function for UART */

static void uart_cb(const struct device *dev, struct uart_event *evt, void *user_data)

{

int ret = 0;

switch (evt->type) {

case UART_RX_RDY:

#if defined (CONFIG_BOARD_NRF7002DK_NRF5340_CPUAPP)|| defined (CONFIG_BOARD_NRF7002DK_NRF5340_CPUAPP_NS)

if((evt->data.rx.len) == 1){

if(evt->data.rx.buf[evt->data.rx.offset] == '1')

gpio_pin_toggle_dt(&led0);

else if (evt->data.rx.buf[evt->data.rx.offset] == '2')

gpio_pin_toggle_dt(&led1);

}

#else

if((evt->data.rx.len>0)&&((evt->data.rx.buf[evt->data.rx.len+evt->data.rx.offset-1]=='\n')&&(evt->data.rx.buf[evt->data.rx.len+evt->data.rx.offset-2]=='\r'))){

printk("\r\nUART message Received\r\n");

ret = find_data(evt->data.rx.buf,"scan");

evt->data.rx.buf = "";

evt->data.rx.len = 0;

evt->data.rx.offset = 0;

}

#endif

break;

case UART_RX_DISABLED:

uart_rx_enable(dev ,rx_buf,sizeof rx_buf,RECEIVE_TIMEOUT);

break;

default:

break;

}

int main(void)

{

int ret;

/* STEP 4.2 - Verify that the UART device is ready */

if (!device_is_ready(uart)){

printk("UART device not ready\r\n");

return 1 ;

}

/* STEP 5.2 - Verify that the LED devices are ready */

if (!device_is_ready(led0.port)){

printk("GPIO device is not ready\r\n");

return 1;

}

/* STEP 6 - Configure the GPIOs of the LEDs */

#if defined (CONFIG_BOARD_NRF7002DK_NRF5340_CPUAPP)|| defined (CONFIG_BOARD_NRF7002DK_NRF5340_CPUAPP_NS)

ret = gpio_pin_configure_dt(&led0, GPIO_OUTPUT_ACTIVE);

if (ret < 0) {

return 1 ;

}

ret = gpio_pin_configure_dt(&led1, GPIO_OUTPUT_ACTIVE);

if (ret < 0) {

return 1 ;

}

#else

ret = gpio_pin_configure_dt(&led0, GPIO_OUTPUT_ACTIVE);

if (ret < 0) {

return 1 ;

}

ret = gpio_pin_configure_dt(&led1, GPIO_OUTPUT_ACTIVE);

if (ret < 0) {

return 1 ;

}

ret = gpio_pin_configure_dt(&led2, GPIO_OUTPUT_ACTIVE);

if (ret < 0) {

return 1 ;

}

#endif

/* STEP 8 - Register the UART callback function */

ret = uart_callback_set(uart, uart_cb, NULL);

if (ret) {

return 1;

}

/* STEP 9.2 - Send the data over UART by calling uart_tx() */

ret = uart_tx(uart, tx_buf, sizeof(tx_buf), SYS_FOREVER_MS);

if (ret) {

return 1;

}

/* STEP 10.3 - Start receiving by calling uart_rx_enable() and pass it the address of the receive buffer */

ret = uart_rx_enable(uart ,rx_buf,sizeof rx_buf,RECEIVE_TIMEOUT);

if (ret) {

return 1;

}

while (1) {

k_msleep(SLEEP_TIME_MS);

}