[Bug 29685] New: DPMS does not turn off LCD backlight on NV18 [GeForce4 MX with AGP8X (Mac)]

[Bug 29685] New: DPMS does not turn off LCD backlight on NV18 [GeForce4 MX with AGP8X (Mac)]

[bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ]

https://bugs.freedesktop.org/show_bug.cgi?id=29685

           Summary: DPMS does not turn off LCD backlight on NV18 [GeForce4
                    MX with AGP8X (Mac)]
           Product: xorg
           Version: 7.5
          Platform: PowerPC
        OS/Version: Linux (All)
            Status: NEW
          Severity: normal
          Priority: medium
         Component: Driver/nouveau
        AssignedTo: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
        ReportedBy: rechner-dLI15PoULosb1SvskN2V4Q@public.gmane.org
         QAContact: xorg-team-go0+a7rfsptAfugRpC6u6w@public.gmane.org


I'm using bug #19356, which describes the same problem with a different iMac,
as an orientation for my bug description.

Expected:
$ xset dpms force standby;
should turn off LCD backlight.

Actual Result:
Screen is black, but LCD backlight stays on.

Workaround:
Use nv driver.

Some information on my system is already in bug #29455 (video bios dump,
register dump, "lspci -vvnn" output).

I downloaded, compiled, and ran radeontool, as was described in bug #19356:

nouveau driver:

DPMS on:
0x0060081c      0xa2565051 (-1571401647)
0x000010f0      0x84830535 (-2071788235)
DPMS off:
0x0060081c      0xa2565055 (-1571401643)
0x000010f0      0x84830535 (-2071788235)

nv driver:

DPMS on:
0x0060081c      0xa2565051 (-1571401647)
0x000010f0      0x84550535 (-2074802891)
DPMS off:
0x0060081c      0xa2565050 (-1571401648)
0x000010f0      0x04550535 (72680757)

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 29685] DPMS does not turn off LCD backlight on NV18 [GeForce4 MX with AGP8X (Mac)]

[bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ]

https://bugs.freedesktop.org/show_bug.cgi?id=29685

--- Comment #1 from Francisco Jerez <currojerez-sGOZH3hwPm2sTnJN9+BGXg@public.gmane.org> 2010-08-20 05:35:50 PDT ---
Created an attachment (id=38013)
 View: https://bugs.freedesktop.org/attachment.cgi?id=38013
 Review: https://bugs.freedesktop.org/review?bug=29685&attachment=38013

nouveau_imac_backlight.patch

The attached patch should fix it, can you give it a try?

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 29685] DPMS does not turn off LCD backlight on NV18 [GeForce4 MX with AGP8X (Mac)]

[bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ]

https://bugs.freedesktop.org/show_bug.cgi?id=29685

--- Comment #2 from Vlado Plaga <rechner@vlado-do.de> 2010-08-21 04:59:32 PDT ---
(In reply to comment #1)

> The attached patch should fix it, can you give it a try?

Yes! Great! Thank you! Now for me nouveau is superior to nv in every aspect...
except for "installation", maybe. ;-)

First I tried the patch with Ubuntu 10.10's kernel sources (2.6.35), and it was
applied without complaint, but the compilation failed with

drivers/gpu/drm/nouveau/nv04_dfp.c: In function ‘nv04_dfp_update_backlight’:
drivers/gpu/drm/nouveau/nv04_dfp.c:454: error: implicit declaration of function
‘nv_mask’
make[5]: *** [drivers/gpu/drm/nouveau/nv04_dfp.o] Fehler 1
make[4]: *** [drivers/gpu/drm/nouveau] Fehler 2
make[3]: *** [drivers/gpu/drm] Fehler 2
make[2]: *** [drivers/gpu] Fehler 2
make[1]: *** [drivers] Fehler 2
make[1]: Verlasse Verzeichnis '/usr/src/linux-source-2.6.35'
make: *** [debian/stamp/build/kernel] Fehler 2

Then I tried the official 2.6.36-rc1, and the backlight can now be switched off
properly. Unfortunately the EDID error mentioned in bug #29455 is also back
again.

Will that bug be fixed in 2.6.36-rc2? Can I just replace the "gpu/drm/nouveau"
directory? Or should I take the whole linux tree from nouveau's git again?

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau

[Bug 29685] DPMS does not turn off LCD backlight on NV18 [GeForce4 MX with AGP8X (Mac)]

[bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ]

https://bugs.freedesktop.org/show_bug.cgi?id=29685

Vlado Plaga <rechner-dLI15PoULosb1SvskN2V4Q@public.gmane.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

[Bug 29685] DPMS does not turn off LCD backlight on NV18 [GeForce4 MX with AGP8X (Mac)]

[bugzilla-daemon-CC+yJ3UmIYqDUpFQwHEjaQ]

https://bugs.freedesktop.org/show_bug.cgi?id=29685

--- Comment #3 from Francisco Jerez <currojerez@riseup.net> 2010-08-21 14:45:01 PDT ---
(In reply to comment #2)
> (In reply to comment #1)
> 
> > The attached patch should fix it, can you give it a try?
> 
> Yes! Great! Thank you! Now for me nouveau is superior to nv in every aspect...
> except for "installation", maybe. ;-)
> 
Ok, pushed.

> First I tried the patch with Ubuntu 10.10's kernel sources (2.6.35), and it was
> applied without complaint, but the compilation failed with
> 
> drivers/gpu/drm/nouveau/nv04_dfp.c: In function ‘nv04_dfp_update_backlight’:
> drivers/gpu/drm/nouveau/nv04_dfp.c:454: error: implicit declaration of function
> ‘nv_mask’
> make[5]: *** [drivers/gpu/drm/nouveau/nv04_dfp.o] Fehler 1
> make[4]: *** [drivers/gpu/drm/nouveau] Fehler 2
> make[3]: *** [drivers/gpu/drm] Fehler 2
> make[2]: *** [drivers/gpu] Fehler 2
> make[1]: *** [drivers] Fehler 2
> make[1]: Verlasse Verzeichnis '/usr/src/linux-source-2.6.35'
> make: *** [debian/stamp/build/kernel] Fehler 2
> 
49eed80ad0cfa5d5d02992e8088463cbf1a5ff85 is what you're missing.

> Then I tried the official 2.6.36-rc1, and the backlight can now be switched off
> properly. Unfortunately the EDID error mentioned in bug #29455 is also back
> again.
> 
Try cherry-picking c80da92edec995a39afe2a8c2ede2fe2e2b30fc0 from the nouveau
kernel tree.

> Will that bug be fixed in 2.6.36-rc2?

Not sure if it will be fixed for RC2 but it will probably be for one of the
next RCs.

> Can I just replace the "gpu/drm/nouveau"
> directory? Or should I take the whole linux tree from nouveau's git again?

That works sometimes, if the internal APIs haven't been broken between the
kernel versions you're trying to mix, but no guarantees.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Nouveau mailing list
Nouveau@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nouveau