From: Ye Li Date: Wed, 10 Jul 2019 10:24:15 +0000 (+0000) Subject: i2c: mxc_i2c: Remove i2c_idle_bus from probe X-Git-Tag: v2019.10-rc4~33^2~1 X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=d7d864017d7f21a4cc3ff2a902024724c26ad213;p=oweals%2Fu-boot.git i2c: mxc_i2c: Remove i2c_idle_bus from probe i2c_idle_bus is already used in i2c_init_transfer. So before each transfer if the bus is not ready, the i2c_idle_bus will be used to force idle. It is unnecessary to call it again in probe. We found a issue when enabling i2c mux with the mxc_i2c. The mxc_i2c is probed after mux probing. However, at this moment the mux is still in idle state not select any port. So if we call i2c_idle_bus in probe, it will fail and cause mxc_i2c probe failed. Signed-off-by: Ye Li --- diff --git a/drivers/i2c/mxc_i2c.c b/drivers/i2c/mxc_i2c.c index 73b9807598..20f6dc4ecb 100644 --- a/drivers/i2c/mxc_i2c.c +++ b/drivers/i2c/mxc_i2c.c @@ -917,13 +917,6 @@ static int mxc_i2c_probe(struct udevice *bus) } } - ret = i2c_idle_bus(i2c_bus); - if (ret < 0) { - /* Disable clk */ - enable_i2c_clk(0, bus->seq); - return ret; - } - /* * Pinmux settings are in board file now, until pinmux is supported, * we can set pinmux here in probe function.