feat(joystick): add CRC-8 packet checksum and region-based XY sensiti…#570
Open
veerwang wants to merge 1 commit into
Open
feat(joystick): add CRC-8 packet checksum and region-based XY sensiti…#570veerwang wants to merge 1 commit into
veerwang wants to merge 1 commit into
Conversation
…vity Port control-panel firmware changes into firmware/joystick: - Add CRC-8-CCITT (poly 0x07) checksum over packet[0..8] into packet[9], matching firmware/octoaxes/serial.cpp. Remap a computed 0x00 to 0x01 so that 0 stays reserved as the legacy "no CRC" marker for old firmware. - Add REGION_OVERSEAS build flag: overseas joystick X/Y sensitivity factor is /4 (2x output at the same detent) versus /8 for the domestic default. Provide teensyLC (domestic) and teensyLC_overseas PlatformIO envs. - Mark encoder_step_size volatile since it is shared with the encoder ISR. - Compute the Z encoder step with integer bit-shift (1 << input_sensitivity_z, capped at 256) instead of pow(), avoiding float math. - Add download.sh helper to flash the domestic or overseas build. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…vity
Port control-panel firmware changes into firmware/joystick: