Merge "Bluetooth: Fix BT_EN not made high in specific usecase"

This commit is contained in:
qctecmdr 2021-10-29 17:16:57 -07:00 committed by Gerrit - the friendly Code Review server
commit 55d9b9d21d

View File

@ -422,6 +422,9 @@ static int bt_configure_gpios(int on)
return rc;
}
pr_info("BTON:Turn Bt OFF asserting BT_EN to low\n");
pr_info("bt-reset-gpio(%d) value(%d)\n", bt_reset_gpio,
gpio_get_value(bt_reset_gpio));
rc = gpio_direction_output(bt_reset_gpio, 0);
if (rc) {
pr_err("%s: Unable to set direction\n", __func__);
@ -430,25 +433,27 @@ static int bt_configure_gpios(int on)
PWR_SRC_STATUS_SET(BT_RESET_GPIO,
gpio_get_value(bt_reset_gpio));
msleep(50);
pr_info("BTON:Turn Bt Off bt-reset-gpio(%d) value(%d)\n",
bt_reset_gpio, gpio_get_value(bt_reset_gpio));
pr_info("BTON:Turn Bt OFF post asserting BT_EN to low\n");
pr_info("bt-reset-gpio(%d) value(%d)\n", bt_reset_gpio,
gpio_get_value(bt_reset_gpio));
if (bt_sw_ctrl_gpio >= 0) {
pr_info("BTON:Turn Bt Off\n");
PWR_SRC_STATUS_SET(BT_SW_CTRL_GPIO,
gpio_get_value(bt_sw_ctrl_gpio));
pr_info("bt-sw-ctrl-gpio(%d) value(%d)\n",
pr_info("BTON:Turn Bt OFF bt-sw-ctrl-gpio(%d) value(%d)\n",
bt_sw_ctrl_gpio,
bt_power_src_status[BT_SW_CTRL_GPIO]);
}
if (wl_reset_gpio >= 0)
pr_info("BTON:Turn Bt On wl-reset-gpio(%d) value(%d)\n",
pr_info("BTON:Turn Bt ON wl-reset-gpio(%d) value(%d)\n",
wl_reset_gpio, gpio_get_value(wl_reset_gpio));
if ((wl_reset_gpio < 0) ||
((wl_reset_gpio >= 0) && gpio_get_value(wl_reset_gpio))) {
btpower_set_xo_reset_gpio_state(true);
pr_info("%s: BTON: Asserting BT_EN\n", __func__);
pr_info("BTON: WLAN ON Asserting BT_EN to high\n");
rc = gpio_direction_output(bt_reset_gpio, 1);
if (rc) {
pr_err("%s: Unable to set direction\n", __func__);
@ -458,21 +463,24 @@ static int bt_configure_gpios(int on)
gpio_get_value(bt_reset_gpio));
btpower_set_xo_reset_gpio_state(false);
}
if ((wl_reset_gpio >= 0) && (gpio_get_value(wl_reset_gpio) == 0)) {
if (gpio_get_value(bt_reset_gpio)) {
pr_info("%s: Wlan Off and BT On too close\n", __func__);
pr_info("%s: reset BT_EN, enable it after delay\n", __func__);
pr_info("BTON: WLAN OFF and BT ON are too close\n");
pr_info("reset BT_EN, enable it after delay\n");
rc = gpio_direction_output(bt_reset_gpio, 0);
if (rc) {
pr_err("%s: Unable to set direction\n", __func__);
pr_err("%s: Unable to set direction\n",
__func__);
return rc;
}
PWR_SRC_STATUS_SET(BT_RESET_GPIO,
gpio_get_value(bt_reset_gpio));
}
pr_info("%s:add 100ms delay for AON output to fully discharge\n",
__func__);
pr_info("BTON: WLAN OFF waiting for 100ms delay\n");
pr_info("for AON output to fully discharge\n");
msleep(100);
pr_info("BTON: WLAN OFF Asserting BT_EN to high\n");
btpower_set_xo_reset_gpio_state(true);
rc = gpio_direction_output(bt_reset_gpio, 1);
if (rc) {
@ -484,6 +492,24 @@ static int bt_configure_gpios(int on)
btpower_set_xo_reset_gpio_state(false);
}
/* Below block of code executes if WL_EN is pulled high when
* BT_EN is about to pull high. so above two if conditions are
* not executed.
*/
if (!gpio_get_value(bt_reset_gpio)) {
btpower_set_xo_reset_gpio_state(true);
pr_info("BTON: WLAN ON and BT ON are too close\n");
pr_info("Asserting BT_EN to high\n");
rc = gpio_direction_output(bt_reset_gpio, 1);
if (rc) {
pr_err("%s: Unable to set direction\n", __func__);
return rc;
}
PWR_SRC_STATUS_SET(BT_RESET_GPIO,
gpio_get_value(bt_reset_gpio));
btpower_set_xo_reset_gpio_state(false);
}
msleep(50);
/* Check if SW_CTRL is asserted */
if (bt_sw_ctrl_gpio >= 0) {
@ -515,10 +541,9 @@ static int bt_configure_gpios(int on)
pr_info("BTON:Turn Bt On bt-reset-gpio(%d) value(%d)\n",
bt_reset_gpio, gpio_get_value(bt_reset_gpio));
if (bt_sw_ctrl_gpio >= 0) {
pr_info("BTON:Turn Bt On\n");
PWR_SRC_STATUS_SET(BT_SW_CTRL_GPIO,
gpio_get_value(bt_sw_ctrl_gpio));
pr_info("bt-sw-ctrl-gpio(%d) value(%d)\n",
pr_info("BTON: Turn BT ON bt-sw-ctrl-gpio(%d) value(%d)\n",
bt_sw_ctrl_gpio,
bt_power_src_status[BT_SW_CTRL_GPIO]);
}