Number | Date | Reviewer | Action | Assignee | Response | Resolution |
---|---|---|---|---|---|---|
1 | 6/5/1900 | Rob Lampereur | Create a linkage in the code between the Design Document and the 8051 assembly code. To make the logic flow easier to map between the design document and and code itself. Create labels within the code, which we reference within the document. |
Daniel Blackman | This will be implemented, but in a gradual fashion - with completion planned for when code goes to Level 3. Assigned to Ken Brownsberger: In DCE- SDD there will be placed Subroutine names and package names. |
OPEN |
2 | 6/5/1900 | Rob Lampereur | Describe the Rational behind the Main Loop order, and that it is arbitrary. Basically, add into your comments that these are event driven activities - so the order is arbitrary. | Daniel Blackman | Comment added to MAINROUTINE comment block ----- Michelle (7/27/2000) Add more information - what are the flag settings, what are the inputs and outputs. Say that the sequence is arbitrary - and any sequence of events could be called in any order. Michelle - Please assist by providing some text. Verbiage should clear up the confusion about processing the commands on a once per second boundary. |
REOPENED |
5 | 6/5/1900 | Chris Dorato | For every subroutine in the code, there should be a summary block describing what the subroutine is and what it does, what the inputs and outputs are, and what the revision history is - starting from this point in time onward. | Daniel Blackman | Comment blocks added to every subroutine, some macro definitions, and main loop. To properly fill-in each comment block will take some time, but I will revise comment blocks as development continues... Please add a CALLED BY line, which includes the subroutines that call it. Please add the DM-05 COMMAND (and TLM) names to the appropriate subroutine banners. - Ken B. Can provide INDEX file for OPCODE to COMMAND. |
OPEN |
12 | 6/5/1900 | Art Rankin | For the section of FSW that reads and populates the PH data buffers, look at the BOOT/OPERATE flag - and if in BOOT, do NOT read the PH data and do NOT overwrite the buffer area. | Daniel Blackman | This will be implemented, but has not been added to the code yet. (As of 7/14/2000.) | OPEN |
14 | 6/5/1900 | Ken Brownsberger | If a properly formatted command is sent, but it does not exist in the OPCODE list, then report a diagnostic and an HST error. (Daniel is recommending Diagnostic 11 - command opcode not implemented.) | Daniel Blackman | Separate COS and GALEX command tables have been implemented to support this AI. Examine JMP table to make certain only ICD valid commands are implemented in the JMP table, and that every other OPCODE results in a INVALID COMMAND diagnostic. |
REOPENED |
15 | 6/5/1900 | Daniel Blackman | SPARE commands currently in the command jump table should be replaced with room for future implementation of TBD commands. For now, they should be basically function as a NOOP - but, also report an illegal command diagnostic. (See previous AI). | Daniel Blackman | In progress... Please update the comments - keeping the HISTORY of the previous use of the OPCODE, but now include the message that they are NOT used for COS. --- Where there is a NO-OP for OP-CODE thats not used for COS, replace with a call to MODE. (This performs a NO-OP, and issues a Diagnostic.) Also, all SPARE_CODE OP-CODEs will be removed. |
REOPENED |
16 | 6/5/1900 | Art Rankin | Confirm that a HSK packet is returned as a result of the JMP command. i.e., if we jump from BOOT to OPERATE, we get a HSK packet - without having to send another follow-up command, like a NOOP. |
Daniel Blackman | Daniel is currently looking at this, and will either implement a fix - or will notify Allison that she needs to modify her code to take action if a HSK packet will not be delivered. | OPEN |
17 | 6/5/1900 | Rob Lampereur | Add functionality to make the Test Commands only usable if a TEST ENABLE command (a new command) has been sent. The default is that Test Commands are DISABLED. Also, when dropping to BOOT, Test Commands should be DISABLED. | Daniel Blackman | Test commands will be included in code only by a non-flight compiler directive. Thus, no test commands will be included in the FSW burned into PROM or in the FSW loaded into RAM for on-orbit operations. FSW with test commands would only be loaded during early, ground test and debug situations. See AI #7. | OPEN |
20 | 6/5/1900 | Rob Lampereur | Set the High-water mark to just below MAX in the PROM, but copy it over to the patchable constants area, and use this value for your CHECK_STACK code. This way, you can patch it to a low value to verify that the DIAGNOSTIC 7 function is working properly. Also, add a TLM item for the stack max, so that this can be bundled with DIAG 7 for use with an HST error. |
Daniel Blackman | Ability to set High-water Mark via patchable constant will be tested for viability. If functions properly, with no adverse side effects - will implement, and will use patchable constant value to bundle with DIAG 7 for HST error of this type. | OPEN |
22 | 6/6/1900 | Michelle Troeltzsch | Add more text to the interrupt comments (starting at line 2252 of INITIALZ.LST). Explain what the interrupt does. If need additional space, put in on the line above. |
Daniel Blackman | Comments added. Michelle says, Nice Start... Complete the job. e.g., Timer1, Timer2. Include if the ISR is NOT used for COS. |
REOPENED |
23 | 6/6/1900 | Michelle Troeltzsch | Scrub all bug and debug comments throughout the entire code package. If a problem that your tracking with these kinds of comments still exists at the time the code is ready to go to Level 2 CM, then the problem needs to be tracked officially and paperwork generated. Also, dont leave dangling comments about bugs and possible fixes. If the bug was fixed, update the comment and say it was fixed. |
Daniel Blackman | Debug comments or lines removed throughout. Word 'debug' is used in some comments that do not actually indicate debug code. Other debug code is simply commented out, but preserved to demonstrate how some debug function could be performed. ---- Specific addition on 7/25/2000: Clear up confusion regarding the comment mysterious reset bug on line 3342 of INITIALZ.LST (V17055). Describe WHY you turn off interrupts here. Also, if you want to leave the mysterious reset bug - add additional comments that describe how this bug has been fixed... and why this is no longer a problem. |
REOPENED |
24 | 6/6/1900 | Art Rankin | Generate a lien list of bugs or problems that exist in the code when it is brought to Level 2. Maintain that list and update it appropriately so that it can be reviewed at the time the code is brought to Level 3. (Level 3 = Delivery to BATC.) | Daniel Blackman | This will be done when code goes from Level 1 to Level 2, and will be part of the delivery package when code goes to Level 3. As of 7/14/2000, code is at Level 1 - so no formal, project-wide lien list exists yet. | OPEN |
26 | 6/6/1900 | Ken Brownsberger | Remove the functionality at line 3389 of INITIALZ.LST. Do NOT perform the NOTSAFE. We will ALWAYS shut off HV, Door, etc... | Daniel Blackman | The XSPEC assembly control shows the old code and the new code. I want to document how I would have designed to code to be tolerant of some kinds of errors, because I believe it to be a better, safer design for general use. However, I will acquiesce to the review board and leave XSPEC turned off for flight code and qual testing. XSPEC will be turned on for use in the lab during detector development. Warning: leaving these features out of the boot PROM removes any possibility of allowing the detector to continue transmitting science data after a reset, even a spurious reset caused by SEU. | OPEN |
29 | 6/6/1900 | Rob Lampereur | Lines 4378 of COMMANDS.LST Fix the code if the lock bits are corrected in hardware. Do not unlock the Protected Control Register, unless you need it. If hardware is not corrected, document WHY you are doing what you are doing See also AI 46 |
Daniel Blackman | The purpose of the lockbit is to prevent inadvertent hardware operation of critical items with a COPY or UPLOAD command. As it turns out, the Control Register that has the lock-bit contains the Actuator Enable, Actuator ON and Door Limit Switch Override commands, which are exactly the hardware operations that should only be used in emergency situations - and thus, would benefit with this added layer of security. All commands that go through the Control Registers (both protected and unprotected), by default, unlock and then relock the register - which simplifies the FSW design, and maintains the safety for which the lock-bit was implemented. Resolution of this AI is to update the DCE SDD to match the hardware design, and the FSW design. Also, update the comments in the DCE FSW to clearly describe management of the control register. See UCB PFR#0004 documents the wiring of the Control Registers. |
OPEN |
31 | 6/6/1900 | Rob Lampereur | Lines 2584 and on, in INITIALZ.LST Add comments in FINDRESETTYPE, which indicate what kind of reset took place - and why. |
Daniel Blackman | Comments added ---- Add additional comments describing HOW you determine which kind of reset occurred. e.g., Line #2356 (V17055): If all the bytes in the pattern table match all the bytes in the test table - then we assume that a reset other than a Power-On reset occurred. |
REOPENED |
32 | 6/6/1900 | Michelle Troeltzsch | Change text of Record this dreadful event, to something along the lines of issue a DCE diagnostic, and - if appropriate - Report an HST error. | Daniel Blackman | Replaced the word 'dreadful' with 'diagnostic'. This is a macro that does a general purpose function. Appropriate comments are near the macro call describe the nature of the diagnostic. --- From 7/25/2000 Change comment at line 2521 COMMANDS.LST (V17055) from dreadful to diagnostic. |
REOPENED |
34 | 6/6/1900 | Ken Brownsberger | Assigned to: Ken B. Update DCE-Reset diagram to include the 255 POR check. |
Ken Brownsberger | SDD still needs to be updated. | OPEN |
36 | 6/6/1900 | Ken Brownsberger | Flight CODE should not decide what flight hardware were running on because of hardware talkbacks, and then set behavior for future operations because of this determination. The SETPROJECT, at line 4184, functionality should NOT test the RXD1 line and determine whether or not the DCE is COS or GALEX. A suggestion was made to use conditional compiles to generate code sets that are unique to the different PROJECTS, and different lab hardware environments. This is to remove the possibility of executing the wrong code set for COS on the flight hardware. (Either during test - or worse - on orbit.) |
Daniel Blackman | Implemented as per item 26. ------- Action Reopened: Remove the SETPROJECT functionality - instead, use conditional compiles, so that we can ONLY run COS code. (i.e., ONLY COS code is available in the PROM and the OPERATE images. |
REOPENED |
39 | 6/6/1900 | Rob Lampereur | Line 3204, (CLRSOMEBITS) in INITIALZ.LST. Document which bits youre clearing and why theyre being cleared. |
Daniel Blackman | Comments added ---- Will be discussed when we discuss the CRP algorithm. |
REOPENED |
41 | 6/6/1900 | Ken Brownsberger | Assigned to Ken B: Change 12 seconds to 10 seconds in Figure 4.2-1 of SDD. Remove background calculations in 4.2.2.4.1 of SDD. Remove the Start CRC calculations in 4.2.2.4.2, in the minimal initialization sequence. (Change to Setup or Initialize for CRC calculations.) |
Ken Brownsberger | SDD needs to be updated. | OPEN |
44 | 6/6/1900 | Michelle Troeltzsch | Try to scrub any and all comments in code that are no longer valid - especially those comments that are invalid because the code the comment was referring has been removed. | Daniel Blackman | At least some comments of this sort have been removed from code. Further updates to continue as code is finalized. | OPEN |
46 | 6/6/1900 | Ken Brownsberger | Assigned to Ken B.: Is the protected control register wired incorrectly for the DCE? If so, when is this going to be fixed? Controls 1 should have been locked. Controls 2 currently has the lock. Consult with John Andrews, Barry Welsh, and electrical engineers working at UCB. Daniel currently implements an undesirable workaround because of this problem. Make certain that he only does this if it the hardware is NOT going to be fixed. If the hardware does not get fixed - and the wrong controls register is stuck with the lock... then consider unlocking it initially, and keeping it unlocked indefinitely. |
Ken Brownsberger | See Response for AI#29 | OPEN |
47 | 6/6/1900 | Ken Brownsberger | Assigned to Ken B.: Is there a problem in the Actel for the PROM enable? Does UCB have a problem with the PROM that has not been documented, and that Daniel has created code to fix? |
Ken Brownsberger | See UCB ECO#0028. Currently, PROM enable command causes DCE to hang. Hardware fix will either involve an ACTEL change or a board-level change. In any event, PROM enable functionality (i.e., memory bank switching) is being dropped from the DCE - so the DCE FSW will not support this capability, and the FSW will be changed accordingly to drop these features. |
OPEN |
49 | 6/6/1900 | Allison Elliot | Add a banner comment to the command traffic ISRs on 2307 of INTERNAL.LST. (See previous AI on this topic) | Daniel Blackman | Subroutine banners added, to be filled-in during development | OPEN |
50 | 6/6/1900 | Ken Brownsberger | Assigned to Ken B. Get the COMMON.INC file and scrub it with the existing DCE HW/SW Interface Document. Provide any inputs/updates to Rich B. |
Ken Brownsberger | Open. Due to personnel changes, we are in the process of finding a new author for the DCE HW/SW Interface Document | OPEN |
51 | 6/6/1900 | Art Rankin | Add a counter to the Command Traffic ISR, so that we can keep track of how many Command words are rejected by the CHECKRXD routine. (Line 2417 of INTERNAL.LST) | Daniel Blackman | Not yet implemented as of 7/14/2000. Will be a variable that we can place a memory monitor on to inspect value. | OPEN |
52 | 6/6/1900 | Ken Brownsberger | Assigned to Ken B.: Work with Daniel to fix Logic Flow of Timer Tick ISR in SDD. Current flow diagram does not match 8051 code. |
Ken Brownsberger | SDD needs to be updated. | OPEN |
54 | 6/6/1900 | Allison Elliot | There needs to be logic at line 4489 of COMMAND.LST, so that the SETB F0 is not set when the CRC parameter is not set to zero. |
Daniel Blackman | Done. ---- Needs to be reexamined during review of CRC code. |
REOPENED |
55 | 7/27/2000 | Michelle Troeltzsch | Everywhere there is a JNB based on PROJECT, please indicate exactly where the code will go for COS. (e.g., Say, for COS go here....) --- Also, CLEARLY indicate where the PROJECT bit is set. (Separate activity.) In the subroutine header for ZAP_INTERNAL_MEMORY, tell where to go to see how these bits are set. (e.g., the HW/SW IFD) |
Daniel Blackman | OPEN | |
56 | 7/27/2000 | Will Clement | Issue involving Relocatable Code Segments Will and Daniel need to discuss off-line Wills concern about Relocatable Code Segments. Upon resolution of their discussion, both should contact Ken B. and relay the outcome of their conversation - and what, if any, changes to the DCE FSW are being recommended as a result. |
Daniel Blackman | OPEN | |
57 | 7/27/2000 | Will Clement | Add a LABEL next to every SUBROUTINE name, which has the DM-05 name for the LABEL. | Daniel Blackman | Daniel is approaching limits to the SYMBOL table information in the Archimedes development environment, so this scheme may not be possible. (Possible remedy involves response to AI #58) | OPEN |
58 | 7/27/2000 | Michelle Troeltzsch | Assigned to Will Clement: Examine the possibility of moving to a Franklin Compiler. Portion assigned to Will: Show that an identical HEX file can be created with the Franklin Compiler. |
Will Clement | OPEN | |
59 | 7/27/2000 | Michelle Troeltzsch | Remove all TABS from the code. Make sure no TABS are included in the source code. Suggestions: 1. Use a new editor... 2. Or, create a tool to remove the TABS, and include this tool in the development process. |
Daniel Blackman | OPEN | |
60 | 7/27/2000 | Michelle Troeltzsch | Make sure that all Interrupts that are not currently being used return an HST error if they are inadvertently called. e.g., vT1_ISR, vSER_ISR --- Chris feels that this represents an existing requirement to produce an HST Error when we are executing a section of code unexpectedly. Michelle wants to know which interrupts are occurring, and how many. So make sure we report which ones... and how many you get of them... Requirement: 1. Each different type of unexpected interrupt needs to be uniquely identified and reported. 2. Also, the number of interrupts of each type needs to be identified and maintained in memory. |
Daniel Blackman | OPEN | |
61 | 7/27/2000 | Allison Elliot | Assigned to Ken B.: Provide Art the most recent DCE TLM database. Send it to him in PDF Assigned to Art Rankin: Provide an evaluation of the DCE TLM items - and give suggestions as to which items you feel should and should not be part of the standard packet. Give to this list to Ken. Assigned to Geoff Gaines and John Vallerga: Based on Arts Recommendations, create a list of the FUV DET HSK items that will be telemeter to the Ground as part of the normal HSK packet, and what rate these items need to be updated. |
Art Rankin | Ken B. provided recent DCE TLM items to Art Rankin. Action is currently awaiting Arts recommendations. | OPEN |
63 | 7/27/2000 | Ken Brownsberger | Assigned to Allison: Make sure the following following DCE commands in DM-05, appendix B are corrected: LFMCOPY should be LFDCOPY LFDRSETP should be LFDRSTP LFDRSETW should be LFDRSTW LFMCRC should be LFDCRC LFGSHF should be LFGSHFT LFRMPWR OPCODE should be DA, not AD LFGBWK parameter 3 should be DIR, not empty LFGEWK parameter 3 should be DIR, not empty LFHVOFF should be removed. |
Allison Elliot | OPEN | |
65 | 7/28/2000 | Art Rankin | Possibly implement a BOOT Table // Operate Table approach - possibly as a conditional compile - for the BOOT and Operate commands. | Daniel Blackman | OPEN | |
66 | 7/28/2000 | Will Clement | Recommendation that the CLR A be removed at line 2686 (V17055) within NO_OPER. | Daniel Blackman | OPEN | |
67 | 7/28/2000 | Michelle Troeltzsch | The text for the HKPFULL command needs to be updated to include the text that it is only used to send a HSK packet. It is basically a NO-OP that increments the CMD counters. The text for the NO_OPER command needs to be updated to include text that says it sets a bit so that the CMD REC and CMD EXC counters are not incremented in the CMD Processing Task. As a start, include the headers from the Command Macro Sheets (from DM-05). |
Daniel Blackman | OPEN | |
68 | 7/28/2000 | Allison Elliot | Correct the logic in the start of the UPLOAD command, so that the CRC checking is done properly. i.e., If the CRC parameter is zero - then you ignore the CRC check to decide if the code should be uploaded or not. This CMD is not COS ready and should be re-reviewed when the command logic matches the COS requirement for the UPLOAD command. |
Daniel Blackman | OPEN | |
69 | 7/28/2000 | Michelle Troeltzsch | Michelle wants ZAP_TS30 moved to the COMMANDS.LST file - so its easier for her to find for maintenance and problem solving. Also change the name from ZAP_TS30 to LFDDIAGC, as the name is not used elsewhere. Also, remove the comment that says informally so far... as it no longer applies. --- Michelle wants the inputs and outputs to include anything that a subroutine reads from or writes to... e.g., ZAP_TS30 should include TS30, tDIAG0, tDIAG1, etc... |
Daniel Blackman | OPEN | |
70 | 7/28/2000 | Art Rankin | Assigned to Allison Elliot: Update DM-05 for LFDDIAGC to note that there is a risk of missing an unreported diagnostic during the interval that it takes for this command to execute. |
Allison Elliot | OPEN | |
71 | 7/28/2000 | Allison Elliot | In the command DUMP: Update the comments for StartAddr and Endaddr to refer to Length - as this is how the code works. |
Daniel Blackman | OPEN | |
73 | 7/28/2000 | Daniel Blackman | Add a range check for DUMP length so that if the length is zero, then a diagnostic is issued. i.e., Do not allow a parameter of zero. | Daniel Blackman | OPEN | |
74 | 7/28/2000 | Allison Elliot | DUMP code seems to indicate that you do not, in fact, compute a CRC as part of the code - which you are required to do. Correct the code to compute the CRC. Rearrange the code - which is almost correct, but in the wrong order. (See code surround lines 5226 in COMMANDS.LST (V17055)) |
Daniel Blackman | OPEN | |
75 | 7/28/2000 | Michelle Troeltzsch | Update the comments for SENDHKP (Line 3660 of EXECUTIV.LST (V17055)), so that it clearly states that two bytes are being sent across the interface. | Daniel Blackman | OPEN | |
76 | 7/28/2000 | Will Clement | Clear the SPECIALF bit before you call the TABLE_COPY - so that you are guaranteed that the bit has the correct value for the TABLE_COPY execution. | Daniel Blackman | OPEN | |
77 | 7/28/2000 | Michelle Troeltzsch | The subroutine FROMCODE (including FROMC2DE) should be moved into BYTE_MOVE. (Near line 3411, INTERNAL.LST (V17055)). Also, when moving FROMCODE and FROMC2DE - change the AJMP to a JMP. |
Daniel Blackman | OPEN | |
78 | 7/28/2000 | Michelle Troeltzsch | Consider moving to a more appropriate location, the PUSHDPTR, PUSH DPL, PUSH DPH and POPDPTR, POP DPH, and POP DPL in the LCOPY subroutine. Or, conversely, leave them here and add additional comments to explain why this should be done here. |
Daniel Blackman | OPEN | |
79 | 7/28/2000 | Allison Elliot | Correct Subroutine banner and the descriptive text. (The banner is associated with MSG_JMPG, not JMPCODE.) | Daniel Blackman | OPEN | |
80 | 7/28/2000 | Allison Elliot | Assigned to Ken B. Change the DCE SDD for LFDJMPCS: If in BOOT, do nothing (NOT jump to the predefined entry point of boot.) |
Ken Brownsberger | OPEN | |
81 | 7/28/2000 | Michelle Troeltzsch | Update the comments in ZAPCODECRC so that you are explaining why you suppress the CRC DIAGNOSTICS in this case. |
Daniel Blackman | OPEN | |
82 | 7/28/2000 | Art Rankin | Implement a system - tied to the activity of the CRC checking process - so that we can disable getting an HST Error. And that we can turn it on back on whenever we want. | Daniel Blackman | OPEN | |
83 | 7/28/2000 | Michelle Troeltzsch | In REPORT_DIAGNOSTIC, make sure you implement the scheme whereby not ALL diagnostics result in HST errors. | Daniel Blackman | OPEN | |
84 | 7/28/2000 | Michelle Troeltzsch | In JUSTTOGL, add information that you are incrementing the command executed counter because you are jumping code space, and the usual method of incrementing the command executed counter will be skipped. **** IGNORED **** This incrementing will take place in the LOCAL_INT, but only if the CMD_ERR is set. This has yet to be implemented. |
Daniel Blackman | OPEN | |
85 | 7/28/2000 | Michelle Troeltzsch | In JUSTTOGL, (line 4986 of V17055), describe the importance of the the #0B0h address in the JUMP CODE process. e.g., This corresponds to the loader parameter file, and is either B0 or F0. B0 - Set up reentry to lower code area. F0 - Set up reentry to upper code area. |
Daniel Blackman | OPEN | |
86 | 7/28/2000 | Art Rankin | Assigned to Allison: Talk to Art Rankin and capture his thoughts on how to enhance verbiage in DM-05 for LFDMADDR so that a user has a clear understanding how to set parameters for the command so that they can monitor the memory areas they're interested in... |
Allison Elliot | OPEN | |
87 | 7/28/2000 | Michelle Troeltzsch | For the Memory Monitor Command (LFDMADDR), make sure that you describe in your comments how precedence in the processing of CODE/DATA/INTERNAL/EXTERNAL parameters, results in the proper bits being reflected in housekeeping. |
Daniel Blackman | OPEN | |
88 | 8/1/2000 | Michelle Troeltzsch | Remove AJMP SOFT_RESET at line 2019 of COMMANDS.LST (V17055). This could lead to an infinite loop. Second, change the DIAGNOSTIC at 2022 of COMMANDS.LST (V17055), from Diagnostic 1 (Bad upload length) to a new Diagnostic - e.g., 31 - Reset Command Failed. Third, add the SOFT_RESET to the subroutine banner. |
Daniel Blackman | The changes Daniel has made are fine. Michelle requests that the comments be enhanced to show a calculation for the length of the delay in the second loop. Also, in the Banner, the names LFDRSTP and LFDRSTW need to be corrected. (The E needs to be dropped.) |
OPEN |
89 | 8/1/2000 | Will Clement | Resolve the difference in the number of parameters for SAA_TRIGD MACRO (line 2719 of EXECUTIV.LST V17055) - with the calls to SAA_TRIGD (lines 2895 and 3046 of EXECUTIV.LST V17055.) | Daniel Blackman | OPEN | |
90 | 8/3/2000 | Michelle Troeltzsch | Assigned to Allison Elliot: Update DM-05 for LFDUPLOD... Here is the change: If we pass a CRC parameter of 0, then the HSK item LFMXFER will remain the same as what was previously in HSK. The LFDUPLOD command will NOT update its value. |
Allison Elliot | OPEN | |
91 | 8/3/2000 | Steve Nissan | Add WD_DISABLE and WD_ENABLE to the Subroutine WD_STATE, as they are called from other areas of the code. (e.g., The SETPROJECT subroutine and the Initialization Sequence.) | Daniel Blackman | OPEN | |
92 | 8/3/2000 | Ken Brownsberger | Lines 2774 - 2786 of COMMANDS.LST (V17055) 1. Make sure there is only one return point for the WD_STATE (and subsequent) subroutines. 2. Create a WD_EXIT label for this return. 3. Change the label WD_ENABLE to WD_INIT. 4. Create a new label, WD_ENABLE, which contains the MOV CCAPM4, #48h (line 2783) functionality - so we could call this subroutine for testing purposes. 5. Make sure that the call to WD_ENABLE in SET_PROJECT, is changed to the new label WD_INIT. |
Daniel Blackman | OPEN | |
93 | 8/8/2000 | Ken Brownsberger | 1. Update the comments in the PATCHCST.LST (Line 487 of V17088) to define the memory regions for COS. e.g. There is no MASK A or MASK B regions. 2. Describe how the various regions are linked together. 3. Describe how the computed CRC values for the differnet regions are reported in TLM. |
Daniel Blackman | OPEN | |
94 | 8/8/2000 | Michelle Troeltzsch | Explain in the CALC_CRC banner description, how this command hijacks the background CRC - computes the region specified, then returns control to the background CRC task. | Daniel Blackman | OPEN | |
95 | 8/10/2000 | Michelle Troeltzsch | In CALC_CRC, put a comment describing where the RET at Line 3008 of COMMANDS.LST (in INIT_REGION:) takes us. During todays code walk-thru you showed us that the RET takes us back up to just after the call of COMMAND_VECTOR (at Line 6445.) Note: Line Numbers refer to V17088. |
Daniel Blackman | OPEN | |
96 | 8/10/2000 | Allison Elliot | For the CALC_CRC task, make sure you follow the DM-05 (and DCE SDD) implementation by zeroing out the calculated CRC value reported in HSK (i.e., LFMXFER, but see AI#99) at the start of process. As per the interface agreement, the CS will be monitoring the HSK value - and will use the change of this value from zero to a new value as an indication that the commanded CRC has completed. | Daniel Blackman | OPEN | |
97 | 8/10/2000 | Will Clement | There are two SAVE_CRC subroutine labels in the V17088 code - one in INTERNAL.LST (Line 3223) and one in COMMANDS.LST (Line 5972). Either consollidate to one routine, or rename one of them. If you consolidate the routines, and remove the one found in COMMANDS.lST, make sure you populate the DPTR before you call SAVE_CRC. |
Daniel Blackman | Per verbal discussion during 8/10/2000 telecon, Daniel will probably conslidate them into a single subroutine by removing the one in COMMANDS.LST | OPEN |
98 | 8/10/2000 | Michelle Troeltzsch | Remove the variable tSAVE_CRC and just use tDMSAV. Note: This variable is the used for the TLM identifier LFMXFER (although see name discrepancy in AI #99), and is used for the computed CRC value for both the UPLOAD and the CALC_CRC commands. | Daniel Blackman | OPEN | |
99 | 8/10/2000 | Allison Elliot | Figure out what youre going to call the TLM item which provides the calculated CRC in HSK (This is the TLM used by both the UPLOAD command, and the CALC_CRC command. ) Currently there is a discrepancy in the various documentation sources. For example, your HSK TLM list says in the FUV ICD says it is LFMCRC. This is also what is contained in the UCB Command and Telemetry database. However, within the Diagnostic list of the FUV Detector, it says the TLM item is LFMXFER. Note: DM-05 says LFMXFER, and the DCE SDD says LFMXFER. Thus, it would be preferrable if you chose to call it LFMXFER. |
Geoff Gaines | OPEN |