On Fri, 8 Dec 2023 at 13:09, Johan Hovold <johan@xxxxxxxxxx> wrote: > > On Fri, Dec 08, 2023 at 12:58:29PM +0200, Dmitry Baryshkov wrote: > > On Fri, 8 Dec 2023 at 10:39, Johan Hovold <johan@xxxxxxxxxx> wrote: > > > > > > On Wed, Oct 25, 2023 at 02:49:28PM +0300, Dmitry Baryshkov wrote: > > > > The UCSI firmware on Qualcomm SC8180X, SC8280XP and SM8350 are buggy. > > > > Submitting UCSI_GET_PDOS command for partners which do not actually > > > > support PD and do not have PDOs causes firmware to crash, preventing > > > > further UCSI activity. Firmware on newer platforms have fixed this > > > > issue. In order to still be able to use UCSI functionality on the > > > > mentioned platforms (e.g. to be able to handle USB role switching), > > > > apply a workaround that completely shortcuts UCSI_GET_PDOS command for > > > > the USB-C partner. > > > > > > > > This has been tested on sm8350 only, but should apply to other > > > > platforms. I did not enable UCSI for sc8180x yet, it has slightly > > > > different implementation, which I'd like to get tested first. > > > > > > Has no one tested this on sc8280xp/x13s before merging? > > > > > > I see a bunch of errors with this series applied to 6.7-rc4: > > > > > > [ 11.999960] ucsi_glink.pmic_glink_ucsi pmic_glink.ucsi.0: timeout waiting for UCSI sync write response > > > [ 12.000430] ucsi_glink.pmic_glink_ucsi pmic_glink.ucsi.0: ucsi_handle_connector_change: GET_CONNECTOR_STATUS failed (-110) > > > [ 17.120515] ucsi_glink.pmic_glink_ucsi pmic_glink.ucsi.0: timeout waiting for UCSI sync write response > > > [ 17.124204] ucsi_glink.pmic_glink_ucsi pmic_glink.ucsi.0: GET_CONNECTOR_STATUS failed (-110) > > > [ 23.264792] ucsi_glink.pmic_glink_ucsi pmic_glink.ucsi.0: timeout waiting for UCSI sync write response > > > [ 23.264953] ucsi_glink.pmic_glink_ucsi pmic_glink.ucsi.0: GET_CONNECTOR_STATUS failed (-110) > > > > Can you please post previous messages or is the first timeout the > > first error from ucsi? > > These are all the ucsi messages in the log (dmesg | grep ucsi). > > The first error is sometimes GET_CONNECTOR_STATUS failed (-95) instead: Ack, thank you. This is pending on my side together with the UCSI glink / altmode rework. I hope to have patches for that closer to the NY. > > [ 9.012421] ucsi_glink.pmic_glink_ucsi pmic_glink.ucsi.0: GET_CONNECTOR_STATUS failed (-95) > [ 14.047379] ucsi_glink.pmic_glink_ucsi pmic_glink.ucsi.0: timeout waiting for UCSI sync write response > [ 14.050708] ucsi_glink.pmic_glink_ucsi pmic_glink.ucsi.0: GET_CONNECTOR_STATUS failed (-110) > [ 20.192382] ucsi_glink.pmic_glink_ucsi pmic_glink.ucsi.0: timeout waiting for UCSI sync write response > [ 20.192542] ucsi_glink.pmic_glink_ucsi pmic_glink.ucsi.0: GET_CONNECTOR_STATUS failed (-110) > > I see that one if I boot with only the charger connected, the later -110 > timeouts go away if I disconnect my r8152 ethernet adapter. > > Johan -- With best wishes Dmitry