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

Feature Request/Suggestion: Remember Camera Toggle State or Make it Configurable in Settings #447

Open
ilikenwf opened this issue Mar 6, 2024 · 4 comments

Comments

@ilikenwf
Copy link
Contributor

ilikenwf commented Mar 6, 2024

I rarely if ever use the camera, it would be nice to have the ability to have the EC either use the last used state (enabled/disabled) or allow me to set a default of enabled/disabled in the settings.

I'm using a serw13, however I'm sure it would be welcome in all models with the feature.

@crawfxrd
Copy link
Member

crawfxrd commented Mar 6, 2024

Camera power is always enabled. It's not configurable.

@ilikenwf
Copy link
Contributor Author

ilikenwf commented Mar 6, 2024

Then why does the camera in the serw13 disappear from lsusb when the disable hotkey is used?

@crawfxrd
Copy link
Member

crawfxrd commented Mar 6, 2024

Oh, I didn't even realize that that's what we used that icon for.

Anyways, the only persistent "config" is the custom keyboard layout. Any other settings, like battery charging thresholds, are lost on EC reset.

See: #256

I suppose the first thing to do would to be able to set the default state at build time.

@ilikenwf
Copy link
Contributor Author

ilikenwf commented Mar 6, 2024

My current custom build includes some of the pending pr's for backlight settings on the keyboard, but yes, it's a handy feature, though with the open EC, mostly open UEFI, and in my build the ME fully stripped, there's little to no actual threat at the low level...

Everything threatening is in userspace.

I've been using this for months without issues:

corna/me_cleaner#384

That mention would be better in coreboot I realize, but even so...food for thought. That PR allows both deblobbing the ME somewhat as well as the HAP bit or disable bit or whatever the name of it is now.

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

No branches or pull requests

2 participants