pyadi-iio icon indicating copy to clipboard operation
pyadi-iio copied to clipboard

ad7606 tweaks and updates

Open commodo opened this issue 1 year ago • 2 comments

Description

This changeset mostly targets support for the AD7606C-18 and AD7606C-16 parts.

The upstream support for these parts is pending here:

https://lore.kernel.org/linux-iio/[email protected]/T/#t

But there were a couple of fixes required for existing parts.

For one thing the oversampling attributes need to be made device-level attributes.

There are no 'range' attributes. These are scale attributes.

Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • [x] Bug fix (non-breaking change which fixes an issue)
  • [x] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [ ] This change requires a documentation update

How has this been tested?

Tested with a ZedBoard and an AD7606C-18 FMC board. The HDL is a simplified version that uses the Zynq PS SPI controller, which kind of like using the AD7606 part directly connected to a simple RPi board; no fancy communication.

  • [ ] Test A
  • [ ] Test B

Test Configuration:

  • Hardware: ZedBoard + AD7606C-18
  • OS: Kuiper Linux + upstream IIO kernel + AD7606C-18/16 changes

Documentation

If this is a new feature or example please mention or link any documentation. All new hardware interface classes require documentation.

Checklist:

  • [x] My code follows the style guidelines of this project
  • [x] I have performed a self-review of my own code
  • [x] I have commented my code, particularly in hard-to-understand areas
  • [x] I have signed off all commits and they contain "Signed-off by: "
  • [ ] I have made corresponding changes to the documentation
  • [x] My changes generate no new warnings
  • [ ] I have added tests that prove my fix is effective or that my feature works
  • [ ] New and existing unit tests pass locally with my changes
  • [ ] Any dependent changes have been merged and published in downstream modules

commodo avatar Aug 19 '24 09:08 commodo

Just need an update on the bare exceptions

apologies for the delay; updated; thanks :)

commodo avatar Mar 02 '25 09:03 commodo

apparently my local python environment is all messed up; it took me a while to be able to run the pre-commit stuff;

lucky for docker;

commodo avatar Mar 04 '25 08:03 commodo