Opened 3 years ago

Last modified 3 years ago

#17462 closed bug

edid support digital input types — at Initial Version

Reported by: kallisti5 Owned by: nobody
Priority: normal Milestone: R1/beta4
Component: Drivers/Graphics Version: R1/beta3
Keywords: edid Cc:
Blocked By: Blocking:
Platform: All

Description

Our EDID headers are designed as follows:

headers/private/graphics/common/edid_raw.h:

// display info
typedef struct _PACKED {
        BBITFIELD8_7 (
                input_type : 1,         // 1 : digital
                input_voltage : 2,      // 0=0.7V/0.3V, 1=0.714V/0.286, 
                                                        // 2=1V/0.4V, 3=0.7V/0V
                setup : 1,                      // true if voltage configurable
                sep_sync : 1,
                comp_sync : 1,
                sync_on_green : 1,
                sync_serr : 1
        );

However, per the EDID (1.4) specifications, when input_type changes from 0 (analog) to 1 (digital), the meaning of the bits of the first bitfield change. (see attached screenshot)

We should likely rename these "input_settings : 7", and parse them manually in our code when we translate from edid1_display_raw to edid1_display.

Change History (0)

Note: See TracTickets for help on using tickets.