Hi, On 11/25/20 11:41 AM, Andy Shevchenko wrote: > On Wed, Nov 25, 2020 at 10:37 AM Hans de Goede <hdegoede@xxxxxxxxxx> wrote: >> >> Hi All, >> >> This series improves support for ACPI enumeration of bmc150 accels >> described by an ACPI node with an ACPI hw-id of BOSC0200: >> >> 1. Add support for nodes which describe 2 acceleromers in a single node, > > accelerometers > >> fixing: https://bugzilla.kernel.org/show_bug.cgi?id=198671 >> >> 2. Add support for reading the mount-matrix from the ACPI node. >> >> This is done in patches 2 - 3, patch 1 is a trivial cleanup which I noticed >> could be done while working on this. >> >> Patch 2 is based on an earlier patch for this from Jeremy Cline: >> https://lore.kernel.org/r/010001602cf53153-39ad69f1-1b39-4e6d-a748-9455a16c2fbd-000000@xxxxxxxxxxxxxxxxxxx >> >> That patch was put on hold because normally ACPI nodes which describe >> multiple i2c-clients in a single node are handled by: >> drivers/platform/x86/i2c-multi-instantiate.c >> >> Which I tried to do at first, but as explained in the commit msg >> of the updated patch, that is not possible in this special case >> (because it would cause userspace breakage due to the modalias changing). > > This is marked as patch 1?! Yes my bad, sorry. > Usually --cover-letter produces a correct template... I use --compose, time to switch to --cover-letter instead I guess. Regards, Hans