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

Add RG_GPIO_LED_ACTIVE_LOW option #149

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

tomvanbraeckel
Copy link
Contributor

On the Fri3d Camp 2024 badge, the RG_GPIO_LED is active low, meaning it is on when the output is 0/low and off when the output is 3V3/high.

Therefore, we need a way to invert the regular RG_GPIO_LED.

At first I used RG_GPIO_LED_INVERT but that is less descriptive than RG_GPIO_LED_ACTIVE_LOW.

On the Fri3d Camp 2024 badge, the RG_GPIO_LED is active low,
meaning it is on when the output is 0/low and off when the output
is 3V3/high.

Therefore, we need a way to invert the regular RG_GPIO_LED.

At first I used RG_GPIO_LED_INVERT but that is less descriptive
than RG_GPIO_LED_ACTIVE_LOW.
@ducalex
Copy link
Owner

ducalex commented Aug 5, 2024

I've recently been working on improving LED, to support colors and intensity and flashing patterns. But it's not ready so for now I'm okay with merging your PR.

Though it conflicts with changes I've pushed a few days. I got rid of rg_system_set_led so your patch must be moved to update_indicators essentially.

@tomvanbraeckel
Copy link
Contributor Author

tomvanbraeckel commented Aug 6, 2024

Okay, I merged the dev branch and modified it as I think it should be, although I was a bit confused by the RGB functionality. Hope it's good like this?

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.

2 participants