diff options
author | Martin Roth <martinroth@google.com> | 2015-12-18 10:46:59 -0700 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2015-12-20 02:41:57 +0100 |
commit | 2b2ff7fa6a9af366c1a9b8e4cb6f7f566048b0d7 (patch) | |
tree | 0357bd47d2f2de60ee31ee5a06f00285e4021da9 /src/superio | |
parent | 9f25da1f490e3e6ce02936c8d6404c68699b5aff (diff) |
soc/intel/broadwell: Init var before use, only use when needed
root_port_init_config() pcie.c wasn't initializing a variable before
passing its pointer to pch_iobp_exec(). pch_iobp_exec() wrote the
uninitialized value into a register.
In theory, the register would only be used if data was being written,
and pch_iobp_exec() was being used to read the data, not write it, so
this change shouldn't have any practical effect.
Fixes coverity error:
CID 1293134 (#1 of 1): Uninitialized scalar variable (UNINIT)
Change-Id: I5d17863d904c6b1ceb30d72b94cd7a40c8fbb437
Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://review.coreboot.org/12778
Tested-by: build bot (Jenkins)
Reviewed-by: Duncan Laurie <dlaurie@google.com>
Diffstat (limited to 'src/superio')
0 files changed, 0 insertions, 0 deletions