Fix exe mode when executing on card

mexanick commented 6 years ago

Description

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Motivation and Context

Fixing exe mode to work correctly on the card

How Has This Been Tested?

tested at eagle61 examples of usage: Examples from host:

reg_interface.py -e read "GEM_AMC.GEM_SYSTEM.BOARD_ID" -n eagle61
reg_interface.py -e write "GEM_AMC.GEM_SYSTEM.BOARD_ID 0xfeed" -n eagle61

Examples from card:

reg_interface.py -e read "GEM_AMC.GEM_SYSTEM.BOARD_ID"
reg_interface.py -e write "GEM_AMC.GEM_SYSTEM.BOARD_ID 0xfeed"

Screenshots (if appropriate):

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

mexanick commented 6 years ago

@bdorney I don't understand your comment. Any method can be called in exe mode...

bdorney commented 6 years ago

@bdorney I don't understand your comment. Any method can be called in exe mode...

The comment is that from the help menu of reg_interface.py the syntax expected when using the -e option is not illustrated. Additionally if incorrect syntax is given the output text does not given information to adequately understand the root of the problem.

mexanick commented 6 years ago

this is fixed in last commit, please see above

bdorney commented 6 years ago

Missed the updated commit, excellent. This is exactly what I was hoping for!

cms-gem-daq-project / xhal