2012-05-16 18:18:41

by Tim Gardner

[permalink] [raw]
Subject: 3.4-rc7 backlight regression on Dell XPS M1710

commit b99da31ed8521eb78d5d6930f3128f8ecdb75fae causes the backlight in
my Dell XPS M1710 to stop working. Symptoms are dim display and won't
respond to key brightness events. I bisected and confirmed that
reverting this single patch on top of 3.4-rc7 fixes the problem.

01:00.0 VGA compatible controller [0300]: nVidia Corporation G71
[GeForce Go 7900 GS] [10de:0298] (rev a1)
Subsystem: Dell Device [1028:019b]
Control: I/O+ Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr-
Stepping- SERR- FastB2B- DisINTx-
Status: Cap+ 66MHz- UDF- FastB2B- ParErr- DEVSEL=fast >TAbort- <TAbort-
<MAbort- >SERR- <PERR- INTx-
Latency: 0, Cache Line Size: 64 bytes
Interrupt: pin A routed to IRQ 16
Region 0: Memory at ed000000 (32-bit, non-prefetchable) [size=16M]
Region 1: Memory at d0000000 (64-bit, prefetchable) [size=256M]
Region 3: Memory at ee000000 (64-bit, non-prefetchable) [size=16M]
Region 5: I/O ports at ef00 [size=128]
Expansion ROM at efe00000 [disabled] [size=128K]
Capabilities: <access denied>
Kernel driver in use: nouveau
Kernel modules: nouveau, nvidiafb

rtg
--
Tim Gardner [email protected]


Attachments:
config (140.95 kB)

2012-05-17 00:25:13

by Ben Skeggs

[permalink] [raw]
Subject: Re: 3.4-rc7 backlight regression on Dell XPS M1710

Am Mittwoch, den 16.05.2012, 12:17 -0600 schrieb Tim Gardner:
> commit b99da31ed8521eb78d5d6930f3128f8ecdb75fae causes the backlight in
> my Dell XPS M1710 to stop working. Symptoms are dim display and won't
> respond to key brightness events. I bisected and confirmed that
> reverting this single patch on top of 3.4-rc7 fixes the problem.
That commit is correct, it may have uncovered a bug elsewhere though.

Cheers,
Ben.

>
> 01:00.0 VGA compatible controller [0300]: nVidia Corporation G71
> [GeForce Go 7900 GS] [10de:0298] (rev a1)
> Subsystem: Dell Device [1028:019b]
> Control: I/O+ Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr-
> Stepping- SERR- FastB2B- DisINTx-
> Status: Cap+ 66MHz- UDF- FastB2B- ParErr- DEVSEL=fast >TAbort- <TAbort-
> <MAbort- >SERR- <PERR- INTx-
> Latency: 0, Cache Line Size: 64 bytes
> Interrupt: pin A routed to IRQ 16
> Region 0: Memory at ed000000 (32-bit, non-prefetchable) [size=16M]
> Region 1: Memory at d0000000 (64-bit, prefetchable) [size=256M]
> Region 3: Memory at ee000000 (64-bit, non-prefetchable) [size=16M]
> Region 5: I/O ports at ef00 [size=128]
> Expansion ROM at efe00000 [disabled] [size=128K]
> Capabilities: <access denied>
> Kernel driver in use: nouveau
> Kernel modules: nouveau, nvidiafb
>
> rtg

2012-05-17 06:20:34

by Ben Skeggs

[permalink] [raw]
Subject: Re: 3.4-rc7 backlight regression on Dell XPS M1710

On Thu, May 17, 2012 at 10:30 AM, Ben Skeggs <[email protected]> wrote:
> Am Mittwoch, den 16.05.2012, 12:17 -0600 schrieb Tim Gardner:
>> commit b99da31ed8521eb78d5d6930f3128f8ecdb75fae causes the backlight in
>> my Dell XPS M1710 to stop working. Symptoms are dim display and won't
>> respond to key brightness events. I bisected and confirmed that
>> reverting this single patch on top of 3.4-rc7 fixes the problem.
> That commit is correct, it may have uncovered a bug elsewhere though.
It's been pointed out I should probably be a bit more verbose about
what I'm after here... Apologies :)

The commit that causes the original regression is
a0b25635515ef5049f93b032a1e37f18b16e0f6f. Did your backlight work
correctly before this?

Thanks,
Ben.

>
> Cheers,
> Ben.
>
>>
>> 01:00.0 VGA compatible controller [0300]: nVidia Corporation G71
>> [GeForce Go 7900 GS] [10de:0298] (rev a1)
>> ? ? ? Subsystem: Dell Device [1028:019b]
>> ? ? ? Control: I/O+ Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr-
>> Stepping- SERR- FastB2B- DisINTx-
>> ? ? ? Status: Cap+ 66MHz- UDF- FastB2B- ParErr- DEVSEL=fast >TAbort- <TAbort-
>> <MAbort- >SERR- <PERR- INTx-
>> ? ? ? Latency: 0, Cache Line Size: 64 bytes
>> ? ? ? Interrupt: pin A routed to IRQ 16
>> ? ? ? Region 0: Memory at ed000000 (32-bit, non-prefetchable) [size=16M]
>> ? ? ? Region 1: Memory at d0000000 (64-bit, prefetchable) [size=256M]
>> ? ? ? Region 3: Memory at ee000000 (64-bit, non-prefetchable) [size=16M]
>> ? ? ? Region 5: I/O ports at ef00 [size=128]
>> ? ? ? Expansion ROM at efe00000 [disabled] [size=128K]
>> ? ? ? Capabilities: <access denied>
>> ? ? ? Kernel driver in use: nouveau
>> ? ? ? Kernel modules: nouveau, nvidiafb
>>
>> rtg
>
>
> _______________________________________________
> dri-devel mailing list
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/dri-devel

2012-05-17 14:41:17

by Tim Gardner

[permalink] [raw]
Subject: Re: 3.4-rc7 backlight regression on Dell XPS M1710

On 05/17/2012 12:20 AM, Ben Skeggs wrote:
> On Thu, May 17, 2012 at 10:30 AM, Ben Skeggs<[email protected]> wrote:
>> Am Mittwoch, den 16.05.2012, 12:17 -0600 schrieb Tim Gardner:
>>> commit b99da31ed8521eb78d5d6930f3128f8ecdb75fae causes the backlight in
>>> my Dell XPS M1710 to stop working. Symptoms are dim display and won't
>>> respond to key brightness events. I bisected and confirmed that
>>> reverting this single patch on top of 3.4-rc7 fixes the problem.
>> That commit is correct, it may have uncovered a bug elsewhere though.
> It's been pointed out I should probably be a bit more verbose about
> what I'm after here... Apologies :)
>
> The commit that causes the original regression is
> a0b25635515ef5049f93b032a1e37f18b16e0f6f. Did your backlight work
> correctly before this?
>
> Thanks,
> Ben.
>

I just checked that brightness controls worked with the original 2.6.32
kernel. The original regression commit
a0b25635515ef5049f93b032a1e37f18b16e0f6f went into 3.3, but the
backlight for this laptop has continued to function until commit
b99da31ed8521eb78d5d6930f3128f8ecdb75fae (3.4-rc6).

rtg@m1710:~$ uname -r
2.6.32-41-generic
rtg@m1710:~$ lsmod|egrep ^n
nouveau 515227 2

rtg
--
Tim Gardner [email protected]