td-firmware / firmware / App / Controllers

BPModule.h

Clone Tools

Stats

Commits this week: 3
Total Committers: 1
Last Commit: 17:37
Commits this week: 3

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
Today
3 hrs ago

Vendor - TEL - Varshini Nagabooshanam

Added

Added

3 hrs ago

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

3 hrs ago

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

3 hrs ago

Vendor - TEL - Varshini Nagabooshanam

Yes, removed

Yes, removed

3 hrs ago

Vendor - TEL - Varshini Nagabooshanam

Yes, moved to different place

Yes, moved to different place

3 hrs ago

Vendor - TEL - Varshini Nagabooshanam

Yes, updated

Yes, updated

3 hrs ago

Vendor - TEL - Varshini Nagabooshanam

Used in handle function

Used in handle function

3 hrs ago

Vendor - TEL - Varshini Nagabooshanam

added

added

3 hrs ago

Vendor - TEL - Varshini Nagabooshanam

Yes added

Yes added

5 hrs ago

Sean Nash

SW fault here?

SW fault here?

5 hrs ago

Sean Nash

Do we need this state if all it's doing is going to IDLE?

Do we need this state if all it's doing is going to IDLE?

5 hrs ago

Sean Nash

Might have to wait for module to not be busy before getting the data.

Might have to wait for module to not be busy before getting the data.

5 hrs ago

Sean Nash

I think we may need to give a get data command here.

I think we may need to give a get data command here.

5 hrs ago

Sean Nash

Look for request flag to give measure command and transition to measure state.

Look for request flag to give measure command and transition to measure state.

5 hrs ago

Sean Nash

Write private handle functions for each state and call them in each case.

Write private handle functions for each state and call them in each case.

5 hrs ago

Sean Nash

Should we also check whether bpMeasurementReady is TRUE?

Should we also check whether bpMeasurementReady is TRUE?

5 hrs ago

Sean Nash

Same.

Same.

5 hrs ago

Sean Nash

Same. Set a request flag to TRUE (for Peds this time) and let the state machine do the rest.

Same. Set a request flag to TRUE (for Peds this time) and let the state machine do the rest.

5 hrs ago

Sean Nash

Don't change state from outside signal. We should just set a requestBPMeasurement flag to TRUE in this function (if current state is IDLE) and then when the state machine runs next, the IDLE state ...

Don't change state from outside signal. We should just set a requestBPMeasurement flag to TRUE in this function (if current state is IDLE) and then when the state machine runs next, the IDLE state should look for this flag to be TRUE and transition to MEASURE state (by doing all of these things you have here in this function).

6 hrs ago

Sean Nash

Change comment to "Number of BP driver states."

Change comment to "Number of BP driver states."

6 hrs ago

Vendor - TEL - Varshini Nagabooshanam

Not really, I can change this to private

Not really, I can change this to private

6 hrs ago

Sean Nash

Add a 5th enum for NUM_OF_BP_DRIVER_STATES. And, does this need to be public?

Add a 5th enum for NUM_OF_BP_DRIVER_STATES.
And, does this need to be public?

7 hrs ago

Sean Nash

if syrPumpDACControl is U16 (2 bytes), then nibpCtl should be at 42.

if syrPumpDACControl is U16 (2 bytes), then nibpCtl should be at 42.

7 hrs ago

Sean Nash

Remove extra blank line.

Remove extra blank line.

Monday 18 May
11:44 am

Sean Nash

10:10 am

Vendor - TEL - Varshini Nagabooshanam

Yes Noe said to replace with new register

Yes Noe said to replace with new register

10:09 am

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

10:09 am

Vendor - TEL - Varshini Nagabooshanam

Sure

Sure

Sunday 17 May
10:38 pm

Sean Nash

If we don't set this pwm time anymore, we don't need this. Please make sure we really don't need this anymore though.

If we don't set this pwm time anymore, we don't need this. Please make sure we really don't need this anymore though.

10:37 pm

Sean Nash

I don't think these definitions need to be public. Some of these should probably be inside the FpgaTD.c and some should probably be inside your driver's .c file.

I don't think these definitions need to be public. Some of these should probably be inside the FpgaTD.c and some should probably be inside your driver's .c file.