td-firmware

Clone Tools

Stats

Commits this week: 43
Total Committers: 16
Last Commit: 21 May
Commits this week: 43

Lines of code count not available

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 21 May
10:56 pm

Vendor - TEL - Vijay Pamula

LDT:3970 Addressed Review Comments and Updated Edit Treatment Parameters

Options
    • -0
    • +17
    /firmware/App/Modes/ModeTreatment.c
    • -0
    • +1
    /firmware/App/Modes/ModeTreatment.h
    • -1
    • +1
    /firmware/App/Monitors/Pressures.c
    • -3
    • +3
    /firmware/App/Services/Messaging.c
    • -7
    • +0
    /firmware/App/Services/Messaging.h
    • -40
    • +29
    /firmware/App/Services/TxParams.c
    • -30
    • +10
    /firmware/App/Services/TxParams.h
10:46 pm

Sean Nash

Why do you have {} in the cases?

Why do you have {} in the cases?

10:45 pm

Sean Nash

You are getting bpResults, but not doing anything with them. Should you pass as parameter to checkBloodPressureReading() function?

You are getting bpResults, but not doing anything with them. Should you pass as parameter to checkBloodPressureReading() function?

10:43 pm

Sean Nash

Where are these get functions defined? I can't find them.

Where are these get functions defined? I can't find them.

8:46 pm

Vendor - TEL - Praneeth Bunne

LDT-1886: Resolve merge conflicts

Options
    • -7
    • +11
    /firmware/App/Services/AlarmMgmtSWFaults.h
    • -2
    • +16
    /firmware/App/Services/Messaging.c
8:46 pm

Dara Navaei

LDT-3259 merged staging

Options
    • -2
    • +6
    /firmware/App/Services/AlarmMgmtSWFaults.h
    • -2
    • +16
    /firmware/App/Services/Messaging.c
    • -0
    • +2
    /firmware/source/sys_main.c
8:42 pm

Sean Nash and Vendor - TEL - Arpita Srivastava

4:35 pm

Vendor - TEL - Varshini Nagabooshanam

LDT-4033-patient-vitals-fw-implementation

Options
    • -164
    • +81
    /firmware/App/Controllers/BPModule.c
    • -7
    • +0
    /firmware/App/Controllers/BPModule.h
    • -64
    • +81
    /firmware/App/Drivers/BPDriver.c
    • -1
    • +0
    /firmware/App/Drivers/GLXferPump.c
    • -0
    • +4
    /firmware/App/Modes/ModeTreatment.c
    • -0
    • +2
    /firmware/App/Services/AlarmMgmtSWFaults.h
    • -2
    • +2
    /firmware/App/Services/FpgaTD.c
    • -1
    • +0
    /firmware/App/Services/FpgaTD.h
4:27 pm

Vendor - TEL - Arpita Srivastava

3:23 pm

Dara Navaei

Bamboo Commit: Updated TDCommon.h with build versions from Bamboo.

Options
    • -1
    • +1
    /firmware/App/TDCommon.h
3:22 pm

Dara Navaei

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

Options
    • -4
    • +4
    /firmware/App/Controllers/SyringePump.c
    • -4
    • +4
    /firmware/App/Controllers/SyringePump.h
    • -2
    • +2
    /firmware/App/Modes/ModeFault.c
    • -2
    • +2
    /firmware/App/Modes/ModeService.c
    • -2
    • +2
    /firmware/App/Modes/ModeStandby.c
    • -2
    • +2
    /firmware/App/Modes/ModeTreatment.c
    • -2
    • +2
    /firmware/App/Services/AlarmMgmtSWFaults.h
    • -2
    • +2
    /firmware/App/Services/FpgaTD.c
    • -2
    • +2
    /firmware/App/Services/FpgaTD.h
    • -2
    • +2
    /firmware/App/Services/Messaging.c
    • -2
    • +2
    /firmware/App/Tasks/TaskPriority.c
3:06 pm

Vendor - TEL - Jashwant Gantyada and Sean Nash

1:21 pm

suresh

LDT-4072: code comment updates

Options
    • -20
    • +34
    /firmware/App/Drivers/BatteryDriver.c
    • -1
    • +27
    /firmware/App/Drivers/BatteryDriver.h
    • -38
    • +26
    /firmware/App/Monitors/Battery.c
    • -25
    • +0
    /firmware/App/Monitors/Battery.h
12:11 pm

Vendor - TEL - Varshini Nagabooshanam

Removed

Removed

12:02 pm

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

11:59 am

Dara Navaei

Bamboo Commit: Updated TDCommon.h with build versions from Bamboo.

Options
    • -1
    • +1
    /firmware/App/TDCommon.h
11:58 am

Vendor - TEL - Varshini Nagabooshanam

LDT-3215-measured-venous-pressure-decreases

Options
    • -0
    • +5
    /firmware/App/Controllers/AirPump.c
    • -1
    • +1
    /firmware/App/Services/Messaging.c
11:28 am

Sean Nash

Do we not need the module error alarm?

Do we not need the module error alarm?

11:27 am

Sean Nash

This is not an atomic assignment, so we have to consider thread safety. If this driver is running from General Task (I believe it is since ModeTreatment's exec will be calling the BPModule's exec w...

This is not an atomic assignment, so we have to consider thread safety.
If this driver is running from General Task (I believe it is since ModeTreatment's exec will be calling the BPModule's exec which calls this driver's exec) and if the caller to this function will also be running from General Task (I believe it is since BPModule controller is the caller), then there is no thread safety issue and we should mention in the function header brief that this function should only be called from within the General Task.
If another task is involved (e.g. Priority Task is calling this function), this function could be interrupting the get data state while it is populating bpResults with new results and the caller would get a mix of old and new results (we would not want that).

11:23 am

Sean Nash and Vendor - TEL - Jashwant Gantyada

11:22 am

Vendor - TEL - Varshini Nagabooshanam

LDT-3215-measured-venous-pressure-decreases

Options
    • -10
    • +39
    /firmware/App/Controllers/AirPump.c
    • -4
    • +8
    /firmware/App/Controllers/AirPump.h
    • -6
    • +14
    /firmware/App/Drivers/GLXferPump.h
    • -3
    • +28
    /firmware/App/Services/FpgaTD.c
    • -2
    • +4
    /firmware/App/Services/FpgaTD.h
11:21 am

Sean Nash

same.

same.

11:20 am

Sean Nash

same.

same.

11:20 am

Sean Nash

Just like driver state machine, we should only set a request flag to TRUE here and let the state machine handle the transition.

Just like driver state machine, we should only set a request flag to TRUE here and let the state machine handle the transition.

11:19 am

Sean Nash

How do we get out of Idle state?

How do we get out of Idle state?

11:18 am

Sean Nash

This should be called in all states (so in controller's exec function).

This should be called in all states (so in controller's exec function).

11:17 am

Sean Nash

I don't think we want to zero this count here.

I don't think we want to zero this count here.

11:16 am

Sean Nash

We need to call the driver exec function from this exec function - either before or after the switch statement.

We need to call the driver exec function from this exec function - either before or after the switch statement.

11:16 am

Vendor - TEL - Praneeth Bunne

LDT-1886: Update event related code

Options
    • -10
    • +11
    /firmware/App/Services/StateServices/TubeSetAutoEject.c
    • -10
    • +11
    /firmware/App/Services/StateServices/TubeSetInstall.c
11:10 am

Sean Nash

We need to call this from ModeTreatment transition function.

We need to call this from ModeTreatment transition function.