(Dropping Andrzej, because that address keeps bouncing. Does MAINTAINERS and/or .mailmap need updating?)
Apologies for the double reply here, but I forgot to mention one last thing for now:
On Wed, Oct 20, 2021 at 5:40 PM Sean Paul sean@poorly.run wrote:
On Wed, Oct 20, 2021 at 04:17:28PM -0700, Brian Norris wrote:
Prior to commit 6c836d965bad ("drm/rockchip: Use the helpers for PSR"), "PSR disable" used non-blocking analogix_dp_send_psr_spd(). The refactor accidentally (?) set blocking=true.
IIRC this wasn't accidental.
One other tip that made me think it was accidental was that today, the |blocking| argument to analogix_dp_send_psr_spd() is always true. If non-blocking support was intentionally dropped, it seemed like you should have dropped the non-blocking code too. But that's a weak proof of your intentions :)
Brian