On Wed, Nov 14, 2018 at 12:49:11PM +0200, Jani Nikula wrote:
> On Tue, 13 Nov 2018, Ben Hutchings <ben.hutchings@xxxxxxxxxxxxxxx> 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@xxxxxxxxxxxxxxxxx
>
> >
> > 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
