Commit fb5e4713 authored by Bartosz Golaszewski's avatar Bartosz Golaszewski Committed by Luiz Augusto von Dentz

Bluetooth: qca: don't disable power management for QCA6390

We unnecessarily fallthrough the case for QCA6390 when initializing the
device and hit the condition where - due to the lack of the enable-gpio
- we disable power management despite using the power sequencer. We don't
need to look for clocks on this model so it makes more sense to just
register the hci device and break the switch.
Reported-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Fixes: 9a15ce68 ("Bluetooth: qca: use the power sequencer for QCA6390")
Signed-off-by: default avatarBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Acked-by: default avatarDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Tested-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> # RB5
Signed-off-by: default avatarLuiz Augusto von Dentz <luiz.von.dentz@intel.com>
parent b96bac96
...@@ -2402,7 +2402,13 @@ static int qca_serdev_probe(struct serdev_device *serdev) ...@@ -2402,7 +2402,13 @@ static int qca_serdev_probe(struct serdev_device *serdev)
"bluetooth"); "bluetooth");
if (IS_ERR(qcadev->bt_power->pwrseq)) if (IS_ERR(qcadev->bt_power->pwrseq))
return PTR_ERR(qcadev->bt_power->pwrseq); return PTR_ERR(qcadev->bt_power->pwrseq);
fallthrough;
err = hci_uart_register_device(&qcadev->serdev_hu, &qca_proto);
if (err) {
BT_ERR("qca6390 serdev registration failed");
return err;
}
break;
default: default:
qcadev->bt_en = devm_gpiod_get_optional(&serdev->dev, "enable", qcadev->bt_en = devm_gpiod_get_optional(&serdev->dev, "enable",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment