On Fri, Oct 16, 2026 at 03:40:13PM +0100, Hugo Sales wrote: > On Fri, Oct 16, 2026 at 04:57:30PM +0300, Ville Syrjälä wrote: > > Connectors aren't probed before something actually probes them. > > Typically that would be fbdev. > > I think I may have misspoken before. fbdev's first probe already uses > the device's limits: __drm_fb_helper_initial_config_and_unlock() passes > max_width and max_height. Only drm_fb_helper_hotplug_event() passes the > framebuffer size, so the list starts with every mode and loses the > larger ones on the first hotplug. This series makes the hotplug probe > match the first one
Right, forgot about the initial probe. And I think we definitely want the initial probe to use the max size because we don't want to keep the BIOS framebuffer if it's tiny. A long time ago we kept the BIOS fb no matter what, but that made the console pretty much unusable (due to being too small) on some machines. > > Are you suggesting a different approach? Nah, I guess what you're doing should be fine. -- Ville Syrjälä Intel
