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

Enhance ADRV9009-ZU11EG multi-class to support selective FMComms8 suport

Open tfcollins opened this issue 1 year ago • 3 comments

Addresses #535

tfcollins avatar Mar 11 '24 15:03 tfcollins

Generated documentation for this PR is available at Link

github-actions[bot] avatar Mar 11 '24 15:03 github-actions[bot]

Test Results

1 463 tests  ±0     279 :white_check_mark: ±0   9m 51s :stopwatch: ±0s     1 suites ±0   1 184 :zzz: ±0      1 files   ±0       0 :x: ±0 

Results for commit ecdc8d3c. ± Comparison against base commit 2061827b.

:recycle: This comment has been updated with latest results.

github-actions[bot] avatar Mar 11 '24 15:03 github-actions[bot]

Hey Travis. Sorry for the delay, but finally got around to testing this today. Everything seems to work! I did have to make a few edits to the multi class. There are some checks for fmcomms8 that need to be changed to index into the list. Specifically lines 234, 246, 254, 271, 360, and 374. I just did like you did earlier and changed the for loop to be (e.g.)

for i, dev in enumerate([self.primary] + self.secondaries):
    dev._clock_chip.attrs["sleep_request"].value = "1"
    if self.fmcomms8[i]:
        dev._clock_chip_fmc.attrs["sleep_request"].value = "1"
    dev._clock_chip_carrier.attrs["sleep_request"].value = "1"

cafeclimber avatar May 22 '24 17:05 cafeclimber