Hi Hnas, ... > @@ -109,7 +109,7 @@ static int wmt_check_status(struct wmt_i2c_dev *i2c_dev) > unsigned long wait_result; > > wait_result = wait_for_completion_timeout(&i2c_dev->complete, > - msecs_to_jiffies(500)); > + msecs_to_jiffies(500)); argh!!! This change is not related to this patch. > if (!wait_result) > return -ETIMEDOUT; > ... > @@ -348,18 +370,9 @@ static int wmt_i2c_probe(struct platform_device *pdev) > } > > err = of_property_read_u32(np, "clock-frequency", &clk_rate); > - if (!err && (clk_rate == I2C_MAX_FAST_MODE_FREQ)) > + if (!err && clk_rate == I2C_MAX_FAST_MODE_FREQ) This is not related either. Andi > i2c_dev->tcr = TCR_FAST_MODE;