[v2,1/2] drm/msm/dpu1: don't choke on disabling the writeback connector
György Kurucz
me at kuruczgy.com
Sat Aug 31 18:46:32 UTC 2024
Dear Leonard,
> Do you observe this issue on every suspend-resume cycle?
I just did 10 suspend/resume cycles in a row to double check, and
without this patch the screen never comes back (always have to switch VT
back-and-forth to bring it back). The
[dpu error]connector not connected 3
[drm:drm_mode_config_helper_resume] *ERROR* Failed to resume (-22)
pair of error messages also consistently appears after all resumes.
Though I think e.g. Rob Clark reported that suspend/resume already works
properly for him without this patch, so this experience is not universal
on the Yoga Slim 7x.
> On sc7180 lazor, I do observe that this patch deterministically breaks restoring the CRTC state and functionality after resume. Can you please validate if you observe the same on Lenovo Yoga Slim 7x? Specifically, try set Night Light in your desktop environment to "Always On" and observe whether the screen remains in "Night Light" mode after resume. For lazor, "Night Light" is breaks after applying this patch and even manually toggling it off and on after resume does not restore "Night Light" / CRTC functionality.
Unfortunately I cannot test this, as color temperature adjustments seems
to be completely non-functional for me in the first place. For color
temperature adjustment, I use gammastep on my machines, which uses
wlr_gamma_control_unstable_v1 under the hood. It outputs the following
warnings:
Warning: Zero outputs support gamma adjustment.
Warning: 1/1 output(s) do not support gamma adjustment.
I haven't dug deeper into the cause yet, based on these it seems that
wlroots isn't detecting the display as being gamma-adjustable in the
first place.
Best regards,
György
More information about the dri-devel
mailing list