| Message ID | 20220119061723.2862683-3-adeep@lexina.in |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Bluetooth: hci_h5: btrtl: Add support for RTL8822CS hci_ver 0x08 | expand |
Hi Vyacheslav,
> Add power reset via enable-gpios in h5_btrtl_open function.
this is too sparse commit message. Add explation and if possible also references on why that is needed.
Regards
Marcel
Hi! 19.01.2022 22:32, Marcel Holtmann wrote: > Hi Vyacheslav, > >> Add power reset via enable-gpios in h5_btrtl_open function. > > this is too sparse commit message. Add explation and if possible also references on why that is needed. While testing the RTL8822CS SDIO WiFi/BT adapter, I found that in some cases the kernel could not initialize it. However, manually resetting the adapter via gpio allows it to start correctly. Apparently at system start the adapter is in an undefined state (including the unknown state of gpio after starting uboot). A forced reset helps to initialize the adapter in most cases. Experimentally it was found that 100ms is enough to reset. Since I am new to wireless drivers feel free correct me.
diff --git a/drivers/bluetooth/hci_h5.c b/drivers/bluetooth/hci_h5.c index 34286ffe0568..fdf504b0d265 100644 --- a/drivers/bluetooth/hci_h5.c +++ b/drivers/bluetooth/hci_h5.c @@ -966,6 +966,11 @@ static void h5_btrtl_open(struct h5 *h5) pm_runtime_enable(&h5->hu->serdev->dev); } + /* The controller needs reset to startup */ + gpiod_set_value_cansleep(h5->enable_gpio, 0); + gpiod_set_value_cansleep(h5->device_wake_gpio, 0); + msleep(100); + /* The controller needs up to 500ms to wakeup */ gpiod_set_value_cansleep(h5->enable_gpio, 1); gpiod_set_value_cansleep(h5->device_wake_gpio, 1);
Add power reset via enable-gpios in h5_btrtl_open function. Signed-off-by: Vyacheslav Bocharov <adeep@lexina.in> --- drivers/bluetooth/hci_h5.c | 5 +++++ 1 file changed, 5 insertions(+)