Skip to content

Heinz/can tx#8

Open
Heinzino wants to merge 38 commits into
masterfrom
Heinz/CanTx
Open

Heinz/can tx#8
Heinzino wants to merge 38 commits into
masterfrom
Heinz/CanTx

Conversation

@Heinzino
Copy link
Copy Markdown
Member

@Heinzino Heinzino commented Nov 30, 2024

CAN Communication for the following:

Lights Inputs
Digital Inputs
Analog Inputs
Lights Status

Assumptions:
-Pedal readings only return _P of the pedal for Analog Inputs
-Zoom Zoom corresponds to DriverControls::GREEN_LED ? There's no driver control zoom zoom

  • headlightsSwitch True if HEADLIGHTS_ENABLE is HIGH

This encoding for FORWARD_NEUTRAL_REVERSE_H and FORWARD_NEUTRAL_REVERSE_L

/**Forward and Reverse Encoding from Electrical Team */
 /* 10 forward 
 * 11 reverse
 * 00 TODO: CHECK ASSUMPTION THIS IS NEUTRAL
 * 01 not implemented 
 */

MacKante and others added 30 commits October 30, 2024 13:58
* init: SPI Task Object created

* feat: init logic to read SPI input

* fix: refactor to test debug and pass in hspi_

* fix: decouple pedal reading from print statement

* feat: Add dedicated SPI_TASK_DELAY

* fix: add while(1) loop in task

* fix: Nathan fixes. Read data scaled wrong

* fix: readData bit masking

* feat: print pedal percentage

---------

Co-authored-by: Heinzino <heinzveintimilla@hotmail.com>

/* ------------------------------- Namespace for Alternative IOPin names -------------------------------*/
namespace DriverControls {
constexpr IOPin FORWARD_NEUTRAL_REVERSE_H = IOPin::P00;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could also be an enum

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants