-
Notifications
You must be signed in to change notification settings - Fork 1.8k
drivers:led:ltc3220: Add support for LTC3220 #2795
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
base: main
Are you sure you want to change the base?
Conversation
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
1fea042 to
4f916a6
Compare
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
4f916a6 to
8e703f3
Compare
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
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.
documentations issues seem to be valid.
f0abacc to
6a5509f
Compare
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
6a5509f to
7723173
Compare
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
7723173 to
2442b73
Compare
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
|
Just a nitpick from me: the drivers doc commit should come after the drivers commit. |
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.
Just some minor changes I've missed.
2442b73 to
8e52f1b
Compare
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
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.
Just minor changes. Otherwise, LGTM.
Added implementation for writing to LTC3220 device registers Signed-off-by: Jan Carlo Roleda <[email protected]>
8e52f1b to
ff69c7f
Compare
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
Added driver documentation for LTC3220 LED driver Signed-off-by: Jan Carlo Roleda <[email protected]>
Created LTC3220 project example for Maxim platform Includes tests for the different features of the LTC3220 Signed-off-by: Jan Carlo Roleda <[email protected]>
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.
I just missed this: rename makefile to Makefile for consistency. Everything else, LGTM.
ff69c7f to
76f4f7b
Compare
Added project documentation for LTC3220 Driver Example Signed-off-by: Jan Carlo Roleda <[email protected]>
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
76f4f7b to
6a839ed
Compare
|
/AzurePipelines run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
Pull Request Description
This PR adds driver support for the LTC3220/3220-1 Universal 18-Channel LED Driver with a project example.
Driver Features Implemented
Project Examples Added
PR Type
PR Checklist