Acked-by: Joel Stanley <joel@xxxxxxxxx>
Thanks!
While reviewing I was concerned about the locking in aspeed_i2c_master_xfer. It's a bit hairy, and I am not convinced it is without bugs.
The locking was already in there in aspeed_i2c_master_xfer but it unlocks the protection to make a waiting for a completion. This code is after the waiting so I added the locking to protect 'bus->master_state' variable which can be accessed from driver context and from interrupt context. Thanks, Jae
