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

Initial interlock configuration and RF channel management structure #27

Merged
merged 28 commits into from
Jul 9, 2020

Conversation

ryan-summers
Copy link
Member

@ryan-summers ryan-summers commented Jul 8, 2020

This PR implements the BoosterChannels management object that is responsible for detecting, instantiating, and interfacing to all Booster RF channels.

Instantiating a channel currently implements an algorithm to query for devices on the I2C bus after properly powering up the channel.

The only currently-implemented functionality is the ability to configure interlock thresholds. All thresholds are defaulted to -30dBm.

This structure should allow for extensible control of the channel as the underlying object has all of the available I2C devices instantiated.

The only resource not yet present in RF channels are their reflected and output power ADC inputs. I'll add these in the future when I add support for measuring these signals.

src/rf_channel.rs Outdated Show resolved Hide resolved
src/rf_channel.rs Outdated Show resolved Hide resolved
src/rf_channel.rs Outdated Show resolved Hide resolved
src/rf_channel.rs Outdated Show resolved Hide resolved
src/rf_channel.rs Outdated Show resolved Hide resolved
src/rf_channel.rs Outdated Show resolved Hide resolved
src/rf_channel.rs Outdated Show resolved Hide resolved
Copy link
Member

@jordens jordens left a comment

Choose a reason for hiding this comment

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

Double checked pins, pin modes, power down state and sequence, i2c channels, conversion factors.

src/rf_channel.rs Outdated Show resolved Hide resolved
src/rf_channel.rs Outdated Show resolved Hide resolved
src/booster_channels.rs Outdated Show resolved Hide resolved
src/main.rs Show resolved Hide resolved
src/booster_channels.rs Show resolved Hide resolved
@ryan-summers ryan-summers requested a review from jordens July 9, 2020 11:56
@ryan-summers
Copy link
Member Author

Note that I realized there's an issue with how the analog pins are implemented (since they are downgraded, we can't pass them to the ADC). I'll implement a refactor for this in a follow-on PR.

@ryan-summers ryan-summers merged commit 27f2b50 into develop Jul 9, 2020
@ryan-summers ryan-summers deleted the feature/interlock-config branch July 9, 2020 12:06
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.

2 participants