[Bug 60659] Backlight control interface with nouveau driver for nVidia Quadro 1000m not works

[bugzilla-daemon at bugzilla.kernel.org]

https://bugzilla.kernel.org/show_bug.cgi?id=60659

Aaron Lu <aaron.lu at intel.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |aaron.lu at intel.com

--- Comment #8 from Aaron Lu <aaron.lu at intel.com> ---
(In reply to Ilia Mirkin from comment #7)
> In the larger sense of "are you having a problem", yes there is a bug
> somewhere.
> 
> But IMO it's not in the nouveau driver. Perhaps discuss this with the person
> on the other bug who suggested there was an issue in nouveau... I'm not sure

Oh, it's me who suggests there was an issue in nouveau since I don't know
nouveau has a logic to check if ACPI video will support backlight control on
this system and then do things accordingly.

#ifdef CONFIG_ACPI
        if (acpi_video_backlight_support()) {
                NV_INFO(drm, "ACPI backlight interface available, "
                             "not registering our own\n");
                return 0;
        }
#endif

I'm not sure if this is correct, I think nouveau can always create backlight
interface for user to use no matter if other module do that or not. Just my
personal opinion.

-- 
You are receiving this mail because:
You are watching the assignee of the bug.