-
Notifications
You must be signed in to change notification settings - Fork 857
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 support for ADUM770x/78xx #2698
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Pop Ioan Daniel <[email protected]>
Signed-off-by: Pop Ioan Daniel <[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.
Stopping the review for now as this needs a lot of rework...
@@ -54,6 +54,7 @@ config IIO_ALL_ADI_DRIVERS | |||
imply AD7923 | |||
imply AD7944 | |||
imply AD7949 | |||
imply AD7405 |
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.
This should a separate patch
help | ||
Say yes here to build support for Analog Devices AD7405 | ||
high speed, low noise, low distortion, 16-bit, Sigma- | ||
Delta modulator. Provides direct access via sysfs. |
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.
also a separate patch... please do not mix code with devicetree stuff
#include <linux/iio/buffer-dma.h> | ||
#include <linux/iio/buffer-dmaengine.h> | ||
|
||
#include <linux/fpga/adi-axi-common.h> |
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.
Not seeing any reason for not being used the new backend framework to support this part. SO that we can actually upstream this
static struct iio_chan_spec_ext_info ad7405_ext_info[] = { | ||
|
||
IIO_ENUM("dec_rate", IIO_SHARED_BY_ALL, &ad7405_dec_rate_iio_enum), | ||
IIO_ENUM_AVAILABLE("dec_rate", IIO_SHARED_BY_ALL, &ad7405_dec_rate_iio_enum), |
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.
Is this somehow affecting the sampling frequency? In some other parts where we can decimate the ODR, this is implemented with the standard sampling frequency attribute.
PR Description
Driver support and devicetree files for the ADUM770x/78xx ADCs
Datasheet: ADUM7701
PR Type
PR Checklist