linux icon indicating copy to clipboard operation
linux copied to clipboard

AD9213 driver

Open danmois opened this issue 8 months ago • 9 comments

PR Description

The AD9213 is a single, 12-bit, 6 GSPS/10.25 GSPS, radio frequency (RF) analog-to-digital converter (ADC) with a 6.5 GHz input bandwidth. The AD9213 supports high dynamic range frequency and time domain applications requiring wide instantaneous bandwidth and low conversion error rates (CER). The AD9213 features a 16-lane JESD204B interface to support maximum bandwidth capability.

PR Type

  • [ ] Bug fix (a change that fixes an issue)
  • [x] New feature (a change that adds new functionality)
  • [ ] Breaking change (a change that affects other repos or cause CIs to fail)

PR Checklist

  • [x] I have conducted a self-review of my own code changes
  • [x] I have tested the changes on the relevant hardware
  • [ ] I have updated the documentation outside this repo accordingly (if there is the case)

danmois avatar May 26 '25 17:05 danmois

V2

  • remove devm_clk_get in ADF4371 driver
  • make OSCOUT1 IIO channel
  • update AD9213 driver to use regmap, device properties, cleanup.h, error handling

danmois avatar Jun 02 '25 12:06 danmois

V3

  • remove HMC7044 unused code

danmois avatar Jun 02 '25 12:06 danmois

V4

  • remove HMC7044 unused code in hmc7044_clk_set_rate

danmois avatar Jun 03 '25 08:06 danmois

V5

  • formatting adjustments

danmois avatar Jun 04 '25 06:06 danmois

Please look at my comment about commit subjects in the first version of the series. Also, can we have this without JESD FSM? What I'm thinking is to upstream the driver without FSM and then backport it with it. At least, the driver would not be completely out of tree

I doubt that this can be done without the FSM. It assures that the correct probing order is maintained. Will experiment if removing it is possible.

danmois avatar Jun 11 '25 08:06 danmois

I doubt that this can be done without the FSM. It assures that the correct probing order is maintained. Will experiment if removing it is possible.

Yeah, it would be just so we could upstream a version of the driver without it :)

nunojsa avatar Jun 12 '25 07:06 nunojsa