[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 2/9] edid: edid_desc_next
From: |
Gerd Hoffmann |
Subject: |
[PATCH 2/9] edid: edid_desc_next |
Date: |
Tue, 16 Mar 2021 15:38:05 +0100 |
Add helper function to find the next free desc block.
Needed when we start to use the dta descriptor entries.
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
hw/display/edid-generate.c | 41 ++++++++++++++++++++++++--------------
1 file changed, 26 insertions(+), 15 deletions(-)
diff --git a/hw/display/edid-generate.c b/hw/display/edid-generate.c
index 1665b7cbb29a..ef7a70fc9654 100644
--- a/hw/display/edid-generate.c
+++ b/hw/display/edid-generate.c
@@ -143,6 +143,17 @@ static void edid_checksum(uint8_t *edid)
}
}
+static uint8_t *edid_desc_next(uint8_t *edid, uint8_t *dta, uint8_t *desc)
+{
+ if (desc == NULL) {
+ return NULL;
+ }
+ if (desc + 18 + 18 < edid + 127) {
+ return desc + 18;
+ }
+ return NULL;
+}
+
static void edid_desc_type(uint8_t *desc, uint8_t type)
{
desc[0] = 0;
@@ -299,7 +310,7 @@ uint32_t qemu_edid_dpi_to_mm(uint32_t dpi, uint32_t res)
void qemu_edid_generate(uint8_t *edid, size_t size,
qemu_edid_info *info)
{
- uint32_t desc = 54;
+ uint8_t *desc = edid + 54;
uint8_t *xtra3 = NULL;
uint8_t *dta = NULL;
uint32_t width_mm, height_mm;
@@ -400,32 +411,32 @@ void qemu_edid_generate(uint8_t *edid, size_t size,
/* =============== descriptor blocks =============== */
- edid_desc_timing(edid + desc, info->prefx, info->prefy,
+ edid_desc_timing(desc, info->prefx, info->prefy,
width_mm, height_mm);
- desc += 18;
+ desc = edid_desc_next(edid, dta, desc);
- edid_desc_ranges(edid + desc);
- desc += 18;
+ edid_desc_ranges(desc);
+ desc = edid_desc_next(edid, dta, desc);
if (info->name) {
- edid_desc_text(edid + desc, 0xfc, info->name);
- desc += 18;
+ edid_desc_text(desc, 0xfc, info->name);
+ desc = edid_desc_next(edid, dta, desc);
}
if (info->serial) {
- edid_desc_text(edid + desc, 0xff, info->serial);
- desc += 18;
+ edid_desc_text(desc, 0xff, info->serial);
+ desc = edid_desc_next(edid, dta, desc);
}
- if (desc < 126) {
- xtra3 = edid + desc;
+ if (desc) {
+ xtra3 = desc;
edid_desc_xtra3_std(xtra3);
- desc += 18;
+ desc = edid_desc_next(edid, dta, desc);
}
- while (desc < 126) {
- edid_desc_dummy(edid + desc);
- desc += 18;
+ while (desc) {
+ edid_desc_dummy(desc);
+ desc = edid_desc_next(edid, dta, desc);
}
/* =============== finish up =============== */
--
2.30.2
- [PATCH 0/9] edid: windows fixes, Gerd Hoffmann, 2021/03/16
- [PATCH 3/9] edid: move xtra3 descriptor, Gerd Hoffmann, 2021/03/16
- [PATCH 4/9] edid: use dta extension block descriptors, Gerd Hoffmann, 2021/03/16
- [PATCH 2/9] edid: edid_desc_next,
Gerd Hoffmann <=
- [PATCH 8/9] edid: allow arbitrary-length checksums, Gerd Hoffmann, 2021/03/16
- [PATCH 1/9] qemu-edid: use qemu_edid_size(), Gerd Hoffmann, 2021/03/16
- [PATCH 9/9] edid: add support for DisplayID extension (5k resolution), Gerd Hoffmann, 2021/03/16
- [PATCH 5/9] edid: prefer standard timings, Gerd Hoffmann, 2021/03/16
- [PATCH 6/9] edid: Make refresh rate configurable, Gerd Hoffmann, 2021/03/16
- [PATCH 7/9] edid: move timing generation into a separate function, Gerd Hoffmann, 2021/03/16
- Re: [PATCH 0/9] edid: windows fixes, no-reply, 2021/03/16