-
Notifications
You must be signed in to change notification settings - Fork 141
Added Rx1-Rx2 phase inversion enable enhancement to AD9361 system object. #75
base: master
Are you sure you want to change the base?
Conversation
| isOutput = false; | ||
| end | ||
|
|
||
| properties (Hidden) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be a logical
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And probably nontunable
|
|
||
| properties (Hidden) | ||
| %EnableRx1Rx2PhaseInversion | ||
| % If enabled, RX1 and RX2 are phase aligned |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would recommend being more specific with this comment. Since it might give a wrong impression
| '', 'EnableRx1Rx2PhaseInversion'); | ||
| obj.EnableRx1Rx2PhaseInversion = value; | ||
| if obj.ConnectedToDevice | ||
| obj.setDebugAttributeLongLong('adi,rx1-rx2-phase-inversion-enable',value); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note that this won't actually update the part unless initialize is called
| obj.setDebugAttributeLongLong('loopback', obj.LoopbackMode); | ||
|
|
||
| % Enable Rx1-Rx2 Phase Inversion | ||
| obj.setDebugAttributeLongLong('adi,rx1-rx2-phase-inversion-enable', obj.EnableRx1Rx2PhaseInversion); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This won't do anything unless CustomAGC is enabled
No description provided.