[PATCH v2 3/4] staging: greybus: operation: add private data with get/set accessors

Bryan O'Donoghue pure.logic at nexus-software.ie
Mon Nov 6 01:26:10 UTC 2017

Asynchronous operation completion handler's lives are made easier if there
is a generic pointer that can store private data associated with the
operation. This patch adds a pointer field to struct gb_operation and
get/set methods to access that pointer.

Signed-off-by: Bryan O'Donoghue <pure.logic at nexus-software.ie>
Cc: Johan Hovold <johan at kernel.org>
Cc: Alex Elder <elder at kernel.org>
Cc: Greg Kroah-Hartman <gregkh at linuxfoundation.org>
Cc: Mitch Tasman <tasman at leaflabs.com>
Cc: greybus-dev at lists.linaro.org
Cc: devel at driverdev.osuosl.org
Cc: linux-kernel at vger.kernel.org
Reviewed-by: Johan Hovold <johan at kernel.org>
 drivers/staging/greybus/operation.h | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/drivers/staging/greybus/operation.h b/drivers/staging/greybus/operation.h
index 7529f01..bfec1e9 100644
--- a/drivers/staging/greybus/operation.h
+++ b/drivers/staging/greybus/operation.h
@@ -105,6 +105,8 @@ struct gb_operation {
 	int			active;
 	struct list_head	links;		/* connection->operations */
+	void			*private;
 static inline bool
@@ -206,6 +208,17 @@ static inline int gb_operation_unidirectional(struct gb_connection *connection,
 			request, request_size, GB_OPERATION_TIMEOUT_DEFAULT);
+static inline void *gb_operation_get_data(struct gb_operation *operation)
+	return operation->private;
+static inline void gb_operation_set_data(struct gb_operation *operation,
+					 void *data)
+	operation->private = data;
 int gb_operation_init(void);
 void gb_operation_exit(void);

More information about the devel mailing list