-
Notifications
You must be signed in to change notification settings - Fork 19
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
Modified the AD405x Documentation #103
Conversation
@@ -22,10 +24,10 @@ Introduction | |||
============ | |||
|
|||
This page gives an overview of using the ARM platforms supported (default is Mbed) |
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 default platform mbed in ad405x? We usually have it as stm for newer projects
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.
It was for AD4052
The changes in this commit were pushed to bitbucket by Ribhu. Now that AD4050 is released too, it was decided to merge it to Github.
@@ -50,13 +52,13 @@ to communicates with IIO device. | |||
Hardware Connections | |||
==================== | |||
|
|||
Required: SDP-K1 (or alternative Mbed enabled controller board, or an STM32 board ), EVAL-AD4052 board | |||
Required: SDP-K1 (or alternative Mbed enabled controller board, or an STM32 board ), EVAL-AD4052 or EVAL-AD4050 board |
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.
Should we remove the term "mbed" ?
b4026e9
to
ae65e18
Compare
ae65e18
to
fc72a4c
Compare
Removed the Selection of active device section, as the firmwares have been merged |
Communication Interface | ||
======================= | ||
|
||
SDP-K1 is powered through USB connection from the computer. SDP-K1 MCU board |
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 think we can include the previous rst for communication interface as VCOM support is present for STM32 as well now
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 think we'll have to remove mentions of mbed from the file too then?
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.
Can we update the image as VCOM support is now present
fc72a4c
to
d9bc870
Compare
Added AD4050 as another supported device Signed-off-by: Anush Shetty <anush.shetty@analog.com>
d9bc870
to
ccc0744
Compare
PR LGTM and can be merged |
Shall I merge this PR, @Shetty-Anush as the part is already released? |
Added AD4050 as another supported device