On 4/12/12 5:35 PM, Adam Jackson wrote: > @@ -1030,6 +1026,10 @@ drm_dmt_modes_for_range(struct drm_connector *connector, struct edid *edid, > > for (i = 0; i< drm_num_dmt_modes; i++) { > if (mode_in_range(drm_dmt_modes + i, edid, timing)) { > + if (mode_is_rb(&drm_dmt_modes[i])&& > + !drm_monitor_supports_rb(edid)) > + continue; > + > newmode = drm_mode_duplicate(dev,&drm_dmt_modes[i]); > if (newmode) { > drm_mode_probed_add(connector, newmode); This chunk is actually unnecessary, upon second reading. mode_in_range() already checks this. New series in a moment that addresses the extra mode list from HP. - ajax