diff options
| author | Christoph Niedermaier <cniedermaier@dh-electronics.de> | 2018-11-05 09:48:34 +0100 |
|---|---|---|
| committer | Jonathan Corbet <corbet@lwn.net> | 2018-11-06 15:36:18 +0100 |
| commit | d2f112a5681078aa2150f218074f76617fea4d74 (patch) | |
| tree | 3df1cfe1b211827eb77446a0f9b7e9673b26c554 /Documentation/EDID/1680x1050.S | |
| parent | Linux 4.20-rc1 (diff) | |
| download | linux-d2f112a5681078aa2150f218074f76617fea4d74.tar.xz linux-d2f112a5681078aa2150f218074f76617fea4d74.zip | |
Docs/EDID: Fixed erroneous bits of XOFFSET, XPULSE, YOFFSET and YPULSE
The problem was found when EDID data sets for displays other
than the provided samples were generated. The patch has
no effect on the provided samples that still match the data
used in drivers/gpu/drm/drm_edid_load.c.
The provided samples use small values for XOFFSET, XPULSE,
YOFFSET and YPULSE, where the error doesn't occur. This fix
corrects the use of that values in case of high values, because
the most significant bits were treated incorrectly.
So in edid.S msbs4 should use bit 8 and 9 of XOFFSET and XPULS.
For YOFFSET and YPULSE msbs4 should use bit 4 and 5.
lsbs2 was introduced for a better overview, without
functional change.
Removing also the useless value 63 of all files, because
it is added in the *.S description files and then it is
subtracted in edid.S.
Signed-off-by: Christoph Niedermaier <cniedermaier@dh-electronics.de>
Reviewed-by: Carsten Emde <C.Emde@osadl.org>
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
Diffstat (limited to 'Documentation/EDID/1680x1050.S')
| -rw-r--r-- | Documentation/EDID/1680x1050.S | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Documentation/EDID/1680x1050.S b/Documentation/EDID/1680x1050.S index b0d7c69282b4..ec679507e33b 100644 --- a/Documentation/EDID/1680x1050.S +++ b/Documentation/EDID/1680x1050.S @@ -31,8 +31,8 @@ #define YBLANK 39 #define XOFFSET 104 #define XPULSE 176 -#define YOFFSET (63+3) -#define YPULSE (63+6) +#define YOFFSET 3 +#define YPULSE 6 #define DPI 96 #define VFREQ 60 /* Hz */ #define TIMING_NAME "Linux WSXGA" |