On Wed, Dec 13, 2017 at 05:23:02PM +0200, Andy Shevchenko wrote: > On Mon, 2017-12-04 at 13:36 +0100, Wolfram Sang wrote: > > No reason to have them undefined, so let's add them. > > > > > int (*recover_bus)(struct i2c_adapter *); > > > > - int (*get_scl)(struct i2c_adapter *); > > - void (*set_scl)(struct i2c_adapter *, int val); > > - int (*get_sda)(struct i2c_adapter *); > > + int (*get_scl)(struct i2c_adapter *adap); > > + void (*set_scl)(struct i2c_adapter *adap, int val); > > + int (*get_sda)(struct i2c_adapter *adap); > > > > void (*prepare_recovery)(struct i2c_adapter *); > > void (*unprepare_recovery)(struct i2c_adapter *); > > It seems inconsistent with the rest of the members even from this > visible piece.
Agreed. Either overlooked, or lost during rebase. Any way, will fix!
signature.asc
Description: PGP signature
