linux-kernel - Re: partial bluetooth success on n900 [was Re: bluetooth/uart timeout handling]

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]

Message-ID: <20161214133158.mwvs574l4otbc7hf@earth>
Date:   Wed, 14 Dec 2016 14:31:58 +0100
From:   Sebastian Reichel <sre@...nel.org>
To:     Pavel Machek <pavel@....cz>
Cc:     Pali Rohár <pali.rohar@...il.com>,
        tony@...mide.com, kernel list <linux-kernel@...r.kernel.org>,
        linux-arm-kernel <linux-arm-kernel@...ts.infradead.org>,
        linux-omap@...r.kernel.org, khilman@...nel.org,
        aaro.koskinen@....fi, ivo.g.dimitrov.75@...il.com,
        patrikbachan@...il.com, serge@...lyn.com, abcloriens@...il.com
Subject: Re: partial bluetooth success on n900 [was Re: bluetooth/uart
 timeout handling]

Hi Pali & Pavel,

On Wed, Dec 14, 2016 at 01:53:23PM +0100, Pavel Machek wrote:
> > > [  220.248596] tty ttyO1: Radio packet sent
> > > [  220.249328] Bluetooth: hci0: Frame reassembly failed (-84)
> > > [  220.272949] tty ttyO1: wakeup received: 1 -> 0
> > > [  221.283477] tty ttyO1: radio packet timeout!
> > > [  221.283630] enqueue: hu c304cc80 skb cd4a9b40
> > > [  223.363372] Bluetooth: hci0 command 0xfc18 tx timeout
> > > pavel@...0:~$
> > 
> > In log are still some failures, but ... is bluetooth working now?

I could scan for devices. The code is still racy, though. It's
most likely related to the newly introduced idle code. (Without
sending the BT module to correctly idle the bcm2048 does not
work correctly at all)

I was quite busy the last few weeks and did not manage to find
much time for kernel work. Now I will first have to catch up
with my power-supply tree.

> It is... for Sebastian. I'm playing with camera now.
> 
> > I see that you applied this patch: 
> > https://git.kernel.org/cgit/linux/kernel/git/pavel/linux-n900.git/commit/?id=051aa3fbf03ac770d8344690f5a936a7f04c6884
> > 
> > Looks like that pinmux is in DTS file incorrect. Can somebody verify it? 
> > Maybe Tony?
> 
> Yes, it is. Sebastian was pretty certain about that.

Yes, I'm certain. The bootloader enables the pullup resistors.
Note, that the wrong DTS entry is not in mainline. My bluetooth
branch has a fixed DTS patch instead of a fixup patch on top of
the broken one:

https://git.kernel.org/cgit/linux/kernel/git/sre/linux-n900.git/commit/?h=nokia-bluetooth-dev&id=6b63c111a979d100cfbdd76cb4a6bbadace35216

-- Sebastian

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