-
Notifications
You must be signed in to change notification settings - Fork 587
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
New Device: LED segment display 5641AS #2350
base: main
Are you sure you want to change the base?
Conversation
@dotnet-policy-service agree |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed markdown syntax for local image
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, couple of questions and remarks. It looks good all up.
…s including removal of special overload for DP handling; added ReadMe for sample
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, looks good. Good addition!
@Ellerbach there is a build failure in |
New device
LedSegmentDisplay5641AS
added. The device has fewer display characteristics (no colon or degrees) and while other devices in the Display category were driven by backpacks over I2C, this code is full GPIO only. In addition to the device, a dedicated pin schemeLedSegmentDisplay5641ASPinScheme
is for mapping board pins to device pins, and a helper class translatesFont
s to segment assignments. Readme also updated.My first contribution so looking for constructive feedback where necessary.
Microsoft Reviewers: Open in CodeFlow