diff options
author | Martin Roth <martinroth@google.com> | 2016-01-16 09:22:22 -0700 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2016-01-16 21:03:30 +0100 |
commit | 5d6929b1d7066bc7f1963eb2a511cdf734f33f94 (patch) | |
tree | 8a86e2c7a9f6549d180fc416d713c2990c389e4e /util/superiotool/winbond.c | |
parent | c5dfdc5992cf1d9ec6b6e5b4f24a7d2fab85b024 (diff) | |
download | coreboot-5d6929b1d7066bc7f1963eb2a511cdf734f33f94.tar.xz |
intel/skylake: Fix uninitialized variable warning
I don't think the warning is valid, because we already verify
that num_channels is 2 or 4 as soon as we enter the function.
Adding the default case makes the compiler happy.
Fixes warning:
src/soc/intel/skylake/nhlt/dmic.c: In function 'nhlt_soc_add_dmic_array':
src/soc/intel/skylake/nhlt/dmic.c:100:2: error: 'formats' may be used
uninitialized in this function [-Werror=maybe-uninitialized]
return nhlt_endpoint_add_formats(endp, formats, num_formats);
^
Change-Id: Idc22c8478ff666af8915d780d7553909c3163690
Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://review.coreboot.org/13021
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'util/superiotool/winbond.c')
0 files changed, 0 insertions, 0 deletions