Re: drm-radeon-testing commit 8131660130d3dc2629a150d47e6a04c8a1a6244b breaks LVDS on my PowerBook

2010-02-10 Thread Alex Deucher
2010/2/10 Michel Dänzer :
> On Wed, 2010-01-27 at 11:52 -0500, Alex Deucher wrote:
>> 2010/1/27 Michel Dänzer :
>> > On Wed, 2010-01-27 at 10:30 -0500, Alex Deucher wrote:
>> >> 2010/1/27 Michel Dänzer :
>> >> >
>> >> > your commit 8131660130d3dc2629a150d47e6a04c8a1a6244b ('drm/radeon/kms:
>> >> > add support for hw i2c on r1xx-r5xx') breaks LVDS on my PowerBook. It
>> >> > looks kinda like blooming, though I can actually make out a higly
>> >> > distorted part of what it's supposed to display, and there's a
>> >> > panel/backlight blinking around every second or so.
>> >> >
>> >>
>> >> Hmm, maybe the mac's edid is wired up such that hw i2c doesn't work or
>> >> the prescale calculation for your system needs adjusting.  Switching
>> >> back to sw i2c should fix it in the meantime.  Set the hw_capable bool
>> >> in the radeon_i2c_bus_rec struct to false.
>> >
>> > Presumably that would work, as the previous commit ('drm/radeon/kms: add
>> > radeon i2c algo') works? [0]
>> >
>> >> For the prescale:
>> >> u32 prescale = (((rdev->clock.default_sclk * 10)/(4 * 128 * 100) + 1)
>> >> << 8) + 128;
>> >> does rdev->clock.default_sclk get initialized properly on the mac?  If
>> >> not, you can use radeon_get_engine_clock to read back the value from
>> >> the regs.
>> >
>> > Looks like it in /sys/kernel/debug/dri/0/radeon_pm_info:
>> >
>> > state: PM_STATE_DISABLED
>> > default engine clock: 445500 kHz
>> > current engine clock: 445500 kHz
>> > default memory clock: 240750 kHz
>> > current memory clock: 240750 kHz
>> >
>>
>> you could also try the prescale algo from r520:
>> i2c_clock = 50;
>> prescale = (127 << 8) + ((rdev->clock.default_sclk * 10) / (4 * 127 *
>> i2c_clock));
>
> Sorry I didn't have time to try your suggestions. However, I'm happy to
> report that this problem is fixed in current drm-radeon-testing.

excellent.  I think it was the prescale calculation:
e6346ab00d9138f0303d79b3638bbc143266186e

Alex

--
SOLARIS 10 is the OS for Data Centers - provides features such as DTrace,
Predictive Self Healing and Award Winning ZFS. Get Solaris 10 NOW
http://p.sf.net/sfu/solaris-dev2dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: drm-radeon-testing commit 8131660130d3dc2629a150d47e6a04c8a1a6244b breaks LVDS on my PowerBook

2010-02-10 Thread Michel Dänzer
On Wed, 2010-01-27 at 11:52 -0500, Alex Deucher wrote: 
> 2010/1/27 Michel Dänzer :
> > On Wed, 2010-01-27 at 10:30 -0500, Alex Deucher wrote:
> >> 2010/1/27 Michel Dänzer :
> >> >
> >> > your commit 8131660130d3dc2629a150d47e6a04c8a1a6244b ('drm/radeon/kms:
> >> > add support for hw i2c on r1xx-r5xx') breaks LVDS on my PowerBook. It
> >> > looks kinda like blooming, though I can actually make out a higly
> >> > distorted part of what it's supposed to display, and there's a
> >> > panel/backlight blinking around every second or so.
> >> >
> >>
> >> Hmm, maybe the mac's edid is wired up such that hw i2c doesn't work or
> >> the prescale calculation for your system needs adjusting.  Switching
> >> back to sw i2c should fix it in the meantime.  Set the hw_capable bool
> >> in the radeon_i2c_bus_rec struct to false.
> >
> > Presumably that would work, as the previous commit ('drm/radeon/kms: add
> > radeon i2c algo') works? [0]
> >
> >> For the prescale:
> >> u32 prescale = (((rdev->clock.default_sclk * 10)/(4 * 128 * 100) + 1)
> >> << 8) + 128;
> >> does rdev->clock.default_sclk get initialized properly on the mac?  If
> >> not, you can use radeon_get_engine_clock to read back the value from
> >> the regs.
> >
> > Looks like it in /sys/kernel/debug/dri/0/radeon_pm_info:
> >
> > state: PM_STATE_DISABLED
> > default engine clock: 445500 kHz
> > current engine clock: 445500 kHz
> > default memory clock: 240750 kHz
> > current memory clock: 240750 kHz
> >
> 
> you could also try the prescale algo from r520:
> i2c_clock = 50;
> prescale = (127 << 8) + ((rdev->clock.default_sclk * 10) / (4 * 127 *
> i2c_clock));

Sorry I didn't have time to try your suggestions. However, I'm happy to
report that this problem is fixed in current drm-radeon-testing.


-- 
Earthling Michel Dänzer   |http://www.vmware.com
Libre software enthusiast |  Debian, X and DRI developer














--
SOLARIS 10 is the OS for Data Centers - provides features such as DTrace,
Predictive Self Healing and Award Winning ZFS. Get Solaris 10 NOW
http://p.sf.net/sfu/solaris-dev2dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: drm-radeon-testing commit 8131660130d3dc2629a150d47e6a04c8a1a6244b breaks LVDS on my PowerBook

2010-01-27 Thread Alex Deucher
2010/1/27 Michel Dänzer :
> On Wed, 2010-01-27 at 10:30 -0500, Alex Deucher wrote:
>> 2010/1/27 Michel Dänzer :
>> >
>> > your commit 8131660130d3dc2629a150d47e6a04c8a1a6244b ('drm/radeon/kms:
>> > add support for hw i2c on r1xx-r5xx') breaks LVDS on my PowerBook. It
>> > looks kinda like blooming, though I can actually make out a higly
>> > distorted part of what it's supposed to display, and there's a
>> > panel/backlight blinking around every second or so.
>> >
>>
>> Hmm, maybe the mac's edid is wired up such that hw i2c doesn't work or
>> the prescale calculation for your system needs adjusting.  Switching
>> back to sw i2c should fix it in the meantime.  Set the hw_capable bool
>> in the radeon_i2c_bus_rec struct to false.
>
> Presumably that would work, as the previous commit ('drm/radeon/kms: add
> radeon i2c algo') works? [0]
>

Out of curiosity, does hw i2c work on the DVI port?  That would rule
out the prescale.

Alex

>> For the prescale:
>> u32 prescale = (((rdev->clock.default_sclk * 10)/(4 * 128 * 100) + 1)
>> << 8) + 128;
>> does rdev->clock.default_sclk get initialized properly on the mac?  If
>> not, you can use radeon_get_engine_clock to read back the value from
>> the regs.
>
> Looks like it in /sys/kernel/debug/dri/0/radeon_pm_info:
>
> state: PM_STATE_DISABLED
> default engine clock: 445500 kHz
> current engine clock: 445500 kHz
> default memory clock: 240750 kHz
> current memory clock: 240750 kHz
>
>
> [0] Though actually I notice there's a warning in dmesg with that as
> well:
>
> [    0.355965] [drm]     TV1: INTERNAL_DAC2
> [    0.356037]
> [    0.356040] =
> [    0.356072] [ INFO: possible recursive locking detected ]
> [    0.356096] 2.6.32.6 #434
> [    0.356112] -
> [    0.356136] swapper/1 is trying to acquire lock:
> [    0.356158]  (&adap->bus_lock){+.+...}, at: [] 
> i2c_transfer+0x84/0x178
> [    0.356219]
> [    0.356221] but task is already holding lock:
> [    0.356249]  (&adap->bus_lock){+.+...}, at: [] 
> i2c_transfer+0x84/0x178
> [    0.356294]
> [    0.356296] other info that might help us debug this:
> [    0.356328] 1 lock held by swapper/1:
> [    0.356346]  #0:  (&adap->bus_lock){+.+...}, at: [] 
> i2c_transfer+0x84/0x178
> [    0.356396]
> [    0.356397] stack backtrace:
> [    0.356422] Call Trace:
> [    0.356447] [ef871b50] [c00094fc] show_stack+0x7c/0x194 (unreliable)
> [    0.356485] [ef871b90] [c007ebe4] __lock_acquire+0x15c0/0x16a8
> [    0.356515] [ef871c20] [c007edb8] lock_acquire+0xec/0x128
> [    0.356549] [ef871c60] [c040bfa4] mutex_lock_nested+0x5c/0x2c8
> [    0.356580] [ef871cc0] [c0355910] i2c_transfer+0x84/0x178
> [    0.356612] [ef871cf0] [c02aa56c] radeon_i2c_xfer+0x44/0x80
> [    0.356643] [ef871d10] [c0355938] i2c_transfer+0xac/0x178
> [    0.356679] [ef871d40] [c026267c] drm_do_probe_ddc_edid+0x50/0x70
> [    0.356710] [ef871d70] [c02626e0] drm_ddc_read_edid+0x44/0xd8
> [    0.356741] [ef871d90] [c026285c] drm_get_edid+0xe8/0x1a0
> [    0.356770] [ef871db0] [c02a7b10] radeon_modeset_init+0x690/0x794
> [    0.356802] [ef871e10] [c028a964] radeon_driver_load_kms+0x150/0x1b0
> [    0.356841] [ef871e30] [c0257c8c] drm_get_dev+0x36c/0x45c
> [    0.356875] [ef871e60] [c0412180] radeon_pci_probe+0x1c/0x2c
> [    0.356912] [ef871e70] [c020c93c] local_pci_probe+0x24/0x34
> [    0.356943] [ef871e80] [c020dc78] pci_device_probe+0x70/0xb0
> [    0.356977] [ef871eb0] [c02f3e24] driver_probe_device+0x13c/0x2a8
> [    0.357008] [ef871ed0] [c02f4008] __driver_attach+0x78/0xb4
> [    0.357038] [ef871ef0] [c02f3218] bus_for_each_dev+0x68/0xc0
> [    0.357067] [ef871f20] [c02f3ad4] driver_attach+0x24/0x34
> [    0.357097] [ef871f30] [c02f2918] bus_add_driver+0x15c/0x324
> [    0.357127] [ef871f50] [c02f442c] driver_register+0xcc/0x184
> [    0.357158] [ef871f70] [c020df78] __pci_register_driver+0x74/0x108
> [    0.357190] [ef871f90] [c0252024] drm_init+0x74/0x108
> [    0.357224] [ef871fb0] [c05c3794] radeon_init+0xa4/0xb8
> [    0.357254] [ef871fc0] [c0003eec] do_one_initcall+0x70/0x210
> [    0.357289] [ef871fe0] [c05a21dc] kernel_init+0xbc/0x130
> [    0.357318] [ef871ff0] [c0014120] kernel_thread+0x4c/0x68
> [    0.651929] [drm] Determined LVDS native mode details from EDID
>
>
> --
> Earthling Michel Dänzer           |                http://www.vmware.com
> Libre software enthusiast         |          Debian, X and DRI developer
>

--
The Planet: dedicated and managed hosting, cloud storage, colocation
Stay online with enterprise data centers and the best network in the business
Choose flexible plans and management services without long-term contracts
Personal 24x7 support from experience hosting pros just a phone call away.
http://p.sf.net/sfu/theplanet-com
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/

Re: drm-radeon-testing commit 8131660130d3dc2629a150d47e6a04c8a1a6244b breaks LVDS on my PowerBook

2010-01-27 Thread Alex Deucher
2010/1/27 Michel Dänzer :
> On Wed, 2010-01-27 at 10:30 -0500, Alex Deucher wrote:
>> 2010/1/27 Michel Dänzer :
>> >
>> > your commit 8131660130d3dc2629a150d47e6a04c8a1a6244b ('drm/radeon/kms:
>> > add support for hw i2c on r1xx-r5xx') breaks LVDS on my PowerBook. It
>> > looks kinda like blooming, though I can actually make out a higly
>> > distorted part of what it's supposed to display, and there's a
>> > panel/backlight blinking around every second or so.
>> >
>>
>> Hmm, maybe the mac's edid is wired up such that hw i2c doesn't work or
>> the prescale calculation for your system needs adjusting.  Switching
>> back to sw i2c should fix it in the meantime.  Set the hw_capable bool
>> in the radeon_i2c_bus_rec struct to false.
>
> Presumably that would work, as the previous commit ('drm/radeon/kms: add
> radeon i2c algo') works? [0]
>
>> For the prescale:
>> u32 prescale = (((rdev->clock.default_sclk * 10)/(4 * 128 * 100) + 1)
>> << 8) + 128;
>> does rdev->clock.default_sclk get initialized properly on the mac?  If
>> not, you can use radeon_get_engine_clock to read back the value from
>> the regs.
>
> Looks like it in /sys/kernel/debug/dri/0/radeon_pm_info:
>
> state: PM_STATE_DISABLED
> default engine clock: 445500 kHz
> current engine clock: 445500 kHz
> default memory clock: 240750 kHz
> current memory clock: 240750 kHz
>

you could also try the prescale algo from r520:
i2c_clock = 50;
prescale = (127 << 8) + ((rdev->clock.default_sclk * 10) / (4 * 127 *
i2c_clock));

>
> [0] Though actually I notice there's a warning in dmesg with that as
> well:

hmmm, I'll take a look.

>
> [    0.355965] [drm]     TV1: INTERNAL_DAC2
> [    0.356037]
> [    0.356040] =
> [    0.356072] [ INFO: possible recursive locking detected ]
> [    0.356096] 2.6.32.6 #434
> [    0.356112] -
> [    0.356136] swapper/1 is trying to acquire lock:
> [    0.356158]  (&adap->bus_lock){+.+...}, at: [] 
> i2c_transfer+0x84/0x178
> [    0.356219]
> [    0.356221] but task is already holding lock:
> [    0.356249]  (&adap->bus_lock){+.+...}, at: [] 
> i2c_transfer+0x84/0x178
> [    0.356294]
> [    0.356296] other info that might help us debug this:
> [    0.356328] 1 lock held by swapper/1:
> [    0.356346]  #0:  (&adap->bus_lock){+.+...}, at: [] 
> i2c_transfer+0x84/0x178
> [    0.356396]
> [    0.356397] stack backtrace:
> [    0.356422] Call Trace:
> [    0.356447] [ef871b50] [c00094fc] show_stack+0x7c/0x194 (unreliable)
> [    0.356485] [ef871b90] [c007ebe4] __lock_acquire+0x15c0/0x16a8
> [    0.356515] [ef871c20] [c007edb8] lock_acquire+0xec/0x128
> [    0.356549] [ef871c60] [c040bfa4] mutex_lock_nested+0x5c/0x2c8
> [    0.356580] [ef871cc0] [c0355910] i2c_transfer+0x84/0x178
> [    0.356612] [ef871cf0] [c02aa56c] radeon_i2c_xfer+0x44/0x80
> [    0.356643] [ef871d10] [c0355938] i2c_transfer+0xac/0x178
> [    0.356679] [ef871d40] [c026267c] drm_do_probe_ddc_edid+0x50/0x70
> [    0.356710] [ef871d70] [c02626e0] drm_ddc_read_edid+0x44/0xd8
> [    0.356741] [ef871d90] [c026285c] drm_get_edid+0xe8/0x1a0
> [    0.356770] [ef871db0] [c02a7b10] radeon_modeset_init+0x690/0x794
> [    0.356802] [ef871e10] [c028a964] radeon_driver_load_kms+0x150/0x1b0
> [    0.356841] [ef871e30] [c0257c8c] drm_get_dev+0x36c/0x45c
> [    0.356875] [ef871e60] [c0412180] radeon_pci_probe+0x1c/0x2c
> [    0.356912] [ef871e70] [c020c93c] local_pci_probe+0x24/0x34
> [    0.356943] [ef871e80] [c020dc78] pci_device_probe+0x70/0xb0
> [    0.356977] [ef871eb0] [c02f3e24] driver_probe_device+0x13c/0x2a8
> [    0.357008] [ef871ed0] [c02f4008] __driver_attach+0x78/0xb4
> [    0.357038] [ef871ef0] [c02f3218] bus_for_each_dev+0x68/0xc0
> [    0.357067] [ef871f20] [c02f3ad4] driver_attach+0x24/0x34
> [    0.357097] [ef871f30] [c02f2918] bus_add_driver+0x15c/0x324
> [    0.357127] [ef871f50] [c02f442c] driver_register+0xcc/0x184
> [    0.357158] [ef871f70] [c020df78] __pci_register_driver+0x74/0x108
> [    0.357190] [ef871f90] [c0252024] drm_init+0x74/0x108
> [    0.357224] [ef871fb0] [c05c3794] radeon_init+0xa4/0xb8
> [    0.357254] [ef871fc0] [c0003eec] do_one_initcall+0x70/0x210
> [    0.357289] [ef871fe0] [c05a21dc] kernel_init+0xbc/0x130
> [    0.357318] [ef871ff0] [c0014120] kernel_thread+0x4c/0x68
> [    0.651929] [drm] Determined LVDS native mode details from EDID
>
>
> --
> Earthling Michel Dänzer           |                http://www.vmware.com
> Libre software enthusiast         |          Debian, X and DRI developer
>

--
The Planet: dedicated and managed hosting, cloud storage, colocation
Stay online with enterprise data centers and the best network in the business
Choose flexible plans and management services without long-term contracts
Personal 24x7 support from experience hosting pros just a phone call away.
http://p.sf.net/sfu/theplanet-com
--
___
Dri-d

Re: drm-radeon-testing commit 8131660130d3dc2629a150d47e6a04c8a1a6244b breaks LVDS on my PowerBook

2010-01-27 Thread Michel Dänzer
On Wed, 2010-01-27 at 10:30 -0500, Alex Deucher wrote: 
> 2010/1/27 Michel Dänzer :
> >
> > your commit 8131660130d3dc2629a150d47e6a04c8a1a6244b ('drm/radeon/kms:
> > add support for hw i2c on r1xx-r5xx') breaks LVDS on my PowerBook. It
> > looks kinda like blooming, though I can actually make out a higly
> > distorted part of what it's supposed to display, and there's a
> > panel/backlight blinking around every second or so.
> >
> 
> Hmm, maybe the mac's edid is wired up such that hw i2c doesn't work or
> the prescale calculation for your system needs adjusting.  Switching
> back to sw i2c should fix it in the meantime.  Set the hw_capable bool
> in the radeon_i2c_bus_rec struct to false.

Presumably that would work, as the previous commit ('drm/radeon/kms: add
radeon i2c algo') works? [0]

> For the prescale:
> u32 prescale = (((rdev->clock.default_sclk * 10)/(4 * 128 * 100) + 1)
> << 8) + 128;
> does rdev->clock.default_sclk get initialized properly on the mac?  If
> not, you can use radeon_get_engine_clock to read back the value from
> the regs.

Looks like it in /sys/kernel/debug/dri/0/radeon_pm_info:

state: PM_STATE_DISABLED
default engine clock: 445500 kHz
current engine clock: 445500 kHz
default memory clock: 240750 kHz
current memory clock: 240750 kHz


[0] Though actually I notice there's a warning in dmesg with that as
well:

[0.355965] [drm] TV1: INTERNAL_DAC2
[0.356037]
[0.356040] =
[0.356072] [ INFO: possible recursive locking detected ]
[0.356096] 2.6.32.6 #434
[0.356112] -
[0.356136] swapper/1 is trying to acquire lock:
[0.356158]  (&adap->bus_lock){+.+...}, at: [] 
i2c_transfer+0x84/0x178
[0.356219]
[0.356221] but task is already holding lock:
[0.356249]  (&adap->bus_lock){+.+...}, at: [] 
i2c_transfer+0x84/0x178
[0.356294]
[0.356296] other info that might help us debug this:
[0.356328] 1 lock held by swapper/1:
[0.356346]  #0:  (&adap->bus_lock){+.+...}, at: [] 
i2c_transfer+0x84/0x178
[0.356396]
[0.356397] stack backtrace:
[0.356422] Call Trace:
[0.356447] [ef871b50] [c00094fc] show_stack+0x7c/0x194 (unreliable)
[0.356485] [ef871b90] [c007ebe4] __lock_acquire+0x15c0/0x16a8
[0.356515] [ef871c20] [c007edb8] lock_acquire+0xec/0x128
[0.356549] [ef871c60] [c040bfa4] mutex_lock_nested+0x5c/0x2c8
[0.356580] [ef871cc0] [c0355910] i2c_transfer+0x84/0x178
[0.356612] [ef871cf0] [c02aa56c] radeon_i2c_xfer+0x44/0x80
[0.356643] [ef871d10] [c0355938] i2c_transfer+0xac/0x178
[0.356679] [ef871d40] [c026267c] drm_do_probe_ddc_edid+0x50/0x70
[0.356710] [ef871d70] [c02626e0] drm_ddc_read_edid+0x44/0xd8
[0.356741] [ef871d90] [c026285c] drm_get_edid+0xe8/0x1a0
[0.356770] [ef871db0] [c02a7b10] radeon_modeset_init+0x690/0x794
[0.356802] [ef871e10] [c028a964] radeon_driver_load_kms+0x150/0x1b0
[0.356841] [ef871e30] [c0257c8c] drm_get_dev+0x36c/0x45c
[0.356875] [ef871e60] [c0412180] radeon_pci_probe+0x1c/0x2c
[0.356912] [ef871e70] [c020c93c] local_pci_probe+0x24/0x34
[0.356943] [ef871e80] [c020dc78] pci_device_probe+0x70/0xb0
[0.356977] [ef871eb0] [c02f3e24] driver_probe_device+0x13c/0x2a8
[0.357008] [ef871ed0] [c02f4008] __driver_attach+0x78/0xb4
[0.357038] [ef871ef0] [c02f3218] bus_for_each_dev+0x68/0xc0
[0.357067] [ef871f20] [c02f3ad4] driver_attach+0x24/0x34
[0.357097] [ef871f30] [c02f2918] bus_add_driver+0x15c/0x324
[0.357127] [ef871f50] [c02f442c] driver_register+0xcc/0x184
[0.357158] [ef871f70] [c020df78] __pci_register_driver+0x74/0x108
[0.357190] [ef871f90] [c0252024] drm_init+0x74/0x108
[0.357224] [ef871fb0] [c05c3794] radeon_init+0xa4/0xb8
[0.357254] [ef871fc0] [c0003eec] do_one_initcall+0x70/0x210
[0.357289] [ef871fe0] [c05a21dc] kernel_init+0xbc/0x130
[0.357318] [ef871ff0] [c0014120] kernel_thread+0x4c/0x68
[0.651929] [drm] Determined LVDS native mode details from EDID


-- 
Earthling Michel Dänzer   |http://www.vmware.com
Libre software enthusiast |  Debian, X and DRI developer

--
The Planet: dedicated and managed hosting, cloud storage, colocation
Stay online with enterprise data centers and the best network in the business
Choose flexible plans and management services without long-term contracts
Personal 24x7 support from experience hosting pros just a phone call away.
http://p.sf.net/sfu/theplanet-com
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: drm-radeon-testing commit 8131660130d3dc2629a150d47e6a04c8a1a6244b breaks LVDS on my PowerBook

2010-01-27 Thread Alex Deucher
2010/1/27 Michel Dänzer :
>
> Hi Alex,
>
>
> your commit 8131660130d3dc2629a150d47e6a04c8a1a6244b ('drm/radeon/kms:
> add support for hw i2c on r1xx-r5xx') breaks LVDS on my PowerBook. It
> looks kinda like blooming, though I can actually make out a higly
> distorted part of what it's supposed to display, and there's a
> panel/backlight blinking around every second or so.
>

Hmm, maybe the mac's edid is wired up such that hw i2c doesn't work or
the prescale calculation for your system needs adjusting.  Switching
back to sw i2c should fix it in the meantime.  Set the hw_capable bool
in the radeon_i2c_bus_rec struct to false.  For the prescale:
u32 prescale = (((rdev->clock.default_sclk * 10)/(4 * 128 * 100) + 1)
<< 8) + 128;
does rdev->clock.default_sclk get initialized properly on the mac?  If
not, you can use radeon_get_engine_clock to read back the value from
the regs.

Alex

> These errors from dmesg might be relevant (nothing's connected to the
> DVI connector):
>
> [    0.355445] [drm] Radeon Display Connectors
> [    0.355473] [drm] Connector 0:
> [    0.355490] [drm]   LVDS
> [    0.355509] [drm]   DDC: 0x64 0x64 0x64 0x64 0x64 0x64 0x64 0x64
> [    0.355533] [drm]   Encoders:
> [    0.30] [drm]     LCD1: INTERNAL_LVDS
> [    0.355570] [drm] Connector 1:
> [    0.355587] [drm]   DVI-I
> [    0.355602] [drm]   HPD2
> [    0.355620] [drm]   DDC: 0x60 0x60 0x60 0x60 0x60 0x60 0x60 0x60
> [    0.355645] [drm]   Encoders:
> [    0.355661] [drm]     DFP2: INTERNAL_DVO1
> [    0.355681] [drm]     CRT1: INTERNAL_DAC1
> [    0.355700] [drm] Connector 2:
> [    0.355716] [drm]   S-video
> [    0.355732] [drm]   Encoders:
> [    0.355748] [drm]     TV1: INTERNAL_DAC2
> [    0.591623] [drm:edid_is_valid] *ERROR* Raw EDID:
> [    0.591652] <3>a1 ff bf bf fd ff ff ff ff ff ff ff fd ff ff ff  
> 
> [    0.591661] <3>ff ff ff ff ff ef a1 ff ff ff a1 7f ff ff a1 ff  
> 
> [    0.591669] <3>fb ff a1 ff ff ff a1 ff ff ff a1 fe ff ff a1 ff  
> 
> [    0.591678] <3>f7 ff ff ff ff ff ff ff ff bf bf ff ff ff ff fd  
> 
> [    0.591686] <3>ff ff ff ff ef ff ff ff ff a1 ff ff ff a1 ff fb  
> 
> [    0.591695] <3>ff a1 ff ff ff a1 ff ff ff a1 fe ff ff a1 ff ff  
> 
> [    0.591704] <3>ff ff ff f7 ff ff ff ff bf bf ff ff ff ff fd ff  
> 
> [    0.591726] <3>ff ff ff ef ff ff ff ef a1 ff ff ff a1 7f fb ff  
> 
> [    0.591733]
> [    0.634535] [drm:edid_is_valid] *ERROR* Raw EDID:
> [    0.634559] <3>a1 ff ff ff fd ff ff ff fd ff ff ff ff ef a1 ff  
> 
> [    0.634568] <3>ff ff a1 7f ff ff a1 ff fb ff a1 ff ff ff a1 ff  
> 
> [    0.634576] <3>ff df a1 fe ff ff a1 ff ff ff ff ff 08 ff ff ff  
> 
> [    0.634585] <3>ff bf bf ff ff 00 00 04 10 4e 4e 9c 00 00 00 00  
> .NN.
> [    0.634594] <3>10 a1 0e 0f 01 a1 83 80 20 a1 16 7c 0a a1 87 f5   
> ..|
> [    0.634602] <3>94 a1 57 4f 8c a1 26 27 50 a1 54 00 00 00 00 01  
> ..WO..&'P.T.
> [    0.634611] <3>01 01 01 01 01 01 01 01 01 01 01 01 01 01 01 01  
> 
> [    0.634620] <3>a1 01 01 01 a1 b8 28 a0 a1 40 51 c0 a1 10 30 40  
> ..(@q...0@
> [    0.634627]
> [    0.676987] [drm:edid_is_valid] *ERROR* Raw EDID:
> [    0.677010] <3>a1 89 ff a1 ff ff ff a1 7f fb ff a1 ff ff ff a1  
> 
> [    0.677019] <3>ff ff ff a1 fe ff ff a1 ff ff ff ff ff ff ff ff  
> 
> [    0.677028] <3>ff ff bf bf ff ff ff ff fd ff ff ff ff ef ff ff  
> 
> [    0.677036] <3>ff ff a1 7f ff ff a1 7f fb ff a1 ff ff ff a1 ff  
> 
> [    0.677045] <3>ff df a1 fe ff ff a1 ff ff ff ff ff f7 ff ff ff  
> 
> [    0.677054] <3>ff bf bf ff ff ff ff fd ff ff ff ff ff ff ff ff  
> 
> [    0.677062] <3>ef a1 ff ff ff a1 7f ff ff a1 ff fb ff a1 ff ff  
> 
> [    0.677071] <3>ff a1 ff ff ff a1 fe ff ff a1 ff f7 ff ff ff ff  
> 
> [    0.677078]
> [    0.719430] [drm:edid_is_valid] *ERROR* Raw EDID:
> [    0.719453] <3>a1 ff df a1 ff ff df a1 fe ff ff a1 ff ff ff ff  
> 
> [    0.719462] <3>ff ff ff ff ff ff bf bf ff ff ff ff fd ff ff ff  
> 
> [    0.719471] <3>ff ef ff ff ff ff a1 ff ff ff a1 ff fb ff a1 ff  
> 
> [    0.719479] <3>ff ff a1 ff ff ff a1 fe ff ff a1 ff ff ff ff ff  
> 
> [    0.719488] <3>f7 ff ff ff ff bf bf ff ff ff ff fd ff ff ff ff  
> 
> [    0.719497] <3>ef ff ff ff ff a1 ff ff ff a1 7f fb ff a1 ff ff  
> 
> [    0.719505] <3>ff a1 ff ff df a1 ff ff ff a1 fe ff ff a1 ff f7  
> 
> [    0.719514] <3>ff ff ff ff ff ff ff ff bf bf fd ff ff ff fd ff  
> 
> [    0.719521]
> [    0.761872] [drm:edid_is_valid] *ERROR* Raw EDID:
> [    0.761895] <3>a1 ff ff bf bf ff ff ff ff fd ff ff ff ff ef ff  
> 
> [    0.761904] <3>f

drm-radeon-testing commit 8131660130d3dc2629a150d47e6a04c8a1a6244b breaks LVDS on my PowerBook

2010-01-27 Thread Michel Dänzer

Hi Alex,


your commit 8131660130d3dc2629a150d47e6a04c8a1a6244b ('drm/radeon/kms:
add support for hw i2c on r1xx-r5xx') breaks LVDS on my PowerBook. It
looks kinda like blooming, though I can actually make out a higly
distorted part of what it's supposed to display, and there's a
panel/backlight blinking around every second or so.

These errors from dmesg might be relevant (nothing's connected to the
DVI connector):

[0.355445] [drm] Radeon Display Connectors
[0.355473] [drm] Connector 0:
[0.355490] [drm]   LVDS
[0.355509] [drm]   DDC: 0x64 0x64 0x64 0x64 0x64 0x64 0x64 0x64
[0.355533] [drm]   Encoders:
[0.30] [drm] LCD1: INTERNAL_LVDS
[0.355570] [drm] Connector 1:
[0.355587] [drm]   DVI-I
[0.355602] [drm]   HPD2
[0.355620] [drm]   DDC: 0x60 0x60 0x60 0x60 0x60 0x60 0x60 0x60
[0.355645] [drm]   Encoders:
[0.355661] [drm] DFP2: INTERNAL_DVO1
[0.355681] [drm] CRT1: INTERNAL_DAC1
[0.355700] [drm] Connector 2:
[0.355716] [drm]   S-video
[0.355732] [drm]   Encoders:
[0.355748] [drm] TV1: INTERNAL_DAC2
[0.591623] [drm:edid_is_valid] *ERROR* Raw EDID:
[0.591652] <3>a1 ff bf bf fd ff ff ff ff ff ff ff fd ff ff ff  

[0.591661] <3>ff ff ff ff ff ef a1 ff ff ff a1 7f ff ff a1 ff  

[0.591669] <3>fb ff a1 ff ff ff a1 ff ff ff a1 fe ff ff a1 ff  

[0.591678] <3>f7 ff ff ff ff ff ff ff ff bf bf ff ff ff ff fd  

[0.591686] <3>ff ff ff ff ef ff ff ff ff a1 ff ff ff a1 ff fb  

[0.591695] <3>ff a1 ff ff ff a1 ff ff ff a1 fe ff ff a1 ff ff  

[0.591704] <3>ff ff ff f7 ff ff ff ff bf bf ff ff ff ff fd ff  

[0.591726] <3>ff ff ff ef ff ff ff ef a1 ff ff ff a1 7f fb ff  

[0.591733] 
[0.634535] [drm:edid_is_valid] *ERROR* Raw EDID:
[0.634559] <3>a1 ff ff ff fd ff ff ff fd ff ff ff ff ef a1 ff  

[0.634568] <3>ff ff a1 7f ff ff a1 ff fb ff a1 ff ff ff a1 ff  

[0.634576] <3>ff df a1 fe ff ff a1 ff ff ff ff ff 08 ff ff ff  

[0.634585] <3>ff bf bf ff ff 00 00 04 10 4e 4e 9c 00 00 00 00  
.NN.
[0.634594] <3>10 a1 0e 0f 01 a1 83 80 20 a1 16 7c 0a a1 87 f5   
..|
[0.634602] <3>94 a1 57 4f 8c a1 26 27 50 a1 54 00 00 00 00 01  
..WO..&'P.T.
[0.634611] <3>01 01 01 01 01 01 01 01 01 01 01 01 01 01 01 01  

[0.634620] <3>a1 01 01 01 a1 b8 28 a0 a1 40 51 c0 a1 10 30 40  
..(@q...0@
[0.634627] 
[0.676987] [drm:edid_is_valid] *ERROR* Raw EDID:
[0.677010] <3>a1 89 ff a1 ff ff ff a1 7f fb ff a1 ff ff ff a1  

[0.677019] <3>ff ff ff a1 fe ff ff a1 ff ff ff ff ff ff ff ff  

[0.677028] <3>ff ff bf bf ff ff ff ff fd ff ff ff ff ef ff ff  

[0.677036] <3>ff ff a1 7f ff ff a1 7f fb ff a1 ff ff ff a1 ff  

[0.677045] <3>ff df a1 fe ff ff a1 ff ff ff ff ff f7 ff ff ff  

[0.677054] <3>ff bf bf ff ff ff ff fd ff ff ff ff ff ff ff ff  

[0.677062] <3>ef a1 ff ff ff a1 7f ff ff a1 ff fb ff a1 ff ff  

[0.677071] <3>ff a1 ff ff ff a1 fe ff ff a1 ff f7 ff ff ff ff  

[0.677078] 
[0.719430] [drm:edid_is_valid] *ERROR* Raw EDID:
[0.719453] <3>a1 ff df a1 ff ff df a1 fe ff ff a1 ff ff ff ff  

[0.719462] <3>ff ff ff ff ff ff bf bf ff ff ff ff fd ff ff ff  

[0.719471] <3>ff ef ff ff ff ff a1 ff ff ff a1 ff fb ff a1 ff  

[0.719479] <3>ff ff a1 ff ff ff a1 fe ff ff a1 ff ff ff ff ff  

[0.719488] <3>f7 ff ff ff ff bf bf ff ff ff ff fd ff ff ff ff  

[0.719497] <3>ef ff ff ff ff a1 ff ff ff a1 7f fb ff a1 ff ff  

[0.719505] <3>ff a1 ff ff df a1 ff ff ff a1 fe ff ff a1 ff f7  

[0.719514] <3>ff ff ff ff ff ff ff ff bf bf fd ff ff ff fd ff  

[0.719521] 
[0.761872] [drm:edid_is_valid] *ERROR* Raw EDID:
[0.761895] <3>a1 ff ff bf bf ff ff ff ff fd ff ff ff ff ef ff  

[0.761904] <3>ff ff ff a1 ff ff ff a1 ff ff ff a1 ff ff ff a1  

[0.761912] <3>ff ff df a1 fe ff ff a1 ff ff ff ff ff f7 ff ff  

[0.761921] <3>ff ff bf bf ff ff ff ff fd ff ff ff ff ef ff ff  

[0.761930] <3>ff ff a1 ff ff ff a1 7f ff ff a1 ff ff ff a1 ff  

[0.761938] <3>ff df a1 ff ff ff a1 fe ff ff a1 ff f7 ff ff ff  

[0.761947] <3>ff bf bf ff ff bf bf fd ff ff ff ff ff ff ff ff  

[0.761956] <3>ef a1 ff ff ff a1 7f ff ff a1 ff fb ff a1 ff ff  

[0.761963] 
[0.761977] radeon :00:10.0: LVDS-1: EDID invalid.
[0.762960] [drm:radeon_dvi_detect] *ERROR* DVI-I-1: probed a monitor but 
no|invalid EDID