Haven't looked at primParams_sct.h properly, maybe Alan would comment on the current RW_MODULE_DATA and RW_MODULE_VARIABLE.
Never used by SCT
- SET_MESSAGE_MASK
- PAUSE_LIST
- COPY_MEMORY
- TRANS_SERIAL_DATA
- CONFIG_SLAVE
The SCT code doesn't use
- ECHO
- WRITE_BUFFER
- MEMORY_TEST
- SET_LED
- FLASH_LED
but these look useful enough to keep around. Of course MEMORY_TEST could be changed, as long as it does the same job.
Primitive by primitive notes
MODULE_MASK
Could do with some renaming and splitting into 3 or 4 primitives. I've just started looking at it properly, it's needed for reconfiguring modules on the fly. I don't know how SCT specific it is, but I've got some code which splits it into separate functions, basically so I could easily see which parameters were being used where.
SET_TRIGGER
This is used to move the slaves to a different bin when they're histogramming in "LOCAL" mode (ie external triggers). All the other parameters aren't used (a relic from pre Douglas days). In fact there seems to be half of a SET_BIN primitive, which would be more useful.
START_SLAVE_EXECUTING
The only used parameter is the slave number.
SET_ROD_MODE
I think could be simplified, all we need is to move between PHYSICS and CALIBRATION and to CAPTURE data from INMEMs. We don't need any of the modify mode with strange options.