Hi Biju, On Mon, Aug 28, 2023 at 2:43 PM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote: > > Subject: Re: [PATCH v2] iio: chemical: vz89x: Convert enum->pointer for > > data in the match tables > > > > On Fri, 18 Aug 2023 20:04:29 +0100 > > Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote: > > > > > Convert enum->pointer for data in the match tables, so that > > > device_get_match_data() can do match against OF/ACPI/I2C tables, once > > > i2c bus type match support added to it. > > > > > > Replace enum->struct *vz89x_chip_data for data in the match table. > > > Simplify the probe() by replacing device_get_match_data() and ID > > > lookup for retrieving data by i2c_get_match_data(). > > > > > > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx> > > > Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > > > > Biju, > > > > Make sure you cc the driver authors etc. > > Normally, I ran a script against the patch to get details and additionally I add Geert, renesas-soc , Andy in Cc list. I don't think there is a need to add renesas-soc (or me, FWIW ;-), unless the specific device is used on Renesas platforms. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds