| Message ID | 20250326124944.6338-1-johan+linaro@kernel.org (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | soc: qcom: pmic_glink: enable UCSI on sc8280xp | expand |
On 3/26/25 1:49 PM, Johan Hovold wrote: > Commit ad3dd9592b2a ("soc: qcom: pmic_glink: disable UCSI on sc8280xp") > disabled UCSI shortly after it had been enabled to fix a regression that > was observed on the Lenovo ThinkPad X13s. > > Specifically, disconnecting an external display would trigger a system > error and hypervisor reset but no one cared enough to track down the bug > at the time. > > The same issue was recently observed on X Elite machines, and commit > f47eba045e6c ("usb: typec: ucsi: Set orientation as none when connector > is unplugged") worked around the underlying issue by setting the > connector orientation to 'none' on disconnect events to avoid having the > PHY driver crash the machine in one orientation. > > Enable UCSI support also on sc8280xp now that the DisplayPort disconnect > crashes are gone. > > Cc: Abel Vesa <abel.vesa@linaro.org> > Cc: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > Signed-off-by: Johan Hovold <johan+linaro@kernel.org> > --- Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> Konrad
On Wed, Mar 26, 2025 at 01:49:44PM +0100, Johan Hovold wrote: > Commit ad3dd9592b2a ("soc: qcom: pmic_glink: disable UCSI on sc8280xp") > disabled UCSI shortly after it had been enabled to fix a regression that > was observed on the Lenovo ThinkPad X13s. > > Specifically, disconnecting an external display would trigger a system > error and hypervisor reset but no one cared enough to track down the bug > at the time. > > The same issue was recently observed on X Elite machines, and commit > f47eba045e6c ("usb: typec: ucsi: Set orientation as none when connector > is unplugged") worked around the underlying issue by setting the > connector orientation to 'none' on disconnect events to avoid having the > PHY driver crash the machine in one orientation. > > Enable UCSI support also on sc8280xp now that the DisplayPort disconnect > crashes are gone. > > Cc: Abel Vesa <abel.vesa@linaro.org> > Cc: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > Signed-off-by: Johan Hovold <johan+linaro@kernel.org> > --- > drivers/soc/qcom/pmic_glink.c | 4 ---- > 1 file changed, 4 deletions(-) > Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
On 25-03-26 13:49:44, Johan Hovold wrote: > Commit ad3dd9592b2a ("soc: qcom: pmic_glink: disable UCSI on sc8280xp") > disabled UCSI shortly after it had been enabled to fix a regression that > was observed on the Lenovo ThinkPad X13s. > > Specifically, disconnecting an external display would trigger a system > error and hypervisor reset but no one cared enough to track down the bug > at the time. > > The same issue was recently observed on X Elite machines, and commit > f47eba045e6c ("usb: typec: ucsi: Set orientation as none when connector > is unplugged") worked around the underlying issue by setting the > connector orientation to 'none' on disconnect events to avoid having the > PHY driver crash the machine in one orientation. > > Enable UCSI support also on sc8280xp now that the DisplayPort disconnect > crashes are gone. > > Cc: Abel Vesa <abel.vesa@linaro.org> > Cc: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > Signed-off-by: Johan Hovold <johan+linaro@kernel.org> Reviewed-by: Abel Vesa <abel.vesa@linaro.org>
diff --git a/drivers/soc/qcom/pmic_glink.c b/drivers/soc/qcom/pmic_glink.c index cde19cdfd3c7..0a6d325b195c 100644 --- a/drivers/soc/qcom/pmic_glink.c +++ b/drivers/soc/qcom/pmic_glink.c @@ -371,15 +371,11 @@ static void pmic_glink_remove(struct platform_device *pdev) __pmic_glink = NULL; } -static const unsigned long pmic_glink_sc8280xp_client_mask = BIT(PMIC_GLINK_CLIENT_BATT) | - BIT(PMIC_GLINK_CLIENT_ALTMODE); - static const unsigned long pmic_glink_sm8450_client_mask = BIT(PMIC_GLINK_CLIENT_BATT) | BIT(PMIC_GLINK_CLIENT_ALTMODE) | BIT(PMIC_GLINK_CLIENT_UCSI); static const struct of_device_id pmic_glink_of_match[] = { - { .compatible = "qcom,sc8280xp-pmic-glink", .data = &pmic_glink_sc8280xp_client_mask }, { .compatible = "qcom,pmic-glink", .data = &pmic_glink_sm8450_client_mask }, {} };
Commit ad3dd9592b2a ("soc: qcom: pmic_glink: disable UCSI on sc8280xp") disabled UCSI shortly after it had been enabled to fix a regression that was observed on the Lenovo ThinkPad X13s. Specifically, disconnecting an external display would trigger a system error and hypervisor reset but no one cared enough to track down the bug at the time. The same issue was recently observed on X Elite machines, and commit f47eba045e6c ("usb: typec: ucsi: Set orientation as none when connector is unplugged") worked around the underlying issue by setting the connector orientation to 'none' on disconnect events to avoid having the PHY driver crash the machine in one orientation. Enable UCSI support also on sc8280xp now that the DisplayPort disconnect crashes are gone. Cc: Abel Vesa <abel.vesa@linaro.org> Cc: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> Signed-off-by: Johan Hovold <johan+linaro@kernel.org> --- drivers/soc/qcom/pmic_glink.c | 4 ---- 1 file changed, 4 deletions(-)