Hi Hans, > The recently added H5_WAKEUP_DISABLE h5->flags flag gets checked in > h5_btrtl_open(), but it gets set in h5_serdev_probe() *after* > calling hci_uart_register_device() and thus after h5_btrtl_open() > is called, set this flag earlier. > > Also on devices where suspend/resume involves fully re-probing the HCI, > runtime-pm suspend should not be used, make the runtime-pm setup > conditional on the H5_WAKEUP_DISABLE flag too. > > This fixes the HCI being removed and then re-added every 10 seconds > because it was being reprobed as soon as it was runtime-suspended. > > Cc: Archie Pusaka <apusaka@xxxxxxxxxxxx> > Fixes: 66f077dde749 ("Bluetooth: hci_h5: add WAKEUP_DISABLE flag") > Fixes: d9dd833cf6d2 ("Bluetooth: hci_h5: Add runtime suspend") > Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> > --- > drivers/bluetooth/hci_h5.c | 20 +++++++++++--------- > 1 file changed, 11 insertions(+), 9 deletions(-) patch has been applied to bluetooth-next tree. Regards Marcel