diff options
author | Patrick Georgi <pgeorgi@google.com> | 2014-12-15 22:52:14 +0100 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2014-12-16 21:31:35 +0100 |
commit | be71ee5dec8f038cdd0e98efe93dadf0128d0f24 (patch) | |
tree | 266c419ccd70a23d3080245711d2f6d28e70b7a0 /src/include | |
parent | 46c85d7dfbe6b789f24f724dee189d529c215c76 (diff) | |
download | coreboot-be71ee5dec8f038cdd0e98efe93dadf0128d0f24.tar.xz |
edid: remove float use
First, we don't want floats in our code base.
Second, the calculation of the aspect ratio was wacky,
using a value guaranteed to be 0 for aspect ratio calculation.
While at it, define the aspect_* fields to be in tenths, to
provide some additional resolution. They were like that already
but we now also commit to that.
Change-Id: I5511adf4bf76cdd6a69240491372f220ef1aa687
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-on: http://review.coreboot.org/7803
Reviewed-by: Marc Jones <marc.jones@se-eng.com>
Reviewed-by: Julius Werner <jwerner@chromium.org>
Tested-by: build bot (Jenkins)
Diffstat (limited to 'src/include')
-rw-r--r-- | src/include/edid.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/include/edid.h b/src/include/edid.h index 867a82ff28..9a1fda5d44 100644 --- a/src/include/edid.h +++ b/src/include/edid.h @@ -89,6 +89,7 @@ struct edid { u32 bytes_per_line; /* it is unlikely we need these things. */ /* if one of these is non-zero, use that one. */ + /* they're aspect * 10 to provide some additional resolution */ unsigned int aspect_landscape; unsigned int aspect_portrait; const char *range_class; |