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.

Sign up for GitHub

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

Jump to bottom

Review code to eliminate public items that don't need to be public #8

Open
BrentSeidel opened this issue Sep 6, 2024 · 1 comment
Open

Review code to eliminate public items that don't need to be public #8

BrentSeidel opened this issue Sep 6, 2024 · 1 comment
Assignees
@BrentSeidel
Labels
enhancement

Comments

@BrentSeidel
Copy link
Owner

BrentSeidel commented Sep 6, 2024 •
edited
Loading

During initial development many devices had constants and datatypes exposed in the public interface that are only needed internally. These should be moved to the private section of the spec or to the package body.

Common:

Driver Status
TB6612 No change
ADS1015 Done
BME280 Done
BMP180 Done
L3GD20H Done
LSM303DLHC Done
MCP4725 Done
MCP23008 Done
MCP23017 Done
PCA9685 Done
RA8875 Pending

Linux:

Driver Status
AIN Pending
GPIO Pending
I2C Pending
LED Pending
PWM Pending
SPI Pending

Arduinio Due will be handled separately once I get a chance to build a run-time for it.
@BrentSeidel BrentSeidel self-assigned this Sep 6, 2024
BrentSeidel added a commit that referenced this issue Sep 11, 2024
@BrentSeidel
Move MCP23017 register address constants to private section per issue #8
4e72cc5 
.  Add function to determine if a MCP23017 is present and update documentation.
@BrentSeidel
Copy link
Owner Author

BME280 driver has been updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BrentSeidel BrentSeidel

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@BrentSeidel