RE: [PATCH RFC 2/2] i2c: Add i2c_device_get_match_data() callback

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Geert,

Thanks for the feedback.

> Subject: Re: [PATCH RFC 2/2] i2c: Add i2c_device_get_match_data()
> callback
> 
> Hi Biju,
> 
> On Sun, Jul 23, 2023 at 10:37 AM Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> wrote:
> > Add i2c_device_get_match_data() callback to struct bus_type().
> >
> > While at it, introduced i2c_get_match_data_helper() to avoid code
> > duplication with i2c_get_match_data().
> >
> > Signed-off-by: Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx>
> > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> 
> Thanks for your patch!
> 
> > --- a/drivers/i2c/i2c-core-base.c
> > +++ b/drivers/i2c/i2c-core-base.c
> > @@ -114,20 +114,41 @@ const struct i2c_device_id *i2c_match_id(const
> > struct i2c_device_id *id,  }  EXPORT_SYMBOL_GPL(i2c_match_id);
> >
> > +static void *i2c_get_match_data_helper(struct i2c_driver *driver,
> 
> static const void *

I missed this.

> 
> > +                                      const struct i2c_client
> > +*client) {
> > +       const struct i2c_device_id *match;
> > +
> > +       match = i2c_match_id(driver->id_table, client);
> > +       if (!match)
> > +               return NULL;
> > +
> > +       return (const void *)match->driver_data;
> 
> I guess your compiler didn't complain about the const/non-const
> conversion when returning because it inlined the function?

It complained. Somehow, I didn't notice that warning before sending the patch.

> 
> > +}
> > +
> > +static const void *i2c_device_get_match_data(const struct device
> > +*dev) {
> > +       const struct i2c_client *client = to_i2c_client(dev);

Not sure, non-const i2c_verify_client(dev)to be used here??

> > +       const struct i2c_driver *driver;
> > +
> > +       if (!dev->driver)
> > +               return NULL;
> > +
> > +       driver = to_i2c_driver(dev->driver);
> > +       if (!driver)
> > +               return NULL;
> 
> "driver" can never be NULL here.

OK, will remove this.

Cheers,
Biju

> 
> > +
> > +       return i2c_get_match_data_helper(driver, client); }
> > +
> 




[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux