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

2011-07-06 Thread Alex Deucher
On Wed, Jul 6, 2011 at 11:39 AM, Alex Deucher wrote: > On Wed, Jul 6, 2011 at 6:09 AM,   wrote: >> From: Thomas Reim >> >>    Some integrated ATI Radeon chipset implementations >>    with add-on HDMI card (e. g. Asus M2A-VM HDMI) indicate the availability >>    of a DDC even when the add-on card

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

2011-07-06 Thread Alex Deucher
On Wed, Jul 6, 2011 at 6:09 AM, wrote: > From: Thomas Reim > >    Some integrated ATI Radeon chipset implementations >    with add-on HDMI card (e. g. Asus M2A-VM HDMI) indicate the availability >    of a DDC even when the add-on card is not plugged in or HDMI is disabled >    in BIOS setup. In

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

2011-07-06 Thread Thomas Reim
Here are the kernel logs with and without the patch for better visualisation of the changes: 1. dmesg after applying the patch: [0.00] Initializing cgroup subsys cpuset [0.00] Initializing cgroup subsys cpu [0.00] Linux version 3.0-3-generic (root@Mark-Aurel) (gcc version

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

2011-07-06 Thread Thomas Reim
Dear Alex, sorry for the long delay. I have updated the patch according to the comments in your last e-mail (see below). I've added function radeon_connector_needs_extended_probe() and flag radeon_connector->requires_extended_probe that provide the means to force on a board,connector basis to che

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

2011-06-27 Thread Jean Delvare
On Fri, 24 Jun 2011 09:36:56 -0400, Alex Deucher wrote: > On Fri, Jun 24, 2011 at 12:02 AM, Thomas Reim wrote: > >> > diff --git a/drivers/gpu/drm/radeon/radeon_i2c.c > >> > b/drivers/gpu/drm/radeon/radeon_i2c.c > >> > index 781196d..7e93cf9 100644 > >> > --- a/drivers/gpu/drm/radeon/radeon_i2c.c

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

2011-06-24 Thread Alex Deucher
On Fri, Jun 24, 2011 at 12:02 AM, Thomas Reim wrote: > Hi Alex, > > thank you for your feedback. > >> Thinking about it more, it might be cleaner to just make the extended >> mode a flag, e.g., >> dret = radeon_ddc_probe_extended(radeon_connector, >> radeon_connector->requires_extended_probe); >>

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

2011-06-23 Thread Thomas Reim
Hi Alex, thank you for your feedback. > Thinking about it more, it might be cleaner to just make the extended > mode a flag, e.g., > dret = radeon_ddc_probe_extended(radeon_connector, > radeon_connector->requires_extended_probe); > and handle the extended fetch in the same probe function. > OK.

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

2011-06-23 Thread Alex Deucher
On Thu, Jun 23, 2011 at 6:05 PM, wrote: > From: Thomas Reim > > Some integrated ATI Radeon chipset implementations > (e. g. Asus M2A-VM HDMI, RS690) indicate the availability > of a DDC even when there's no monitor connected. > In this case, drm_get_edid() and drm_edid_block_valid() > periodical