diff options
author | Stewart Smith <stewart@linux.ibm.com> | 2018-11-29 15:28:32 +1100 |
---|---|---|
committer | Stewart Smith <stewart@linux.ibm.com> | 2018-12-10 22:21:12 -0600 |
commit | babd5a27dc06143b3b8860eb121aa382e1315022 (patch) | |
tree | ff6a0d0ea855452694246dc1e364c05abc49448d /hdata | |
parent | 99fc78dbee4af123bffb5ca6a9e17a423fa2d013 (diff) | |
download | skiboot-babd5a27dc06143b3b8860eb121aa382e1315022.zip skiboot-babd5a27dc06143b3b8860eb121aa382e1315022.tar.gz skiboot-babd5a27dc06143b3b8860eb121aa382e1315022.tar.bz2 |
hdata/iohub.c: remove condition that was always true
Caught by static analysis. The previous if() condition was ensuring lxr
was not null, so we don't need this additional check.
Signed-off-by: Stewart Smith <stewart@linux.ibm.com>
Diffstat (limited to 'hdata')
-rw-r--r-- | hdata/iohub.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/hdata/iohub.c b/hdata/iohub.c index f37f5ab..e16fb0b 100644 --- a/hdata/iohub.c +++ b/hdata/iohub.c @@ -82,8 +82,7 @@ static bool io_get_lx_info(const void *kwvpd, unsigned int kwvpd_sz, return false; } - if (lxr) - memcpy(lxrbuf, lxr, sizeof(uint32_t)*2); + memcpy(lxrbuf, lxr, sizeof(uint32_t)*2); prlog(PR_DEBUG, "CEC: LXRn=%d LXR=%08x%08x\n", lx_idx, lxrbuf[0], lxrbuf[1]); prlog(PR_DEBUG, "CEC: LX Info added to %llx\n", (long long)hn); |