Re: [PATCH 4/5] fbdev: Make BIOS EDID reading configurable

["Antonino A. Daplas" <adaplas@gmail.com>]

Andrew Morton wrote:
> "Antonino A. Daplas" <adaplas@gmail.com> wrote:
>>> Also, is it not possible to defer this operation until we've parsed the
>>> command line, control it with a kernel boot parameter?
>>>
>> I thought of that, but it seems a bit complicated to me to parse boot
>> options in assembly so I left this for another time.
> 
> Well I'm wondering whether we can instead do this query later on, after
> we've entered start_kernel().

The query is done while machine is in real mode (in arch/i386/boot/video.S).

> 
>> The firmware EDID is only used as fallback, or if drivers have no DDC/I2C
>> support. Currently, all drivers that use the firmware EDID also have I2C
>> support in place. And chances are, if DDC transfers failed, the firmware
>> EDID will also be absent.
> 
> hm.  Don't you think `default y' would be better?  As it is, people's stuff
> will mysteriously stop working.

Hmm.  You're right, 'default y' looks better. Would you like me to send you
a patch, or would you rather take care of it?

Tony



-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642