Difference: Amc13CppProductionSoftwareToDoList (1 vs. 34)

Using a FW file called

AMC13T2Goldenv0xfff7_6slx45t.mcs

Programming AMC13T2Goldenv0xfff7_6slx45t.mcs to flash address 0x200000...
Erasing flash sector at address 0x200000
Erasing flash sector at address 0x240000
Erasing flash sector at address 0x280000
Erasing flash sector at address 0x2c0000
Erasing flash sector at address 0x300000
Erasing flash sector at address 0x340000

...

Using AMC13 software ver:48372
Read firmware versions 0x225c 0xfff7

From a main T2 FW file:

Programming AMC13T2v0x0031_6slx45t.mcs to flash address 0x200000...

After taking out the 0xfff7 the problem goes away:

Programming AMC13T2Golden_6slx45t.mcs to flash address 0x80000...
Erasing flash sector at address 0x80000

• Dan would like a command in AMC13Tool2.exe that print a history of the commands you entered.

• Add options to "localL1A" command: (Semiray-Owen) DONE!

• When looking at all the registers by throwing the wild card in AMC13Tool2.exe (rv *status) we noticed that some of the registers that appeared could not be found by looking for those registers with the rv command. For example:
  • 'STATUS.TTC.SGL_BIT_ERRORS_HI' appears when using the wild card but
  • > rv STATUS.TTC.SGL_BIT_ERRORS_HI gives us nothing in the tool.

• Add options to "localL1A" command:
  • when invoked with no arguments it displays the current settings
  • localL1A reset should restore defaults (set 0x1c to 0)
• Fix 'bgo' command issues:
  • in 'list' remove 'ena' column, replace with 'repeat'
  • last argument to bgo setting command should be one of "off", "single" or "repeat"
  • allow things like bgo 0 off, bgo 1 repeat which don't change other settings
  • add new command like bgo l1a_offset which sets register CONF.TTC.BGO.L1A_OFFSET

• Fix MCS file reader so it give sensible error messages when the files is not an MCS file or has other formatting problems.

* AMC13Tool2 pv command breaks if a file in the current directory is not named correctly. eg "AMC13_T1_test.mcs" This will cause an exception and kill the "pv" command even if you aren't going to use that file.

2015-08-06, djarcaro

AMC13 build warnings:

• src/common/Launcher_commands_control.cc:741: warning: no return statement in function returning non-void
• src/common/Launcher_commands_status.cc:55: warning: comparison between signed and unsigned integer expressions
• src/common/Launcher_commands.cc:64:39: warning: backslash and newline separated by space

* AMC13Tool2 pv command breaks if a file in the current directory is not named correctly. eg "AMC13_T1_test.mcs" This will cause an exception and kill the "pv" command even if you aren't going to use that file.

• AMC13ToolFlash doesn't work with IP address names or connection files with the "-c" option

• Add "dump" command to AMC13Tool2. I suggest the following behavior:

If invoked with no arguments, perform the equivalent of the following commands:

  > rv 0 0x2000
  > rs 0 0x1000

  AMC13XGDump_2015-01-03_131532.txt

It should print a message such as this:

Dump written to /home/someuser/working/directory/AMC13XGDump_2015-01-03_131532.txt.
Please e-mail to wusx@bu.edu with details of the conditions under which the dump was captured and the problems experienced.

The command should take a file name as an optional argument and if specified write the dump to that file.

• Add "dump" command to AMC13Tool2. I suggest the following behavior:

If invoked with no arguments, perform the equivalent of the following commands:

  > rv 0 0x2000
  > rs 0 0x1000

  AMC13XGDump_2015-01-03_131532.txt

It should print a message such as this:

Dump written to /home/someuser/working/directory/AMC13XGDump_2015-01-03_131532.txt.
Please e-mail to wusx@bu.edu with details of the conditions under which the dump was captured and the problems experienced.

The command should take a file name as an optional argument and if specified write the dump to that file.

To Do

• AMC13Tool2 doesn't handle 9 bit serial numbers in a few cases:
  • fv command
  • flash programming for T2 claims no matching files (probably a 25 vs 45 problem)

Done

2015-07-13, hazen Done

Fix bugs in AMC13ToolFlash:

• Environment variable AMC13_ADDRESS_TABLE_PATH should tolerate value ending with "/" or no "/"
• On the command line, after -c one should be able to specify an IP address with trailing "/c" to use the control hub.

2014-10-31, hazen Done

Display actual state names in TTS status. Probably this is a "format" attribute, with two new values for "raw ttc" and "encoded ttc".

Display SVN version and/or tag in status display.

2015-09-11, hazen Done

• Adding "nzr" option to status display code to display table row only if the specified item is non-zero. This should allow to handle things like the L1A history (more) gracefully

• Adding "nzr" option to status display code to display table row only if the specified item is non-zero. This should allow to handle things like the L1A history (more) gracefully

• HyperDAQ improvements
  • Provide menu with level-of-detail select, TTC history, L1A history, register dump, data dump options
  • Restructure code so that top-level page with menu is rendered in DTCManager.cc.
    • Remove HTML header output from Status::Report() (or make optional)

AMC13 build warnings:

• src/common/Launcher_commands_control.cc:741: warning: no return statement in function returning non-void
• src/common/Launcher_commands_status.cc:55: warning: comparison between signed and unsigned integer expressions
• src/common/Launcher_commands.cc:64:39: warning: backslash and newline separated by space

• On the command line, after -c one should be able to specify an IP address with trailing "/c" to use the control hub.

In AMC13Tool2 and AMC13ToolFlash:

• Done 7/30/2015 -Zach Collins Restore the function from the old AMC13Tool where you can specify a serial number on the command line and have the tool calculate the default IP address for a connection. This could just be an option for the "-c" switch, e.g. "-c 53" instead of "-c 192.168.1.27".

Fix bugs in AMC13ToolFlash:

• Environment variable AMC13_ADDRESS_TABLE_PATH should tolerate value ending with "/" or no "/"
• On the command line, after -c one should be able to specify an IP address with trailing "/c" to use the control hub

In AMC13Tool2 and AMC13ToolFlash:

• Restore the function from the old AMC13Tool where you can specify a serial number on the command line and have the tool calculate the default IP address for a connection. This could just be an option for the "-c" switch, e.g. "-c 53" instead of "-c 192.168.1.27".

Display actual state names in TTS status. Probably this is a "format" attribute, with two new values for "raw ttc" and "encoded ttc".

Display SVN version and/or tag in status display.

Status display, AMC_Links table:

• Link_Enable is redundant
• AMC_Words, AMC_Events, AMC_Headers etc should be zero-suppressed

When "T" (ttc-loop-back) is enabled, "L" (local triggers) must be also. localTtcSignalEnable() should also call genInternalL1asEnable()

2014-08-06, hazen

Use address table names or #define constants instead of hard-wired values in the flash programming code.

Add support for file paths in flash programming code (need to separate the file name from the path before parsing the file name). Check for and support "Golden" and "Header" files.

selectMcsFile should also take a directory path.

Provide return value for flash verify functions

Add FLASH programming. (Needs to be done before initial release)

void AMC13::daqLinkEnable(bool b) { if(b) write(T1, "EVB.ENABLE_DAQLSC", 1); else write(T1, "EVB.ENABLE_DAQLSC", 0); }

This is a small thing, but I'd suggest the more concise:

void AMC13::daqLinkEnable(bool b) { write(T1, "EVB.ENABLE_DAQLSC", b); }

Please change the AMCInputEnable to take a bit mask instead of a list.

Please add an SFPOutputEnable to enable SFPs, taking a 3-bit mask (writes to CONF.SFP.ENABLE_MASK).

A number of control functions implemented with hard coded addresses (due to no equivalent register).

Should add relevant registers to new address table and use named registers in control functions.

Add 64-bit versions of read and write methods. Apparently under some new C++ rules you aren't allowed to convert uint32_t* to uint64_t* without using a union or something.

(for now, maybe just add -fno-strict-aliasing to Makefile!)

2014-06-10, hazen:

Check for block read capability and automatically choose between block read and single-word reads. e.g.:

 
  if( m_T1 -> getNode(reg).getMode() == uhal::defs::INCREMENTAL))
    // do block read
  else
    // do single reads

2041-06-10, hazen:

Break up block reads greater than 0x400 32-bit words into multiple reads. File a bug report to UHAL developers that reads greater than about 0x500 words causes the following exception:

terminate called after throwing an instance of 'uhal::exception::IPbusCoreResponseCodeSet'
  what():  Returned Header, 0x20180706 ( transaction id = 0x00000018, transaction type = 0x00, word count = 7 ) \
  had response field = 0x06 indicating an error (1025 bytes into IPbus reply payload)
Original sent header was 0x2018080F, 9 bytes into IPbus send payload

-- EricHazen - 10 Jun 2014