Skip to content
This repository was archived by the owner on Apr 8, 2026. It is now read-only.

[DRAFT] Bootloader Upgrade#14

Draft
eiln wants to merge 3 commits intoPurdueElectricRacing:masterfrom
eiln:eileen/bootloader
Draft

[DRAFT] Bootloader Upgrade#14
eiln wants to merge 3 commits intoPurdueElectricRacing:masterfrom
eiln:eileen/bootloader

Conversation

@eiln
Copy link
Copy Markdown

@eiln eiln commented Oct 19, 2024

Add standalone BL module bl.py
Depends on eileen/bootloader firmware branch
Breaks daqapp UI BL button due to API changes

eiln added 3 commits October 19, 2024 11:49
Depends on eileen/bootloader firmware branch
Also daqapp UI BL is broken because of API changes

Signed-off-by: Eileen Yoon <eyn@gmx.com>
Canable firmware with gs_usb backend has a bug with high bus loads
due to finite queue size. Solve by using native linux sockets instead

Signed-off-by: Eileen Yoon <eyn@gmx.com>
Signed-off-by: Eileen Yoon <eyn@gmx.com>
Comment thread bl.py
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.

It's been a while so I might have forgotten a bit about why you made this file, but iirc it was because we weren't acking packets we sent on the RX line. If this is the only issue that prompted the creation of this file, I'd rather fix the underlying daqapp problem in communication/can_bus.py rather than bring bootloader out into it's own module. Boothloader isn't the only thing that is sending to the car thru DAQ

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.

Also would rather this be bootloader.py, full name is very helpful for understanding

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants