Hi Melissa,
On Wed, Mar 18, 2026 at 01:27:11PM -0300, Melissa Wen wrote:
> For suspend/resume to work correctly, do for colorop state the same we
> do for plane/crtc/connector states: duplicate the state of colorops in a
> color pipeline if it's in use by a given plane when suspending and
> restore cached colorop states when resuming.
>
> Fixes: 2afc3184f3b3 ("drm/plane: Add COLOR PIPELINE property")
> Reviewed-by: Harry Wentland <[email protected]>
> Reviewed-by: Alex Hung <[email protected]>
> Reviewed-by: Chaitanya Kumar Borah <[email protected]>
> Signed-off-by: Melissa Wen <[email protected]>Only slightly related to this patch itself, so this discussion must not block merging this patch. I've been looking at revamping the state allocation / reset recently and came across the colorop. It's not clear to me if it's something that should be expected to be reset across a suspend/resume cycle or if it's something that should be maintained across that cycle. What is your opinion on that? Thanks! Maxime
signature.asc
Description: PGP signature
