Comment 17 for bug 77739

Revision history for this message
yleetiny (yleetiny) wrote :

Sebastian : could you post here the result of the command "lshal".

Laurent : I think that commenting out the exit line, will be necessary for some VAIO model, because the laptop_panel is partially detected as a sony's panel ("linux.sysfs_path"). And thus, it uses it's behaviour ( the "echo" way and then exit)
For example, my lshal laptop_panel part is :

udi = '/org/freedesktop/Hal/devices/computer_backlight'
  info.capabilities = {'laptop_panel'} (string list)
  info.category = 'laptop_panel' (string)
  info.interfaces = {'org.freedesktop.Hal.Device.LaptopPanel'} (string list)
  info.parent = '/org/freedesktop/Hal/devices/computer' (string)
  info.product = 'Generic Backlight Device' (string)
  info.udi = '/org/freedesktop/Hal/devices/computer_backlight' (string)
  laptop_panel.access_method = 'general' (string)
  laptop_panel.num_levels = 8 (0x8) (int)
  linux.hotplug_type = 2 (0x2) (int)
  linux.subsystem = 'backlight' (string)
  linux.sysfs_path = '/sys/class/backlight/sony' (string)
  org.freedesktop.Hal.Device.LaptopPanel.method_argnames = {'brightness_value', ''} (string list)
  org.freedesktop.Hal.Device.LaptopPanel.method_execpaths = {'hal-system-lcd-set-brightness', 'hal-system-lcd-get-brightness'} (string list)
  org.freedesktop.Hal.Device.LaptopPanel.method_names = {'SetBrightness', 'GetBrightness'} (string list)
  org.freedesktop.Hal.Device.LaptopPanel.method_signatures = {'i', ''} (string list)

If we comment or delete the "exits", no harm will be done since you have the same command in the end of the files...(I admit, it's not a clean).
So deleting the "exits" seems to me the best way to just make more things works (because i absolutely don't how to change the detection of the "sysfs" path ).