ASoC: Intel: remove status, it is shadowing status of a higher scope
authorColin Ian King <colin.king@canonical.com>
Fri, 9 Sep 2016 09:09:29 +0000 (10:09 +0100)
committerMark Brown <broonie@kernel.org>
Sat, 24 Sep 2016 18:56:28 +0000 (19:56 +0100)
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>
sound/soc/intel/atom/sst/sst_acpi.c

index 9718e824922465121b7bf406cb3fe8bdb36ea9a3..ba5c0d71720ab8548fddbb42cd79dd90c869abab 100644 (file)
@@ -249,7 +249,6 @@ static int is_byt_cr(struct device *dev, bool *bytcr)
                        { X86_VENDOR_INTEL, 6, 55 }, /* Valleyview, Bay Trail */
                        {}
                };
-               int status;
                u32 bios_status;
 
                if (!x86_match_cpu(cpu_ids) || !iosf_mbi_available()) {