On Tuesday, 6 October 2020, 08:05:28 CEST, Uwe Kleine-König wrote: > Could we please move clearing an irq to a dedicated function? Such that > it looks like: > > /* check for arbitration lost */ > if (temp & I2SR_IAL) { > i2c_imx_clear_irq(i2c_imx, I2SR_IAL); > return -EAGAIN; > } done Changes in v2: --------------- - Don't accidently clear additional status flags on Vybrid (reported by Uwe Kleine-Koenig) Changes in v3: --------------- - dedicated function for clearing an irq