On Mon, Aug 8, 2022 at 1:07 AM Jakob Hauser <jahau@xxxxxxxxxxxxxx> wrote: ... > + yas5xx->chip = id->driver_data; > + yas5xx->chip_info = &yas5xx_chip_info_tbl[yas5xx->chip]; I don't see how ->chip is being used, I would expect it is the part of chip_info, if it's really needed. That said, please make it directly a pointer, so the above becomes: ... ->chip_info = (const struct ...)id->driver_data; ... > + {"yas530", yas530 }, > + {"yas532", yas532 }, > + {"yas533", yas533 }, Read above and here: yas53... ==> (kernel_ulong_t)&...[yas53...] -- With Best Regards, Andy Shevchenko