greybus: count rather than list protocol users
We don't really need a list of protocol users, we can just keep
track of how many there are. Get rid of the list and use a count
instead.
Also, have gb_protocol_get() return the protocol rather than assigning
a passed-in connection pointer's protocol. Make a comparable change
to the gb_protocol_put() interface.
Get rid of gb_protocol_find() (the version that locks), because it
is no longer needed.
Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>
diff --git a/drivers/staging/greybus/protocol.c b/drivers/staging/greybus/protocol.c
index 704b180..347d52c 100644
--- a/drivers/staging/greybus/protocol.c
+++ b/drivers/staging/greybus/protocol.c
@@ -24,18 +24,6 @@
return NULL;
}
-/* This is basically for debug */
-static struct gb_protocol *gb_protocol_find(u8 id, u8 major, u8 minor)
-{
- struct gb_protocol *protocol;
-
- spin_lock_irq(&gb_protocols_lock);
- protocol = _gb_protocol_find(id, major, minor);
- spin_unlock_irq(&gb_protocols_lock);
-
- return protocol;
-}
-
/* Returns true if protocol was succesfully registered, false otherwise */
bool gb_protocol_register(u8 id, u8 major, u8 minor)
{
@@ -49,7 +37,6 @@
protocol->id = id;
protocol->major = major;
protocol->minor = minor;
- INIT_LIST_HEAD(&protocol->connections);
spin_lock_irq(&gb_protocols_lock);
existing = _gb_protocol_find(id, major, minor);
@@ -68,64 +55,64 @@
/* Returns true if successful, false otherwise */
bool gb_protocol_deregister(struct gb_protocol *protocol)
{
+ u8 protocol_count;
+
spin_lock_irq(&gb_protocols_lock);
- if (list_empty(&protocol->connections))
- list_del(&protocol->links);
- else
- protocol = NULL; /* Protocol is still in use */
+ protocol = _gb_protocol_find(protocol->id, protocol->major,
+ protocol->minor);
+ if (protocol) {
+ protocol_count = protocol->count;
+ if (!protocol_count)
+ list_del(&protocol->links);
+ }
spin_unlock_irq(&gb_protocols_lock);
kfree(protocol);
- return protocol != NULL;
+ return protocol && !protocol_count;
}
-/* Returns true if successful, false otherwise */
-bool
-gb_protocol_get(struct gb_connection *connection, u8 id, u8 major, u8 minor)
+/* Returns the requested protocol if available, or a null pointer */
+struct gb_protocol *gb_protocol_get(u8 id, u8 major, u8 minor)
{
struct gb_protocol *protocol;
-
- /* Sanity */
- if (!list_empty(&connection->protocol_links) ||
- !connection->protocol->id) {
- gb_connection_err(connection,
- "connection already has protocol");
- return false;
- }
+ u8 protocol_count;
spin_lock_irq(&gb_protocols_lock);
protocol = _gb_protocol_find(id, major, minor);
- if (protocol)
- list_add(&connection->protocol_links, &protocol->connections);
+ if (protocol) {
+ protocol_count = protocol->count;
+ if (protocol_count != U8_MAX)
+ protocol->count++;
+ }
spin_unlock_irq(&gb_protocols_lock);
- connection->protocol = protocol;
- return protocol != NULL;
+ if (protocol)
+ WARN_ON(protocol_count == U8_MAX);
+ else
+ pr_err("protocol id %hhu version %hhu.%hhu not found\n",
+ id, major, minor);
+
+ return protocol;
}
-void gb_protocol_put(struct gb_connection *connection)
+void gb_protocol_put(struct gb_protocol *protocol)
{
- struct gb_protocol *protocol = connection->protocol;
u8 major = protocol->major;
u8 minor = protocol->minor;
-
- /* Sanity checks */
- if (list_empty(&connection->protocol_links)) {
- gb_connection_err(connection,
- "connection protocol not recorded");
- return;
- }
- if (!protocol) {
- gb_connection_err(connection, "connection has no protocol");
- return;
- }
- if (gb_protocol_find(protocol->id, major, minor) != protocol) {
- gb_connection_err(connection, "connection protocol not found");
- return;
- }
+ u8 protocol_count;
spin_lock_irq(&gb_protocols_lock);
- list_del(&connection->protocol_links);
- connection->protocol = NULL;
+ protocol = _gb_protocol_find(protocol->id, protocol->major,
+ protocol->minor);
+ if (protocol) {
+ protocol_count = protocol->count;
+ if (protocol_count)
+ protocol->count--;
+ }
spin_unlock_irq(&gb_protocols_lock);
+ if (protocol)
+ WARN_ON(!protocol_count);
+ else
+ pr_err("protocol id %hhu version %hhu.%hhu not found\n",
+ protocol->id, major, minor);
}