Re: [PATCH V6 2/2] i2c/designware: Provide i2c bus recovery support

[Wolfram Sang <w.sang-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 1105 bytes --]

> >> +             if (adap->bus_recovery_info &&
> >> +                             adap->bus_recovery_info->recover_bus) {
> >> +                     dev_dbg(dev->dev, "try i2c bus recovery\n");
> >> +                     adap->bus_recovery_info->recover_bus(adap);
> >> +             }
> >> +
> >
> > This should be in the core?
> 
> Because wait_for_completion() would fail in controllers, so i kept this
> code here. How will we come to know about xfer failure in core?

Somehow true. We need the clarification of .timeout and .retries in the
i2c subsystem first, then we can think of returning a specific
-Esomething here which could indicate that a recovery might help. Might
require another thinking if that should be different from -ETIMEDOUT.
Still, we need the cleanup first, and because this may take a while, it
is not your problem.

So, we could start like above and fix users later.


Regards,

   Wolfram

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]