samples: intel_s1000/i2s: fix uninitialized use warning

dcpleung · galak · commit 66aefdbb93a7 · 2019-11-13T06:26:10.000-06:00
One of the return variable is declared and used, but never
assigned values. So fix it.

Signed-off-by: Daniel Leung &lt;daniel.leung@intel.com&gt;
diff --git a/samples/boards/intel_s1000_crb/i2s/src/i2s_sample.c b/samples/boards/intel_s1000_crb/i2s/src/i2s_sample.c
@@ -292,11 +292,13 @@ static void i2s_stop_audio(void)
 {
 	int ret;
 
-	if (i2s_trigger(spk_i2s_dev, I2S_DIR_TX, I2S_TRIGGER_STOP)) {
+	ret = i2s_trigger(spk_i2s_dev, I2S_DIR_TX, I2S_TRIGGER_STOP);
+	if (ret) {
 		LOG_ERR("spk_i2s_dev stop failed with code %d", ret);
 	}
 
-	if (i2s_trigger(host_i2s_dev, I2S_DIR_RX, I2S_TRIGGER_STOP)) {
+	ret = i2s_trigger(host_i2s_dev, I2S_DIR_RX, I2S_TRIGGER_STOP);
+	if (ret) {
 		LOG_ERR("host_i2s_dev stop failed with code %d", ret);
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -292,11 +292,13 @@ static void i2s_stop_audio(void)`
`292`	`292`	`{`
`293`	`293`	`int ret;`
`294`	`294`
`295`		`- if (i2s_trigger(spk_i2s_dev, I2S_DIR_TX, I2S_TRIGGER_STOP)) {`
	`295`	`+ ret = i2s_trigger(spk_i2s_dev, I2S_DIR_TX, I2S_TRIGGER_STOP);`
	`296`	`+ if (ret) {`
`296`	`297`	`LOG_ERR("spk_i2s_dev stop failed with code %d", ret);`
`297`	`298`	`}`
`298`	`299`
`299`		`- if (i2s_trigger(host_i2s_dev, I2S_DIR_RX, I2S_TRIGGER_STOP)) {`
	`300`	`+ ret = i2s_trigger(host_i2s_dev, I2S_DIR_RX, I2S_TRIGGER_STOP);`
	`301`	`+ if (ret) {`
`300`	`302`	`LOG_ERR("host_i2s_dev stop failed with code %d", ret);`
`301`	`303`	`}`
`302`	`304`