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

Add SAADC driver for nRF52 #8643

Closed
rodrigopex opened this issue Jun 29, 2018 · 3 comments
Closed

Add SAADC driver for nRF52 #8643

rodrigopex opened this issue Jun 29, 2018 · 3 comments
Labels
platform: nRF Nordic nRFx

Comments

@rodrigopex
Copy link
Contributor

rodrigopex commented Jun 29, 2018

I have the SAADC working on my personal project using NRFX Hal, and I would like to "port" this code and make a driver for SAADC on Zephyr. Would I create a specific driver for SAADC cloning the ADC one or should I put the driver as another ADC driver implementation?

@carlescufi
Copy link
Member

Hi @rodrigopex. Once this PR is merged you should have what you need:
https://github.com/zephyrproject-rtos/zephyr/pull/7691/commits

Mind closing this issue if that covers your requirement?

@carlescufi carlescufi added the platform: nRF Nordic nRFx label Jun 29, 2018
@rodrigopex
Copy link
Contributor Author

Thank you. I will cover what I need.

@rodrigopex
Copy link
Contributor Author

@carlescufi I have already done some code for this issue. As it is almost ready based on the #7691, I will discard my code. I did a sample for the ADC and I was wondering if it would be useful as we don't have any ADC sample code. This is the link for the branch with the implementation I did: https://github.com/rodrigopex/zephyr/tree/saadc_driver. It is really simple and basic but it solves what I need. It is using the "old" ADC API.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform: nRF Nordic nRFx
Projects
None yet
Development

No branches or pull requests

2 participants