Skip to content

xfree86: ddc: new entry point for EDID parsing#1881

Closed
metux wants to merge 0 commit intomasterfrom
pr/xfree86-ddc-new-entry-point-for-edid-parsing-_2026-01-27_20-29-45
Closed

xfree86: ddc: new entry point for EDID parsing#1881
metux wants to merge 0 commit intomasterfrom
pr/xfree86-ddc-new-entry-point-for-edid-parsing-_2026-01-27_20-29-45

Conversation

@metux
Copy link
Contributor

@metux metux commented Jan 27, 2026

The old ones didn't know the block size, so couldn't deduce the block
type version. With upcoming new features, eg. HDR, we need to know the
block type version in order to know what we can extract from it.

This new function should now be used by all drivers, the old ones shall
be phased out.

That commit should be backported to 25.0 and 25.1 releases, so drivers
can remain compatible with all existing release lines.

Signed-off-by: Enrico Weigelt, metux IT consult info@metux.net

@metux metux self-assigned this Jan 27, 2026
@metux metux requested a review from a team January 27, 2026 19:30
@metux metux force-pushed the pr/xfree86-ddc-new-entry-point-for-edid-parsing-_2026-01-27_20-29-45 branch 2 times, most recently from 48fdb88 to 580262b Compare January 28, 2026 13:04
@metux
Copy link
Contributor Author

metux commented Jan 28, 2026

@cepelinas9000 this is preparation for your HDR work.
You can add the stuff from your xf86DoInterpretHDRMetadata() into the new xf86ParseEDID() function.

@stefan11111
Copy link
Contributor

With upcoming new features, eg. HDR,

Are you planning to add hdr support to the modesetting driver only, or to other drivers too?

@metux
Copy link
Contributor Author

metux commented Feb 2, 2026

With upcoming new features, eg. HDR,

Are you planning to add hdr support to the modesetting driver only, or to other drivers too?

IMHO we should start with modesetting and then take care of the others. (IIRC there're only two relevant ones: amd and intel).

@metux metux force-pushed the pr/xfree86-ddc-new-entry-point-for-edid-parsing-_2026-01-27_20-29-45 branch 2 times, most recently from a82187c to 8dadfc4 Compare February 2, 2026 10:22
@metux
Copy link
Contributor Author

metux commented Feb 2, 2026

@cepelinas9000 is working on HDR stuff, I'm only doing some infrastructure preparations here.
I'm also going to add some more accessor functions, so we can get rid of some ugly macros and hopefully have the whole EDID-internal structures encapsulated / hidden away from the drivers.

@metux metux force-pushed the pr/xfree86-ddc-new-entry-point-for-edid-parsing-_2026-01-27_20-29-45 branch 3 times, most recently from dd62695 to 1b4759d Compare February 3, 2026 15:27
@metux metux closed this Feb 3, 2026
@metux metux force-pushed the pr/xfree86-ddc-new-entry-point-for-edid-parsing-_2026-01-27_20-29-45 branch from 1b4759d to 3cb0735 Compare February 3, 2026 16:04
@metux
Copy link
Contributor Author

metux commented Feb 3, 2026

Merged via #1933

@metux
Copy link
Contributor Author

metux commented Feb 3, 2026

Also going to backport it to 25.0 and 25.1

@metux
Copy link
Contributor Author

metux commented Feb 3, 2026

backports:
#1934
#1935

@metux metux deleted the pr/xfree86-ddc-new-entry-point-for-edid-parsing-_2026-01-27_20-29-45 branch February 3, 2026 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants