From e6ab7a154c5dbfc0ad08bf1570efea22f2c7ee6c Mon Sep 17 00:00:00 2001 From: Vaibhav Agarwal Date: Wed, 4 May 2016 16:29:20 +0530 Subject: [PATCH] greybus: audio: Cleanup GB Audio bundle driver. Remove unnecessary lock & list maintained in GB audio bundle driver Signed-off-by: Vaibhav Agarwal Signed-off-by: Greg Kroah-Hartman --- drivers/staging/greybus/audio_module.c | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/drivers/staging/greybus/audio_module.c b/drivers/staging/greybus/audio_module.c index 05bda486f490..1e29901024cd 100644 --- a/drivers/staging/greybus/audio_module.c +++ b/drivers/staging/greybus/audio_module.c @@ -14,9 +14,6 @@ #include "audio_apbridgea.h" #include "audio_manager.h" -static DEFINE_MUTEX(gb_codec_list_lock); -static LIST_HEAD(gb_codec_list); - /* * gb_snd management functions */ @@ -241,16 +238,13 @@ static int gb_audio_probe(struct gb_bundle *bundle, if (bundle->num_cports < 2) return -ENODEV; - mutex_lock(&gb_codec_list_lock); /* * There can be only one Management connection and any number of data * connections. */ gbmodule = devm_kzalloc(dev, sizeof(*gbmodule), GFP_KERNEL); - if (!gbmodule) { - mutex_unlock(&gb_codec_list_lock); + if (!gbmodule) return -ENOMEM; - } gbmodule->num_data_connections = bundle->num_cports - 1; mutex_init(&gbmodule->lock); @@ -348,7 +342,6 @@ static int gb_audio_probe(struct gb_bundle *bundle, gbmodule->manager_id = gb_audio_manager_add(&desc); dev_dbg(dev, "Add GB Audio device:%s\n", gbmodule->name); - mutex_unlock(&gb_codec_list_lock); return 0; @@ -378,7 +371,6 @@ destroy_connections: gb_connection_destroy(gbmodule->mgmt_connection); devm_kfree(dev, gbmodule); - mutex_unlock(&gb_codec_list_lock); return ret; } @@ -388,8 +380,6 @@ static void gb_audio_disconnect(struct gb_bundle *bundle) struct gbaudio_module_info *gbmodule = greybus_get_drvdata(bundle); struct gbaudio_data_connection *dai, *_dai; - mutex_lock(&gb_codec_list_lock); - gbaudio_unregister_module(gbmodule); /* inform uevent to above layers */ @@ -409,7 +399,6 @@ static void gb_audio_disconnect(struct gb_bundle *bundle) gbmodule->mgmt_connection = NULL; devm_kfree(&bundle->dev, gbmodule); - mutex_unlock(&gb_codec_list_lock); } static const struct greybus_bundle_id gb_audio_id_table[] = { -- 2.39.5