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

update API and examples for egpio #249

Merged
merged 2 commits into from
Jul 2, 2024
Merged

update API and examples for egpio #249

merged 2 commits into from
Jul 2, 2024

Conversation

ChrisDick
Copy link
Contributor

also fix units of Battery current.

Description

add new elements to the API for the the LS gpio and update examples. correct the units for the battery charge current from mA to A.

also fix units of Battery current.
Copy link
Collaborator

@npentrel npentrel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm % nits. Is there a reason you'd prefer to change the capitalisation of gpio? I would like to keep it consistent and currently it's capitalised everywhere

docs/tildagon-apps/reference/badge-hardware.md Outdated Show resolved Hide resolved
docs/tildagon-apps/reference/badge-hardware.md Outdated Show resolved Hide resolved
docs/tildagon-apps/reference/badge-hardware.md Outdated Show resolved Hide resolved
@ChrisDick
Copy link
Contributor Author

I renamed it to match the name of the module. Do you think that matters? not sure it's wise changing the name in the sw, but we could while it's still in rc. I don't mind either way, happy to replace all occurrences in the docs if that makes more sense to keep consistency. Another solution is to make them all eGPIO except in the methods section and make that one egpio.ePin?

@npentrel
Copy link
Collaborator

npentrel commented Jul 2, 2024

@ChrisDick let's go with your suggestion and make them all eGPIO except in the methods section and make that one egpio.ePin in code formatting

@npentrel
Copy link
Collaborator

npentrel commented Jul 2, 2024

Merging for now

@npentrel npentrel merged commit b71c181 into main Jul 2, 2024
3 checks passed
@npentrel npentrel deleted the ext-gpio-c branch July 2, 2024 17:21
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