[BUG] als device not working properly #111

hazemcg · 2020-01-12T04:40:25Z

Clight version the issue has been seen with:

4.0

Used distribution:

Arch Linux

Describe the bug

Auto back light only works with webcam and not als device.
When als device is not working, sometimes the screen will turn black and will be unable to turn back to normal without putting the computer to sleep and waking the computer. Moving the mouse does not do anything.

Expected behavior
Normal use of als device with auto backlight working properly.

To Reproduce
Comment/uncomment sensor_devname=video0

clight.log

FedeDP · 2020-01-12T10:16:47Z

Hi! Thanks for opening this issue.
I don't quite understand: is ALS capture working for you?

clight.log

I can see from your log that a clogged capture is always detected, thus screen backlight is not changed. Your shutter threshold is

Shutter threshold: 0.10

Are you in a very dark room?

hazemcg · 2020-01-12T11:07:14Z

I don't quite understand: is ALS capture working for you?

ALS capture does not work. It only works with the webcam.

I can see from your log that a clogged capture is always detected, thus screen backlight is not changed. Your shutter threshold is

Shutter threshold: 0.10

Clogged capture only shows when I try to use ALS capture.

Are you in a very dark room?

I was using the laptop with the lights on.

FedeDP · 2020-01-12T11:20:30Z

Ok!

ALS capture does not work. It only works with the webcam.

CAn you share output of:
busctl call org.clightd.clightd /org/clightd/clightd/Sensor/Als org.clightd.clightd.Sensor IsAvailable "s" ""
and
ls -ahl /sys/bus/iio/devices
?

Btw Clightd is undergoing a full rework for its als device support: FedeDP/Clightd#34.

It should be working fine using clightd-git version; care to test?

hazemcg · 2020-01-12T11:58:27Z

busctl call org.clightd.clightd /org/clightd/clightd/Sensor/Als org.clightd.clightd.Sensor IsAvailable "s" ""

sb "/dev/iio:device0" true

ls -ahl /sys/bus/iio/devices

total 0 drwxr-xr-x 2 root root 0 Jan 12 06:47 . drwxr-xr-x 4 root root 0 Jan 12 06:47 .. lrwxrwxrwx 1 root root 0 Jan 12 06:47 iio:device0 -> ../../../devices/LNXSYSTM:00/LNXSYBUS:00/ACPI0008:00/iio:device0

It should be working fine using clightd-git version; care to test?

I have switched to the clightd-git version and it still seems like the backlight does not change.

FedeDP · 2020-01-12T14:29:27Z

I have switched to the clightd-git version and it still seems like the backlight does not change.

Did you reboot after?

What about udevadm info -a /sys/bus/iio/devices/iio:device0 output?

hazemcg · 2020-01-12T16:40:09Z

Did you reboot after?

Yeah I tried to reboot before checking to see if it works but no luck.

What about udevadm info -a /sys/bus/iio/devices/iio:device0 output?

`
Udevadm info starts with the device specified by the devpath and then
walks up the chain of parent devices. It prints for every device
found, all possible attributes in the udev rules key format.
A rule to match, can be composed by the attributes of the device
and the attributes from one single parent device.

looking at device '/devices/LNXSYSTM:00/LNXSYBUS:00/ACPI0008:00/iio:device0':
KERNEL=="iio:device0"
SUBSYSTEM=="iio"
DRIVER==""
ATTR{in_illuminance_raw}=="400"
ATTR{name}=="acpi-als"
ATTR{in_illuminance_input}=="400"

looking at parent device '/devices/LNXSYSTM:00/LNXSYBUS:00/ACPI0008:00':
KERNELS=="ACPI0008:00"
SUBSYSTEMS=="acpi"
DRIVERS=="acpi_als"
ATTRS{hid}=="ACPI0008"
ATTRS{status}=="11"
ATTRS{path}=="_SB_.ALS_"

looking at parent device '/devices/LNXSYSTM:00/LNXSYBUS:00':
KERNELS=="LNXSYBUS:00"
SUBSYSTEMS=="acpi"
DRIVERS==""
ATTRS{hid}=="LNXSYBUS"
ATTRS{path}=="_SB_"

looking at parent device '/devices/LNXSYSTM:00':
KERNELS=="LNXSYSTM:00"
SUBSYSTEMS=="acpi"
DRIVERS==""
ATTRS{hid}=="LNXSYSTM"
ATTRS{path}==""

`

FedeDP · 2020-01-12T16:42:52Z

Ok, it seems fine.
So, what does busctl call org.clightd.clightd /org/clightd/clightd/Sensor/Als org.clightd.clightd.Sensor Capture "sis" "" 5 "" returns?

hazemcg · 2020-01-12T16:48:53Z

sis" "" 5 "" sad "/dev/iio:device0" 5 0.8 0.8 0.8 0.8 0.8

FedeDP · 2020-01-12T17:05:57Z

That's great; it means clightd is actually working fine.
Can you share a new clight log with new clightd?

hazemcg · 2020-01-12T17:20:36Z

clight.log

FedeDP · 2020-01-12T17:35:21Z

Isn't it working fine now?

(I)[12:15:50]{backlight.c:247} Ambient brightness: 0.800 (-0.058 screen compensation) -> Backlight pct: 0.906.

It seems like your Sensor is always reporting 0.8 (ie: 400 lux) value. In fact it reported the same in your above udev output:

ATTR{in_illuminance_input}=="400"

hazemcg · 2020-01-12T17:41:08Z

It seems like your Sensor is always reporting 0.8 (ie: 400 lux) value. In fact it reported the same in your above udev output:

ATTR{in_illuminance_input}=="400"

The lux value doesn't seem to change even when I run clight in a dark room.

FedeDP · 2020-01-12T17:43:02Z

Can you run udevadm info -a /sys/bus/iio/devices/iio:device0 in 2 very different conditions and report output here?
Btw it seems like your sensor is reporting wrong values, ie:

ATTR{in_illuminance_input}=="400"

does not depend on clight; it is a sensor driver reading exposed by udev.

hazemcg · 2020-01-12T17:51:48Z

With large amounts of light:
looking at device '/devices/LNXSYSTM:00/LNXSYBUS:00/ACPI0008:00/iio:device0': KERNEL=="iio:device0" SUBSYSTEM=="iio" DRIVER=="" ATTR{name}=="acpi-als" ATTR{in_illuminance_input}=="400" ATTR{in_illuminance_raw}=="400"

With low amount of light:

looking at device '/devices/LNXSYSTM:00/LNXSYBUS:00/ACPI0008:00/iio:device0': KERNEL=="iio:device0" SUBSYSTEM=="iio" DRIVER=="" ATTR{in_illuminance_input}=="400" ATTR{in_illuminance_raw}=="400" ATTR{name}=="acpi-als"

It outputs the same thing in both conditions.

FedeDP · 2020-01-12T17:58:46Z

Unfortunately it seems like a bug in your ALS driver, or worse your ALS hardware is misbehaving :(

hazemcg · 2020-01-13T10:49:25Z

I looked into it and it seems that the als device on my laptop is disabled by default. I just needed to make an ACPI call to enable the sensor _SB.PCI1.LPCB.EC0.ALSC and now it seems to work properly.

FedeDP · 2020-01-13T11:07:20Z

Wow that's great, thanks! We can close this then :)
Added this information to Clightd wiki page: https://github.com/FedeDP/Clightd/wiki/Sensors#als-devices.

hazemcg · 2020-01-13T15:41:04Z

Also to keep the als device enabled, I had to make a systemd service launch a script that does the acpi call at boot.

FedeDP · 2020-01-13T16:26:25Z

Mind to share the service? I will add it to Clightd wiki page! Thank you!

hazemcg · 2020-01-13T19:23:59Z

Service:

als-enable.service

[Unit]
Description=Enables ALS device
[Service]
ExecStart=pathtoscript
[Install]
WantedBy=multi-user.target

Script:

als-enable

echo \\_SB.PCI0.LPCB.EC0.ALSC 0x1 > /proc/acpi/call

hazemcg changed the title ~~[BUG]~~ [BUG] als device not working properly Jan 12, 2020

FedeDP closed this as completed Jan 14, 2020

guillaumeboehm mentioned this issue Aug 14, 2022

ALS not working FedeDP/Clightd#88

Closed

This issue was closed.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[BUG] als device not working properly #111

[BUG] als device not working properly #111

hazemcg commented Jan 12, 2020

FedeDP commented Jan 12, 2020

hazemcg commented Jan 12, 2020

FedeDP commented Jan 12, 2020 •

edited

Loading

hazemcg commented Jan 12, 2020 •

edited

Loading

FedeDP commented Jan 12, 2020

hazemcg commented Jan 12, 2020 •

edited

Loading

FedeDP commented Jan 12, 2020

hazemcg commented Jan 12, 2020

FedeDP commented Jan 12, 2020

hazemcg commented Jan 12, 2020

FedeDP commented Jan 12, 2020

hazemcg commented Jan 12, 2020

FedeDP commented Jan 12, 2020

hazemcg commented Jan 12, 2020 •

edited

Loading

FedeDP commented Jan 12, 2020

hazemcg commented Jan 13, 2020

FedeDP commented Jan 13, 2020

hazemcg commented Jan 13, 2020

FedeDP commented Jan 13, 2020

hazemcg commented Jan 13, 2020 •

edited

Loading

[BUG] als device not working properly #111

[BUG] als device not working properly #111

Comments

hazemcg commented Jan 12, 2020

FedeDP commented Jan 12, 2020

hazemcg commented Jan 12, 2020

FedeDP commented Jan 12, 2020 • edited Loading

hazemcg commented Jan 12, 2020 • edited Loading

FedeDP commented Jan 12, 2020

hazemcg commented Jan 12, 2020 • edited Loading

FedeDP commented Jan 12, 2020

hazemcg commented Jan 12, 2020

FedeDP commented Jan 12, 2020

hazemcg commented Jan 12, 2020

FedeDP commented Jan 12, 2020

hazemcg commented Jan 12, 2020

FedeDP commented Jan 12, 2020

hazemcg commented Jan 12, 2020 • edited Loading

FedeDP commented Jan 12, 2020

hazemcg commented Jan 13, 2020

FedeDP commented Jan 13, 2020

hazemcg commented Jan 13, 2020

FedeDP commented Jan 13, 2020

hazemcg commented Jan 13, 2020 • edited Loading

FedeDP commented Jan 12, 2020 •

edited

Loading

hazemcg commented Jan 12, 2020 •

edited

Loading

hazemcg commented Jan 12, 2020 •

edited

Loading

hazemcg commented Jan 12, 2020 •

edited

Loading

hazemcg commented Jan 13, 2020 •

edited

Loading