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

Unconditionally enable ChibiOS syscalls #8268

Merged
merged 2 commits into from
Feb 29, 2020
Merged

Conversation

zvecr
Copy link
Member

@zvecr zvecr commented Feb 29, 2020

Description

As ChibiOS syscalls is already used within in a few scenarios, this enables by default so that we avoid some of the issues within #7059.

Been using this for a while now and not seen issues 🤷‍♂️, but then we only alloc in limited circumstances ( otherwise a lot more builds would be failing).

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

Issues Fixed or Closed by This PR

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

@zvecr zvecr added the core label Feb 29, 2020
@zvecr zvecr requested a review from a team February 29, 2020 10:42
@zvecr zvecr mentioned this pull request Feb 29, 2020
13 tasks
@tzarc
Copy link
Member

tzarc commented Feb 29, 2020

Whoops, just realised I just included similar changes into a new PR of mine: #8272
That said, this looks out of date with respect to current master.

@zvecr
Copy link
Member Author

zvecr commented Feb 29, 2020

Currently in the process of rebasing everything.... including all the non PRed stuff ive been sat on for the last couple of months.

This should be good to go now.

@tzarc
Copy link
Member

tzarc commented Feb 29, 2020

Yep, all good, go for merge!

@tzarc
Copy link
Member

tzarc commented Feb 29, 2020

I'd been using these in some shape or form for a few months now, too.

@tzarc tzarc merged commit 8d9c800 into qmk:master Feb 29, 2020
cmaier pushed a commit to cmaier/qmk_firmware that referenced this pull request Mar 1, 2020
* Enable syscalls all the time

* fix whitespace
rhruiz pushed a commit to rhruiz/qmk_firmware that referenced this pull request Mar 1, 2020
* Enable syscalls all the time

* fix whitespace
HokieGeek pushed a commit to HokieGeek/qmk_firmware that referenced this pull request Mar 5, 2020
* Enable syscalls all the time

* fix whitespace
c0psrul3 pushed a commit to c0psrul3/qmk_firmware that referenced this pull request Mar 23, 2020
* Enable syscalls all the time

* fix whitespace
drashna pushed a commit to zsa/qmk_firmware that referenced this pull request Mar 26, 2020
* Enable syscalls all the time

* fix whitespace
sowbug pushed a commit to sowbug/qmk_firmware that referenced this pull request Apr 2, 2020
* Enable syscalls all the time

* fix whitespace
kylekuj pushed a commit to kylekuj/qmk_firmware that referenced this pull request Apr 21, 2020
* Enable syscalls all the time

* fix whitespace
@zvecr zvecr deleted the feature/syscalls branch April 28, 2020 01:14
jakeisnt pushed a commit to jakeisnt/qmk_firmware that referenced this pull request Aug 20, 2020
* Enable syscalls all the time

* fix whitespace
BorisTestov pushed a commit to BorisTestov/qmk_firmware that referenced this pull request May 23, 2024
* Enable syscalls all the time

* fix whitespace
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Using malloc on ARM causes build failure
2 participants