[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20230804203134.GA618419@madhu-kernel>
Date: Fri, 4 Aug 2023 15:31:34 -0500
From: Madhumitha Prabakaran <madhumithabiw@...il.com>
To: vaibhav.sr@...il.com, mgreer@...malcreek.com, johan@...nel.org,
elder@...nel.org, gregkh@...uxfoundation.org,
greybus-dev@...ts.linaro.org, linux-staging@...ts.linux.dev,
linux-kernel@...r.kernel.org
Cc: skhan@...uxfoundation.org, ivan.orlov0322@...il.com
Subject: [PATCH] staging: greybus: Refactor gb_audio_gb_get_topology() into
separate calls
Refactor gb_audio_gb_get_topology() into separate calls for better modularity.
Signed-off-by: Madhumitha Prabakaran <madhumithabiw@...il.com>
---
drivers/staging/greybus/audio_gb.c | 67 +++++++++++++++++++-----------
1 file changed, 42 insertions(+), 25 deletions(-)
diff --git a/drivers/staging/greybus/audio_gb.c b/drivers/staging/greybus/audio_gb.c
index 9d8994fdb41a..a48ddadd6f1e 100644
--- a/drivers/staging/greybus/audio_gb.c
+++ b/drivers/staging/greybus/audio_gb.c
@@ -8,39 +8,56 @@
#include <linux/greybus.h>
#include "audio_codec.h"
-/* TODO: Split into separate calls */
-int gb_audio_gb_get_topology(struct gb_connection *connection,
- struct gb_audio_topology **topology)
+int gb_audio_gb_get_topology_size(struct gb_connection *connection, u16 *size)
{
- struct gb_audio_get_topology_size_response size_resp;
- struct gb_audio_topology *topo;
- u16 size;
- int ret;
+ struct gb_audio_get_topology_size_response size_resp;
+ int ret;
- ret = gb_operation_sync(connection, GB_AUDIO_TYPE_GET_TOPOLOGY_SIZE,
- NULL, 0, &size_resp, sizeof(size_resp));
- if (ret)
- return ret;
+ ret = gb_operation_sync(connection, GB_AUDIO_TYPE_GET_TOPOLOGY_SIZE,
+ NULL, 0, &size_resp, sizeof(size_resp));
+ if (ret)
+ return ret;
- size = le16_to_cpu(size_resp.size);
- if (size < sizeof(*topo))
- return -ENODATA;
+ *size = le16_to_cpu(size_resp.size);
+ return 0;
+}
- topo = kzalloc(size, GFP_KERNEL);
- if (!topo)
- return -ENOMEM;
+struct gb_audio_topology *gb_audio_gb_alloc_topology(u16 size)
+{
+ struct gb_audio_topology *topo;
- ret = gb_operation_sync(connection, GB_AUDIO_TYPE_GET_TOPOLOGY, NULL, 0,
- topo, size);
- if (ret) {
- kfree(topo);
- return ret;
- }
+ if (size < sizeof(struct gb_audio_topology))
+ return NULL;
- *topology = topo;
+ topo = kzalloc(size, GFP_KERNEL);
+ return topo;
+}
- return 0;
+int gb_audio_gb_get_topology(struct gb_connection *connection,
+ struct gb_audio_topology **topology)
+{
+ u16 size;
+ int ret;
+
+ ret = gb_audio_gb_get_topology_size(connection, &size);
+ if (ret)
+ return ret;
+
+ *topology = gb_audio_gb_alloc_topology(size);
+ if (!*topology)
+ return -ENOMEM;
+
+ ret = gb_operation_sync(connection, GB_AUDIO_TYPE_GET_TOPOLOGY,
+ NULL, 0, *topology, size);
+ if (ret) {
+ kfree(*topology);
+ *topology = NULL;
+ return ret;
+ }
+
+ return 0;
}
+
EXPORT_SYMBOL_GPL(gb_audio_gb_get_topology);
int gb_audio_gb_get_control(struct gb_connection *connection,
--
2.25.1
Powered by blists - more mailing lists