HD-DEN-14001_HD Battery Driver Update

Activity

HD-DEN-14001-1 58

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 32m 1 OK, I see what you wanted. I have a couple of questions: ...
    Reviewer - Complete 1h 24m 35 RESOLVED in CODE WALKTHROUGH.
    wbracken  (deleted user)
    Reviewer completed
    Reviewer - Complete 1h 1m 14 RESOLVED IN CODE WALKTHROUGH
    Reviewer - 87% reviewed 36m 6 Done.
    Reviewer - 0% reviewed 12m 2 Did we lose 1 second of time or is this the 2nd alarm? Co...
    Total   3h 47m 58  
    #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.
    /AlarmDefs.h Changed 4
    /MsgDefs.h Changed
    Open in IDE #permalink
    /firmware/App/Controllers/DialInFlow.c Changed 2
    Open in IDE #permalink
    /firmware/App/Drivers/Battery.c Changed 47
    /firmware/App/Drivers/Battery.h Changed 3
    /firmware/App/Services/SystemComm.c Changed
    Open in IDE #permalink
    /firmware/.../Services/SystemCommMessages.c Changed
    Open in IDE #permalink
    /firmware/App/Tasks/TaskBG.c Changed 2

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time