Skip to content

Make ring buffer head and tail indexes volatile #303

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

Merged

Conversation

sandeepmistry
Copy link
Contributor

I was having some issues running some MKRGSM lib examples with the latest SAMD core master. 503a9b3 introduced an issue, however making the ring buffer head and tail indexes volatile allows things to work again. They probably should have been volatile in the original non-templated version.

@ArduinoBot
Copy link

✅ Build completed.

⬇️ Build URL: http://downloads.arduino.cc/PR/samd/package_samd-b188_index.json

ℹ️ To test this build:

  1. Open the Preferences of the Arduino IDE.
  2. Add the Build URL above in the Additional Boards Manager URLs field, and click OK.
  3. Open the Boards Manager (menu Tools->Board->Board Manager...)
  4. Install Arduino SAMD core - Pull Request Make ring buffer head and tail indexes volatile #303
  5. Select one of the boards under SAMD Pull Request Make ring buffer head and tail indexes volatile #303 in Tools->Board menu
  6. Compile/Upload as usual

@sandeepmistry sandeepmistry merged commit 69fcf8d into arduino:master Feb 14, 2018
@cmaglie cmaglie deleted the ringbuffer-volatile-head-tail branch July 11, 2018 15:03
@cmaglie cmaglie added this to the Release 1.6.19 milestone Jul 11, 2018
boseji pushed a commit to go-ut/combined-ArduinoCore-samd that referenced this pull request Nov 10, 2021
* more Adafruit_TinyUSB include

* add metro m0 m4 with tinyusb to ci

* correct fqbn

* support ci with tinyusb variant

bump zeroDMA and fix warnings with USBHost library example

* install missing libraries

* more libraries

* update tinyusb library to 1.2.0

increase version to 1.7.3
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.

4 participants