[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4F5D3C69.7050308@osadl.org>
Date: Mon, 12 Mar 2012 00:59:37 +0100
From: Carsten Emde <C.Emde@...dl.org>
To: Alan Cox <alan@...rguk.ukuu.org.uk>
CC: David Airlie <airlied@...ux.ie>,
Thomas Gleixner <tglx@...utronix.de>,
DRI <dri-devel@...ts.freedesktop.org>,
LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/3] drivers-gpu-drm-allow-to-load-edid-firmware.patch
Alan,
>> This patch allows to load an EDID data set via the firmware interface.
>> It contains data sets of frequently used screen resolutions (1024x768,
>> 1280x1024, 1680x1050 and 1920x1080). The requested EDID data are
>> specified as a module parameter of the drm_kms_helper module, e.g.
>> options drm_kms_helper edid_firmware=edid/1280x1024.bin or as kernel
>> command line parameter.
> [..]
> Given the EDID is tiny and is data not code wouldn't it be simpler (and
> smaller) if this option compiled in a few generic EDIDs to use ?
I liked the idea so much that I gave it a try. The patch looks much
better now, but has the same functionality as before. As the only
disadvantage, the patch no longer contains the template to produce a
particular EDID. But this code may be made available elsewhere - no need
to have it in the kernel.
Hope you like it.
Thanks,
-Carsten.
View attachment "drivers-gpu-drm-allow-to-load-edid-firmware.patch" of type "text/x-patch" (15366 bytes)
Powered by blists - more mailing lists