Hi Heiko, On Tue Oct 8, 2024 at 9:28 PM CEST, Heiko Stuebner wrote: > On Tue, 8 Oct 2024 13:15:35 +0200, Diederik de Haas wrote: > > This is a set of 4 small device-tree validation fixes. > > > > Patch 1 adds the power-domains property to the csi dphy node on rk356x. > > Patch 2 removes the 2nd interrupt from the hdmi node on rk3328. > > Patch 3 replaces 'wake' with 'wakeup' on PineNote BT node. > > Patch 4 replaces 'reset-gpios' with 'shutdown-gpios' on brcm BT nodes. > > Applied, thanks! > > [2/4] arm64: dts: rockchip: Remove hdmi's 2nd interrupt on rk3328 > commit: de50a7e3681771c6b990238af82bf1dea9b11b21 > [3/4] arm64: dts: rockchip: Fix wakeup prop names on PineNote BT node > commit: 87299d6ee95a37d2d576dd8077ea6860f77ad8e2 > [4/4] arm64: dts: rockchip: Fix reset-gpios property on brcm BT nodes > commit: 2b6a3f857550e52b1cd4872ebb13cb3e3cf12f5f Please revert the 4th patch. I must have messed up my testing previously, but BT does not work on the PineNote with the 4th patch applied and does work with it reverted. With the 4th patch applied I get this: [ 20.298465] Bluetooth: hci0: command 0x0c03 tx timeout [ 20.299036] Bluetooth: hci0: BCM: Reset failed (-110) And then running the `list` command in `bluetoothctl` returns no controllers and `hciconfig -a hci0` reports `DOWN` and when trying to reset hci0, I get that timeout error again. With the 4th patch reverted, the controller does work. I also get a HUGE amount of 'spam' when running `bluetoothctl` like: [bluetooth]# [CHG] Controller 90:E8:68:B9:60:46 UUIDs: 0000110e-0000-1000-8000-00805f9b34fb [bluetooth]# [CHG] Controller 90:E8:68:B9:60:46 UUIDs: 0000112d-0000-1000-8000-00805f9b34fb [bluetooth]# [CHG] Controller 90:E8:68:B9:60:46 UUIDs: 00001200-0000-1000-8000-00805f9b34fb [bluetooth]# [CHG] Controller 90:E8:68:B9:60:46 UUIDs: 0000111f-0000-1000-8000-00805f9b34fb [bluetooth]# [CHG] Controller 90:E8:68:B9:60:46 UUIDs: 0000110c-0000-1000-8000-00805f9b34fb [bluetooth]# [CHG] Controller 90:E8:68:B9:60:46 UUIDs: 00001800-0000-1000-8000-00805f9b34fb [bluetooth]# [CHG] Controller 90:E8:68:B9:60:46 UUIDs: 00001801-0000-1000-8000-00805f9b34fb [bluetooth]# [CHG] Controller 90:E8:68:B9:60:46 UUIDs: 0000180a-0000-1000-8000-00805f9b34fb [bluetooth]# [CHG] Controller 90:E8:68:B9:60:46 UUIDs: 03b80e5a-ede8-4b33-a751-6ce34ec4c700 and this is only a tiny fraction. So there's definitely room for improvement there, but at least it does work. Dunno yet what to make of it as this will re-introduce the DT validation error, but working hardware seems still be preferable to non-working HW. Sorry about this. Regards, Diederik
Attachment:
signature.asc
Description: PGP signature
