edid-generator icon indicating copy to clipboard operation
edid-generator copied to clipboard

No support for interlacing

Open Ketorin opened this issue 8 years ago • 5 comments

Would it be a lot of work if support for interlaced modes was added?

Some people would need it, I would need it. (CRT projector, 1080 lines interlaced.)

Meanwhile, I'm studying your work and alternative ways to spoof the EDID.

Ketorin avatar Nov 05 '16 14:11 Ketorin

Ah, yeah, interlaced should be supported as is, what modeline are you handing it? Ty, Trevor

akatrevorjay avatar Dec 29 '16 09:12 akatrevorjay

Hi,

sorry to necro this issue, but I have a fix ! First apply this patch :

diff --git a/edid.S b/edid.S
index ef082dc..d1adf9e 100644
--- a/edid.S
+++ b/edid.S
@@ -237,7 +237,7 @@ y_border:   .byte   0
    Bit 1       If analog sync: Sync on all 3 RGB lines (else green only)
    Digital: HSync polarity (1=positive)
    Bit 0       2-way line-interleaved stereo, if bits 4-3 are not 00. */
-features:      .byte   0x18+(VSYNC_POL<<2)+(HSYNC_POL<<1)
+features:      .byte   FEATURES+(VSYNC_POL<<2)+(HSYNC_POL<<1)
 
 descriptor2:   .byte   0,0     /* Not a detailed timing descriptor */
                .byte   0       /* Must be zero */

This allows to use a #define FEATURES inside your .S like:

#define FEATURES 0x18 | 1<<7

See how I've enabled the interlace bit (number 7) for the features byte with 1<<7. I'm not sure how the sync bits (the 0x18) work, they could have stayed in the edid.S rather than in my own edid .S

I can submit a PR if you wish.

substring avatar Apr 16 '19 15:04 substring

Please could this patch be integrated? Without interlace support people trying to use this great tool for interlaced monitors/TVs are stuck.

STPKITT avatar Oct 27 '23 21:10 STPKITT

@STPKITT @substring Want to put up a patch? I'm happy to accept this if you can wire it up as an option to declare this

akatrevorjay avatar Nov 15 '23 03:11 akatrevorjay

Oh God that was aeons ago ... May have a look at it eventually, but it would be best to add a commandline parameter to ask for an interlaced resolution. Just applying the change like this would make all resolutions interlaced, and we don't want that

substring avatar Nov 15 '23 16:11 substring