FWTS should have a test to check that ACPI backlight interface works correctly

Bug #1250429 reported by Brendan Donegan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Firmware Test Suite
Fix Released
High
Colin Ian King

Bug Description

We've hit on a system during testing that seems to have a broken ACPI interface for the system backlight. I believe it's important this interface works, since it will be used by gnome-settings-daemon where it exists - due to logic in that code that dictates where multiple backlight interfaces are available, the one of type 'firmware' (as shown in /sys/class/backlight/<backlight>/type) is chosen. This is typically 'acpi_video0'. On the system in question, using the slider in gnome-control-center will update '/sys/class/backlight/acpi_video0/brightness', but this is not reflected in 'actual_brightness' and more importantly not reflected in the backlight hardware.

I think there should be a new test which checks that, if the system has a firmware interface to the backlight then updating 'brightness' will reflect in 'actual_brightness'.

Changed in fwts:
status: New → In Progress
importance: Undecided → High
assignee: nobody → Colin King (colin-king)
Revision history for this message
Colin Ian King (colin-king) wrote :

We have a fwts "brightness" test, but this is an interactive test because it also exercises the brightness control keys as well inspecting the brightness levels settings. It's broken, but I've just got a fix to it (LP: #1250517).

I guess you want this as a non-interactive test?

Revision history for this message
Brendan Donegan (brendan-donegan) wrote :

Yeah I think it would be good to separate the two so we could have this in an automated test suite.

Revision history for this message
Colin Ian King (colin-king) wrote :

OK - I will figure out some way to do this w/o breaking older functionality

Revision history for this message
Colin Ian King (colin-king) wrote :

I'll break up the original brightness tests into 2 parts:

original tests 1 and 2 go into a new autobrightness tests and remove from old brightness test

original tests 3, 4 and 5 keep in the old brightness test

Revision history for this message
Colin Ian King (colin-king) wrote :
Keng-Yu Lin (lexical)
Changed in fwts:
milestone: none → 13.11.00
Revision history for this message
Colin Ian King (colin-king) wrote :

Fix committed, commit 41c4b07d2177ee250ce01d9b41a6c99bfe060e4a

Changed in fwts:
status: In Progress → Fix Committed
Changed in fwts:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.