[PATCH] drm/radeon: Extended DDC Probing for Toshiba L300D Radeon Mobility X1100 HDMI-A Connector

[Alex Deucher]

On Wed, Aug 17, 2011 at 5:03 AM, Thomas Reim <reimth at googlemail.com> wrote:
>   Toshiba Satellite L300D with ATI Mobility Radeon X1100 sends data
>   to i2c bus for a HDMI connector that is not implemented/existent
>   on the notebook's board.

If the laptop doesn't actually have the port, it would be better to
just add a connector table quirk to just remove it.  However, the HDMI
port may be on a docking station.

Alex

>
>   Fix by applying extented DDC probing for this connector.
>
>   Requires [PATCH] drm/radeon: Extended DDC Probing for Connectors
>   with Improperly Wired DDC Lines
>
>   Tested for kernel 2.6.38 on Toshiba Satellite L300D notebook
>
>   BugLink: http://bugs.launchpad.net/bugs/826677
>
> Signed-off-by: Thomas Reim <reimth at gmail.com>
> Acked-by: Chris Routh <routhy at gmail.com>
> Cc: <stable at kernel.org>
> ---
>  drivers/gpu/drm/radeon/radeon_connectors.c |   10 ++++++++++
>  1 files changed, 10 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_connectors.c b/drivers/gpu/drm/radeon/radeon_connectors.c
> index 6d6b5f1..ef5a6f0 100644
> --- a/drivers/gpu/drm/radeon/radeon_connectors.c
> +++ b/drivers/gpu/drm/radeon/radeon_connectors.c
> @@ -464,6 +464,16 @@ static bool radeon_connector_needs_extended_probe(struct radeon_device *dev,
>                    (supported_device == ATOM_DEVICE_DFP2_SUPPORT))
>                        return true;
>        }
> +       /* TOSHIBA Satellite L300D with ATI Mobility Radeon x1100
> +        * (RS690M) sends data to i2c bus for a HDMI connector that
> +        * is not implemented */
> +       if ((dev->pdev->device == 0x791f) &&
> +           (dev->pdev->subsystem_vendor == 0x1179) &&
> +           (dev->pdev->subsystem_device == 0xff68)) {
> +               if ((connector_type == DRM_MODE_CONNECTOR_HDMIA) &&
> +                   (supported_device == ATOM_DEVICE_DFP2_SUPPORT))
> +                       return true;
> +       }
>
>        /* Default: no EDID header probe required for DDC probing */
>        return false;
> --
> 1.7.1
>
>