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.

Sign up for GitHub

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

Jump to bottom

API refactor #851

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

API refactor #851

wants to merge 7 commits into from

Commits on Sep 27, 2024

  1. Move API definition to "HALs" 

    Avoid device-specific memory things in the main library
    @Daft-Freak
    Daft-Freak committed Sep 27, 2024
    Configuration menu
    Copy the full SHA
    47e6e74 View commit details
    Browse the repository at this point in the history

  2. Split API into const and data parts 

    I'd prefer to not allocate 2k of RAM for something that is mostly constants.

The STM32 parts are somewhat hacky
    @Daft-Freak
    Daft-Freak committed Sep 27, 2024
    Configuration menu
    Copy the full SHA
    a911c5d View commit details
    Browse the repository at this point in the history

  3. sdl: make api_const really const 

    Might be usable in global initialisers... still undefined though.
    @Daft-Freak
    Daft-Freak committed Sep 27, 2024
    Configuration menu
    Copy the full SHA
    0641636 View commit details
    Browse the repository at this point in the history

  4. pico: make api_const really const

    @Daft-Freak
    Daft-Freak committed Sep 27, 2024
    Configuration menu
    Copy the full SHA
    25a61e7 View commit details
    Browse the repository at this point in the history

  5. Remove the compatibility padding for non-stm32 

    Not needed elsewhere. Reduces RAM usage for the API to 64 bytes
    @Daft-Freak
    Daft-Freak committed Sep 27, 2024
    Configuration menu
    Copy the full SHA
    569ff5c View commit details
    Browse the repository at this point in the history

  6. pico: init can_launch 

    These patches predate it, so add it to squash the warnings.
    @Daft-Freak
    Daft-Freak committed Sep 27, 2024
    Configuration menu
    Copy the full SHA
    8dfbbf3 View commit details
    Browse the repository at this point in the history

  7. sdl: init can_launch 

    Same as the pico patch
    @Daft-Freak
    Daft-Freak authored Sep 27, 2024
    Configuration menu
    Copy the full SHA
    9a70076 View commit details
    Browse the repository at this point in the history