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

Added macro NRF52_SERIES when checking for _delay_us to work with NRF52840 #208

Open
wants to merge 6 commits into
base: development
Choose a base branch
from

Conversation

jkano
Copy link

@jkano jkano commented May 13, 2020

Any pull request raised here MUST be submitted according to here MUST be submitted according to this template or it will be flagged with 'Not enough information'. No action will be taken till all the prerequisite information (according to the right template) is provided. If no information is provided for over a month after the 'Not enough information' label is applied, the issue will be closed.

Pull request details

  • Please check if the PR fulfills these requirements
  • The commit message/s explain/s all the changes clearly
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
  • Bug fix
  • Added feature
  • Documentation update
  • Other - Please explain here:
  • What is the current behavior? (You can also link to an open issue here)
    Compilation error described here Compilation error on NRF52840 #207

  • What is the new behavior? (if this is a feature change)

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
    Nothing.

  • Other information:


DO NOT DELETE OR EDIT anything below this

Note 1: Make sure to add all the information needed to understand the bug so that someone can help. If any essential information is missing we'll add the 'Needs more information' label and close the issue until there is enough information.

Note 2: For support questions (for example, tutorials on how to use the library), please use the Arduino Forums. This repository's issues are reserved for feature requests and bug reports.


GitHub issue state GitHub issue title GitHub issue author GitHub issue label GitHub issue comments GitHub issue age GitHub issue last updateGitHub pull request check contexts

Marzogh and others added 5 commits April 20, 2019 22:55
Updated pull requests and issue templates
Latest stable changes from development - Custom SPI pins and multiple SPI support for ESP32
Final code for v3.4.0 from Development
Final code for v3.4.0 from Stable
@TheBengineer
Copy link

Thanks for making this fix!

Copy link
Owner

@Marzogh Marzogh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this :)

@Marzogh Marzogh self-assigned this Sep 16, 2021
@Marzogh Marzogh added bugfix Fixes an existing bug in the current version of the code enhancement An enhancement to the library's function is proposed labels Sep 16, 2021
@Marzogh Marzogh added this to the v3.4.1 milestone Sep 16, 2021
@Marzogh Marzogh linked an issue Sep 16, 2021 that may be closed by this pull request
@Marzogh Marzogh changed the base branch from master to development September 16, 2021 04:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Fixes an existing bug in the current version of the code enhancement An enhancement to the library's function is proposed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compilation error on NRF52840
4 participants