Commit 8d46ec49 authored by Johan Hovold's avatar Johan Hovold Committed by Greg Kroah-Hartman
Browse files

greybus: bundle: remove private data field



Remove the private data field from the bundle structure as it is no
longer needed. Bundle drivers can use the driver data field in the
bundle device.

Update the only current user to use the connection private data until it
has been converted to a bundle driver.

Signed-off-by: default avatarJohan Hovold <johan@hovoldconsulting.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@google.com>
parent 5ef545fb
Loading
Loading
Loading
Loading
+0 −1
Original line number Diff line number Diff line
@@ -29,7 +29,6 @@ struct gb_bundle {
	u8			*state;

	struct list_head	links;	/* interface->bundles */
	void			*private;
};
#define to_gb_bundle(d) container_of(d, struct gb_bundle, dev)

+2 −4
Original line number Diff line number Diff line
@@ -1121,7 +1121,7 @@ static int gb_loopback_connection_init(struct gb_connection *connection)
	gb->file = debugfs_create_file(name, S_IFREG | S_IRUGO, gb_dev.root, gb,
				       &gb_loopback_debugfs_latency_ops);
	gb->connection = connection;
	connection->bundle->private = gb;
	connection->private = gb;

	gb->id = ida_simple_get(&loopback_ida, 0, 0, GFP_KERNEL);
	if (gb->id < 0) {
@@ -1177,7 +1177,6 @@ static int gb_loopback_connection_init(struct gb_connection *connection)
	ida_simple_remove(&loopback_ida, gb->id);
out_ida:
	debugfs_remove(gb->file);
	connection->bundle->private = NULL;
out_kzalloc:
	kfree(gb);

@@ -1186,13 +1185,12 @@ static int gb_loopback_connection_init(struct gb_connection *connection)

static void gb_loopback_connection_exit(struct gb_connection *connection)
{
	struct gb_loopback *gb = connection->bundle->private;
	struct gb_loopback *gb = connection->private;
	unsigned long flags;

	if (!IS_ERR_OR_NULL(gb->task))
		kthread_stop(gb->task);

	connection->bundle->private = NULL;
	kfifo_free(&gb->kfifo_lat);
	kfifo_free(&gb->kfifo_ts);
	gb_connection_latency_tag_disable(connection);