diff options
author | Colin Ian King <colin.king@canonical.com> | 2016-09-09 05:09:29 -0400 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2016-09-24 14:56:28 -0400 |
commit | dc995069c675af71a2ecf2ade0995df084da3e2e (patch) | |
tree | 5f75610dc0e36cda89ae2849ca438ed8e5e3ea0e /sound | |
parent | fe27f4e0545d3fc1b0518fafb4fe0460d757651d (diff) |
ASoC: Intel: remove status, it is shadowing status of a higher scope
The second declaration of status is shadowing the status of a higher
scope. This uninitialized status results in garbage being returned
by the !x86_match_cpu(cpu_ids) || !iosf_mbi_available() return exit
path. Fix this by removing the extraneous second declaration of
status.
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound')
-rw-r--r-- | sound/soc/intel/atom/sst/sst_acpi.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/sound/soc/intel/atom/sst/sst_acpi.c b/sound/soc/intel/atom/sst/sst_acpi.c index 9718e8249224..ba5c0d71720a 100644 --- a/sound/soc/intel/atom/sst/sst_acpi.c +++ b/sound/soc/intel/atom/sst/sst_acpi.c | |||
@@ -249,7 +249,6 @@ static int is_byt_cr(struct device *dev, bool *bytcr) | |||
249 | { X86_VENDOR_INTEL, 6, 55 }, /* Valleyview, Bay Trail */ | 249 | { X86_VENDOR_INTEL, 6, 55 }, /* Valleyview, Bay Trail */ |
250 | {} | 250 | {} |
251 | }; | 251 | }; |
252 | int status; | ||
253 | u32 bios_status; | 252 | u32 bios_status; |
254 | 253 | ||
255 | if (!x86_match_cpu(cpu_ids) || !iosf_mbi_available()) { | 254 | if (!x86_match_cpu(cpu_ids) || !iosf_mbi_available()) { |