[coreboot-gerrit] Change in coreboot[master]: drivers/intel/gma/edid: Use byte swap function

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38647 )

Change subject: drivers/intel/gma/edid: Use byte swap function
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/38647/1/src/drivers/intel/gma/edid.... 
File src/drivers/intel/gma/edid.c:

https://review.coreboot.org/c/coreboot/+/38647/1/src/drivers/intel/gma/edid.... 
PS1, Line 96: edid
> Call me cursed, buuut... […]

I am not so good in assembly. Can you take a look please?

```
 .LBE128:
 .LBE127:
-       .loc 1 94 3 is_stmt 1 discriminator 3 view .LVU157
-       addl    $4, %ebp
-       .loc 1 94 15 is_stmt 0 discriminator 3 view .LVU158
-       movb    %al, -4(%ebp)
-       .loc 1 95 3 is_stmt 1 discriminator 3 view .LVU159
-       .loc 1 96 28 is_stmt 0 discriminator 3 view .LVU160
-       movl    %eax, %edx
-       .loc 1 95 19 discriminator 3 view .LVU161
-       movb    %ah, -3(%ebp)
-       .loc 1 96 3 is_stmt 1 discriminator 3 view .LVU162
-       .loc 1 96 28 is_stmt 0 discriminator 3 view .LVU163
-       shrl    $16, %edx
-       .loc 1 96 19 discriminator 3 view .LVU164
-       movb    %dl, -2(%ebp)
-       .loc 1 97 3 is_stmt 1 discriminator 3 view .LVU165
-       .loc 1 97 35 is_stmt 0 discriminator 3 view .LVU166
-       shrl    $24, %eax
+       .loc 1 96 3 is_stmt 1 discriminator 3 view .LVU159
+       .loc 1 96 15 is_stmt 0 discriminator 3 view .LVU160
+       movl    76(%esp), %edx
+       .loc 1 96 17 discriminator 3 view .LVU161
+       bswap   %eax
 .LVL45:
-       .loc 1 97 19 discriminator 3 view .LVU167
-       movb    %al, -1(%ebp)
+       .loc 1 96 15 discriminator 3 view .LVU162
+       movb    %al, (%edx,%ebp,4)
```

I guess the `movb` is the interesting part?