-
Notifications
You must be signed in to change notification settings - Fork 11
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/base peripheral #150
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Update peripheral classes to use new base_peripheral as the base class
finger563
added
enhancement
New feature or request
nfc
peripherals
refactor
aw9523
ndef
ads7138
tla2528
TLA2528 analog/digital expander
ads1x15
drv2605
mcp23x17
mt6701
as5600
st25dv
ft5x06
touch driver
tt21100
touch driver
qwiicnes
gt911
touch driver
bm8563
real time clock peripheral driver
t-keyboard
max1704x
base peripheral
labels
Feb 12, 2024
…t6701 config (read -> read_register)
✅Static analysis result - no issues found! ✅ |
* Updated tla2528 example to test all analog input channels
12 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
ads1x15
ads7138
as5600
aw9523
base peripheral
bm8563
real time clock peripheral driver
drv2605
enhancement
New feature or request
ft5x06
touch driver
gt911
touch driver
max1704x
mcp23x17
mt6701
ndef
nfc
peripherals
qwiicnes
refactor
st25dv
t-keyboard
tla2528
TLA2528 analog/digital expander
tt21100
touch driver
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
base_peripheral
component (espp::BasePeripheral
) which contains:base_config_
: address and raw I/O functionsbase_mutex_
: std::recursive_mutex for protecting I/O / access to the peripheralespp::BaseComponent
(logger and associated functions)Motivation and Context
Similar to #149 - this PR creates a
espp::BasePeripheral
component with config, members, and helper functions for creating a thread-safe peripheral driver. It refactors the existing peripheral components to use this as the new base.How has this been tested?
bm8563
examplebldc_motor
examplebldc_haptics
examplett21100
examplest25dv
exampledrv2605
exampletla2528
exampleaw9523
examplemax1704x
examplemcp23x17
exampleads1x15
examplegt911
examplet_keyboard
exampleScreenshots (if appropriate, e.g. schematic, board, console logs, lab pictures):
Types of changes
Checklist:
Software
.github/workflows/build.yml
file to add my new test to the automated cloud build github action.