github/esphome
2
0
Fork 0
mirror of https://github.com/esphome/esphome.git synced 2026-09-03 19:46:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

[uart] Validate fixed UART settings at config time for fixed-baud components (#17207)

Browse Source
This commit is contained in:
Jonathan Swoboda
2026-06-25 10:19:05 -04:00
committed by GitHub
parent 4f70f6b2a6
commit 18c7f60410
19 changed files with 77 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files
esphome/components/bl0940/sensor.py
+11
View File
@@ -211,6 +211,17 @@ CONFIG_SCHEMA = (
.add_extra(set_reference_values)
)
# BL0940 datasheet: 4800 baud, 8 data bits, no parity (stop bits are 1.5 -- not
# representable in the uart schema, so it isn't asserted).
FINAL_VALIDATE_SCHEMA = uart.final_validate_device_schema(
"bl0940",
baud_rate=4800,
data_bits=8,
parity="NONE",
require_rx=True,
require_tx=True,
)
async def to_code(config):
var = cg.new_Pvariable(config[CONF_ID])
esphome/components/midea/climate.py
+5
View File
@@ -260,6 +260,11 @@ async def power_inv_to_code(var, config, args):
pass
FINAL_VALIDATE_SCHEMA = uart.final_validate_device_schema(
"midea", baud_rate=9600, require_rx=True, require_tx=True
)
async def to_code(config):
var = await climate.new_climate(config)
await cg.register_component(var, config)
esphome/components/pzem004t/sensor.py
+4
View File
@@ -58,6 +58,10 @@ CONFIG_SCHEMA = (
.extend(uart.UART_DEVICE_SCHEMA)
)
FINAL_VALIDATE_SCHEMA = uart.final_validate_device_schema(
"pzem004t", baud_rate=9600, require_rx=True, require_tx=True
)
async def to_code(config):
var = cg.new_Pvariable(config[CONF_ID])
esphome/components/rdm6300/__init__.py
+4
View File
@@ -29,6 +29,10 @@ CONFIG_SCHEMA = (
.extend(uart.UART_DEVICE_SCHEMA)
)
FINAL_VALIDATE_SCHEMA = uart.final_validate_device_schema(
"rdm6300", baud_rate=9600, require_rx=True
)
async def to_code(config):
var = cg.new_Pvariable(config[CONF_ID])
esphome/components/rf_bridge/__init__.py
+10
View File
@@ -80,6 +80,16 @@ _CALLBACK_AUTOMATIONS = (
),
)
FINAL_VALIDATE_SCHEMA = uart.final_validate_device_schema(
"rf_bridge",
baud_rate=19200,
require_rx=True,
require_tx=True,
data_bits=8,
parity="NONE",
stop_bits=1,
)
async def to_code(config):
var = cg.new_Pvariable(config[CONF_ID])
esphome/components/rf_bridge/rf_bridge.cpp
+1 -4
View File
@@ -195,10 +195,7 @@ void RFBridgeComponent::learn() {
this->flush();
}
void RFBridgeComponent::dump_config() {
ESP_LOGCONFIG(TAG, "RF_Bridge:");
this->check_uart_settings(19200);
}
void RFBridgeComponent::dump_config() { ESP_LOGCONFIG(TAG, "RF_Bridge:"); }
void RFBridgeComponent::start_advanced_sniffing() {
ESP_LOGI(TAG, "Advanced Sniffing on");
esphome/components/sds011/sds011.cpp
-1
View File
@@ -73,7 +73,6 @@ void SDS011Component::dump_config() {
this->update_interval_min_, ONOFF(this->rx_mode_only_));
LOG_SENSOR(" ", "PM2.5", this->pm_2_5_sensor_);
LOG_SENSOR(" ", "PM10.0", this->pm_10_0_sensor_);
this->check_uart_settings(9600);
}
void SDS011Component::loop() {
esphome/components/sds011/sensor.py
+18
View File
@@ -63,6 +63,24 @@ CONFIG_SCHEMA = cv.All(
)
def _final_validate(config):
# In the default mode setup() writes config commands, so tx is required;
# rx_only mode never writes, so tx is optional.
uart.final_validate_device_schema(
"sds011",
baud_rate=9600,
require_rx=True,
require_tx=not config.get(CONF_RX_ONLY, False),
data_bits=8,
parity="NONE",
stop_bits=1,
)(config)
return config
FINAL_VALIDATE_SCHEMA = _final_validate
async def to_code(config):
# Pop update_interval before register_component so it doesn't generate
# a set_update_interval call — sds011 handles this via set_update_interval_min
esphome/components/senseair/senseair.cpp
-1
View File
@@ -146,7 +146,6 @@ bool SenseAirComponent::senseair_write_command_(const uint8_t *command, uint8_t
void SenseAirComponent::dump_config() {
ESP_LOGCONFIG(TAG, "SenseAir:");
LOG_SENSOR(" ", "CO2", this->co2_sensor_);
this->check_uart_settings(9600);
}
} // namespace esphome::senseair
esphome/components/senseair/sensor.py
+10
View File
@@ -51,6 +51,16 @@ CONFIG_SCHEMA = (
.extend(uart.UART_DEVICE_SCHEMA)
)
FINAL_VALIDATE_SCHEMA = uart.final_validate_device_schema(
"senseair",
baud_rate=9600,
require_rx=True,
require_tx=True,
data_bits=8,
parity="NONE",
stop_bits=1,
)
async def to_code(config):
var = cg.new_Pvariable(config[CONF_ID])
esphome/components/shelly_dimmer/light.py
+4
View File
@@ -186,6 +186,10 @@ CONFIG_SCHEMA = (
.extend(uart.UART_DEVICE_SCHEMA)
)
FINAL_VALIDATE_SCHEMA = uart.final_validate_device_schema(
"shelly_dimmer", baud_rate=115200, require_rx=True, require_tx=True
)
async def to_code(config):
fw_hex = get_firmware(config[CONF_FIRMWARE])
esphome/components/sm300d2/sensor.py
+4
View File
@@ -88,6 +88,10 @@ CONFIG_SCHEMA = cv.All(
.extend(uart.UART_DEVICE_SCHEMA)
)
FINAL_VALIDATE_SCHEMA = uart.final_validate_device_schema(
"sm300d2", baud_rate=9600, require_rx=True, data_bits=8, parity="NONE", stop_bits=1
)
async def to_code(config):
var = cg.new_Pvariable(config[CONF_ID])
esphome/components/sm300d2/sm300d2.cpp
-1
View File
@@ -100,7 +100,6 @@ void SM300D2Sensor::dump_config() {
LOG_SENSOR(" ", "PM10", this->pm_10_0_sensor_);
LOG_SENSOR(" ", "Temperature", this->temperature_sensor_);
LOG_SENSOR(" ", "Humidity", this->humidity_sensor_);
this->check_uart_settings(9600);
}
} // namespace esphome::sm300d2
tests/components/bl0940/test.esp32-idf.yaml
+1 -1
View File
@@ -3,6 +3,6 @@ substitutions:
rx_pin: GPIO14
packages:
uart: !include ../../test_build_components/common/uart/esp32-idf.yaml
uart_4800: !include ../../test_build_components/common/uart_4800/esp32-idf.yaml
<<: !include common.yaml
tests/components/bl0940/test.esp8266-ard.yaml
+1 -1
View File
@@ -3,6 +3,6 @@ substitutions:
rx_pin: GPIO3
packages:
uart: !include ../../test_build_components/common/uart/esp8266-ard.yaml
uart_4800: !include ../../test_build_components/common/uart_4800/esp8266-ard.yaml
<<: !include common.yaml
tests/components/bl0940/test.rp2040-ard.yaml
+1 -1
View File
@@ -3,6 +3,6 @@ substitutions:
rx_pin: GPIO5
packages:
uart: !include ../../test_build_components/common/uart/rp2040-ard.yaml
uart_4800: !include ../../test_build_components/common/uart_4800/rp2040-ard.yaml
<<: !include common.yaml
tests/components/rf_bridge/test.esp32-idf.yaml
+1 -1
View File
@@ -1,4 +1,4 @@
packages:
uart: !include ../../test_build_components/common/uart/esp32-idf.yaml
uart_19200: !include ../../test_build_components/common/uart_19200/esp32-idf.yaml
<<: !include common.yaml
tests/components/rf_bridge/test.esp8266-ard.yaml
+1 -1
View File
@@ -1,4 +1,4 @@
packages:
uart: !include ../../test_build_components/common/uart/esp8266-ard.yaml
uart_19200: !include ../../test_build_components/common/uart_19200/esp8266-ard.yaml
<<: !include common.yaml
tests/components/rf_bridge/test.rp2040-ard.yaml
+1 -1
View File
@@ -1,4 +1,4 @@
packages:
uart: !include ../../test_build_components/common/uart/rp2040-ard.yaml
uart_19200: !include ../../test_build_components/common/uart_19200/rp2040-ard.yaml
<<: !include common.yaml