Re: [PATCH 0/5] Fixes and additions to EDID generation

[Daniel Vetter <daniel@ffwll.ch>]

On Wed, Nov 14, 2018 at 12:49:11PM +0200, Jani Nikula wrote:
> On Tue, 13 Nov 2018, Ben Hutchings <ben.hutchings@codethink.co.uk> wrote:
> > This series adds two more EDIDs that I needed for a project some time ago,
> > and fixes some problems that I found along the way.
> 
> See also [1]. The patches there have been applied via the linux-doc
> tree.
> 
> > There isn't a listed maintainer for Documentation/EDID but it looks
> > like it falls under the DRM umbrella.  Please let me know if this the
> > wrong place.
> 
> Cc: linux-doc, Jon and Dave.
> 
> The Documentation/EDID directory was merged by Dave along with the EDID
> firmware loader mechanism in da0df92b5731 ("drm: allow loading an EDID
> as firmware to override broken monitor").
> 
> I think it would be reasonable to handle the EDID stuff via dri-devel
> and the drm trees, and modify MAINTAINERS to reflect that. Thoughts?
> 
> (And in the long run we should use or write a tool to generate EDIDs
> instead of using assembly with macros. And like Jon already mentioned in
> the other thread, even the assembly probably shouldn't be under
> Documentation. But I digress.)

If we do that, maybe stuff the EDID things into Documentation/gpu? Would
need an ack from media folks I think, but imo would make sense.
-Daniel

> 
> 
> BR,
> Jani.
> 
> [1] http://mid.mail-archive.com/1541407715-5417-1-git-send-email-cniedermaier@dh-electronics.de
> 
> >
> > Ben.
> >
> > Ben Hutchings (5):
> >   drm: EDID: Remove a mess involving the number 63
> >   drm: EDID: Fix bit masking of {X,Y}{OFFSET,PULSE}
> >   drm: EDID: Don't force make to be silent
> >   drm: EDID: Add a 1280x720 (720p) EDID
> >   drm: EDID: Add a 1280x768 ("WXGA") EDID
> >
> >  Documentation/EDID/1024x768.S  |  4 ++--
> >  Documentation/EDID/1280x1024.S |  4 ++--
> >  Documentation/EDID/1280x720.S  | 45 ++++++++++++++++++++++++++++++++++++++++++
> >  Documentation/EDID/1280x768.S  | 45 ++++++++++++++++++++++++++++++++++++++++++
> >  Documentation/EDID/1600x1200.S |  4 ++--
> >  Documentation/EDID/1680x1050.S |  4 ++--
> >  Documentation/EDID/1920x1080.S |  4 ++--
> >  Documentation/EDID/800x600.S   |  4 ++--
> >  Documentation/EDID/HOWTO.txt   |  4 ++--
> >  Documentation/EDID/Makefile    | 12 +++++------
> >  Documentation/EDID/edid.S      | 10 ++++------
> >  11 files changed, 114 insertions(+), 26 deletions(-)
> >  create mode 100644 Documentation/EDID/1280x720.S
> >  create mode 100644 Documentation/EDID/1280x768.S
> 
> -- 
> Jani Nikula, Intel Open Source Graphics Center

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch