LEAHI-DD-FIRMWARE-LDT-3344_Firmware support for Beta 2 FPGA updates.

Activity

LEAHI-DD-FIRMWARE-LDT-3344-1 136

Keyboard shortcuts  
  • Summarize the review outcomes (optional)
     
    #permalink

    Details

    Warning: no files are visible, they have all been filtered.
    Participant Role Time Spent Comments Latest Comment
    Author & Moderator 5h 8m 46 removed.
    Reviewer - 19% reviewed 1h 33m 16 This is total mess up . that is what I want it a clean im...
    Reviewer - 0% reviewed 0m    
    Reviewer - 0% reviewed      
    Reviewer - Complete 4h 58m 15 the 2nd if statement for beta 1.0 is an ask from systems ...
    Reviewer - Complete 2h 44m 38 Noe says we can distinguish between beta 1.9 and 2.0 w/ I...
    Reviewer - 0% reviewed      
    Reviewer - Complete 5h 12m 21 Mask value seems wrong. please update per HDD.
    Reviewer - 0% reviewed 32m    
    Reviewer - 30% reviewed 19m    
    Reviewer - 0% reviewed      
    Total   20h 29m 136  
    #permalink

    Objectives

    There are no specific objectives for this review.

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    There are no general comments on this review.
    /firmware/App/Controllers/RinsePump.c Changed 40
    Open in IDE #permalink
    /firmware/App/Controllers/RinsePump.h Changed
    Open in IDE #permalink
    /firmware/App/Controllers/Valves.c Changed
    Open in IDE #permalink
    /firmware/App/Drivers/FluidPump.c Changed
    Open in IDE #permalink
    /firmware/App/Drivers/GPIO.c Changed 5
    Open in IDE #permalink
    /firmware/App/Drivers/GPIO.h Changed
    Open in IDE #permalink
    /firmware/.../FPModes/StateFlushConcentrate.c Changed
    Open in IDE #permalink
    /firmware/App/Monitors/Level.c Changed 17
    Open in IDE #permalink
    /firmware/App/Services/AlarmMgmtSWFaults.h Changed
    Open in IDE #permalink
    /firmware/App/Services/FpgaDD.c Changed 57
    Open in IDE #permalink
    /firmware/App/Services/FpgaDD.h Changed 2
    Open in IDE #permalink
    /firmware/App/Tasks/TaskPriority.c Changed
    Open in IDE #permalink
    /AlarmDefs.h Changed 6
    Open in IDE #permalink
    /DDDefs.h Changed 4
    Open in IDE #permalink
    /FPGA.c Changed
    Open in IDE #permalink
    /TestSupport.c Changed 5
    Open in IDE #permalink
    /TestSupport.h Changed
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time