diff options
| author | Ronald G. Minnich <rminnich@google.com> | 2013-06-28 14:33:30 -0700 |
|---|---|---|
| committer | Stefan Reinauer <stefan.reinauer@coreboot.org> | 2013-12-12 22:21:22 +0100 |
| commit | 4c5b161e3f0a2daa7d37bb9d77b5dba468dd3c0e (patch) | |
| tree | f3a8b0a30f13c00b07837c4bb5cdf11d93ddc60b /src/include/edid.h | |
| parent | 3ece50d9db35eecef557fc6ad8cc404224cfc9ac (diff) | |
| download | coreboot-4c5b161e3f0a2daa7d37bb9d77b5dba468dd3c0e.tar.gz coreboot-4c5b161e3f0a2daa7d37bb9d77b5dba468dd3c0e.tar.bz2 coreboot-4c5b161e3f0a2daa7d37bb9d77b5dba468dd3c0e.zip | |
EDID: add fields specialized to the needs of framebuffers
Now that we have horizontal display areas that are not multiples of 32 bytes,
things are more complex. We add three struct members (x, y resolution and
bytes per line) which are to be filled in by the mainboard as it sets the mode.
In future, the EDID code may take a stab at initializing these but the values are
context-dependent.
Change-Id: Ib9102d6bbf8c66931f5adb1029a04b881a982cfe
Signed-off-by: Ronald G. Minnich <rminnich@google.com>
Reviewed-on: https://gerrit.chromium.org/gerrit/60514
Tested-by: Ronald G. Minnich <rminnich@chromium.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Commit-Queue: Ronald G. Minnich <rminnich@chromium.org>
Reviewed-on: http://review.coreboot.org/4336
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
Diffstat (limited to 'src/include/edid.h')
| -rw-r--r-- | src/include/edid.h | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/src/include/edid.h b/src/include/edid.h index 7d4736dc334e..ce1f615709c3 100644 --- a/src/include/edid.h +++ b/src/include/edid.h @@ -55,6 +55,14 @@ struct edid { unsigned int vso; unsigned int vspw; unsigned int vborder; + /* 3 variables needed for coreboot framebuffer. + * In most cases, they are the same as the ha + * and va variables, but not always, as in the + * case of a 1366 wide display. + */ + u32 x_resolution; + u32 y_resolution; + u32 bytes_per_line; /* it is unlikely we need these things. */ /* if one of these is non-zero, use that one. */ unsigned int aspect_landscape; |