On Fri, Feb 10, 2017 at 08:39:58AM +0100, Andrzej Hajda wrote:
> On 09.02.2017 17:27, Wolfram Sang wrote:
> > On Thu, Jan 05, 2017 at 01:06:53PM +0100, Andrzej Hajda wrote:
> >> In case of arbitration lost adequate interrupt sometimes is not signaled. As
> >> a result transfer timeouts and is not retried, as it should. To avoid such
> >> cases code is added to check transaction status in case of every interrupt.
> >>
> >> Signed-off-by: Andrzej Hajda <a.hajda@xxxxxxxxxxx>
> >> ---
> >> +
> >> + switch (trans_status & HSI2C_MASTER_ST_MASK) {
> >> + case HSI2C_MASTER_ST_LOSE:
> >> + i2c->state = -EAGAIN;
> >> + goto stop;
> >> + }
> > Why not using if() instead of switch() as in the rest of the driver?
>
> Following reasons (not serious ones):
I see. It is probably a taste thing, but a switch() with just one case
in it looks like something has been forgotten :) We can change it back
to switch() if ever more states need to be added.
Given that I am not super strict with the 80 char limit anyhow and it
will miss the limit by a few chars only, I'd prefer the if statement
actually.
Other than this, I am OK with applying the patch for 4.11. Can you
resend with the above change?
Thanks,
Wolfram
Attachment:
signature.asc
Description: PGP signature
