From mboxrd@z Thu Jan  1 00:00:00 1970
From: Thomas Reim <thomas.reim@nepomuc.de>
Subject: Re: [PATCH 1/1] drm/radeon: Fix Asus M2A-VM HDMI EDID error
 flooding problem
Date: Tue, 21 Jun 2011 20:03:03 +0200
Message-ID: <1308679383.5981.7.camel@Mark-Aurel.gas.de>
References: <1308670305-7096-1-git-send-email-rdratlos@yahoo.co.uk>
	 <BANLkTi=cna6rLmwLCcPZsR-AMZKp4HVPDg@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Return-path: <linux-kernel-owner@vger.kernel.org>
In-Reply-To: <BANLkTi=cna6rLmwLCcPZsR-AMZKp4HVPDg@mail.gmail.com>
Sender: linux-kernel-owner@vger.kernel.org
To: Alex Deucher <alexdeucher@gmail.com>
Cc: Thomas Reim <reimth@googlemail.com>, Dave Airlie <airlied@redhat.com>, Mario Kleiner <mario.kleiner@tuebingen.mpg.de>, Jean Delvare <khali@linux-fr.org>, Tyson Whitehead <twhitehead@gmail.com>, Jason Wessel <jason.wessel@windriver.com>, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Thomas Reim <rdratlos@yahoo.co.uk>
List-Id: dri-devel@lists.freedesktop.org

Dear Alex, 

yes, the proposed fix should also fix the 'drm/radeon: workaround a hw
bug on some radeon chipsets with all-0 EDIDs.' issue. The trick is, that
we check within the Radeon domain directly on i2c interface level, if an
EDID can be retrieved at all, before we hand over to the main drm edid
functions. 

If you can provide some logs from Dave, I can double-check.

Regards,

  Thomas

> On Tue, Jun 21, 2011 at 11:31 AM, Thomas Reim <reimth@googlemail.com> wrote:
> > Some integrated ATI Radeon chipset implementations
> > (e. g. Asus M2A-VM HDMI) indicate the availability
> > of a DDC even when there's no monitor connected.
> > In this case, drm_get_edid and drm_edid_block_valid
> > periodically dump data and kernel errors into system
> > log files and onto terminals, which lead to an unacceptable
> > system behaviour.
> >
> > Tested since kernel 2.35 on Asus M2A-VM HDMI board
> >
> > Signed-off-by: Thomas Reim <rdratlos@yahoo.co.uk>
> 
> Does this patch fix the issue:
> http://git.kernel.org/?p=linux/kernel/git/airlied/drm-2.6.git;a=commitdiff;h=4a9a8b71e12d41abb71c4e741bff524f016cfef4
> 
> Alex
> 
> > ---
> >  drivers/gpu/drm/radeon/radeon_connectors.c |   10 +++++
> >  drivers/gpu/drm/radeon/radeon_display.c    |   11 +++++
> >  drivers/gpu/drm/radeon/radeon_i2c.c        |   60 ++++++++++++++++++++++++++++
> >  drivers/gpu/drm/radeon/radeon_mode.h       |    1 +
> >  4 files changed, 82 insertions(+), 0 deletions(-)
> >