lkml.org 
[lkml]   [2021]   [Apr]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v10] i2c: virtio: add a virtio i2c frontend driver
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.

[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2021-04-15 10:20    [W:0.148 / U:0.628 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site