Hello Claudius! It has been a while ;) On 04/11/2020 11:26:27+0100, Claudius Heine wrote: > Hi, > > this patch introduces I2C support to the RX6110 RTC driver and also adds > an ACPI identifier to it. > > Since we are also pushing the coreboot changes for the ACPI table > upstream in parallel, we are free to name this ACPI entry however we > like it seems. So any feedback on that would be welcome ;) > I don't care too much about ACPI so if you are really looking for advice there, I guess you should ask seom of the ACPI guys (but I guess you are free to choose whatever you want). -- Alexandre Belloni, Bootlin Embedded Linux and Kernel engineering https://bootlin.com