From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1033270AbdAFT0a (ORCPT ); Fri, 6 Jan 2017 14:26:30 -0500 Received: from mout.kundenserver.de ([212.227.17.24]:61210 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751997AbdAFT0U (ORCPT ); Fri, 6 Jan 2017 14:26:20 -0500 From: Arnd Bergmann To: Rob Herring Cc: Greg Kroah-Hartman , Marcel Holtmann , Jiri Slaby , Sebastian Reichel , "Dr . H . Nikolaus Schaller" , Peter Hurley , Andy Shevchenko , Alan Cox , Loic Poulain , Pavel Machek , NeilBrown , Linus Walleij , linux-bluetooth@vger.kernel.org, linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 0/9] Serial slave device bus Date: Fri, 06 Jan 2017 20:25:34 +0100 Message-ID: <1877696.ddOr49d5xy@wuerfel> User-Agent: KMail/5.1.3 (Linux/4.4.0-34-generic; KDE/5.18.0; x86_64; ; ) In-Reply-To: <20170106162635.19677-1-robh@kernel.org> References: <20170106162635.19677-1-robh@kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:h405Y/2niqf6kmqwsC5U2iBOGxsG8cgf6/uXBTpt/EndeGQ7Eih CF3PQ945zVm0NJA8F2YBgVtZxBGWveRt9upttsVsM82Tenf4rdv+3f7C2YwtCCDBqNDZ5SR eogmpyXzXBnoB3s/fyrKZqPf/CTpxv+2ILVt4ffFRUh3IEtItBuCpRrY/A9Gpr6kE8mRgUq 9fAI1bjNYBYKvsytDNvMg== X-UI-Out-Filterresults: notjunk:1;V01:K0:ELZ9lUHxmyM=:aupA7F8XlvvHgCgtoXndyy haOM0A9XeFOD6cecYV+7Tqdk89PFAtdvIDbTjwwq4Y3RsCfwdGjpyIgRg6d++YSz21oGS0jqo rvpOIV3WqPT1trvxwvEBQQYqY4xfGKQ+gPLx9XhT3YzGwAnGWUTKvu1cI2Z+UHhAg0SpMSh+S pRsWCmirN8G9yDHD/oxamMfD3eFfIYMf7EOYqZtIDw46Voqc9Myl1LyM4imfMl7FJOLRG1csc 5tMbXglK4OwmlvsXwmhOdjywiQSgGxqoaVMURkxSGvmHln/ZvizPcFuMp90d8Y1Kcm45307ID UC+x422vSgi1s7gAT2vy2XGFjC8N84iTRyalVy6qLs52/fM+M6LLlsbxJqvr+5XtYy+TTAeyZ iNHvXEtsG1l0MdLnRBwlmFDz62CVL87hBcEGe/Sg95knYPhsa7iUKyH7j/zLXSw/QerafxlML rfMgrKrBOlJvr0FbHGqgDWVV1LbXsypv+SV7l27orfHXFSb1aTS9GjB9Z9D3+MU3vm2+52Tpn Oqa4RbH7uTuc96dcRu5/8zVsyjnoM1nILSGT1IGAs+PFCnO29H/2KBTn6kgdM753CdAleaUKy ciywcFHCUZGbRS6n4NNT07JhneK5OM+X09FO9qC6GG9mtVBnxJzNYlXnErV9MisLgV23X7tXP cw2PpolIXIwiTr5FNO4848JHwCVRIaN+f6aoULXjbW/D+0VYD02CYue0aw97Zc8HVEgx2edRs jMTLsUMbT1lHFvPg Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Friday, January 6, 2017 10:26:26 AM CET Rob Herring wrote: > Here goes another attempt at a serial device bus (aka uart slaves, tty > slaves, etc.). > > After some discussions with Dmitry at LPC, I decided to move away from > extending serio and moved back to making a new bus type instead. He didn't > think using serio was a good fit, and serio has a number of peculiarities > in regards to sysfs and it's driver model. I don't think we want to inherit > those for serial slave devices. Using serio was originally my idea, but since you seem to have discussed this in more detail than I ever had, the new version is certainly fine with me too. > This version sits on top of tty_port rather than uart_port as Alan > requested. Once I created a struct tty rather than moving everything > needed to tty_port, it became a lot easier and less invasive to the tty > core code. > > I have hacked up versions of the BT ldisc and TI ST drivers moved over to > use the serdev bus. I have BT working on the HiKey board which has TI BT. > With the serdev bus support, it eliminates the need for the TI userspace > UIM daemon. > > This series and the mentioned drivers can be found here[1]. I took a quick look at the series and have no immediate concerns, just one detail about the DT binding that seems odd to me. Arnd