drivers: i2s: Do not ignore the return value from queue_put.

cdesjardins · carlescufi · commit 84572a6532ac · 2024-09-05T09:50:08.000+02:00
If failure occurs the caller can free the buffer.

Signed-off-by: Chris Desjardins &lt;chris@arch-embedded.com&gt;
diff --git a/drivers/i2s/i2s_ll_stm32.c b/drivers/i2s/i2s_ll_stm32.c
@@ -480,9 +480,7 @@ static int i2s_stm32_write(const struct device *dev, void *mem_block,
 	}
 
 	/* Add data to the end of the TX queue */
-	queue_put(&dev_data->tx.mem_block_queue, mem_block, size);
-
-	return 0;
+	return queue_put(&dev_data->tx.mem_block_queue, mem_block, size);
 }
 
 static const struct i2s_driver_api i2s_stm32_driver_api = {

Original file line number	Diff line number	Diff line change
`@@ -480,9 +480,7 @@ static int i2s_stm32_write(const struct device dev, void mem_block,`
`480`	`480`	`}`
`481`	`481`
`482`	`482`	`/* Add data to the end of the TX queue */`
`483`		`- queue_put(&dev_data->tx.mem_block_queue, mem_block, size);`
`484`		`-`
`485`		`- return 0;`
	`483`	`+ return queue_put(&dev_data->tx.mem_block_queue, mem_block, size);`
`486`	`484`	`}`
`487`	`485`
`488`	`486`	`static const struct i2s_driver_api i2s_stm32_driver_api = {`