In the function oaktrail_lvds_mode_set, I don't think that the following code makes any sense:
/* Find the connector we're trying to set up */ list_for_each_entry(connector, &mode_config->connector_list, head) { if (!connector->encoder || connector->encoder->crtc != crtc) continue; }
if (!connector) { DRM_ERROR("Couldn't find connector when setting mode"); return; }
drm_connector_property_get_value( connector, dev->mode_config.scaling_mode_property, &v);
The initial loop is a no-op, because it always continues. The test !connector can never be true, because at the end of a list_for_each_entry connector points to the list head, and calling drm_connector_property_get_value on the list head probably does not make sense.
julia