Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
User-Agent: Cyrus-JMAP/3.3.0-622-g4a97c0b-fm-20201115.001-g4a97c0b3
Mime-Version: 1.0
Message-Id: <1b4fe1fd-592d-4a88-b156-fbf6af2df428@www.fastmail.com>
In-Reply-To: <052ecf4d-9e08-2c08-8a06-c30ba2b28d82@redhat.com>
References: <20201203212640.663931-1-luzmaximilian@gmail.com>
 <20201206070705.GA686270@unreal>
 <052ecf4d-9e08-2c08-8a06-c30ba2b28d82@redhat.com>
Date:   Sun, 06 Dec 2020 17:58:32 +0900
From:   =?UTF-8?Q?Bla=C5=BE_Hrastnik?= <blaz@mxxn.io>
To:     "Hans de Goede" <hdegoede@redhat.com>,
        "Leon Romanovsky" <leon@kernel.org>,
        "Maximilian Luz" <luzmaximilian@gmail.com>,
        "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>
Cc:     lkml <linux-kernel@vger.kernel.org>,
        "Mark Gross" <mgross@linux.intel.com>,
        "Andy Shevchenko" <andy.shevchenko@gmail.com>,
        =?UTF-8?Q?Barnab=C3=A1s_P=C5=91cze?= <pobrn@protonmail.com>,
        "Arnd Bergmann" <arnd@arndb.de>, "Rob Herring" <robh@kernel.org>,
        "Jiri Slaby" <jirislaby@kernel.org>,
        "Rafael J . Wysocki" <rjw@rjwysocki.net>,
        "Len Brown" <lenb@kernel.org>,
        "Steven Rostedt" <rostedt@goodmis.org>,
        "Ingo Molnar" <mingo@redhat.com>,
        "Masahiro Yamada" <masahiroy@kernel.org>,
        "Michal Marek" <michal.lkml@markovi.net>,
        "Jonathan Corbet" <corbet@lwn.net>,
        "Dorian Stoll" <dorian.stoll@tmsp.io>,
        platform-driver-x86@vger.kernel.org, linux-serial@vger.kernel.org,
        linux-acpi@vger.kernel.org, linux-kbuild@vger.kernel.org,
        linux-doc@vger.kernel.org
Subject: =?UTF-8?Q?Re:_[PATCH_v2_0/9]_Add_support_for_Microsoft_Surface_System_Ag?=
 =?UTF-8?Q?gregator_Module?=
Content-Type: text/plain
Precedence: bulk

> 
> > More on that, the whole purpose of proposed interface is to debug and
> > not intended to be used by any user space code.
> 
> The purpose is to provide raw access to the Surface Serial Hub protocol,
> just like we provide raw access to USB devices and have hidraw devices.
> 
> So this goes a litle beyond just debugging; and eventually the choice
> may be made to implement some functionality with userspace drivers,
> just like we do for some HID and USB devices.
> 
> Still I agree with you that adding new userspace API is something which
> needs to be considered carefully. So I will look at this closely when
> reviewing this set.

To add to that: this was previously a debugfs interface but was moved to misc after review on the initial RFC:
https://lkml.org/lkml/2020/9/24/96