Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

filter-brightness-als: support droid4 ALS #8

Merged
merged 1 commit into from
Apr 6, 2020
Merged

Conversation

MerlijnWajer
Copy link
Member

Supporting dimming the display backlight brightness, the LEDs and the
keyboard backlight based upon the Ambient Light Sensor in the Droid 4.

The values in the profiles are very much work in progress, and you
currently need a very strong light to be able to see this display when
the brightness level is not set to a high value.

Probably the mapping needs to be inverted, and I need to do further
testing based upon ambient light values.

@MerlijnWajer
Copy link
Member Author

Looks like the Droid 4 ALS just works. This PR adds support for it, but note that:

  • The values are just guestimated (guess-estimated) and likely wrong.
  • This is not a "generic" way to do it, in the sense that it won't work out of the box on other devices, but it's a start, right?

@MerlijnWajer
Copy link
Member Author

(So ... don't merge as it)

@MerlijnWajer MerlijnWajer changed the title WIP: filter-brightness-als: support droid4 filter-brightness-als: support droid4 ALS Apr 5, 2020
@MerlijnWajer
Copy link
Member Author

Updated PR, now works quite well, at least in direct sunlight.

Will test in a dark room tonight, but I think it should be ready for merging.

Perhaps we can also attempt a generic way to do this, if/when a max ALS value is reported in sysfs, and we know the scale.

But the sysfs interfaces are not generic/standardised, so that doesn't help...

@MerlijnWajer MerlijnWajer force-pushed the droid4-als branch 2 times, most recently from 7a99032 to c1998e2 Compare April 5, 2020 17:17
@MerlijnWajer
Copy link
Member Author

Wondering if we should use in_intensity_raw or in_illuminance_input. Not sure if they represent the same sensor.

@MerlijnWajer MerlijnWajer force-pushed the droid4-als branch 4 times, most recently from 12ce25d to 06fd83c Compare April 5, 2020 18:58
Supporting dimming the display backlight brightness, the LEDs and the
keyboard backlight based upon the Ambient Light Sensor in the Droid 4.

The values in the profiles are work in progress, I have yet to test
very dark rooms.
@MerlijnWajer MerlijnWajer merged commit 816a8d8 into master Apr 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant