Paul Menzel has uploaded this change for review.

View Change

drivers/intel/gma/edid: Use unsigned type for count variable

No real difference in the assembly though.

@@ -799,8 +799,8 @@
.string "i"
.byte 0x1
.byte 0x44
- .byte 0x6
- .long 0x4d
+ .byte 0xf
+ .long 0x60
.long .LLST25
.long .LVUS25
.uleb128 0xe

Change-Id: I340a2752f955e5e4d2198a81408b3663b1488b80
Signed-off-by: Paul Menzel <pmenzel@molgen.mpg.de>
---
M src/drivers/intel/gma/edid.c
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/45/38645/1
diff --git a/src/drivers/intel/gma/edid.c b/src/drivers/intel/gma/edid.c
index 4d4aec3..7e6c840 100644
--- a/src/drivers/intel/gma/edid.c
+++ b/src/drivers/intel/gma/edid.c
@@ -65,7 +65,7 @@

void intel_gmbus_read_edid(u8 *mmio, u8 bus, u8 slave, u8 *edid, u32 edid_size)
{
- int i;
+ unsigned int i;

slave &= 0x7f;
edid_size &= 0x1fc;

To view, visit change 38645. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I340a2752f955e5e4d2198a81408b3663b1488b80
Gerrit-Change-Number: 38645
Gerrit-PatchSet: 1
Gerrit-Owner: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-MessageType: newchange