xref: /linux/drivers/gpu/drm/drm_writeback.c (revision 0500c04ea14a4143edf902d087079c4e7b2f0229)
1935774cdSBrian Starkey // SPDX-License-Identifier: GPL-2.0
2935774cdSBrian Starkey /*
3935774cdSBrian Starkey  * (C) COPYRIGHT 2016 ARM Limited. All rights reserved.
4935774cdSBrian Starkey  * Author: Brian Starkey <brian.starkey@arm.com>
5935774cdSBrian Starkey  *
6935774cdSBrian Starkey  * This program is free software and is provided to you under the terms of the
7935774cdSBrian Starkey  * GNU General Public License version 2 as published by the Free Software
8935774cdSBrian Starkey  * Foundation, and any use by you of this program is subject to the terms
9935774cdSBrian Starkey  * of such GNU licence.
10935774cdSBrian Starkey  */
11935774cdSBrian Starkey 
12*0500c04eSSam Ravnborg #include <linux/dma-fence.h>
13*0500c04eSSam Ravnborg 
14935774cdSBrian Starkey #include <drm/drm_crtc.h>
15*0500c04eSSam Ravnborg #include <drm/drm_device.h>
16*0500c04eSSam Ravnborg #include <drm/drm_drv.h>
17935774cdSBrian Starkey #include <drm/drm_modeset_helper_vtables.h>
18935774cdSBrian Starkey #include <drm/drm_property.h>
19935774cdSBrian Starkey #include <drm/drm_writeback.h>
20935774cdSBrian Starkey 
21935774cdSBrian Starkey /**
22935774cdSBrian Starkey  * DOC: overview
23935774cdSBrian Starkey  *
24935774cdSBrian Starkey  * Writeback connectors are used to expose hardware which can write the output
25935774cdSBrian Starkey  * from a CRTC to a memory buffer. They are used and act similarly to other
26935774cdSBrian Starkey  * types of connectors, with some important differences:
2773915b2bSBoris Brezillon  *
2873915b2bSBoris Brezillon  * * Writeback connectors don't provide a way to output visually to the user.
2973915b2bSBoris Brezillon  *
3071a5cb3eSAlexandru Gheorghe  * * Writeback connectors are visible to userspace only when the client sets
3171a5cb3eSAlexandru Gheorghe  *   DRM_CLIENT_CAP_WRITEBACK_CONNECTORS.
3273915b2bSBoris Brezillon  *
3373915b2bSBoris Brezillon  * * Writeback connectors don't have EDID.
34935774cdSBrian Starkey  *
35935774cdSBrian Starkey  * A framebuffer may only be attached to a writeback connector when the
36935774cdSBrian Starkey  * connector is attached to a CRTC. The WRITEBACK_FB_ID property which sets the
37935774cdSBrian Starkey  * framebuffer applies only to a single commit (see below). A framebuffer may
38935774cdSBrian Starkey  * not be attached while the CRTC is off.
39935774cdSBrian Starkey  *
40b13cc8ddSBrian Starkey  * Unlike with planes, when a writeback framebuffer is removed by userspace DRM
41b13cc8ddSBrian Starkey  * makes no attempt to remove it from active use by the connector. This is
42b13cc8ddSBrian Starkey  * because no method is provided to abort a writeback operation, and in any
43b13cc8ddSBrian Starkey  * case making a new commit whilst a writeback is ongoing is undefined (see
44b13cc8ddSBrian Starkey  * WRITEBACK_OUT_FENCE_PTR below). As soon as the current writeback is finished,
45b13cc8ddSBrian Starkey  * the framebuffer will automatically no longer be in active use. As it will
46b13cc8ddSBrian Starkey  * also have already been removed from the framebuffer list, there will be no
47b13cc8ddSBrian Starkey  * way for any userspace application to retrieve a reference to it in the
48b13cc8ddSBrian Starkey  * intervening period.
49b13cc8ddSBrian Starkey  *
50935774cdSBrian Starkey  * Writeback connectors have some additional properties, which userspace
51935774cdSBrian Starkey  * can use to query and control them:
52935774cdSBrian Starkey  *
53935774cdSBrian Starkey  *  "WRITEBACK_FB_ID":
54935774cdSBrian Starkey  *	Write-only object property storing a DRM_MODE_OBJECT_FB: it stores the
55935774cdSBrian Starkey  *	framebuffer to be written by the writeback connector. This property is
56935774cdSBrian Starkey  *	similar to the FB_ID property on planes, but will always read as zero
57935774cdSBrian Starkey  *	and is not preserved across commits.
58935774cdSBrian Starkey  *	Userspace must set this property to an output buffer every time it
59935774cdSBrian Starkey  *	wishes the buffer to get filled.
60935774cdSBrian Starkey  *
61935774cdSBrian Starkey  *  "WRITEBACK_PIXEL_FORMATS":
62935774cdSBrian Starkey  *	Immutable blob property to store the supported pixel formats table. The
63935774cdSBrian Starkey  *	data is an array of u32 DRM_FORMAT_* fourcc values.
64935774cdSBrian Starkey  *	Userspace can use this blob to find out what pixel formats are supported
65935774cdSBrian Starkey  *	by the connector's writeback engine.
66b13cc8ddSBrian Starkey  *
67b13cc8ddSBrian Starkey  *  "WRITEBACK_OUT_FENCE_PTR":
68b13cc8ddSBrian Starkey  *	Userspace can use this property to provide a pointer for the kernel to
69b13cc8ddSBrian Starkey  *	fill with a sync_file file descriptor, which will signal once the
70b13cc8ddSBrian Starkey  *	writeback is finished. The value should be the address of a 32-bit
71b13cc8ddSBrian Starkey  *	signed integer, cast to a u64.
72b13cc8ddSBrian Starkey  *	Userspace should wait for this fence to signal before making another
73b13cc8ddSBrian Starkey  *	commit affecting any of the same CRTCs, Planes or Connectors.
74b13cc8ddSBrian Starkey  *	**Failure to do so will result in undefined behaviour.**
75b13cc8ddSBrian Starkey  *	For this reason it is strongly recommended that all userspace
76b13cc8ddSBrian Starkey  *	applications making use of writeback connectors *always* retrieve an
77b13cc8ddSBrian Starkey  *	out-fence for the commit and use it appropriately.
78b13cc8ddSBrian Starkey  *	From userspace, this property will always read as zero.
79935774cdSBrian Starkey  */
80935774cdSBrian Starkey 
81b13cc8ddSBrian Starkey #define fence_to_wb_connector(x) container_of(x->lock, \
82b13cc8ddSBrian Starkey 					      struct drm_writeback_connector, \
83b13cc8ddSBrian Starkey 					      fence_lock)
84b13cc8ddSBrian Starkey 
85b13cc8ddSBrian Starkey static const char *drm_writeback_fence_get_driver_name(struct dma_fence *fence)
86b13cc8ddSBrian Starkey {
87b13cc8ddSBrian Starkey 	struct drm_writeback_connector *wb_connector =
88b13cc8ddSBrian Starkey 		fence_to_wb_connector(fence);
89b13cc8ddSBrian Starkey 
90b13cc8ddSBrian Starkey 	return wb_connector->base.dev->driver->name;
91b13cc8ddSBrian Starkey }
92b13cc8ddSBrian Starkey 
93b13cc8ddSBrian Starkey static const char *
94b13cc8ddSBrian Starkey drm_writeback_fence_get_timeline_name(struct dma_fence *fence)
95b13cc8ddSBrian Starkey {
96b13cc8ddSBrian Starkey 	struct drm_writeback_connector *wb_connector =
97b13cc8ddSBrian Starkey 		fence_to_wb_connector(fence);
98b13cc8ddSBrian Starkey 
99b13cc8ddSBrian Starkey 	return wb_connector->timeline_name;
100b13cc8ddSBrian Starkey }
101b13cc8ddSBrian Starkey 
102b13cc8ddSBrian Starkey static bool drm_writeback_fence_enable_signaling(struct dma_fence *fence)
103b13cc8ddSBrian Starkey {
104b13cc8ddSBrian Starkey 	return true;
105b13cc8ddSBrian Starkey }
106b13cc8ddSBrian Starkey 
107b13cc8ddSBrian Starkey static const struct dma_fence_ops drm_writeback_fence_ops = {
108b13cc8ddSBrian Starkey 	.get_driver_name = drm_writeback_fence_get_driver_name,
109b13cc8ddSBrian Starkey 	.get_timeline_name = drm_writeback_fence_get_timeline_name,
110b13cc8ddSBrian Starkey 	.enable_signaling = drm_writeback_fence_enable_signaling,
111b13cc8ddSBrian Starkey 	.wait = dma_fence_default_wait,
112b13cc8ddSBrian Starkey };
113b13cc8ddSBrian Starkey 
114935774cdSBrian Starkey static int create_writeback_properties(struct drm_device *dev)
115935774cdSBrian Starkey {
116935774cdSBrian Starkey 	struct drm_property *prop;
117935774cdSBrian Starkey 
118935774cdSBrian Starkey 	if (!dev->mode_config.writeback_fb_id_property) {
119935774cdSBrian Starkey 		prop = drm_property_create_object(dev, DRM_MODE_PROP_ATOMIC,
120935774cdSBrian Starkey 						  "WRITEBACK_FB_ID",
121935774cdSBrian Starkey 						  DRM_MODE_OBJECT_FB);
122935774cdSBrian Starkey 		if (!prop)
123935774cdSBrian Starkey 			return -ENOMEM;
124935774cdSBrian Starkey 		dev->mode_config.writeback_fb_id_property = prop;
125935774cdSBrian Starkey 	}
126935774cdSBrian Starkey 
127935774cdSBrian Starkey 	if (!dev->mode_config.writeback_pixel_formats_property) {
128935774cdSBrian Starkey 		prop = drm_property_create(dev, DRM_MODE_PROP_BLOB |
129935774cdSBrian Starkey 					   DRM_MODE_PROP_ATOMIC |
130935774cdSBrian Starkey 					   DRM_MODE_PROP_IMMUTABLE,
131935774cdSBrian Starkey 					   "WRITEBACK_PIXEL_FORMATS", 0);
132935774cdSBrian Starkey 		if (!prop)
133935774cdSBrian Starkey 			return -ENOMEM;
134935774cdSBrian Starkey 		dev->mode_config.writeback_pixel_formats_property = prop;
135935774cdSBrian Starkey 	}
136935774cdSBrian Starkey 
137b13cc8ddSBrian Starkey 	if (!dev->mode_config.writeback_out_fence_ptr_property) {
138b13cc8ddSBrian Starkey 		prop = drm_property_create_range(dev, DRM_MODE_PROP_ATOMIC,
139b13cc8ddSBrian Starkey 						 "WRITEBACK_OUT_FENCE_PTR", 0,
140b13cc8ddSBrian Starkey 						 U64_MAX);
141b13cc8ddSBrian Starkey 		if (!prop)
142b13cc8ddSBrian Starkey 			return -ENOMEM;
143b13cc8ddSBrian Starkey 		dev->mode_config.writeback_out_fence_ptr_property = prop;
144b13cc8ddSBrian Starkey 	}
145b13cc8ddSBrian Starkey 
146935774cdSBrian Starkey 	return 0;
147935774cdSBrian Starkey }
148935774cdSBrian Starkey 
149935774cdSBrian Starkey static const struct drm_encoder_funcs drm_writeback_encoder_funcs = {
150935774cdSBrian Starkey 	.destroy = drm_encoder_cleanup,
151935774cdSBrian Starkey };
152935774cdSBrian Starkey 
153935774cdSBrian Starkey /**
154935774cdSBrian Starkey  * drm_writeback_connector_init - Initialize a writeback connector and its properties
155935774cdSBrian Starkey  * @dev: DRM device
156935774cdSBrian Starkey  * @wb_connector: Writeback connector to initialize
157935774cdSBrian Starkey  * @con_funcs: Connector funcs vtable
158935774cdSBrian Starkey  * @enc_helper_funcs: Encoder helper funcs vtable to be used by the internal encoder
159935774cdSBrian Starkey  * @formats: Array of supported pixel formats for the writeback engine
160935774cdSBrian Starkey  * @n_formats: Length of the formats array
161935774cdSBrian Starkey  *
162935774cdSBrian Starkey  * This function creates the writeback-connector-specific properties if they
163935774cdSBrian Starkey  * have not been already created, initializes the connector as
164935774cdSBrian Starkey  * type DRM_MODE_CONNECTOR_WRITEBACK, and correctly initializes the property
165935774cdSBrian Starkey  * values. It will also create an internal encoder associated with the
166935774cdSBrian Starkey  * drm_writeback_connector and set it to use the @enc_helper_funcs vtable for
167935774cdSBrian Starkey  * the encoder helper.
168935774cdSBrian Starkey  *
169935774cdSBrian Starkey  * Drivers should always use this function instead of drm_connector_init() to
170935774cdSBrian Starkey  * set up writeback connectors.
171935774cdSBrian Starkey  *
172935774cdSBrian Starkey  * Returns: 0 on success, or a negative error code
173935774cdSBrian Starkey  */
174935774cdSBrian Starkey int drm_writeback_connector_init(struct drm_device *dev,
175935774cdSBrian Starkey 				 struct drm_writeback_connector *wb_connector,
176935774cdSBrian Starkey 				 const struct drm_connector_funcs *con_funcs,
177935774cdSBrian Starkey 				 const struct drm_encoder_helper_funcs *enc_helper_funcs,
178935774cdSBrian Starkey 				 const u32 *formats, int n_formats)
179935774cdSBrian Starkey {
180935774cdSBrian Starkey 	struct drm_property_blob *blob;
181935774cdSBrian Starkey 	struct drm_connector *connector = &wb_connector->base;
182935774cdSBrian Starkey 	struct drm_mode_config *config = &dev->mode_config;
183935774cdSBrian Starkey 	int ret = create_writeback_properties(dev);
184935774cdSBrian Starkey 
185935774cdSBrian Starkey 	if (ret != 0)
186935774cdSBrian Starkey 		return ret;
187935774cdSBrian Starkey 
188935774cdSBrian Starkey 	blob = drm_property_create_blob(dev, n_formats * sizeof(*formats),
189935774cdSBrian Starkey 					formats);
190935774cdSBrian Starkey 	if (IS_ERR(blob))
191935774cdSBrian Starkey 		return PTR_ERR(blob);
192935774cdSBrian Starkey 
193935774cdSBrian Starkey 	drm_encoder_helper_add(&wb_connector->encoder, enc_helper_funcs);
194935774cdSBrian Starkey 	ret = drm_encoder_init(dev, &wb_connector->encoder,
195935774cdSBrian Starkey 			       &drm_writeback_encoder_funcs,
196935774cdSBrian Starkey 			       DRM_MODE_ENCODER_VIRTUAL, NULL);
197935774cdSBrian Starkey 	if (ret)
198935774cdSBrian Starkey 		goto fail;
199935774cdSBrian Starkey 
200935774cdSBrian Starkey 	connector->interlace_allowed = 0;
201935774cdSBrian Starkey 
202935774cdSBrian Starkey 	ret = drm_connector_init(dev, connector, con_funcs,
203935774cdSBrian Starkey 				 DRM_MODE_CONNECTOR_WRITEBACK);
204935774cdSBrian Starkey 	if (ret)
205935774cdSBrian Starkey 		goto connector_fail;
206935774cdSBrian Starkey 
207cde4c44dSDaniel Vetter 	ret = drm_connector_attach_encoder(connector,
208935774cdSBrian Starkey 						&wb_connector->encoder);
209935774cdSBrian Starkey 	if (ret)
210935774cdSBrian Starkey 		goto attach_fail;
211935774cdSBrian Starkey 
212935774cdSBrian Starkey 	INIT_LIST_HEAD(&wb_connector->job_queue);
213935774cdSBrian Starkey 	spin_lock_init(&wb_connector->job_lock);
214935774cdSBrian Starkey 
215b13cc8ddSBrian Starkey 	wb_connector->fence_context = dma_fence_context_alloc(1);
216b13cc8ddSBrian Starkey 	spin_lock_init(&wb_connector->fence_lock);
217b13cc8ddSBrian Starkey 	snprintf(wb_connector->timeline_name,
218b13cc8ddSBrian Starkey 		 sizeof(wb_connector->timeline_name),
219b13cc8ddSBrian Starkey 		 "CONNECTOR:%d-%s", connector->base.id, connector->name);
220b13cc8ddSBrian Starkey 
221b13cc8ddSBrian Starkey 	drm_object_attach_property(&connector->base,
222b13cc8ddSBrian Starkey 				   config->writeback_out_fence_ptr_property, 0);
223b13cc8ddSBrian Starkey 
224935774cdSBrian Starkey 	drm_object_attach_property(&connector->base,
225935774cdSBrian Starkey 				   config->writeback_fb_id_property, 0);
226935774cdSBrian Starkey 
227935774cdSBrian Starkey 	drm_object_attach_property(&connector->base,
228935774cdSBrian Starkey 				   config->writeback_pixel_formats_property,
229935774cdSBrian Starkey 				   blob->base.id);
230935774cdSBrian Starkey 	wb_connector->pixel_formats_blob_ptr = blob;
231935774cdSBrian Starkey 
232935774cdSBrian Starkey 	return 0;
233935774cdSBrian Starkey 
234935774cdSBrian Starkey attach_fail:
235935774cdSBrian Starkey 	drm_connector_cleanup(connector);
236935774cdSBrian Starkey connector_fail:
237935774cdSBrian Starkey 	drm_encoder_cleanup(&wb_connector->encoder);
238935774cdSBrian Starkey fail:
239935774cdSBrian Starkey 	drm_property_blob_put(blob);
240935774cdSBrian Starkey 	return ret;
241935774cdSBrian Starkey }
242935774cdSBrian Starkey EXPORT_SYMBOL(drm_writeback_connector_init);
243935774cdSBrian Starkey 
2449d2230dcSLaurent Pinchart int drm_writeback_set_fb(struct drm_connector_state *conn_state,
2459d2230dcSLaurent Pinchart 			 struct drm_framebuffer *fb)
2469d2230dcSLaurent Pinchart {
2479d2230dcSLaurent Pinchart 	WARN_ON(conn_state->connector->connector_type != DRM_MODE_CONNECTOR_WRITEBACK);
2489d2230dcSLaurent Pinchart 
2499d2230dcSLaurent Pinchart 	if (!conn_state->writeback_job) {
2509d2230dcSLaurent Pinchart 		conn_state->writeback_job =
2519d2230dcSLaurent Pinchart 			kzalloc(sizeof(*conn_state->writeback_job), GFP_KERNEL);
2529d2230dcSLaurent Pinchart 		if (!conn_state->writeback_job)
2539d2230dcSLaurent Pinchart 			return -ENOMEM;
2549d2230dcSLaurent Pinchart 
2559d2230dcSLaurent Pinchart 		conn_state->writeback_job->connector =
2569d2230dcSLaurent Pinchart 			drm_connector_to_writeback(conn_state->connector);
2579d2230dcSLaurent Pinchart 	}
2589d2230dcSLaurent Pinchart 
2599d2230dcSLaurent Pinchart 	drm_framebuffer_assign(&conn_state->writeback_job->fb, fb);
2609d2230dcSLaurent Pinchart 	return 0;
2619d2230dcSLaurent Pinchart }
2629d2230dcSLaurent Pinchart 
2639d2230dcSLaurent Pinchart int drm_writeback_prepare_job(struct drm_writeback_job *job)
2649d2230dcSLaurent Pinchart {
2659d2230dcSLaurent Pinchart 	struct drm_writeback_connector *connector = job->connector;
2669d2230dcSLaurent Pinchart 	const struct drm_connector_helper_funcs *funcs =
2679d2230dcSLaurent Pinchart 		connector->base.helper_private;
2689d2230dcSLaurent Pinchart 	int ret;
2699d2230dcSLaurent Pinchart 
2709d2230dcSLaurent Pinchart 	if (funcs->prepare_writeback_job) {
2719d2230dcSLaurent Pinchart 		ret = funcs->prepare_writeback_job(connector, job);
2729d2230dcSLaurent Pinchart 		if (ret < 0)
2739d2230dcSLaurent Pinchart 			return ret;
2749d2230dcSLaurent Pinchart 	}
2759d2230dcSLaurent Pinchart 
2769d2230dcSLaurent Pinchart 	job->prepared = true;
2779d2230dcSLaurent Pinchart 	return 0;
2789d2230dcSLaurent Pinchart }
2799d2230dcSLaurent Pinchart EXPORT_SYMBOL(drm_writeback_prepare_job);
2809d2230dcSLaurent Pinchart 
281935774cdSBrian Starkey /**
282935774cdSBrian Starkey  * drm_writeback_queue_job - Queue a writeback job for later signalling
283935774cdSBrian Starkey  * @wb_connector: The writeback connector to queue a job on
28497eb9eaeSLaurent Pinchart  * @conn_state: The connector state containing the job to queue
285935774cdSBrian Starkey  *
28697eb9eaeSLaurent Pinchart  * This function adds the job contained in @conn_state to the job_queue for a
28797eb9eaeSLaurent Pinchart  * writeback connector. It takes ownership of the writeback job and sets the
28897eb9eaeSLaurent Pinchart  * @conn_state->writeback_job to NULL, and so no access to the job may be
28997eb9eaeSLaurent Pinchart  * performed by the caller after this function returns.
290935774cdSBrian Starkey  *
291935774cdSBrian Starkey  * Drivers must ensure that for a given writeback connector, jobs are queued in
292935774cdSBrian Starkey  * exactly the same order as they will be completed by the hardware (and
293935774cdSBrian Starkey  * signaled via drm_writeback_signal_completion).
294935774cdSBrian Starkey  *
295935774cdSBrian Starkey  * For every call to drm_writeback_queue_job() there must be exactly one call to
296935774cdSBrian Starkey  * drm_writeback_signal_completion()
297935774cdSBrian Starkey  *
298935774cdSBrian Starkey  * See also: drm_writeback_signal_completion()
299935774cdSBrian Starkey  */
300935774cdSBrian Starkey void drm_writeback_queue_job(struct drm_writeback_connector *wb_connector,
30197eb9eaeSLaurent Pinchart 			     struct drm_connector_state *conn_state)
302935774cdSBrian Starkey {
30397eb9eaeSLaurent Pinchart 	struct drm_writeback_job *job;
304935774cdSBrian Starkey 	unsigned long flags;
305935774cdSBrian Starkey 
30697eb9eaeSLaurent Pinchart 	job = conn_state->writeback_job;
30797eb9eaeSLaurent Pinchart 	conn_state->writeback_job = NULL;
30897eb9eaeSLaurent Pinchart 
309935774cdSBrian Starkey 	spin_lock_irqsave(&wb_connector->job_lock, flags);
310935774cdSBrian Starkey 	list_add_tail(&job->list_entry, &wb_connector->job_queue);
311935774cdSBrian Starkey 	spin_unlock_irqrestore(&wb_connector->job_lock, flags);
312935774cdSBrian Starkey }
313935774cdSBrian Starkey EXPORT_SYMBOL(drm_writeback_queue_job);
314935774cdSBrian Starkey 
315e482ae9bSLaurent Pinchart void drm_writeback_cleanup_job(struct drm_writeback_job *job)
316e482ae9bSLaurent Pinchart {
3179d2230dcSLaurent Pinchart 	struct drm_writeback_connector *connector = job->connector;
3189d2230dcSLaurent Pinchart 	const struct drm_connector_helper_funcs *funcs =
3199d2230dcSLaurent Pinchart 		connector->base.helper_private;
3209d2230dcSLaurent Pinchart 
3219d2230dcSLaurent Pinchart 	if (job->prepared && funcs->cleanup_writeback_job)
3229d2230dcSLaurent Pinchart 		funcs->cleanup_writeback_job(connector, job);
3239d2230dcSLaurent Pinchart 
324e482ae9bSLaurent Pinchart 	if (job->fb)
325e482ae9bSLaurent Pinchart 		drm_framebuffer_put(job->fb);
326e482ae9bSLaurent Pinchart 
327e482ae9bSLaurent Pinchart 	kfree(job);
328e482ae9bSLaurent Pinchart }
329e482ae9bSLaurent Pinchart EXPORT_SYMBOL(drm_writeback_cleanup_job);
330e482ae9bSLaurent Pinchart 
331935774cdSBrian Starkey /*
332935774cdSBrian Starkey  * @cleanup_work: deferred cleanup of a writeback job
333935774cdSBrian Starkey  *
334935774cdSBrian Starkey  * The job cannot be cleaned up directly in drm_writeback_signal_completion,
335935774cdSBrian Starkey  * because it may be called in interrupt context. Dropping the framebuffer
336935774cdSBrian Starkey  * reference can sleep, and so the cleanup is deferred to a workqueue.
337935774cdSBrian Starkey  */
338935774cdSBrian Starkey static void cleanup_work(struct work_struct *work)
339935774cdSBrian Starkey {
340935774cdSBrian Starkey 	struct drm_writeback_job *job = container_of(work,
341935774cdSBrian Starkey 						     struct drm_writeback_job,
342935774cdSBrian Starkey 						     cleanup_work);
343935774cdSBrian Starkey 
344e482ae9bSLaurent Pinchart 	drm_writeback_cleanup_job(job);
345e482ae9bSLaurent Pinchart }
346935774cdSBrian Starkey 
347935774cdSBrian Starkey /**
348935774cdSBrian Starkey  * drm_writeback_signal_completion - Signal the completion of a writeback job
349935774cdSBrian Starkey  * @wb_connector: The writeback connector whose job is complete
350b13cc8ddSBrian Starkey  * @status: Status code to set in the writeback out_fence (0 for success)
351935774cdSBrian Starkey  *
352935774cdSBrian Starkey  * Drivers should call this to signal the completion of a previously queued
353935774cdSBrian Starkey  * writeback job. It should be called as soon as possible after the hardware
354935774cdSBrian Starkey  * has finished writing, and may be called from interrupt context.
355935774cdSBrian Starkey  * It is the driver's responsibility to ensure that for a given connector, the
356935774cdSBrian Starkey  * hardware completes writeback jobs in the same order as they are queued.
357935774cdSBrian Starkey  *
358935774cdSBrian Starkey  * Unless the driver is holding its own reference to the framebuffer, it must
359935774cdSBrian Starkey  * not be accessed after calling this function.
360935774cdSBrian Starkey  *
361935774cdSBrian Starkey  * See also: drm_writeback_queue_job()
362935774cdSBrian Starkey  */
363935774cdSBrian Starkey void
364b13cc8ddSBrian Starkey drm_writeback_signal_completion(struct drm_writeback_connector *wb_connector,
365b13cc8ddSBrian Starkey 				int status)
366935774cdSBrian Starkey {
367935774cdSBrian Starkey 	unsigned long flags;
368935774cdSBrian Starkey 	struct drm_writeback_job *job;
369935774cdSBrian Starkey 
370935774cdSBrian Starkey 	spin_lock_irqsave(&wb_connector->job_lock, flags);
371935774cdSBrian Starkey 	job = list_first_entry_or_null(&wb_connector->job_queue,
372935774cdSBrian Starkey 				       struct drm_writeback_job,
373935774cdSBrian Starkey 				       list_entry);
374b13cc8ddSBrian Starkey 	if (job) {
375935774cdSBrian Starkey 		list_del(&job->list_entry);
376b13cc8ddSBrian Starkey 		if (job->out_fence) {
377b13cc8ddSBrian Starkey 			if (status)
378b13cc8ddSBrian Starkey 				dma_fence_set_error(job->out_fence, status);
379b13cc8ddSBrian Starkey 			dma_fence_signal(job->out_fence);
380b13cc8ddSBrian Starkey 			dma_fence_put(job->out_fence);
381b13cc8ddSBrian Starkey 		}
382b13cc8ddSBrian Starkey 	}
383935774cdSBrian Starkey 	spin_unlock_irqrestore(&wb_connector->job_lock, flags);
384935774cdSBrian Starkey 
385935774cdSBrian Starkey 	if (WARN_ON(!job))
386935774cdSBrian Starkey 		return;
387935774cdSBrian Starkey 
388935774cdSBrian Starkey 	INIT_WORK(&job->cleanup_work, cleanup_work);
389935774cdSBrian Starkey 	queue_work(system_long_wq, &job->cleanup_work);
390935774cdSBrian Starkey }
391935774cdSBrian Starkey EXPORT_SYMBOL(drm_writeback_signal_completion);
392b13cc8ddSBrian Starkey 
393b13cc8ddSBrian Starkey struct dma_fence *
394b13cc8ddSBrian Starkey drm_writeback_get_out_fence(struct drm_writeback_connector *wb_connector)
395b13cc8ddSBrian Starkey {
396b13cc8ddSBrian Starkey 	struct dma_fence *fence;
397b13cc8ddSBrian Starkey 
398b13cc8ddSBrian Starkey 	if (WARN_ON(wb_connector->base.connector_type !=
399b13cc8ddSBrian Starkey 		    DRM_MODE_CONNECTOR_WRITEBACK))
400b13cc8ddSBrian Starkey 		return NULL;
401b13cc8ddSBrian Starkey 
402b13cc8ddSBrian Starkey 	fence = kzalloc(sizeof(*fence), GFP_KERNEL);
403b13cc8ddSBrian Starkey 	if (!fence)
404b13cc8ddSBrian Starkey 		return NULL;
405b13cc8ddSBrian Starkey 
406b13cc8ddSBrian Starkey 	dma_fence_init(fence, &drm_writeback_fence_ops,
407b13cc8ddSBrian Starkey 		       &wb_connector->fence_lock, wb_connector->fence_context,
408b13cc8ddSBrian Starkey 		       ++wb_connector->fence_seqno);
409b13cc8ddSBrian Starkey 
410b13cc8ddSBrian Starkey 	return fence;
411b13cc8ddSBrian Starkey }
412b13cc8ddSBrian Starkey EXPORT_SYMBOL(drm_writeback_get_out_fence);
413