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

Commit Message

Dan Carpenter Feb. 21, 2014, 8:52 a.m. UTC

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

Signed-off-by: Dan Carpenter <[e-mail]>

Comments

Mark Brown Feb. 25, 2014, 11:46 p.m. UTC | #1

On Fri, Feb 21, 2014 at 11:52:40AM +0300, Dan Carpenter wrote:
> There were some curly braces intended here, but the code actually
> works the same either way so it's not a bug.

Applied, thanks.

Patch

diff --git a/sound/soc/intel/sst-firmware.c b/sound/soc/intel/sst-firmware.c
index b6f9b5ecb66a..51b56a473bc5 100644
--- a/sound/soc/intel/sst-firmware.c
+++ b/sound/soc/intel/sst-firmware.c
@@ -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;