Hi Hans, ... > > > @@ -58,6 +58,8 @@ struct viai2c { > > > u16 tcr; > > > int irq; > > > u16 cmd_status; > > > + ktime_t ti; > > > + ktime_t to; > > don't these need some arbitration? > > > > I don't think it needs to be arbitration. > the controllers are independent of each other, > each access is locked using __i2c_lock_bus_helper(). > Am I missing something? no, it's fine, it's me who missed that. Do you mind writing a comment? Thanks, Andi