Skip to content

Add Zucrow interface and native tests - #13

Open
wadhwat wants to merge 7 commits into
mainfrom
features/zucrow-tests
Open

Add Zucrow interface and native tests#13
wadhwat wants to merge 7 commits into
mainfrom
features/zucrow-tests

Conversation

@wadhwat

@wadhwat wadhwat commented Sep 30, 2026 •
edited
Loading

Copy link
Copy Markdown
  • Add the Zucrow interface for fault and sync signaling plus dual-channel valve-angle DAC output.
  • Initialize the interface before other devices on the shared SPI bus and integrate it with the engine controller.

RobertJN64
RobertJN64 requested changes Sep 30, 2026
View reviewed changes
Comment thread firmware/lib/zucrow_interface/ZucrowInterface.h Outdated
Comment thread firmware/lib/zucrow_interface/ZucrowInterface.cpp Outdated
Comment thread firmware/lib/zucrow_interface/ZucrowInterface.cpp Outdated
Comment thread firmware/lib/zucrow_interface/ZucrowInterface.cpp
Comment thread firmware/platformio.ini Outdated
Comment thread firmware/lib/zucrow_interface/ZucrowInterface.cpp Outdated
@wadhwat
wadhwat force-pushed the features/zucrow-tests branch from e139a58 to 6894327 Compare October 3, 2026 15:45
wadhwat added 4 commits October 3, 2026 12:03
@wadhwat
Add Zucrow interface module
85d4aa9
@wadhwat
Fix Zucrow output init order so pins boot at their intended levels
16ca35a 
stm32duino only enables a GPIO port's clock inside pinMode(), so digitalWrite() before pinMode() is dropped and the pin comes up LOW. That made the fault line to Zucrow boot as OK instead of FAULT, and left the DAC selected.
@wadhwat
Tidy Zucrow interface: expose full-scale angle in header, use float l…
119fd5b 
…iterals
@wadhwat
cleaned up superfluous comments and made bit numbers clearer
e631e6d
@wadhwat
wadhwat force-pushed the features/zucrow-tests branch from 6894327 to e631e6d Compare October 3, 2026 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jacobh460 jacobh460 jacobh460 left review comments
@RobertJN64 RobertJN64 RobertJN64 requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@wadhwat @RobertJN64 @jacobh460