Skip to contentSkip to conversation

ASoC: Intel: sst-firmware: missing curly braces (harmless)

PATCH Applied Dan Carpenter 2 participants 2 messages Feb 21, 2014 – Feb 25, 2014

Subsystems (4)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 ASoC: Intel: sst-firmware: missing curly braces (harmless)

There were some curly braces intended here, but the code actually works the same either way so it's not a bug.

sound/soc/intel/sst-firmware.c +2 -1
index b6f9b5ecb66a..51b56a473bc5 100644
@@ -293,7 +293,7 @@ static int block_module_prepare(struct sst_module *module)
 	/* enable each block so that's it'e ready for module P/S data */
 	list_for_each_entry(block, &module->block_list, module_list) {
 
-		if (block->ops && block->ops->enable)
+		if (block->ops && block->ops->enable) {
 			ret = block->ops->enable(block);
 			if (ret < 0) {
 				dev_err(module->dsp->dev,
@@ -301,6 +301,7 @@ static int block_module_prepare(struct sst_module *module)
 					block->type, block->index);
 				goto err;
 			}
+		}
 	}
 	return ret;
 
maintainer Applied
#1 lore raw
Quoted 2 lines · Dan Carpenter

There were some curly braces intended here, but the code actually works the same either way so it's not a bug.

Applied, thanks.