diff mbox

drm/radeon: Fix ECS A740GM-M DVI-D EDID error flooding problem

Message ID 1311150845-19198-1-git-send-email-reimth@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Thomas Reim July 20, 2011, 8:34 a.m. UTC
From: Thomas Reim <reimth@gmail.com>

   ECS A740GM-M with ATI RADEON 2100 sends data to i2c bus
   for a DVI connector that is not implemented/existent on the board.

   Fix by applying extented DDC probing for this connector.

   Requires [PATCH] drm/radeon: Fix Asus M2A-VM HDMI EDID error flooding problem

   BugLink: http://bugs.launchpad.net/bugs/810926

Signed-off-by: Thomas Reim <reimth@gmail.com>
---
 drivers/gpu/drm/radeon/radeon_connectors.c |    9 +++++++++
 1 files changed, 9 insertions(+), 0 deletions(-)

Comments

Thomas Reim July 20, 2011, 10:18 p.m. UTC | #1
Von: Stephen Micheals <stephen.micheals@gmail.com>
An: thomas.reim@nepomuc.de
Betreff: Re: [PATCH] drm/radeon: Fix ECS A740GM-M DVI-D EDID error
flooding problem
Datum: Wed, 20 Jul 2011 13:12:26 -0700

Acked-by: Stephen Michaels <Stephen.Micheals@gmail.com>

On Wed, Jul 20, 2011 at 1:34 AM,  <reimth@googlemail.com> wrote:
> From: Thomas Reim <reimth@gmail.com>
>
>   ECS A740GM-M with ATI RADEON 2100 sends data to i2c bus
>   for a DVI connector that is not implemented/existent on the board.
>
>   Fix by applying extented DDC probing for this connector.
>
>   Requires [PATCH] drm/radeon: Fix Asus M2A-VM HDMI EDID error flooding problem
>
>   BugLink: http://bugs.launchpad.net/bugs/810926
>
> Signed-off-by: Thomas Reim <reimth@gmail.com>
> ---
>  drivers/gpu/drm/radeon/radeon_connectors.c |    9 +++++++++
>  1 files changed, 9 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_connectors.c b/drivers/gpu/drm/radeon/radeon_connectors.c
> index 2e70be2..82dacc6 100644
> --- a/drivers/gpu/drm/radeon/radeon_connectors.c
> +++ b/drivers/gpu/drm/radeon/radeon_connectors.c
> @@ -449,6 +449,15 @@ static bool radeon_connector_needs_extended_probe(struct radeon_device *dev,
>                    (supported_device == ATOM_DEVICE_DFP2_SUPPORT))
>                        return true;
>        }
> +       /* ECS A740GM-M with ATI RADEON 2100 sends data to i2c bus
> +        * for a DVI connector that is not implemented */
> +       if ((dev->pdev->device == 0x796e) &&
> +           (dev->pdev->subsystem_vendor == 0x1019) &&
> +           (dev->pdev->subsystem_device == 0x2615)) {
> +               if ((connector_type == DRM_MODE_CONNECTOR_DVID) &&
> +                   (supported_device == ATOM_DEVICE_DFP2_SUPPORT))
> +                       return true;
> +       }
>
>        /* Default: no EDID header probe required for DDC probing */
>        return false;
> --
> 1.7.1
>
>
Alex Deucher July 26, 2011, 1:05 p.m. UTC | #2
On Wed, Jul 20, 2011 at 4:34 AM,  <reimth@googlemail.com> wrote:
> From: Thomas Reim <reimth@gmail.com>
>
>   ECS A740GM-M with ATI RADEON 2100 sends data to i2c bus
>   for a DVI connector that is not implemented/existent on the board.
>
>   Fix by applying extented DDC probing for this connector.
>
>   Requires [PATCH] drm/radeon: Fix Asus M2A-VM HDMI EDID error flooding problem
>
>   BugLink: http://bugs.launchpad.net/bugs/810926
>
> Signed-off-by: Thomas Reim <reimth@gmail.com>

Reviewed-by: Alex Deucher <alexander.deucher@amd.com>

> ---
>  drivers/gpu/drm/radeon/radeon_connectors.c |    9 +++++++++
>  1 files changed, 9 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_connectors.c b/drivers/gpu/drm/radeon/radeon_connectors.c
> index 2e70be2..82dacc6 100644
> --- a/drivers/gpu/drm/radeon/radeon_connectors.c
> +++ b/drivers/gpu/drm/radeon/radeon_connectors.c
> @@ -449,6 +449,15 @@ static bool radeon_connector_needs_extended_probe(struct radeon_device *dev,
>                    (supported_device == ATOM_DEVICE_DFP2_SUPPORT))
>                        return true;
>        }
> +       /* ECS A740GM-M with ATI RADEON 2100 sends data to i2c bus
> +        * for a DVI connector that is not implemented */
> +       if ((dev->pdev->device == 0x796e) &&
> +           (dev->pdev->subsystem_vendor == 0x1019) &&
> +           (dev->pdev->subsystem_device == 0x2615)) {
> +               if ((connector_type == DRM_MODE_CONNECTOR_DVID) &&
> +                   (supported_device == ATOM_DEVICE_DFP2_SUPPORT))
> +                       return true;
> +       }
>
>        /* Default: no EDID header probe required for DDC probing */
>        return false;
> --
> 1.7.1
>
>
diff mbox

Patch

diff --git a/drivers/gpu/drm/radeon/radeon_connectors.c b/drivers/gpu/drm/radeon/radeon_connectors.c
index 2e70be2..82dacc6 100644
--- a/drivers/gpu/drm/radeon/radeon_connectors.c
+++ b/drivers/gpu/drm/radeon/radeon_connectors.c
@@ -449,6 +449,15 @@  static bool radeon_connector_needs_extended_probe(struct radeon_device *dev,
 		    (supported_device == ATOM_DEVICE_DFP2_SUPPORT))
 			return true;
 	}
+	/* ECS A740GM-M with ATI RADEON 2100 sends data to i2c bus
+	 * for a DVI connector that is not implemented */
+	if ((dev->pdev->device == 0x796e) &&
+	    (dev->pdev->subsystem_vendor == 0x1019) &&
+	    (dev->pdev->subsystem_device == 0x2615)) {
+		if ((connector_type == DRM_MODE_CONNECTOR_DVID) &&
+		    (supported_device == ATOM_DEVICE_DFP2_SUPPORT))
+			return true;
+	}
 
 	/* Default: no EDID header probe required for DDC probing */
 	return false;