Difference: Amc13CppProductionSoftwareToDoList (1 vs. 34)

Revision 3425 Jan 2018 - DanielArcaro

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"

To Do

Added:
>
>
2018-01-25 (djarcaro) The flash programming incorrectly selects an address when the name is slightly off. Example below:

Using a FW file called

AMC13T2Goldenv0xfff7_6slx45t.mcs
causes the FW to be written to the main T2 FW location:

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
 2016-06-10 (Owen)
  • Dan would like a command in AMC13Tool2.exe that print a history of the commands you entered.

Revision 3312 Jun 2016 - SemirayGirgisChyla

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"

To Do

Revision 3210 Jun 2016 - OwenBurek

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"

To Do

Added:
>
>
2016-06-10 (Owen)
  • Dan would like a command in AMC13Tool2.exe that print a history of the commands you entered.

 2016-05-31 (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

Revision 3109 Jun 2016 - SemirayGirgisChyla

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"

To Do

Line: 9 to 9
  2016-05-25 (hazen)
Changed:
<
<
  • Add options to "localL1A" command:
>
>
  • Add options to "localL1A" command: (Semiray-Owen) DONE!
 
    • when invoked with no arguments it displays the current settings
    • localL1A reset should restore defaults (set 0x1c to 0)
  • Fix 'bgo' command issues:

Revision 3003 Jun 2016 - SemirayGirgisChyla

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"

To Do

Changed:
<
<
2016-05-31 (Semiray-Owen)
>
>
2016-05-31 (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.

Revision 2931 May 2016 - SemirayGirgisChyla

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"

To Do

Added:
>
>
2016-05-31 (Semiray-Owen)
  • 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.
 2016-05-25 (hazen)

  • Add options to "localL1A" command:

Revision 2825 May 2016 - EricHazen

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"

To Do

Added:
>
>
2016-05-25 (hazen)

  • 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
 2016-04-25

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

Revision 2725 Apr 2016 - EricHazen

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"

To Do

Added:
>
>
2016-04-25

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

Revision 2610 Mar 2016 - DanielGastler

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"

To Do

Deleted:
<
<
2015-12-18 FIXED dgastler 2016-01-08

* 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-11-30
Line: 23 to 20
 
    • Restructure code so that top-level page with menu is rendered in DTCManager.cc.
      • Remove HTML header output from Status::Report() (or make optional)
Deleted:
<
<
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
 2014-08-22, hazen

Status display, AMC_Links table:

Line: 72 to 61
 

Done

Added:
>
>
2015-12-18 FIXED dgastler 2016-01-08

* 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
 2015-11-01

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

Revision 2508 Jan 2016 - DanielGastler

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"

To Do

Changed:
<
<
2015-12-18
>
>
2015-12-18 FIXED dgastler 2016-01-08
  * 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.

Revision 2418 Dec 2015 - DanielGastler

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"

To Do

Added:
>
>
2015-12-18

* 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-11-30

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

Revision 2330 Nov 2015 - EricHazen

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"

To Do

Changed:
<
<
2015-11-01

  • 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
with results sent to a file named as follows:
  AMC13XGDump_2015-01-03_131532.txt
in the current working directory.
>
>
2015-11-30
 
Changed:
<
<
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.

>
>
  • AMC13ToolFlash doesn't work with IP address names or connection files with the "-c" option
  2015-09-16
Line: 87 to 68
 

Done

Added:
>
>
2015-11-01

  • 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
with results sent to a file named as follows:
  AMC13XGDump_2015-01-03_131532.txt
in the current working directory.

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.

 2015-07-13, hazen Done

Fix bugs in AMC13ToolFlash:

Revision 2203 Nov 2015 - EricHazen

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
Deleted:
<
<
 

To Do

Added:
>
>
2015-11-01

  • 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
with results sent to a file named as follows:
  AMC13XGDump_2015-01-03_131532.txt
in the current working directory.

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.

 2015-09-16

  • AMC13Tool2 doesn't handle 9 bit serial numbers in a few cases:

Revision 2116 Sep 2015 - EricHazen

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
Deleted:
<
<
TO DO
 
Changed:
<
<
2015-09-11, hazen
>
>

To Do

 
Changed:
<
<
  • 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
>
>
2015-09-16

  • 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)
  2015-09-08
Line: 21 to 24
 
  • 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
Deleted:
<
<
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.

 2014-08-22, hazen

Status display, AMC_Links table:

Line: 74 to 63
  A similar ticket has been submitted https://svnweb.cern.ch/trac/cactus/ticket/635
Changed:
<
<
DONE
>
>

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
  In AMC13Tool2 and AMC13ToolFlash:

Revision 2011 Sep 2015 - EricHazen

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
TO DO
Added:
>
>
2015-09-11, hazen

  • 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
 2015-09-08

Revision 1911 Sep 2015 - DanielArcaro

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
TO DO
Line: 17 to 17
 
  • 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
Changed:
<
<
2015-07-13, hazen Done?
>
>
2015-07-13, hazen Done
  Fix bugs in AMC13ToolFlash:

Revision 1808 Sep 2015 - EricHazen

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
TO DO
Added:
>
>
2015-09-08

  • 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)
 2015-08-06, djarcaro

AMC13 build warnings:

Line: 10 to 17
 
  • 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
Changed:
<
<
2015-07-13, hazen
>
>
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.
Changed:
<
<
2014-10-31, hazen
>
>
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".

Revision 1706 Aug 2015 - DanielArcaro

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
TO DO
Added:
>
>
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
 2015-07-13, hazen

Fix bugs in AMC13ToolFlash:

Revision 1604 Aug 2015 - DanielArcaro

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
TO DO
Line: 7 to 7
 Fix bugs in AMC13ToolFlash:

  • Environment variable AMC13_ADDRESS_TABLE_PATH should tolerate value ending with "/" or no "/"
Changed:
<
<
  • On the command line, after -c one should be able to specify an IP address with trailing "/c" to use the control hub
>
>
  • 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

Revision 1530 Jul 2015 - ZachCollins

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
Added:
>
>
TO DO
 2015-07-13, hazen

Fix bugs in AMC13ToolFlash:

Line: 7 to 9
 
  • 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
Deleted:
<
<
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".

 2014-10-31, hazen

Display actual state names in TTS status. Probably this is a "format" attribute,

Line: 59 to 55
  A similar ticket has been submitted https://svnweb.cern.ch/trac/cactus/ticket/635
Added:
>
>
DONE

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".
 -- EricHazen - 10 Jun 2014 \ No newline at end of file

Revision 1413 Jul 2015 - EricHazen

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
Added:
>
>
2015-07-13, hazen

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".

 2014-10-31, hazen

Display actual state names in TTS status. Probably this is a "format" attribute,

Revision 1331 Oct 2014 - EricHazen

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
2014-10-31, hazen
Line: 32 to 32
  4. Provide return value for flash verify functions
Deleted:
<
<
2014-06-17, hazen, dzou:

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

DONE

2014-06-13, hazen, dzou:

1. I think you have left out parts of the register names in many cases.

For example, below you want I believe "CONF.EVB.ENABLE_DAQLSC".

EDIT 1: Fixed

2. 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); }

EDIT 2: Fixed

3. Please change the AMCInputEnable to take a bit mask instead of a list. EDIT 3: FIXED

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

2014-06-10, dzou:

1. 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.

EDIT 1: Fixed

2014-06-10, hazen:

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
That is fine if a register name is used. For the address-based block reads, it is more

FIXED(2014-06-10, dzou): Address block read do not require register to have incremental mode. Register block read now checks for mode==incremental and will do a address block read if there is no mode == incremental

 2041-06-10, hazen:

Break up block reads greater than 0x400 32-bit words into multiple reads.

Revision 1231 Oct 2014 - EricHazen

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
Added:
>
>
2014-10-31, hazen

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.

 2014-08-22, hazen

Status display, AMC_Links table:

Revision 1122 Aug 2014 - EricHazen

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
Added:
>
>
2014-08-22, hazen

Status display, AMC_Links table:

  • Link_Enable is redundant
  • AMC_Words, AMC_Events, AMC_Headers etc should be zero-suppressed
 2014-08-07, hazen

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

Revision 1014 Aug 2014 - DavidZou

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
2014-08-07, hazen
Line: 7 to 7
  2014-08-06, hazen
Changed:
<
<
Use address table names or #define constants instead of hard-wired values
>
>
1. Use address table names or #define constants instead of hard-wired values
 in the flash programming code.
Changed:
<
<
Add support for file paths in flash programming code (need to separate the file name
>
>
2. 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.
Changed:
<
<
selectMcsFile should also take a directory path.
>
>
3. selectMcsFile should also take a directory path.
 
Changed:
<
<
Provide return value for flash verify functions
>
>
4. Provide return value for flash verify functions
  2014-06-17, hazen, dzou:

Revision 908 Aug 2014 - EricHazen

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
2014-08-07, hazen
Line: 22 to 22
  Add FLASH programming. (Needs to be done before initial release)
Added:
>
>
DONE
 2014-06-13, hazen, dzou:

1. I think you have left out parts of the register names in many cases.

Revision 807 Aug 2014 - EricHazen

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
Changed:
<
<
2014-08--6, hazen
>
>
2014-08-07, hazen

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.

Revision 706 Aug 2014 - EricHazen

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
Added:
>
>
2014-08--6, 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

 2014-06-17, hazen, dzou:

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

Revision 617 Jun 2014 - DavidZou

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
Added:
>
>
2014-06-17, hazen, dzou:

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

 2014-06-13, hazen, dzou:
Changed:
<
<
I think you have left out parts of the register names in many cases.
>
>
1. I think you have left out parts of the register names in many cases.
  For example, below you want I believe "CONF.EVB.ENABLE_DAQLSC".
Changed:
<
<
EDIT: Fixed
>
>
EDIT 1: Fixed
 
Added:
>
>
2.
  void AMC13::daqLinkEnable(bool b) { if(b) write(T1, "EVB.ENABLE_DAQLSC", 1);
Line: 20 to 25
  write(T1, "EVB.ENABLE_DAQLSC", b); }
Changed:
<
<
EDIT: Fixed
>
>
EDIT 2: Fixed
 
Changed:
<
<
Please change the AMCInputEnable to take a bit mask instead of a list.
>
>
3. Please change the AMCInputEnable to take a bit mask instead of a list. EDIT 3: FIXED
 
Changed:
<
<
Please add an SFPOutputEnable to enable SFPs, taking a 3-bit mask
>
>
4. Please add an SFPOutputEnable to enable SFPs, taking a 3-bit mask
 (writes to CONF.SFP.ENABLE_MASK).
Added:
>
>
EDIT 4: FIXED
  2014-06-10, dzou:
Changed:
<
<
A number of control functions implemented with hard coded addresses (due to no equivalent register).
>
>
1. 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.
 
Changed:
<
<
Should add relevant registers to new address table and use named registers in control functions.
>
>
EDIT 1: Fixed
  2014-06-10, hazen:

Revision 517 Jun 2014 - DavidZou

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
2014-06-13, hazen, dzou:
Added:
>
>
 I think you have left out parts of the register names in many cases.
Added:
>
>
 For example, below you want I believe "CONF.EVB.ENABLE_DAQLSC".
Added:
>
>
EDIT: Fixed
  void AMC13::daqLinkEnable(bool b) { if(b)
Line: 17 to 20
  write(T1, "EVB.ENABLE_DAQLSC", b); }
Added:
>
>
EDIT: Fixed
 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

Revision 416 Jun 2014 - DavidZou

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
Added:
>
>
2014-06-13, hazen, dzou: I think you have left out parts of the register names in many cases. For example, below you want I believe "CONF.EVB.ENABLE_DAQLSC".

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).

 2014-06-10, dzou:

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

Revision 310 Jun 2014 - DavidZou

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
Added:
>
>
2014-06-10, dzou:

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.

 2014-06-10, hazen:

Add 64-bit versions of read and write methods. Apparently under some new C++

Line: 20 to 26
 That is fine if a register name is used. For the address-based block reads, it is more
Added:
>
>
FIXED(2014-06-10, dzou): Address block read do not require register to have incremental mode. Register block read now checks for mode==incremental and will do a address block read if there is no mode == incremental
 2041-06-10, hazen:

Break up block reads greater than 0x400 32-bit words into multiple reads.

Revision 210 Jun 2014 - DavidZou

Line: 1 to 1
 
META TOPICPARENT name="AMC13CppProductionSoftware"
2014-06-10, hazen:
Line: 32 to 32
 Original sent header was 0x2018080F, 9 bytes into IPbus send payload
Changed:
<
<
>
>
A similar ticket has been submitted https://svnweb.cern.ch/trac/cactus/ticket/635
  -- EricHazen - 10 Jun 2014

Revision 110 Jun 2014 - EricHazen

Line: 1 to 1
Added:
>
>
META TOPICPARENT name="AMC13CppProductionSoftware"
2014-06-10, hazen:

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
That is fine if a register name is used. For the address-based block reads, it is more

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

 
This site is powered by the TWiki collaboration platform Powered by PerlCopyright © 2008-2019 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.
Ideas, requests, problems regarding TWiki? Send feedback