Hi Guenter, [...] > > > +static const struct hwmon_channel_info *hwmon_info[] = { > > > + NULL > > > +}; > > > > just: > > > > static const struct hwmon_channel_info *hwmon_info[] = { }; > > > > would do. > > > > What would be the point ? This is just a placeholder, and we need > a NULL entry at the end. right... this series needs to be read from the end to the beginning :) Andi