| From f9d72d31e765df62f49ba18bc48bd82fe7c73059 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: [PATCH] ASoC: Intel: mrfld: return error codes when an error occurs |
| |
| 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: Paul Gortmaker <paul.gortmaker@windriver.com> |
| |
| diff --git a/sound/soc/intel/atom/sst/sst_pci.c b/sound/soc/intel/atom/sst/sst_pci.c |
| index d952719bc098..5862fe968083 100644 |
| --- 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(struct intel_sst_drv *ctx) |
| dev_dbg(ctx->dev, "DRAM Ptr %p\n", ctx->dram); |
| do_release_regions: |
| pci_release_regions(pci); |
| - return 0; |
| + return ret; |
| } |
| |
| /* |
| -- |
| 2.7.4 |
| |
