hwmon: pmbus: Fix -EIO seen on pli1209
authorPatrick Rudolph <patrick.rudolph@9elements.com>
Thu, 17 Aug 2023 09:25:26 +0000 (11:25 +0200)
committerGuenter Roeck <linux@roeck-us.net>
Mon, 21 Aug 2023 13:04:31 +0000 (06:04 -0700)
commite7593bda6a2e2e46521567f0ddea940f4acda0db
tree2ca4353050fd66d2c7792c83e60ddabc6fc452d4
parent3fd2188e588ffd8d45df5ac92ab1089328211a86
hwmon: pmbus: Fix -EIO seen on pli1209

After doing performance optimizations the pli1209 driver failed to
probe with a probabilty of 2%. It wasn't able to read the PMBUS_OPERATION
register due to an -EIO error.

An investigation showed that the PLI1209 takes 230 usec to execute the
CLEAR_FAULTS command. During that time it's busy and NACKs all requests
on the SMBUS interface. It also NACKs reads on PMBUS_STATUS_BYTE
making it impossible to poll the BUSY flag.

Add a custom write_data function to just wait for not BUSY unconditionally
after sending a CLEAR_FAULTS command.

TEST: Verified using an I2C bus analyser that no more NACKs are seen after
      sending a CLEAR_FAULTS command.

Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>
Signed-off-by: Naresh Solanki <Naresh.Solanki@9elements.com>
Link: https://lore.kernel.org/r/20230817092527.808631-3-Naresh.Solanki@9elements.com
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
drivers/hwmon/pmbus/pli1209bc.c