td-firmware / firmware / App / Controllers

Ejector.c

Clone Tools

Stats

Commits this week: 3
Total Committers: 2
Last Commit: 11 Dec
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
Thursday 11 Dec
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
9:48 am

Dara Navaei

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

Options
    • -1
    • +1
    ./Ejector.c
  2. … 3 more files in changeset.
9:45 am

Dara Navaei

Merge branch 'LDT-1886-blood-set-auto-load-and-auto-eject-implement-ejector-sensor' into develop

Options
    • -2
    • +2
    ./Ejector.c
  2. … 3 more files in changeset.
Friday 05 Dec
2:55 pm

Sean Nash

Should we comment this out before merging?

Should we comment this out before merging?

2:11 pm

Sean Nash

2:10 pm

Dara Navaei

LDT-1886 updated the ejector and valves modules after testing

Options
    • -2
    • +2
    ./Ejector.c
  2. … 2 more files in changeset.
Tuesday 02 Dec
9:50 am

Dara Navaei

Done

Done

9:49 am

Dara Navaei

Done

Done

9:16 am

Sean Nash

Align =

Align =

9:14 am

Sean Nash

Should be 2 blank lines above/below test banner.

Should be 2 blank lines above/below test banner.

Monday 01 Dec
9:44 am

Dara Navaei

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

Options
    • -4
    • +4
    ./Ejector.c
  2. … 95 more files in changeset.
9:43 am

Dara Navaei

Done

Done

9:11 am

Sean Nash

Should we initialize to inactive (and use #define)?

Should we initialize to inactive (and use #define)?

Wednesday 26 Nov
8:35 pm

Dara Navaei

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

Options
    • -4
    • +4
    ./Ejector.c
  2. … 90 more files in changeset.
Tuesday 25 Nov
4:13 pm

Dara Navaei

LDT-1886 added more comments

Options
    • -0
    • +2
    ./Ejector.c
4:01 pm

Dara Navaei

Took note that the FPGA interface is at the driver level.

Took note that the FPGA interface is at the driver level.

4:01 pm

Dara Navaei

Done

Done

3:50 pm

Dara Navaei

LDT-1886 updated the ejectors code. Transferred the FPGA interface to the ejector motor level.

Options
    • -58
    • +4
    ./Ejector.c
  2. … 7 more files in changeset.
1:56 pm

Sean Nash

Should this be an enum at driver level? Driver get would return enum so controller doesn't have to figure out what it means.

Should this be an enum at driver level? Driver get would return enum so controller doesn't have to figure out what it means.

1:48 pm

Sean Nash

Use U32 unless there is a good reason to use U08.

Use U32 unless there is a good reason to use U08.

1:46 pm

Sean Nash

Why bypassing driver here?

Why bypassing driver here?

1:45 pm

Sean Nash

Why are we bypassing driver here?

Why are we bypassing driver here?

1:36 pm

Dara Navaei

LDT-1886 added the ejector optical sensors

Options
    • -8
    • +63
    ./Ejector.c
  2. … 4 more files in changeset.
1:24 pm

Sean Nash

Then your doxygen return comment looks backward.

Then your doxygen return comment looks backward.

1:23 pm

Sean Nash

Why is controller bypassing driver to get to fpga?

Why is controller bypassing driver to get to fpga?

11:10 am

Dara Navaei

According to the HDD, it is a '1' when inactive and becomes a '0' when active. It becomes active when it reaches the position.

According to the HDD, it is a '1' when inactive and becomes a '0' when active. It becomes active when it reaches the position.

9:25 am

Sean Nash

Seems backward. Why is inactive = TRUE? I would expect TRUE to indicate that the optical sensor is detecting home (retracted) position.

Seems backward. Why is inactive = TRUE? I would expect TRUE to indicate that the optical sensor is detecting home (retracted) position.

4:20 am

Dara Navaei

LEAHI-TD-FIRMWARE-LDT-1886_Blood Set Auto-Load and Auto-Eject - TD
LEAHI-TD-FIRMWARE-LDT-1886_Blood Set Auto-Load and Auto-Eject - TD