On Wed, 14 May 2014, Knut Petersen <Knut_Petersen@xxxxxxxxxxx> wrote:
> On 13.05.2014 22:24, Jesse Barnes wrote:
>> On Tue, 13 May 2014 16:50:12 +0100
>> Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> wrote:
>>>> diff --git a/drivers/gpu/drm/i915/intel_fbdev.c b/drivers/gpu/drm/i915/intel_fbdev.c
>>>> index 1d3f0a9cce31..05beec4badd5 100644
>>>> --- a/drivers/gpu/drm/i915/intel_fbdev.c
>>>> +++ b/drivers/gpu/drm/i915/intel_fbdev.c
>>>> @@ -397,6 +397,14 @@ static bool intel_fb_initial_config(struct drm_fb_helper *fb_helper,
>>>> height);
>>>> }
>>>>
>>>> + /* No preferred mode marked by the EDID? Are there any modes? */
>>>> + if (!modes[i] && !list_empty(&connector->modes)) {
>>>> + DRM_DEBUG_KMS("using first mode listed on connector %s\n",
>>>> + drm_get_connector_name(connector));
>>>> + modes[i] = list_first_entry(struct drm_display_mode,
>>>> + &connector->modes, head);
>>> Please imagine that I wrote this correctly.
>> Imagining you wrote it correctly:
>> Reviewed-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx>
>
> After proper imagination Tested-by: Knut Petersen <Knut_Petersen@xxxxxxxxxxx>.
Pushed the imagined version to -fixes, thanks for the patch, review,
testing, and imagination.
BR,
Jani.
--
Jani Nikula, Intel Open Source Technology Center
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html