Re: [PATCH v5 12/19] drm: writeback: Cleanup job ownership handling when queuing job

Laurent Pinchart laurent.pinchart+renesas@ideasonboard.com writes:

The drm_writeback_queue_job() function takes ownership of the passed job and requires the caller to manually set the connector state writeback_job pointer to NULL. To simplify drivers and avoid errors (such as the missing NULL set in the vc4 driver), pass the connector state pointer to the function instead of the job pointer, and set the writeback_job pointer to NULL internally.

Signed-off-by: Laurent Pinchart laurent.pinchart+renesas@ideasonboard.com

Acked-by: Eric Anholt eric@anholt.net