dd-firmware

Clone Tools

Stats

Commits this week: 22
Total Committers: 17
Last Commit: 05 Jun
Commits this week: 22

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 seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Friday 06 Feb
10:01 am

Raghu Kallala

LDT-3344 added function to initialize FPGA registers with every setConfig call

Options
    • -2
    • +56
    /firmware/App/Monitors/Level.c
    • -11
    • +25
    /firmware/App/Services/FpgaDD.c
    • -0
    • +1
    /firmware/App/Services/FpgaDD.h
9:26 am

Vinayakam Mani

I don't think we need to check test config for variable initialization.

I don't think we need to check test config for variable initialization.

9:13 am

Vendor - TEL - Sameer Poyil

fixed it

fixed it

9:10 am

Vendor - TEL - Sameer Poyil

Fixed it

Fixed it

9:04 am

Vendor - TEL - Sameer Poyil

fixed it

fixed it

9:04 am

Vendor - TEL - Sameer Poyil

fixed it

fixed it

9:03 am

Vendor - TEL - Sameer Poyil

fixed it

fixed it

9:03 am

Vendor - TEL - Sameer Poyil

fixed it

fixed it

9:01 am

Vendor - TEL - Sameer Poyil

fixed it

fixed it

8:58 am

Vendor - TEL - Sameer Poyil

LDT-2004-3 dry bicart review comment implementation

Options
    • -26
    • +37
    /firmware/App/Controllers/BalancingChamber.c
    • -103
    • +144
    /firmware/App/Controllers/DryBiCart.c
    • -3
    • +0
    /firmware/App/DDCommon.h
    • -21
    • +33
    /firmware/App/Modes/ModeGenDialysate.c
    • -10
    • +10
    /firmware/App/Modes/ModePreGenDialysate.c
    • -2
    • +5
    /firmware/App/Tasks/TaskGeneral.c
Thursday 05 Feb
9:14 pm

chaithra bs <ven.cburudugatta@diality.com>

LDT-3223-dialysate-flow-rate---dd---mode-tel-dev-test : peer review findings addressed for ConcentratePumps file

Options
7:27 pm

Vendor - TEL - Sameer Poyil

Since fill and drain is a single time request and it is not a frequent request similar to supply during dialysate generation. I removed and now run as idle

Since fill and drain is a single time request and it is not a frequent request similar to supply during dialysate generation. I removed and now run as idle

7:24 pm

Vendor - TEL - Sameer Poyil

To make it exclusive operation. no previous request should be pending when a new operation is requested. So only one state machine is running at a time. otherwise, while chamber fill is in progress...

To make it exclusive operation. no previous request should be pending when a new operation is requested. So only one state machine is running at a time. otherwise, while chamber fill is in progress, anyone can request drain and both requests will be handled. even through new code run only one state machine at a time, this will ensure that only one request is submitted to drybicart

This was a concern reported by Sean and proposed for restructuring into a single state machine.

3:46 pm

Raghu Kallala

LDT-3344 reverted valve changes

Options
    • -1
    • +1
    /firmware/App/Controllers/Valves.c
3:46 pm

Dara Navaei

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

Options
    • -1
    • +1
    /firmware/App/DDCommon.h
3:44 pm

Dara Navaei

Merged staging

Options
    • -2
    • +43
    /firmware/App/Controllers/Valves.c
    • -1
    • +11
    /firmware/App/Controllers/Valves.h
    • -4
    • +15
    /firmware/App/Modes/ModeStandby.c
    • -2
    • +3
    /firmware/App/Services/AlarmMgmtSWFaults.h
    • -3
    • +3
    /firmware/App/Services/TDInterface.c
3:41 pm

Raghu Kallala

LDT-3344 latest from staging

Options
    • -2
    • +43
    /firmware/App/Controllers/Valves.c
3:19 pm

Vendor - TEL - Jashwant Gantyada

LDT-3344 changes made to registers

Options
    • -1
    • +84
    /firmware/App/Services/FpgaDD.c
2:52 pm

Vendor - TEL - Jashwant Gantyada

LDT-3344-changed registers according to beta 2 hdd

Options
    • -1
    • +1
    /firmware/App/Controllers/Valves.c
    • -3
    • +175
    /firmware/App/Services/FpgaDD.c
2:17 pm

Dara Navaei

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

Raghu Kallala

LDT-3344 fixed set P40 boostpump pwm call

Options
    • -1
    • +3
    /firmware/App/Drivers/FluidPump.c
1:51 pm

Vendor - TEL - Arpita Srivastava

LDT-3222: Removed local variable initialization for testBCValveStatesOverride

Options
    • -3
    • +3
    /firmware/App/Controllers/Valves.c
1:01 pm

Sean Nash

This doesn't look like a valid substate. We should have an enum for valid substates. In default case, we should trigger a s/w fault and also set the state to a valid state (idle or something like t...

This doesn't look like a valid substate. We should have an enum for valid substates. In default case, we should trigger a s/w fault and also set the state to a valid state (idle or something like that).

12:53 pm

Sean Nash

I think /// was correct. // will not be seen by doxygen.

I think /// was correct. // will not be seen by doxygen.

11:03 am

Vinayakam Mani

Not clear, why we are checking fill and supply request here?

Not clear, why we are checking fill and supply request here?

10:24 am

Vinayakam Mani

we should use the functions defined in testsupport.c/h -> getU32OverrideValue, getF32OverrideValue based on data type. this needs to be fixed throughout the file.

we should use the functions defined in testsupport.c/h -> getU32OverrideValue, getF32OverrideValue based on data type. this needs to be fixed throughout the file.

10:19 am

Vinayakam Mani

space after if. please fix it throughout the files.

space after if. please fix it throughout the files.

10:17 am

Vinayakam Mani

I guess, Fill and Drain are priority items than the Supply request, which means upon request of fill and drain, state machine should service that operation. Also, setting supply request should be ...

I guess, Fill and Drain are priority items than the Supply request, which means upon request of fill and drain, state machine should service that operation.

Also, setting supply request should be based on the level sensor later (there shouldn't be any default request -> currently supply is assigned)

9:59 am

Vinayakam Mani

Please introduce test config for drybicart selection.

Please introduce test config for drybicart selection.

9:42 am

Dara Navaei

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

Options
    • -1
    • +1
    /firmware/App/DDCommon.h