feat(rmt_uart): remove parity support

Author: Kainarx

rmt_soft_uart/include/rmt_uart_type.h

@@ -28,14 +28,12 @@ typedef enum {
 } rmt_uart_word_length_t;
 
 typedef enum {
-    RMT_UART_STOP_BITS_1   = 0x1,  /*!< stop bit: 1bit*/
-    RMT_UART_STOP_BITS_2   = 0x2,  /*!< stop bit: 2bits*/
+    RMT_UART_STOP_BITS_1   = 0x0,  /*!< stop bit: 1bit*/
+    RMT_UART_STOP_BITS_2   = 0x1,  /*!< stop bit: 2bits*/
 } rmt_uart_stop_bits_t;
 
 typedef enum {
     RMT_UART_PARITY_DISABLE = 0x0,  /*!< Disable UART parity*/
-    RMT_UART_PARITY_EVEN    = 0x1,  /*!< Enable UART even parity*/
-    RMT_UART_PARITY_ODD     = 0x2,  /*!< Enable UART odd parity*/
 } rmt_uart_parity_t;
 
 /**

rmt_soft_uart/src/rmt_uart.c

@@ -80,9 +80,8 @@ static bool rmt_uart_process_level(uint8_t level, uint32_t duration, int bit_tic
 {
     int bit_count = ROUND_CLOSEST(duration, bit_ticks);
     uint8_t data_bits = uart_device->data_bits;
-    uint8_t has_parity = (uart_device->parity != RMT_UART_PARITY_DISABLE);
     uint8_t stop_bits = (uart_device->stop_bits == RMT_UART_STOP_BITS_1) ? 1 : 2;
-    uint8_t total_bits = 1 + data_bits + has_parity + stop_bits;
+    uint8_t total_bits = 1 + data_bits + stop_bits;
 
     for (int i = 0; i < bit_count; i++) {
         // if the current bit is not a start bit, stop the decoding
@@ -104,23 +103,6 @@ static bool rmt_uart_process_level(uint8_t level, uint32_t duration, int bit_tic
             // extract the data byte
             uint8_t data_byte = (decode_context->raw_data >> 1) & ((1 << data_bits) - 1);
 
-            // check the parity
-            if (has_parity) {
-                uint8_t parity_bit = (decode_context->raw_data >> (1 + data_bits)) & 0x01;
-                uint8_t parity_calc = __builtin_parity(data_byte);
-                bool parity_ok = (uart_device->parity == RMT_UART_PARITY_EVEN)
-                                 ? parity_calc == parity_bit
-                                 : parity_calc != parity_bit;
-
-                // if the parity is wrong, stop the decoding
-                if (!parity_ok) {
-                    ESP_LOGE(TAG, "Parity error @ byte %d", decode_context->byte_pos);
-                    decode_context->bit_pos = 0;
-                    decode_context->raw_data = 0;
-                    return decode_context->continue_on_error;
-                }
-            }
-
             // if the stop bit is wrong, stop the decoding
             if ((decode_context->raw_data >> (total_bits - 1)) != 1) {
                 ESP_LOGE(TAG, "Invalid stop bit @ byte %d", decode_context->byte_pos);
@@ -215,7 +197,6 @@ esp_err_t rmt_new_uart_device(const rmt_uart_config_t *uart_config, rmt_uart_dev
     rmt_uart_device_t *uart_device = heap_caps_calloc(1, sizeof(rmt_uart_device_t), MALLOC_CAP_INTERNAL | MALLOC_CAP_8BIT);
     ESP_RETURN_ON_FALSE(uart_device, ESP_ERR_NO_MEM, TAG, "no mem for rmt_uart_device");
     ESP_RETURN_ON_FALSE(uart_config->data_bits == RMT_UART_DATA_8_BITS, ESP_ERR_INVALID_ARG, TAG, "Invalid data bits");
-    ESP_RETURN_ON_FALSE(uart_config->parity == RMT_UART_PARITY_DISABLE, ESP_ERR_INVALID_ARG, TAG, "Invalid parity");
     uart_device->baud_rate = uart_config->baud_rate;
     uart_device->data_bits = uart_config->data_bits;
     uart_device->stop_bits = uart_config->stop_bits;