Re: [PATCH] drm/i915: no LVDS on Intel SandyBridge SDVs

[Dave Airlie <airlied@gmail.com>]

On Tue, Dec 21, 2010 at 6:05 AM, Jesse Barnes <jbarnes@virtuousgeek.org> wrote:
> Add a DMI match entry for the SDV to indicate it has no LVDS present.

Sounds wrong, generally the SDVs can get an LVDS panel plugged into
them, so they have an LVDS connector.

I'd worry that this would break something upstream that isn't an SDV.

Dave.

>
> Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
>
> diff --git a/drivers/gpu/drm/i915/intel_lvds.c b/drivers/gpu/drm/i915/intel_lvds.c
> index aa23070..2f383f0 100644
> --- a/drivers/gpu/drm/i915/intel_lvds.c
> +++ b/drivers/gpu/drm/i915/intel_lvds.c
> @@ -719,6 +719,14 @@ static const struct dmi_system_id intel_no_lvds[] = {
>                        DMI_MATCH(DMI_PRODUCT_NAME, "U800"),
>                },
>        },
> +       {
> +               .callback = intel_no_lvds_dmi_callback,
> +               .ident = "Intel SandyBridge",
> +               .matches = {
> +                       DMI_MATCH(DMI_SYS_VENDOR, "Intel Corporation"),
> +                       DMI_MATCH(DMI_PRODUCT_NAME, "SandyBridge Platform"),
> +               },
> +       },
>
>        { }     /* terminating entry */
>  };
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
>