diff options
author | Alexandru Gagniuc <mr.nuke.me@gmail.com> | 2014-12-28 20:38:32 -0600 |
---|---|---|
committer | Alexandru Gagniuc <mr.nuke.me@gmail.com> | 2014-12-29 04:06:23 +0100 |
commit | 2e0cf14e144593f8b364f8741f7196d829f3b427 (patch) | |
tree | 8a8ad2ed2e4e34004bad310d29bf57f927dc170f /src/mainboard/lenovo | |
parent | 4851bf28228114016cf69720422c38e92ec091eb (diff) | |
download | coreboot-2e0cf14e144593f8b364f8741f7196d829f3b427.tar.xz |
northbridge/amd/pi/northbridge.c: Remove superfluous logic operand
The "((1ull << (sizeof(modules) * 8)) - 1)" statement evaluates to
0xffffffff, but there's no need to AND with that value, as 'modules'
is already 32-bit. The '&&' is most likely a typo, which meant bitwise
and, as indicated by the structure of thus operation.
Remove this superfluous statement. This also fixes a clang warning.
Change-Id: Ie55bd9f8b0ec5fd41e440f56dcedd40c830bf826
Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
Reviewed-on: http://review.coreboot.org/7965
Reviewed-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Tested-by: build bot (Jenkins)
Diffstat (limited to 'src/mainboard/lenovo')
0 files changed, 0 insertions, 0 deletions