Hi Dmitry, On Thu, Jul 13, 2023 at 10:00:05AM -0700, Dmitry Torokhov wrote: > Hi Jeff, > > On Thu, Jul 13, 2023 at 11:06:46AM -0500, Jeff LaBundy wrote: > > @@ -2460,13 +2451,11 @@ static int iqs7211_probe(struct i2c_client *client) > > > > INIT_LIST_HEAD(&iqs7211->reg_field_head); > > > > - if (client->dev.of_node) > > - dev_id = (enum iqs7211_dev_id)of_device_get_match_data(&client->dev); > > - else > > - dev_id = i2c_match_id(iqs7211_id, client)->driver_data; > > + iqs7211->dev_desc = of_device_get_match_data(&client->dev); > > Can we make it device_get_match_data() instead of using OF-specific > variant? Ah yes, good catch; I'll fix that. > > Thanks. > > -- > Dmitry Kind regards, Jeff LaBundy
