UART: Unable to transmit data

[frosticles]

I'm using an MTK3339 GPS module with UART1 in pins 16 (RX) & 17 (TX). I can read data coming from the GPS module with uart_read_bytes() on UART1, but I can't get uart_write_bytes() working to send commands to the module (see: https://cdn-shop.adafruit.com/datasheets/PMTK_A11.pdf). I've tried sending data as a char array, and also sending the <CR><LF> as litterally "<CR><LF>" or \r\n which I believe mean the same thing. Neither have worked so far, so hopefully someone smarter than me can tell me what I'm doing wrong .

Code: Select all

    
    uart_config_t uart_config = {
        .baud_rate = 9600,
        .data_bits = UART_DATA_8_BITS,
        .parity = UART_PARITY_DISABLE,
        .stop_bits = UART_STOP_BITS_1,
        .flow_ctrl = UART_HW_FLOWCTRL_DISABLE,
        .rx_flow_ctrl_thresh = 122,
    };
    
    uart_param_config(UART_NUM_1, &uart_config);
    uart_set_pin(UART_NUM_1, 17, 16, UART_PIN_NO_CHANGE, UART_PIN_NO_CHANGE);
    uart_driver_install(UART_NUM_1, BUF_SIZE * 2, 0, 0, NULL, 0);
    
    uint8_t* writeBuf = (uint8_t*) "$PMTK220,10000*2F\r\n";
    uart_write_bytes(UART_NUM_1, (const char*)writeBuf, 0);

    unsigned char buf[BUF_SIZE];
    while(1) {
        int size = uart_read_bytes(UART_NUM_1, buf, BUF_SIZE, 1000 / portTICK_RATE_MS);
        if (size >0) {
          printf("%s", buf);
        }
    }

[Ritesh]

I'm using an MTK3339 GPS module with UART1 in pins 16 (RX) & 17 (TX). I can read data coming from the GPS module with uart_read_bytes() on UART1, but I can't get uart_write_bytes() working to send commands to the module (see: https://cdn-shop.adafruit.com/datasheets/PMTK_A11.pdf). I've tried sending data as a char array, and also sending the <CR><LF> as litterally "<CR><LF>" or \r\n which I believe mean the same thing. Neither have worked so far, so hopefully someone smarter than me can tell me what I'm doing wrong .

Code: Select all

    
    uart_config_t uart_config = {
        .baud_rate = 9600,
        .data_bits = UART_DATA_8_BITS,
        .parity = UART_PARITY_DISABLE,
        .stop_bits = UART_STOP_BITS_1,
        .flow_ctrl = UART_HW_FLOWCTRL_DISABLE,
        .rx_flow_ctrl_thresh = 122,
    };
    
    uart_param_config(UART_NUM_1, &uart_config);
    uart_set_pin(UART_NUM_1, 17, 16, UART_PIN_NO_CHANGE, UART_PIN_NO_CHANGE);
    uart_driver_install(UART_NUM_1, BUF_SIZE * 2, 0, 0, NULL, 0);
    
    uint8_t* writeBuf = (uint8_t*) "$PMTK220,10000*2F\r\n";
    uart_write_bytes(UART_NUM_1, (const char*)writeBuf, 0);

    unsigned char buf[BUF_SIZE];
    while(1) {
        int size = uart_read_bytes(UART_NUM_1, buf, BUF_SIZE, 1000 / portTICK_RATE_MS);
        if (size >0) {
          printf("%s", buf);
        }
    }

Hi,

I have looked your code snippet and found issue in your uart_write_bytes call which you have called in your example.

The last argument for uart_write_bytes call is how many data you want to send over UART on which you are transmitting your data.

You have passed it 0 means nothing will be sent over UART as length of data you want send will be passed it as 0 in last argument.

So, Please pass correct bytes of data length whatever you want to send then check both write and receive data communication.

Let me know if you have any doubt or query for that.

[frosticles]

Thanks for the reply, I misread the documents so I've changed it to:

Code: Select all

  uint8_t *writeBuf = (uint8_t *)"$PMTK220,10000*2F\r\n";
  uart_write_bytes(UART_NUM_1, (const char *)writeBuf, 128);

This didn't work at first, but when I moved the uart_write_bytes() call further down the program after I'd read a few bytes, it did work. So it turns out you need a delay after uart_driver_install() before you call uart_write_bytes(). So this does work:

Code: Select all

  uart_param_config(UART_NUM_1, &uart_config);
  uart_set_pin(UART_NUM_1, 17, 16, UART_PIN_NO_CHANGE, UART_PIN_NO_CHANGE);
  uart_driver_install(UART_NUM_1, BUF_SIZE * 2, 0, 0, NULL, 0);
  
  vTaskDelay(100 / portTICK_RATE_MS);
  uint8_t *writeBuf = (uint8_t *)"$PMTK220,100*2F\r\n";
  uart_write_bytes(UART_NUM_1, (const char *)writeBuf, 128);

[ESP_Sprite]

You're now actually sending your command, an enter, plus whatever crap is in memory after the command because you're sending 128 bytes and your writeBuf is only 19 bytes. Suggest replacing the 128 with strlen(writeBuf).