Thread (31 messages) 31 messages, 5 authors, 2020-12-15

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

From: Hans de Goede <hidden>
Date: 2020-12-06 10:43:19
Also in: linux-acpi, linux-kbuild, linux-serial, lkml, platform-driver-x86

Hi,

On 12/6/20 11:33 AM, Leon Romanovsky wrote:
On Sun, Dec 06, 2020 at 11:04:06AM +0100, Hans de Goede wrote:
<snip>
quoted
But there is a difference between being careful and just nacking
it because no new UAPI may be added at all (also see GKH's response).
I saw, the author misunderstood the Greg's comments.
Quoting from patch 8/9:

"
+==============================
+User-Space EC Interface (cdev)
+==============================
+
+The ``surface_aggregator_cdev`` module provides a misc-device for the SSAM
+controller to allow for a (more or less) direct connection from user-space to
+the SAM EC. It is intended to be used for development and debugging, and
+therefore should not be used or relied upon in any other way. Note that this
+module is not loaded automatically, but instead must be loaded manually.
"

If I'm not mistaken that seems to be pretty much what Greg asked for.

Regards,

Hans
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help