Raspberry PI 4B support

[Ciantic]

I'm trying to implement this, but since I have no expertise I got a bit stuck.

So far I have figured out how to get the LOCAL NAME printed out just once, notice: BCM4345C0 this means something is working somehow already.

sudo ./hid_mouse_demo
Packet Log: /tmp/hci_dump.pklg
Raspberry Pi UART type 3
H4, 3000000, BT_REG_EN at GPIO 129, Warm Boot
Power cycle 0
Flow control 1
Terminal device /dev/serial1
baud 13
current serial terminal parameter baudrate: 9600, flow control: None
Local name: BCM4345C0

Steps I did to get this far:

1. In config config.txt

[all]
dtoverlay=dwc2
dtoverlay=miniuart-bt
dtparam=krnbt=off

(Adding dtoverlay=disable-bt didn't help.)

2. ENABLE_CONTROLLER_WARM_BOOT

3. Added constant MODEL_4B

4. hard coded raspi_get_bluetooth_uart_type to return UART_HARDWARE_FLOW

[mringwal]

If you can get the local name, that's already quite good. Could you upload /tmp/hci_dump.plkg please? You can open it with Wireshark for analysis.

Also: you can try the new port/linux which uses the linux kernel driver to talk to the Bluetooth Controller. (You might need to switch to develop branch)

[Ciantic]

@mringwal here is the packet log:

ZIP: hci_dump.zip

Looks like it is not human readable format:

$ cat /tmp/hci_dump.pklg
8g�+7u�btstack_run_loop_posix.c.320: Pipe: in 5, out 48g�+7��btstack_run_loop_posix.c.320: Pipe: in 7, out 6>g�+9 �btstack_chipset_bcm.c.120: chipset-bcm: init folder .Og�+98�btstack_chipset_bcm.c.120: chipset-bcm: init folder /lib/firmware/brcm<g�+A��le_device_db_tlv.c.164: btstack_tlv not initialized?g�+A��l2cap.c.4996: L2CAP_REGISTER_SERVICE psm 0x1 mtu 65535>g�+B�l2cap.c.4996: L2CAP_REGISTER_SERVICE psm 0x13 mtu 100>g�+B)�l2cap.c.4996: L2CAP_REGISTER_SERVICE psm 0x11 mtu 1009g�+BQ�hci.c.5546: hci_power_control: 1, current mode 0Og�+B`�btstack_chipset_bcm.c.120: chipset-bcm: init folder /lib/firmware/brcm9g�+C#�btstack_uart_posix.c.170: h4_set_baudrate 1152008g��7�btstack_uart_posix.c.361: Open tty /dev/serial1*g��a�hci.c.8159: BTSTACK_EVENT_STATE 1
                                                                                                                                                                                          g���
                                                                                                                                                                                              Jg����btstack_run_loop_posix.c.170: POSIX run loop with monotonic clockg��!

                                                                         g��>g��
                                                                                a)g���hci.c.2984: Manufacturer: 0x000f
                                                                                                                      g��
                                                                                                                         g�+    >�
                                                                                                                                  BCM4345C0*g�+ >1�hci.c.2832: local name: BCM4345C0g�+ \j���-g�+       c�Kg�+  c+�hci.c.2546: Local baud rate change to 3000000(w4_send_baud_change)Fg�+       c8�hci_transport_h4.c.178: hci_transport_h4_set_baudrate 3000000:g�+    cC�btstack_uart_posix.c.170: h4_set_baudrate 3000000[g�+        c{�btstack_chipset_bcm.c.131: chipset-bcm: open file /lib/firmware/brcm/BCM4345C0.hcd
                                                                                                                             g�+        c�.�Kg�+        d|�hci_transport_h4.c.263: hci_transport_h4: invalid packet type 0x84Kg�+       d��hci_transport_h4.c.263: hci_transport_h4: invalid packet type 0x8eKg�+       d��hci_transport_h4.c.263: hci_transport_h4: invalid packet type 0x50Kg�+       d��hci_transport_h4.c.263: hci_transport_h4: invalid packet type 0xaeKg�+       e�hci_transport_h4.c.263: hci_transport_h4: invalid packet type 0x802g�+�a�main.c.193: sigint_handler: shutting down9g�+��hci.c.5546: hci_power_control: 0, current mode 1*g�+��hci.c.5304: hci_power_control_offAg�+�(�hci.c.5309: hci_power_control_off - hci_transport closed;g�+�G�hci.c.5316: hci_power_control_off - control closed*g�+�_�hci.c.8159: BTSTACK_EVENT_STATE 0(g�+��main.c.229: Good bye, see you.

[mringwal]

Well, it's a binary format which you can open with Wireshark.

It shows that communication works in general, that it can find the PatchRAM file for the 4345. After sending the command to change the baudrate to 3000000, we get errors like

hci_transport_h4.c.263: hci_transport_h4: invalid packet type 0x84

This indicates that:

• 3 mbps is not supported on that UART, or,
• flow control is not working.

Please try to use 0 as main baud rate to skip the baud rate change. If that works, you can try increasing to 230400 and 921600 to see if that works.

[Ciantic]

@mringwal

WOW! It works!

sudo ./hid_mouse_demo
Packet Log: /tmp/hci_dump.pklg
Raspberry Pi UART type 3
H4, 0, BT_REG_EN at GPIO 129, Warm Boot
Power cycle 0
Flow control 1
Terminal device /dev/serial1
baud 13
current serial terminal parameter baudrate: 9600, flow control: None
Local name: BCM4345C0
BTstack up and running on B8:27:EB:A2:C9:1B.
HID Connected, control mouse using 'a','s',''d', 'w' keys for movement and 'l' and 'r' for buttons...www

I forced transport_config.baudrate_main = 0;

I would like to make PR, but I have to figure out how to make this cleanly so that it works with older models too.

[Ciantic]

I created a draft PR #668 if someone else stumbles here, it explains how I got this functioning.

I don't understand the old logic there, so I have (for now) circumvented it with just if (model == MODEL_4B) { statements in few places.