On Thu, Apr 15, 2021 at 04:15:07PM +0800, Jie Deng wrote: > On 2021/4/15 15:28, Wolfram Sang wrote: > > > > Now that we were able to catch you, I will use the opportunity to > > > clarify the doubts I had. > > > > > > - struct mutex lock in struct virtio_i2c, I don't think this is > > > required since the core takes care of locking in absence of this. > > This is likely correct. > > OK. Then I will remove the lock. Let me have a look first, please. > > > - Use of I2C_CLASS_DEPRECATED flag, I don't think it is required for > > > new drivers. > > This is definately correct :) > > Do you mean a new driver doesn't need to set the following ? > > vi->adap.class = I2C_CLASS_DEPRECATED; > > Just leave the class to be 0 ? Yes. DEPRECATED is only for drivers which used to have a class and then chose to remove it.
Attachment:
signature.asc
Description: PGP signature