On 2021/4/15 16:18, Wolfram Sang wrote:> 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.
Sure. Thank you.
>>>> - 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.
Got it. Thanks for your clarification.