change(i2c): Add check for scl frequency for master_bus_add_device

Closes espressif#12598
movsb · Dec 1, 2023 · eab4fa6 · eab4fa6
1 parent 406e4ac
commit eab4fa6
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/components/driver/i2c/i2c_master.c b/components/driver/i2c/i2c_master.c
@@ -874,6 +874,8 @@ esp_err_t i2c_master_bus_add_device(i2c_master_bus_handle_t bus_handle, const i2
 {
     esp_err_t ret = ESP_OK;
     ESP_RETURN_ON_FALSE((bus_handle != NULL), ESP_ERR_INVALID_ARG, TAG, "this bus is not initialized, please call `i2c_new_master_bus`");
+    ESP_RETURN_ON_FALSE(dev_config, ESP_ERR_INVALID_ARG, TAG, "invalid argument");
+    ESP_RETURN_ON_FALSE(dev_config->scl_speed_hz > 0, ESP_ERR_INVALID_ARG, TAG, "invalid scl frequency");
     if(bus_handle->base->bus_mode != I2C_BUS_MODE_MASTER) {
         ESP_LOGE(TAG, "This is not master bus!");
         return ESP_ERR_INVALID_ARG;