Re: [132/152] drm/i915/bios: Reverse order of 100/120 Mhz SSC clocks

From: François Valenduc
Date: Fri Jan 07 2011 - 15:18:48 EST


Le 06/01/11 01:23, Greg KH a écrit :
> 2.6.36-stable review patch. If anyone has any objections, please let us know.
>
> ------------------
>
> From: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
>
> commit 448f53a1ede54eb854d036abf54573281412d650 upstream.
>
> Fixes the lack of output on the LVDS panel of the Lenovo U160.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=31596
> Reported-and-tested-by: Dirk Gouders <gouders@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
> Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxx>
>
> ---
> drivers/gpu/drm/i915/intel_bios.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- a/drivers/gpu/drm/i915/intel_bios.c
> +++ b/drivers/gpu/drm/i915/intel_bios.c
> @@ -276,7 +276,7 @@ parse_general_features(struct drm_i915_p
> general->ssc_freq ? 66 : 48;
> else if (IS_IRONLAKE(dev_priv->dev) || IS_GEN6(dev))
> dev_priv->lvds_ssc_freq =
> - general->ssc_freq ? 100 : 120;
> + general->ssc_freq ? 120 : 100;
> else
> dev_priv->lvds_ssc_freq =
> general->ssc_freq ? 100 : 96;
>
>

This commit gives a black screen on many computers and has been reverted
between 2.6.37-rc8 and 2.6.37. Why has this been added to this review ?

Thanks,
François Valenduc
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/