ASoC: soc-core: tidyup snd_soc_lookup_component() (5bd7e08b) · Commits · 戴 / test

snd_soc_lookup_component() is using mix of continue and break in the same loop. It is odd. This patch cleanup it. Reported-by:

sound/soc/soc-core.c

+10 −13

Original line number	Diff line number	Diff line
		@@ -360,25 +360,22 @@ struct snd_soc_component snd_soc_lookup_component(struct device dev,
		const char *driver_name)
		{
		struct snd_soc_component *component;
		struct snd_soc_component *ret;
		struct snd_soc_component *found_component;

		ret = NULL;
		found_component = NULL;
		mutex_lock(&client_mutex);
		for_each_component(component) {
		if (dev != component->dev)
		continue;

		if (driver_name &&
		(driver_name != component->driver->name) &&
		(strcmp(component->driver->name, driver_name) != 0))
		continue;

		ret = component;
		if ((dev == component->dev) &&
		(!driver_name \|\|
		(driver_name == component->driver->name) \|\|
		(strcmp(component->driver->name, driver_name) == 0))) {
		found_component = component;
		break;
		}
		}
		mutex_unlock(&client_mutex);

		return ret;
		return found_component;
		}
		EXPORT_SYMBOL_GPL(snd_soc_lookup_component);