Return-Path: Content-Type: text/plain; charset=utf-8 Mime-Version: 1.0 (Mac OS X Mail 10.3 \(3273\)) Subject: Re: [PATCH v3 2/3] ARM: dts: bcm2837-rpi-3-b: Add bcm43438 as serial slave From: Marcel Holtmann In-Reply-To: <92579650-5eb0-cdf2-405a-7c5be9c54770@gmail.com> Date: Thu, 17 Aug 2017 12:35:37 +0200 Cc: Peter Robinson , "devicetree@vger.kernel.org" , Johan Hedberg , Scott Branden , Ray Jui , "open list:BLUETOOTH DRIVERS" , Rob Herring , linux-rpi-kernel@lists.infradead.org Message-Id: <8CF7E37E-BF1C-4E32-AF3A-E44BA2E5A2B9@holtmann.org> References: <1502102366-2760-1-git-send-email-loic.poulain@gmail.com> <1502102366-2760-2-git-send-email-loic.poulain@gmail.com> <8186E2A3-D0AC-498F-8229-78862D363F78@holtmann.org> <392e6d51-33d6-1585-6582-1397a0c64852@gmail.com> <161632C7-EBBF-47F5-86DC-453E10395B54@holtmann.org> <0C198B33-8CB6-4D6A-B89F-35D3CD771135@holtmann.org> <92579650-5eb0-cdf2-405a-7c5be9c54770@gmail.com> To: Loic Poulain , Eric Anholt Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Loic, >>>> < HCI Command: Broadcom Write UART Clock Setting (0x3f|0x0045) plen 1 >>>> 01 . >>>>> HCI Event: Command Complete (0x0e) plen 4 >>>> Broadcom Write UART Clock Setting (0x3f|0x0045) ncmd 1 >>>> Status: Unknown HCI Command (0x01) >>>> >>>> And I am seeing fun stuff like failed frame assembly. >>>> >>>> [ 888.687594] Bluetooth: hci0: BCM: chip id 94 >>>> [ 888.687821] Bluetooth: hci0: BCM43430A1 (001.002.009) build 0182 >>>> [ 892.059023] Bluetooth: hci0: Frame reassembly failed (-84) >>>> [ 892.316936] Bluetooth: hci0: BCM: failed to write clock (-56) >>>> [ 892.429478] Bluetooth: hci0: BCM (001.002.009) build 0182 >>>> >>>> Actually not providing the firmware makes the controller work. It however is stuck ad AA:AA:.. default address. Providing the firmware turns the address active. However then it never completes. >>> I've tried on and off to get the BT working, there seems to lots of >>> options and bits needed including some patches to the bluez [1] stuff >>> but between not quite upstream kernel bits and numerous distros all >>> doing it slightly differently I've never got it to work well. >>> >>> The yocto [1] bits seem fairly representative of some the patches >>> flying around "to get it working" although I'm not sure how many of >>> these are actually required and how many are superfluous with this >>> patch set. There seems to be a firmware required that's not >>> distributed with linux-firmware which would also be nice to resolve. >> Non of these Yocto patches are actually needed. The culprit is the .oper_speed setting to be 4Mbps. Once you reduce that to 921600 thing will start to work smoothly. I sent a patch that takes the .oper_speed out completely and only applies it for the ACPI based devices where we know that it works. >> >> With my patch and the right DT entries for uart0 it actually works with “btattach -B /dev/ttyAMA0 -P bcm”. It will load the firmware, configure it and head towards the right path. >> >> Obviously btattach is only an interim step here. Loic’s patches for serdev integration and changing the DT to expose uart0 as serial-slave for Bluetooth is the right approach. Once Loic’s resends the patches we can get them into bluetooth-next and start merging these towards upstream. After that, Bluetooth should work just out of the box like with any USB dongle. >> >> And the Yocto patches should be abandoned. If using H:5 (aka 3-Wire) instead of H:4 is possible, we could consider it, but as long as the UART wiring doesn’t cause any bit errors, it is not worth it. >> >> That said, I do see a "Bluetooth: hci0: Frame reassembly failed (-84)” error. I need to figure out where that is. Frankly we really need to hexdump the packet when this happens. >> > > I also meet this Frame reassembly failure, Seems we receive a 0x00 byte from the controller (unknown pkt type). that means adding something like this will silence it. +#define BCM_RECV_NULL \ + .type = 0x00, \ + .hlen = 0, \ + .loff = 0, \ + .lsize = 0, \ + .maxlen = 0 + +static int bcm_recv_null(struct hci_dev *hdev, struct sk_buff *skb) +{ + kfree_skb(skb); + return 0; +} + static const struct h4_recv_pkt bcm_recv_pkts[] = { { H4_RECV_ACL, .recv = hci_recv_frame }, { H4_RECV_SCO, .recv = hci_recv_frame }, { H4_RECV_EVENT, .recv = hci_recv_frame }, { BCM_RECV_LM_DIAG, .recv = hci_recv_diag }, + { BCM_RECV_NULL, .recv = bcm_recv_null }, }; It does silence it indeed. Maybe this is some of their markers to ensure that the baud rate change worked. Or it is the indication that the reset completed. Do you happen to know between which commands we receive it? > Regarding the speed, I'm unable to reach 3Mbps, I selected 921600 because this is the baudrate used by raspbian bt script. That is the same I experienced. The 921600 works fine, but trying 3Mbps doesn’t. However it seems with hciattach you can crank it up to 3Mpbs somehow. Maybe the delay is just to short for the UART switching in that case. > Maybe they had some issues at higher speed (empirical value ?), However 2Mbps seems ok on my side (need to double check/adjust). We have to make it a DT max-speed param anyway. So I am not that worried. > In a second step, need to check If we can use hardware flow control, I heard that pin 16&17 are routed to the bcm RTS/CTS. > Since there is no DMA usage, it could help. Eric, any chance you can dig out the recommendation for the Bluetooth controller usage? I would also like to see the recommended PCM settings for this hardware. It bet it is somehow wired up to the sound controller. Regards Marcel