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? Thanks. -- Dmitry