On Wed, Sep 23, 2020 at 6:32 PM Arnd Bergmann <arnd@xxxxxxxx> wrote: > > On Wed, Sep 23, 2020 at 5:15 PM Maximilian Luz <luzmaximilian@xxxxxxxxx> wrote: > > > > Hello, > > > > The Surface System Aggregator Module (we'll refer to it as Surface > > Aggregator or SAM below) is an embedded controller (EC) found on various > > Microsoft Surface devices. Specifically, all 4th and later generation > > Surface devices, i.e. Surface Pro 4, Surface Book 1 and later, with the > > exception of the Surface Go series and the Surface Duo. Notably, it > > seems like this EC can also be found on the ARM-based Surface Pro X [1]. > > I think this should go to drivers/platform/x86 or drivers/platform/surface/ > along with other laptop vendor specific code rather than drivers/misc/. +1 here. drivers/platform/surface is a good place to start. And you may begin with moving a few Surface drivers out of PDx86 to the new folder. -- With Best Regards, Andy Shevchenko