linux-kernel - Re: [PATCH v3 0/5] Serial slave device bus

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 31 Jan 2017 06:44:15 +0100
From:   Sebastian Reichel <sre@...nel.org>
To:     Rob Herring <robh@...nel.org>
Cc:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Marcel Holtmann <marcel@...tmann.org>,
        Jiri Slaby <jslaby@...e.com>, Arnd Bergmann <arnd@...db.de>,
        "Dr . H . Nikolaus Schaller" <hns@...delico.com>,
        Peter Hurley <peter@...leysoftware.com>,
        Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
        Alan Cox <gnomes@...rguk.ukuu.org.uk>,
        Loic Poulain <loic.poulain@...el.com>,
        Pavel Machek <pavel@....cz>, NeilBrown <neil@...wn.name>,
        Linus Walleij <linus.walleij@...aro.org>,
        linux-bluetooth@...r.kernel.org, linux-serial@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 0/5] Serial slave device bus

Hi,

On Tue, Jan 31, 2017 at 04:02:41AM +0100, Sebastian Reichel wrote:
> On Wed, Jan 25, 2017 at 02:23:38PM -0600, Rob Herring wrote:
> > Here's V3 of the serdev bus support with all the review feedback so far
> > incorporated. Greg applied the first 4 patches already. I also found a
> > problem in the tty_port_wakeup changes and have fixed that.
> > 
> > I've added serdev support to the BT hci_ll.c driver as it turns out it works
> > on newer TI chips too, but just needed firmware loading. That avoids the
> > impossible mess of cleaning up the TI-ST driver, and it's in a much closer
> > state to being ready for upstream. I've sent that out separately.
> > 
> > Changelog is in individual patches. Previous versions are here[1][2]. This
> > series and the mentioned drivers can be found here[3].
> > 
> > Rob
> > 
> > [1] http://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1304151.html
> > [2] http://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1311650.html
> > [3] git://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git serial-bus-v4
> 
> I rebased my nokia bluetooth driver on top of [3]. It's not yet
> working, but everything added by this patchset (and
> "bluetooth: hci_uart: add serdev driver support library") seems
> to work as expected, so for the whole series:
> 
> Tested-By: Sebastian Reichel <sre@...nel.org>

FWIW, I got bluetooth on top of your patchset working on N950. N900
is not yet tested and probably does not yet work (it's more picky
about flow control and usage of the wakeup gpios). You can find a
branch with the additional patches @ [4]. I had to add serdev
functions to wait for TX buffer depletion and checking CTS. Apart
from that flow control only worked correctly when also taking care
of RTS (like hci_uart_set_flow_control does).

I noticed, that you did not yet write any DT bindings for your
bluetooth slaves. Did you already decide for a location?

[4] git://git.kernel.org/pub/scm/linux/kernel/git/sre/linux-n900.git serial-bus-v4-nokia-bt

-- Sebastian

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)