diff options
author | Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> | 2024-08-16 09:36:26 -0700 |
---|---|---|
committer | Ilpo Järvinen <ilpo.jarvinen@linux.intel.com> | 2024-08-20 14:19:09 +0300 |
commit | 46ee21e9f59205e54943dfe51b2dc8a9352ca37d (patch) | |
tree | 0a21abaffb885ce348ebabbeb53ac2e7612d5e98 /arch | |
parent | bc923d594db21bee0ead128eb4bb78f7e77467a4 (diff) | |
download | linux-46ee21e9f59205e54943dfe51b2dc8a9352ca37d.tar.gz linux-46ee21e9f59205e54943dfe51b2dc8a9352ca37d.tar.bz2 linux-46ee21e9f59205e54943dfe51b2dc8a9352ca37d.zip |
platform/x86: ISST: Fix return value on last invalid resource
When only the last resource is invalid, tpmi_sst_dev_add() is returing
error even if there are other valid resources before. This function
should return error when there are no valid resources.
Here tpmi_sst_dev_add() is returning "ret" variable. But this "ret"
variable contains the failure status of last call to sst_main(), which
failed for the invalid resource. But there may be other valid resources
before the last entry.
To address this, do not update "ret" variable for sst_main() return
status.
If there are no valid resources, it is already checked for by !inst
below the loop and -ENODEV is returned.
Fixes: 9d1d36268f3d ("platform/x86: ISST: Support partitioned systems")
Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Cc: stable@vger.kernel.org # 6.10+
Link: https://lore.kernel.org/r/20240816163626.415762-1-srinivas.pandruvada@linux.intel.com
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Diffstat (limited to 'arch')
0 files changed, 0 insertions, 0 deletions