[coreboot-gerrit] New patch to review for coreboot: 89f566e autoport/bd82x6x: Fix off-by-one error

Vladimir Serbinenko (phcoder@gmail.com) gerrit at coreboot.org
Fri May 29 22:51:33 CEST 2015


Vladimir Serbinenko (phcoder at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10371

-gerrit

commit 89f566edf0669751aa2ebf5e0f01ff6a590545c8
Author: Vladimir Serbinenko <phcoder at gmail.com>
Date:   Fri May 29 21:43:42 2015 +0200

    autoport/bd82x6x: Fix off-by-one error
    
    Change-Id: Ibd5c8a1364ab183f72e12cc9b450a36200fe3fde
    Signed-off-by: Vladimir Serbinenko <phcoder at gmail.com>
---
 util/autoport/bd82x6x.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/util/autoport/bd82x6x.go b/util/autoport/bd82x6x.go
index 91538bd..4a0f503 100644
--- a/util/autoport/bd82x6x.go
+++ b/util/autoport/bd82x6x.go
@@ -105,7 +105,7 @@ func ich9GetFlashSize(ctx Context) {
 		highflkb := uint32(0)
 		for reg := uint16(0); reg < 5; reg++ {
 			fl := (inteltool.RCBA[0x3854+4*reg] >> 16) & 0x1fff
-			flkb := fl << 2
+			flkb := (fl + 1) << 2
 			if flkb > highflkb {
 				highflkb = flkb
 			}



More information about the coreboot-gerrit mailing list