Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

usb: split parts of usb_desc.h into more headers #1370

Merged
merged 2 commits into from
Feb 12, 2025

Conversation

benma
Copy link
Collaborator

@benma benma commented Feb 11, 2025

@NickeZ

The first commit changes the firmware hashes, but is a trivial change.

The second commit refactors the headers. This commit does not change the hashes. I verified it by making these tags:

bootloader/v1.1.0
firmware/v9.22.0
firmware-btc-only/v9.22.0

Compiling everything, and repeating the same on the previous commit. Both had the same results:

ded91042ae87560a12580c6fb1f6db33caafd06c46ac112d18aacb748348269b  build/bin/firmware.bin
3e5a72915801dca13d88e11052d821d5b2aad2130cb153a398335f97cebc3ecb  build/bin/firmware-btc.bin
b3816a1a56b42dc1ad1a04091d237a9d567d6cad04b57b8c49e18cb1c3dade17  build/bin/bootloader-btc-production.bin
fcddd94ea76e278757caa1b114153c401b94f05e8fbca00c152a7d47fa916721  build/bin/bootloader-production.bin

Keep the main usb_desc.h smaller, with the goal if copy/pasting it
once per platform (BitBox02 and BitBox02 Plus) to add new product
strings for BitBox02 Plus.
@benma benma changed the title usb: split parts of usb_desc.h into usb_size.h and usb_u2f_desc.h usb: split parts of usb_desc.h into more headers Feb 12, 2025
@benma benma requested a review from NickeZ February 12, 2025 09:13
@benma benma marked this pull request as ready for review February 12, 2025 09:13
@benma benma merged commit 7d35ce1 into BitBoxSwiss:master Feb 12, 2025
3 checks passed
@benma benma deleted the desc-split branch February 12, 2025 13:30
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