Re: [PATCH] drm/msm/dp: fix runtime_pm handling in dp_wait_hpd_asserted

2024-02-28 Thread Stephen Boyd
Quoting Dmitry Baryshkov (2024-02-26 14:34:45) > The function dp_wait_hpd_asserted() uses pm_runtime_get_sync() and > doesn't care about the return value. Potentially this can lead to > unclocked access if for some reason resuming of the DP controller fails. > > Change the function to use

Re: [PATCH] drm/msm/dp: fix runtime_pm handling in dp_wait_hpd_asserted

2024-02-26 Thread Abhinav Kumar
On 2/26/2024 2:34 PM, Dmitry Baryshkov wrote: The function dp_wait_hpd_asserted() uses pm_runtime_get_sync() and doesn't care about the return value. Potentially this can lead to unclocked access if for some reason resuming of the DP controller fails. Change the function to use

[PATCH] drm/msm/dp: fix runtime_pm handling in dp_wait_hpd_asserted

2024-02-26 Thread Dmitry Baryshkov
The function dp_wait_hpd_asserted() uses pm_runtime_get_sync() and doesn't care about the return value. Potentially this can lead to unclocked access if for some reason resuming of the DP controller fails. Change the function to use pm_runtime_resume_and_get() and return an error if resume fails.