@@ -94,10 +94,16 @@ EXPORT_SYMBOL_GPL(media_device_request_get);
static void media_device_request_release(struct kref *kref)
{
+ struct media_entity_request_data *data, *next;
struct media_device_request *req =
container_of(kref, struct media_device_request, kref);
struct media_device *mdev = req->mdev;
+ list_for_each_entry_safe(data, next, &req->data, list) {
+ list_del(&data->list);
+ data->release(data);
+ }
+
ida_simple_remove(&mdev->req_ids, req->id);
mdev->ops->req_free(mdev, req);
@@ -109,6 +115,70 @@ void media_device_request_put(struct media_device_request *req)
}
EXPORT_SYMBOL_GPL(media_device_request_put);
+/**
+ * media_device_request_get_entity_data - Get per-entity data
+ * @req: The request
+ * @entity: The entity
+ *
+ * Search and return per-entity data (as a struct media_entity_request_data
+ * instance) associated with the given entity for the request, as previously
+ * registered by a call to media_device_request_set_entity_data().
+ *
+ * The caller is expected to hold a reference to the request. Per-entity data is
+ * not reference counted, the returned pointer will be valid only as long as the
+ * reference to the request is held.
+ *
+ * Return the data instance pointer or NULL if no data could be found.
+ */
+struct media_entity_request_data *
+media_device_request_get_entity_data(struct media_device_request *req,
+ struct media_entity *entity)
+{
+ struct media_entity_request_data *data;
+ unsigned long flags;
+
+ spin_lock_irqsave(&req->mdev->req_lock, flags);
+
+ list_for_each_entry(data, &req->data, list) {
+ if (data->entity == entity)
+ goto done;
+ }
+
+ data = NULL;
+
+done:
+ spin_unlock_irqrestore(&req->mdev->req_lock, flags);
+ return data;
+}
+EXPORT_SYMBOL_GPL(media_device_request_get_entity_data);
+
+/**
+ * media_device_request_set_entity_data - Set per-entity data
+ * @req: The request
+ * @entity: The entity
+ * @data: The data
+ *
+ * Record the given per-entity data as being associated with the entity for the
+ * request.
+ *
+ * Only one per-entity data instance can be associated with a request. The
+ * caller is responsible for enforcing this requirement.
+ *
+ * Ownership of the per-entity data is transferred to the request when calling
+ * this function. The data will be freed automatically when the last reference
+ * to the request is released.
+ */
+void media_device_request_set_entity_data(struct media_device_request *req,
+ struct media_entity *entity, struct media_entity_request_data *data)
+{
+ unsigned long flags;
+
+ spin_lock_irqsave(&req->mdev->req_lock, flags);
+ list_add_tail(&data->list, &req->data);
+ spin_unlock_irqrestore(&req->mdev->req_lock, flags);
+}
+EXPORT_SYMBOL_GPL(media_device_request_set_entity_data);
+
static int media_device_request_alloc(struct media_device *mdev,
struct media_device_fh *fh,
struct media_request_cmd *cmd)
@@ -130,6 +200,7 @@ static int media_device_request_alloc(struct media_device *mdev,
req->mdev = mdev;
req->id = id;
kref_init(&req->kref);
+ INIT_LIST_HEAD(&req->data);
spin_lock_irqsave(&mdev->req_lock, flags);
list_add_tail(&req->list, &mdev->requests);
@@ -153,6 +224,7 @@ static void media_device_request_delete(struct media_device *mdev,
spin_lock_irqsave(&mdev->req_lock, flags);
list_del(&req->list);
+ list_del(&req->fh_list);
spin_unlock_irqrestore(&mdev->req_lock, flags);
media_device_request_put(req);
@@ -284,12 +356,18 @@ static int media_device_close(struct file *filp)
{
struct media_device_fh *fh = media_device_fh(filp);
struct media_device *mdev = to_media_device(fh->fh.devnode);
- struct media_device_request *req, *next;
spin_lock_irq(&mdev->req_lock);
- list_for_each_entry_safe(req, next, &fh->requests, fh_list) {
+ while (!list_empty(&fh->requests)) {
+ struct media_device_request *req;
+
+ req = list_first_entry(&fh->requests, typeof(*req), fh_list);
+ list_del(&req->list);
list_del(&req->fh_list);
+
+ spin_unlock_irq(&mdev->req_lock);
media_device_request_put(req);
+ spin_lock_irq(&mdev->req_lock);
}
spin_unlock_irq(&mdev->req_lock);
@@ -281,6 +281,7 @@ enum media_device_request_state {
* @fh_list: List entry in the media file handle requests list
* @state: The state of the request, MEDIA_DEVICE_REQUEST_STATE_*,
* access to state serialised by mdev->req_lock
+ * @data: Per-entity data list
*/
struct media_device_request {
u32 id;
@@ -289,6 +290,7 @@ struct media_device_request {
struct list_head list;
struct list_head fh_list;
enum media_device_request_state state;
+ struct list_head data;
};
/**
@@ -767,5 +769,10 @@ void media_device_request_get(struct media_device_request *req);
void media_device_request_put(struct media_device_request *req);
void media_device_request_complete(struct media_device *mdev,
struct media_device_request *req);
+struct media_entity_request_data *
+media_device_request_get_entity_data(struct media_device_request *req,
+ struct media_entity *entity);
+void media_device_request_set_entity_data(struct media_device_request *req,
+ struct media_entity *entity, struct media_entity_request_data *data);
#endif
@@ -557,6 +557,18 @@ void media_gobj_create(struct media_device *mdev,
*/
void media_gobj_destroy(struct media_gobj *gobj);
+/*
+ * struct media_entity_request_data - Per-entity request data
+ * @entity: Entity this data belongs to
+ * @release: Release operation to free the data
+ * @list: List entry in the media_device_request data list
+ */
+struct media_entity_request_data {
+ struct media_entity *entity;
+ void (*release)(struct media_entity_request_data *data);
+ struct list_head list;
+};
+
/**
* media_entity_pads_init() - Initialize the entity pads
*