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

BobLight protocol #2917

Merged
merged 17 commits into from
Dec 8, 2022
Merged

BobLight protocol #2917

merged 17 commits into from
Dec 8, 2022

Conversation

blazoncek
Copy link
Collaborator

Add (simplified) BobLight protocol implementation to WLED using usermod approach.

@softhack007
Copy link
Collaborator

👍 will review later this week.
Do you have some more infos about the protocol? timing, error codes, etc?

usermods/boblight/boblight.h Outdated Show resolved Hide resolved
usermods/boblight/boblight.h Outdated Show resolved Hide resolved
@softhack007
Copy link
Collaborator

@blazoncek should we merge this now, or wait a bit in case you're still tinkering on improvements?

@blazoncek
Copy link
Collaborator Author

Let's wait if @Aircoookie has something to say.

@Aircoookie
Copy link
Owner

Couldn't get a boblight daemon to work to test this, but reviewed your code and installed the usermod, found nothing to complain about :)

@Aircoookie Aircoookie merged commit b24c8b3 into main Dec 8, 2022
@blazoncek
Copy link
Collaborator Author

A Windows PC with Kodi installed should do.
But then you merged it. 😁

@blazoncek blazoncek deleted the boblight branch December 8, 2022 21:49
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.

3 participants