diff options
| author | Ahmad Fatoum <ahmad@a3f.at> | 2021-01-01 23:52:26 +0100 |
|---|---|---|
| committer | Sascha Hauer <s.hauer@pengutronix.de> | 2021-01-05 13:11:16 +0100 |
| commit | 5db623db5d54a8b530b04f0f8977fbd60952c15d (patch) | |
| tree | c1073b754bd7b47c18be97a30f56804e8c977993 /drivers/video | |
| parent | 2f523f3a134d0c8133354cf3105a4e944d078f48 (diff) | |
| download | barebox-5db623db5d54a8b530b04f0f8977fbd60952c15d.tar.gz barebox-5db623db5d54a8b530b04f0f8977fbd60952c15d.tar.xz | |
video: edid: make edid_check_header externally visible
For non-i2c EDID use, drivers will want to verify the EDID header. We
already have a function for this, but internal linkage. Change this and
while at it change the type to a more appropriate type.
Signed-off-by: Ahmad Fatoum <ahmad@a3f.at>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'drivers/video')
| -rw-r--r-- | drivers/video/edid.c | 8 | ||||
| -rw-r--r-- | drivers/video/edid.h | 10 |
2 files changed, 14 insertions, 4 deletions
diff --git a/drivers/video/edid.c b/drivers/video/edid.c index 41d40d0297..1baff7317b 100644 --- a/drivers/video/edid.c +++ b/drivers/video/edid.c @@ -223,19 +223,19 @@ static int edid_checksum(unsigned char *edid) return err; } -static int edid_check_header(unsigned char *edid) +bool edid_check_header(unsigned char *edid) { - int i, err = 1, fix = check_edid(edid); + int i, fix = check_edid(edid); if (fix) fix_edid(edid, fix); for (i = 0; i < 8; i++) { if (edid[i] != edid_v1_header[i]) - err = 0; + return false; } - return err; + return true; } /* diff --git a/drivers/video/edid.h b/drivers/video/edid.h index 006d9f2834..60c5b6422e 100644 --- a/drivers/video/edid.h +++ b/drivers/video/edid.h @@ -135,4 +135,14 @@ #define DPMS_SUSPEND (1 << 6) #define DPMS_STANDBY (1 << 7) +/** + * edid_check_header - sanity check the header of the base EDID block + * @raw_edid: pointer to raw base EDID block + * + * Sanity check the header of the base EDID block. + * + * Return: true if the header is perfect, false if any byte is wrong. + */ +bool edid_check_header(unsigned char *edid); + #endif /* __EDID_H__ */ |