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 driver for AD8460 #2406

Merged
merged 5 commits into from
Jan 22, 2025
Merged

Conversation

jemfgeronimo
Copy link
Contributor

@jemfgeronimo jemfgeronimo commented Dec 20, 2024

Pull Request Description

Note: IIO Driver was patterned to the mainline AD8460 Linux driver

The AD8460 is a “bits in, power out” high voltage, high-power, high-speed driver optimized for large output current (up to ±1 A) and high slew rate (up to ±1800 V/μs) at high voltage (up to ±40 V) into capacitive loads. Combining a 14-bit high-speed DAC, a high voltage, high output current (HV-HI) analog driver, and fault monitoring and protection circuits, the AD8460 is ideally suited for high power applications such as arbitrary waveform generation (AWG), programmable power supplies, and high voltage automated test equipment (ATE).

A proprietary high-voltage BCDMOS process, novel high voltage architecture, and thermally enhanced package from Analog Devices Inc. enable this high-performance driver. A digital engine implements user-configurable features: modes for digital input, programmable supply current, and fault monitoring and programmable protection settings for output current, output voltage, and junction temperature. Analog features extend functionality: external compensation enables unlimited capacitive load drive, programmable shutdown delay, and full-scale adjustment. The AD8460 operates on high voltage dual supplies up to ±55 V and a single low voltage supply of 5 V.

PR Type

  • Bug fix (change that fixes an issue)
  • New feature (change that adds new functionality)
  • Breaking change (has dependencies in other repos or will cause CI to fail)

PR Checklist

  • I have followed the Coding style guidelines
  • I have performed a self-review of the changes
  • I have commented my code, at least hard-to-understand parts
  • I have build all projects affected by the changes in this PR
  • I have tested in hardware affected projects, at the relevant boards
  • I have signed off all commits from this PR
  • I have updated the documentation (wiki pages, ReadMe etc), if applies

@jemfgeronimo jemfgeronimo force-pushed the dev/ad8460 branch 3 times, most recently from 513b882 to 8d0d77f Compare December 20, 2024 15:23
@jemfgeronimo jemfgeronimo marked this pull request as ready for review December 21, 2024 05:00
@jemfgeronimo
Copy link
Contributor Author

jemfgeronimo commented Jan 9, 2025

v1 -> v2

  • int -> int32_t
  • ADI BSD -> 3-Clause BSD
  • license year: 2024 -> 2025

@CiprianRegus
Copy link
Contributor

It seems like you have some left over commits.

@jemfgeronimo
Copy link
Contributor Author

It seems like you have some left over commits.

You're right. Fixed. Thanks!

@jemfgeronimo
Copy link
Contributor Author

v2 -> v3

  • removed AD8460_CTRL_REG(x)

Copy link
Contributor

@amiclaus amiclaus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one small comment, otherwise lgtm.

Add initial header and source files for AD8460 driver.

Signed-off-by: John Erasmus Mari Geronimo <Johnerasmusmari.Geronimo@analog.com>
Add initial header and source file for AD8460 IIO driver.

Signed-off-by: John Erasmus Mari Geronimo <Johnerasmusmari.Geronimo@analog.com>
Add README.rst documentation file for AD8460 alongside other
documentation related files.

Signed-off-by: John Erasmus Mari Geronimo <Johnerasmusmari.Geronimo@analog.com>
Add initial project files for both basic and IIO examples for AD8460.

Signed-off-by: John Erasmus Mari Geronimo <Johnerasmusmari.Geronimo@analog.com>
Add README.rst documentation file for project alongside other
documentation related files.

Signed-off-by: John Erasmus Mari Geronimo <Johnerasmusmari.Geronimo@analog.com>
@jemfgeronimo
Copy link
Contributor Author

v3 -> v4

  • int32_t -> int

@jemfgeronimo jemfgeronimo requested a review from amiclaus January 15, 2025 03:42
@kister-jimenez kister-jimenez merged commit b38bf46 into analogdevicesinc:main Jan 22, 2025
11 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants