| From 3025571edd9df653e1ad649f0638368a39d1bbb5 Mon Sep 17 00:00:00 2001 |
| From: Colin Ian King <colin.king@canonical.com> |
| Date: Sat, 8 Feb 2020 22:07:20 +0000 |
| Subject: ASoC: Intel: mrfld: return error codes when an error occurs |
| |
| From: Colin Ian King <colin.king@canonical.com> |
| |
| commit 3025571edd9df653e1ad649f0638368a39d1bbb5 upstream. |
| |
| Currently function sst_platform_get_resources always returns zero and |
| error return codes set by the function are never returned. Fix this |
| by returning the error return code in variable ret rather than the |
| hard coded zero. |
| |
| Addresses-Coverity: ("Unused value") |
| Fixes: f533a035e4da ("ASoC: Intel: mrfld - create separate module for pci part") |
| Signed-off-by: Colin Ian King <colin.king@canonical.com> |
| Acked-by: Cezary Rojewski <cezary.rojewski@intel.com> |
| Acked-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> |
| Link: https://lore.kernel.org/r/20200208220720.36657-1-colin.king@canonical.com |
| Signed-off-by: Mark Brown <broonie@kernel.org> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| sound/soc/intel/atom/sst/sst_pci.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/sound/soc/intel/atom/sst/sst_pci.c |
| +++ b/sound/soc/intel/atom/sst/sst_pci.c |
| @@ -99,7 +99,7 @@ static int sst_platform_get_resources(st |
| dev_dbg(ctx->dev, "DRAM Ptr %p\n", ctx->dram); |
| do_release_regions: |
| pci_release_regions(pci); |
| - return 0; |
| + return ret; |
| } |
| |
| /* |