Re: [PATCH 0/9] Add support for Microsoft Surface System Aggregator Module

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11/24/20 12:59 PM, Hans de Goede wrote:
Hi,

On 11/15/20 8:21 PM, Maximilian Luz wrote:
Hello,

   N.B.: the following text is mostly a repeat of cover letter from the
   previous RFC for the uninitiated, which can be found at

   https://lore.kernel.org/linux-serial/20200923151511.3842150-1-luzmaximilian@xxxxxxxxx/

   See "Changes" below for an overview of differences between the RFC and
   this patchset. I hope I have addressed all comments from that in this
   version, thank you again for those.

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].

<snip>

This patch-set can also be found at the following repository and
reference, if you prefer to look at a kernel tree instead of these
emails:

   https://github.com/linux-surface/kernel tags/s/surface-aggregator/v1

Thanks,
Max

Thank you for your work on this. It would be great if we can get better
support for the Surface line in the mainline kernel.

Since a lot of people have already commented on this series I think that
you have enough feedback to do a v2 addressing that feedback right?

Yes, I'm already working on it.

For now I'm going to assume that you will do a v2 addressing the
initial round of comments and not review this myself (IOW I'll review
this when v2 is posted).

Sure, no need for you to review v1 at this point.
Let me know if you see things differently.

Thanks,
Max



[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux