Skip to content

Restore gamepad; turn off keypad on inappropriate boards; add deprecation notice #5059

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

Closed
wants to merge 2 commits into from

Conversation

dhalbert
Copy link
Collaborator

We have the room now to have both keypad and gamepad on most boards. Restore gamepad so libraries will work on both.

When 7.0.0 final is shipped, we can change library and guide code to use keypad, and remove gamepad and gamepadshift.

  • Revert 04b69cd, part of keypad: support for vector and matrix key scanning #4891, to restore gamepad to most builds.
  • Turn off both gamepad and keypad on builds that don't need them.
  • Add deprecation notice in gamepad and gamepadshift.
  • Update frozen libraries: in particular, the circuitplayground library now does not include were_pressed().

@dhalbert
Copy link
Collaborator Author

gamepad and keypad together are not going to fit on several SAMD21 boards. I'll do something about combining parts of #4940 and this before the next alpha.

@dhalbert dhalbert marked this pull request as draft July 26, 2021 16:03
@dhalbert dhalbert closed this Jul 27, 2021
@dhalbert dhalbert deleted the restore-gamepad branch July 27, 2021 02:02
cwalther pushed a commit to cwalther/circuitpython that referenced this pull request May 28, 2022
mpy-cross will now generate native code based on the size of
mp_code_state_native_t, and the runtime will use this struct to calculate
the offset of the .state field.  This makes native code generation and
execution (which rely on this struct) independent to the settings
MICROPY_STACKLESS and MICROPY_PY_SYS_SETTRACE, both of which change the
size of the mp_code_state_t struct.

Fixes issue adafruit#5059.

Signed-off-by: Damien George <[email protected]>
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