FpgaTD.h

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
move static function after public function

move static function after public function

let public function on top

let public function on top

Let all public functions come on top for all files

Let all public functions come on top for all files

Should we comment this out before merging?

Should we comment this out before merging?

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

  1. … 7 more files in changeset.
Merge branch 'LDT-1886-blood-set-auto-load-and-auto-eject-implement-ejector-sensor' into develop

  1. … 7 more files in changeset.
Done

Done

Done

Done

Align =

Align =

Should be 2 blank lines above/below test banner.

Should be 2 blank lines above/below test banner.

LDT-1886 updated the pinch valves homing sequence

  1. … 5 more files in changeset.
Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

  1. … 95 more files in changeset.
Done

Done

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

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

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

  1. … 90 more files in changeset.
Took note that the FPGA interface is at the driver level.

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

Done

Done

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

  1. … 7 more files in changeset.
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.

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

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

Why bypassing driver here?

Why bypassing driver here?

Why are we bypassing driver here?

Why are we bypassing driver here?

LDT-1886 added the ejector optical sensors

  1. … 4 more files in changeset.
Then your doxygen return comment looks backward.

Then your doxygen return comment looks backward.

Why is controller bypassing driver to get to fpga?

Why is controller bypassing driver to get to fpga?

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.

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.

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
LDT-1886 implemented the ejector optical sensor

  1. … 2 more files in changeset.