td-firmware

Clone Tools

Stats

Commits this week: 30
Total Committers: 8
Last Commit: 16:57
Commits this week: 30

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

Vendor - TEL - Varshini Nagabooshanam

Thanks for the clarification. This handler already follows the single-parameter MESSAGE_T *message pattern, and the message ID is already defined and registered in Messaging.c. I’ll keep using the ...

Thanks for the clarification. This handler already follows the single-parameter MESSAGE_T *message pattern, and the message ID is already defined and registered in Messaging.c. I’ll keep using the existing ID, ensure payload length is validated, and parse the param/value from the payload as expected.

6 hrs ago

Vendor - TEL - Varshini Nagabooshanam

Yes, this is called in OperationModes.c

Yes, this is called in OperationModes.c

6 hrs ago

Vendor - TEL - Varshini Nagabooshanam

Sure.

Sure.

6 hrs ago

Vendor - TEL - Varshini Nagabooshanam

Added.

Added.

6 hrs ago

Vendor - TEL - Varshini Nagabooshanam

Added.

Added.

6 hrs ago

Varshini Nagabooshanam <ven.vnagabooshanam@diality.com>

LDT-473-treatment-parameters-create-rx--

Options
    • -1
    • +2
    /firmware/App/Modes/ModeTxParams.c
    • -16
    • +23
    /firmware/App/Services/TxParams.c
    • -0
    • +9
    /firmware/App/Services/TxParams.h
9 hrs ago

Sean Nash

Not sure how Doxygen will handle this comment between the /// and the const. I think we should remove it.

Not sure how Doxygen will handle this comment between the /// and the const. I think we should remove it.

Sunday 14 Dec
7:26 pm

Dara Navaei

Done

Done

6:48 pm

Dara Navaei

Make ////< to ///< for doxygen to recognize it.

Make ////< to ///< for doxygen to recognize it.

Thursday 11 Dec
11:09 am

Sean Nash

10:15 am

Vendor - TEL - Sameer Poyil

move static function after public function

move static function after public function

10:14 am

Vendor - TEL - Sameer Poyil

let public function on top

let public function on top

10:11 am

Vendor - TEL - Sameer Poyil

Let all public functions come on top for all files

Let all public functions come on top for all files

Tuesday 09 Dec
4:13 pm

Sean Nash

4:13 pm

Sean Nash

3:18 pm

Dara Navaei

LEAHI-TD-FIRMWARE-LDT-2932_(DD, TD) TMP reported by Dialin/ FW is incorrect
LEAHI-TD-FIRMWARE-LDT-2932_(DD, TD) TMP reported by Dialin/ FW is incorrect
9:58 am

Sean Nash

I didn't see this in header file.

I didn't see this in header file.

9:57 am

Sean Nash

I don't see these Lower/Upper Range Limit functions declared in the header file.

I don't see these Lower/Upper Range Limit functions declared in the header file.

9:33 am

Sean Nash

Need to delete this unit (ModeTxParams.c/.h). Also need to remove the mode from the modes enumeration and from OperationModes.c.

Need to delete this unit (ModeTxParams.c/.h). Also need to remove the mode from the modes enumeration and from OperationModes.c.

9:32 am

Sean Nash

I don't think we need this signal function since this is not a mode anymore. Is this function being called somewhere?

I don't think we need this signal function since this is not a mode anymore. Is this function being called somewhere?

Monday 08 Dec
1:11 pm

Vendor - TEL - Varshini Nagabooshanam

Added

Added

1:11 pm

Vendor - TEL - Varshini Nagabooshanam

Added

Added

1:11 pm

Vendor - TEL - Varshini Nagabooshanam

Updated

Updated

1:09 pm

Varshini Nagabooshanam <ven.vnagabooshanam@diality.com>

LDT-473-treatment-parameters-create-rx--

Options
    • -215
    • +716
    /firmware/App/Services/TxParams.c
    • -12
    • +20
    /firmware/App/Services/TxParams.h
11:17 am

Sean Nash

I don't see an isTreatmentParamInRange function. I think we need to copy from the ModeTxParams.c.

I don't see an isTreatmentParamInRange function. I think we need to copy from the ModeTxParams.c.

11:16 am

Sean Nash

I don't see any get functions. I think we need to copy from ModeTxParams.c. Let's make sure we've copied all needed functions from ModeTxParams.c and then don't forget to delete ModeTxParams.c/.h.

I don't see any get functions. I think we need to copy from ModeTxParams.c.
Let's make sure we've copied all needed functions from ModeTxParams.c and then don't forget to delete ModeTxParams.c/.h.

11:14 am

Sean Nash

Missing return statement.

Missing return statement.

11:14 am

Sean Nash

Add a blank line between declaration and if statement.

Add a blank line between declaration and if statement.

10:58 am

Sean Nash

Message handler functions must have a single parameter (message). The payload of the message will have the param and value in it and this function needs to parse them out of the payload. Also need ...

Message handler functions must have a single parameter (message). The payload of the message will have the param and value in it and this function needs to parse them out of the payload. Also need to check payload length first.
We need a new message ID in MsgDefs.h for Dialin to send us this message and we need to add an entry in the message/handler table in Messaging.c.

10:53 am

Sean Nash

Add one more blank line above test support banner.

Add one more blank line above test support banner.