Skip to content

Fixed order of nibbles for forward and regen AC currents in ECU status - #235

Merged
Shrav816 merged 1 commit into
mainfrom
ecu_regen_debugging
Oct 26, 2026
Merged

Shrav816 merged 1 commit into
mainfrom
ecu_regen_debugging

Conversation

@Shrav816

Copy link
Copy Markdown
Contributor

Modified the signal definition for forward and regen AC currents to match firmware definition:

.ForwardRegenCurrentMap = (stateData->forward_ac_i_code << 4) | (stateData->regen_ac_i_code & 0x0F)

with forward AC current representing the first 4 bits and regen AC current representing the last 4 bits. Previously, the order of the nibbles were switched.

@Shrav816
Shrav816 merged commit 4862e84 into main Oct 26, 2026
22 checks passed
@Shrav816
Shrav816 deleted the ecu_regen_debugging branch October 26, 2026 00:16
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.

1 participant