On Wed, Dec 16, 2020 at 8:02 AM Gwendal Grignou <gwendal@xxxxxxxxxxxx> wrote: > > Add timestamp channel in list of channel, to allow retrieving timestamps > when events are produced. ... > /* > * The event buffer contains timestamp and all the data from > * the ACPI0008 block. There are multiple, but so far we only > - * support _ALI (illuminance). Once someone adds new channels > - * to acpi_als_channels[], the evt_buffer below will grow > - * automatically. > + * support _ALI (illuminance): > + * One channel, padding and timestamp. Why on a new line? We have enough space on the previous one, no? > */ -- With Best Regards, Andy Shevchenko