diff options
author | Kimriver Liu <kimriver.liu@siengine.com> | 2024-09-13 11:31:46 +0800 |
---|---|---|
committer | Andi Shyti <andi.shyti@kernel.org> | 2024-09-24 16:23:25 +0200 |
commit | 5d69d5a00f80488ddcb4dee7d1374a0709398178 (patch) | |
tree | 2c0e5dc01b2fc3492a2b68941a41fa83f2fd9f22 /net/smc | |
parent | 4e2c9cd7dce6c7480f236c3ead196ff4e92ed597 (diff) | |
download | linux-stable-5d69d5a00f80488ddcb4dee7d1374a0709398178.tar.gz linux-stable-5d69d5a00f80488ddcb4dee7d1374a0709398178.tar.bz2 linux-stable-5d69d5a00f80488ddcb4dee7d1374a0709398178.zip |
i2c: designware: fix controller is holding SCL low while ENABLE bit is disabled
It was observed that issuing the ABORT bit (IC_ENABLE[1]) will not
work when IC_ENABLE is already disabled.
Check if the ENABLE bit (IC_ENABLE[0]) is disabled when the controller
is holding SCL low. If the ENABLE bit is disabled, the software needs
to enable it before trying to issue the ABORT bit. otherwise,
the controller ignores any write to ABORT bit.
These kernel logs show up whenever an I2C transaction is
attempted after this failure.
i2c_designware e95e0000.i2c: timeout waiting for bus ready
i2c_designware e95e0000.i2c: timeout in disabling adapter
The patch fixes the issue where the controller cannot be disabled
while SCL is held low if the ENABLE bit is already disabled.
Fixes: 2409205acd3c ("i2c: designware: fix __i2c_dw_disable() in case master is holding SCL low")
Signed-off-by: Kimriver Liu <kimriver.liu@siengine.com>
Cc: <stable@vger.kernel.org> # v6.6+
Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Acked-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
Reviewed-by: Andy Shevchenko <andy@kernel.org>
Signed-off-by: Andi Shyti <andi.shyti@kernel.org>
Diffstat (limited to 'net/smc')
0 files changed, 0 insertions, 0 deletions