diff --git a/components/driver/i2c/i2c_private.h b/components/driver/i2c/i2c_private.h
index 2eee1bb2e3..bdf8e09a58 100644
--- a/components/driver/i2c/i2c_private.h
+++ b/components/driver/i2c/i2c_private.h
@@ -91,7 +91,7 @@ struct i2c_bus_t {
     uint32_t clk_src_freq_hz; // Record the clock source frequency
     int sda_num; // SDA pin number
     int scl_num; // SCL pin number
-    bool pull_up_enable; // Disable pull-ups
+    bool pull_up_enable; // Enable pull-ups
     intr_handle_t intr_handle; // I2C interrupt handle
     esp_pm_lock_handle_t pm_lock; // power manange lock
 #if CONFIG_PM_ENABLE
diff --git a/components/driver/i2c/include/driver/i2c_master.h b/components/driver/i2c/include/driver/i2c_master.h
index 0983c0cdfc..48471ddd67 100644
--- a/components/driver/i2c/include/driver/i2c_master.h
+++ b/components/driver/i2c/include/driver/i2c_master.h
@@ -27,7 +27,7 @@ typedef struct {
     int intr_priority;                    /*!< I2C interrupt priority, if set to 0, driver will select the default priority (1,2,3). */
     size_t trans_queue_depth;             /*!< Depth of internal transfer queue, increase this value can support more transfers pending in the background, only valid in asynchronous transaction. (Typically max_device_num * per_transaction)*/
     struct {
-        uint32_t enable_internal_pullup:1;   /*!< Enable internal pullups */
+        uint32_t enable_internal_pullup:1;   /*!< Enable internal pullups. Note: This is not strong enough to pullup buses under high-speed frequency. Recommend proper external pull-up if possible */
     } flags;
 } i2c_master_bus_config_t;