[haiku-bugs] Re: [Haiku] #7787: VESA regression: 640x480 no longer works at boot (but works in Screen prefs!)

  • From: "ttcoder" <trac@xxxxxxxxxxxx>
  • Date: Fri, 15 Jul 2011 16:17:23 -0000

#7787: VESA regression: 640x480 no longer works at boot (but works in Screen
prefs!)
----------------------------------+-----------------------
   Reporter:  ttcoder             |      Owner:  axeld
       Type:  bug                 |     Status:  new
   Priority:  normal              |  Milestone:  R1
  Component:  System/Boot Loader  |    Version:  R1/alpha3
 Resolution:                      |   Keywords:
 Blocked By:                      |   Blocking:
Has a Patch:  0                   |   Platform:  x86
----------------------------------+-----------------------

Comment (by ttcoder):

 Ok so I'm back on track, and working on a first 'real' patch. It will
 address only a tiny parcel of my problem(s), but I believe it's a big
 enough one that it has to be attacked with a cartesian strategy :-]..

 Here's the first problem I want to address:

 - [http://dev.haiku-
 os.org/browser/haiku/trunk/src/system/boot/platform/bios_ia32/video.cpp#L226
 video.cpp:226] is a return statement.

 It should not. Forcing the DETAILED_MONITOR_DESC on the user is deprecated
 in Haiku, as per other discussions in this ticket and others. Such code
 flow is present also in the nVidia driver, which will also need a patch
 later on, so that it calls create_display_modes() (that one being a nice
 guy who uses all 3 EDID sections) but let's focus on the bootloader for
 now, and this ticket here.

 Instead I propose that...

 - [http://dev.haiku-
 os.org/browser/haiku/trunk/src/system/boot/platform/bios_ia32/video.cpp#L212
 find_edid_mode()] applies the "prefer higher resolutions" policy to both
 DETAILED_MONITOR_DESC and STD_TIMING, merged somehow.

 Later on I'll propose a patch to also look in the third VESA_Desc section
 (which is the one that has 1024x768 for me.. Yummy!) but again, let's
 attack these problems one at a time :-)

 Feedback? Shall I start writing this (small) change ?

-- 
Ticket URL: <http://dev.haiku-os.org/ticket/7787#comment:10>
Haiku <http://dev.haiku-os.org>
Haiku - the operating system.

Other related posts: