Re: [PATCH] Bluetooth: hci_serdev: Init hci_uart proto_lock to avoid oops

Hi John, John Stultz reports a boot time crash with the HiKey board (which uses hci_serdev) occurring in hci_uart_tx_wakeup(). That function is contained in hci_ldisc.c, but also called from the newer hci_serdev.c. It acquires the proto_lock in struct hci_uart and it turns

Re: [PATCH] Bluetooth: hci_serdev: Init hci_uart proto_lock to avoid oops

Hi John, John Stultz reports a boot time crash with the HiKey board (which uses hci_serdev) occurring in hci_uart_tx_wakeup(). That function is contained in hci_ldisc.c, but also called from the newer hci_serdev.c. It acquires the proto_lock in struct hci_uart and it turns

Re: [PATCH] Bluetooth: hci_serdev: Init hci_uart proto_lock to avoid oops

On Mon, Dec 11, 2017 at 1:49 PM, John Stultz wrote: > On Thu, Nov 16, 2017 at 10:07 PM, Marcel Holtmann wrote: >> Hi Lukas, >> >>> John Stultz reports a boot time crash with the HiKey board (which uses >>> hci_serdev) occurring in

Re: [PATCH] Bluetooth: hci_serdev: Init hci_uart proto_lock to avoid oops

On Mon, Dec 11, 2017 at 1:49 PM, John Stultz wrote: > On Thu, Nov 16, 2017 at 10:07 PM, Marcel Holtmann wrote: >> Hi Lukas, >> >>> John Stultz reports a boot time crash with the HiKey board (which uses >>> hci_serdev) occurring in hci_uart_tx_wakeup(). That function is >>> contained in

Re: [PATCH] Bluetooth: hci_serdev: Init hci_uart proto_lock to avoid oops

On Thu, Nov 16, 2017 at 10:07 PM, Marcel Holtmann wrote: > Hi Lukas, > >> John Stultz reports a boot time crash with the HiKey board (which uses >> hci_serdev) occurring in hci_uart_tx_wakeup(). That function is >> contained in hci_ldisc.c, but also called from the newer

Re: [PATCH] Bluetooth: hci_serdev: Init hci_uart proto_lock to avoid oops

On Thu, Nov 16, 2017 at 10:07 PM, Marcel Holtmann wrote: > Hi Lukas, > >> John Stultz reports a boot time crash with the HiKey board (which uses >> hci_serdev) occurring in hci_uart_tx_wakeup(). That function is >> contained in hci_ldisc.c, but also called from the newer hci_serdev.c. >> It

Re: [PATCH] Bluetooth: hci_serdev: Init hci_uart proto_lock to avoid oops

Hi Lukas, > John Stultz reports a boot time crash with the HiKey board (which uses > hci_serdev) occurring in hci_uart_tx_wakeup(). That function is > contained in hci_ldisc.c, but also called from the newer hci_serdev.c. > It acquires the proto_lock in struct hci_uart and it turns out that we >

Re: [PATCH] Bluetooth: hci_serdev: Init hci_uart proto_lock to avoid oops

Hi Lukas, > John Stultz reports a boot time crash with the HiKey board (which uses > hci_serdev) occurring in hci_uart_tx_wakeup(). That function is > contained in hci_ldisc.c, but also called from the newer hci_serdev.c. > It acquires the proto_lock in struct hci_uart and it turns out that we >

Re: [PATCH] Bluetooth: hci_serdev: Init hci_uart proto_lock to avoid oops

On Fri, Nov 17, 2017 at 12:54:53AM +0100, Lukas Wunner wrote: > John Stultz reports a boot time crash with the HiKey board (which uses > hci_serdev) occurring in hci_uart_tx_wakeup(). That function is > contained in hci_ldisc.c, but also called from the newer hci_serdev.c. > It acquires the

Re: [PATCH] Bluetooth: hci_serdev: Init hci_uart proto_lock to avoid oops

On Fri, Nov 17, 2017 at 12:54:53AM +0100, Lukas Wunner wrote: > John Stultz reports a boot time crash with the HiKey board (which uses > hci_serdev) occurring in hci_uart_tx_wakeup(). That function is > contained in hci_ldisc.c, but also called from the newer hci_serdev.c. > It acquires the

[PATCH] Bluetooth: hci_serdev: Init hci_uart proto_lock to avoid oops

John Stultz reports a boot time crash with the HiKey board (which uses hci_serdev) occurring in hci_uart_tx_wakeup(). That function is contained in hci_ldisc.c, but also called from the newer hci_serdev.c. It acquires the proto_lock in struct hci_uart and it turns out that we forgot to init the

[PATCH] Bluetooth: hci_serdev: Init hci_uart proto_lock to avoid oops

John Stultz reports a boot time crash with the HiKey board (which uses hci_serdev) occurring in hci_uart_tx_wakeup(). That function is contained in hci_ldisc.c, but also called from the newer hci_serdev.c. It acquires the proto_lock in struct hci_uart and it turns out that we forgot to init the