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

Add NFC library #783

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

Add NFC library #783

wants to merge 2 commits into from

Conversation

robsonos
Copy link

Add support to NFC. Base on arduino/ArduinoCore-primo and Seeed-Studio/Adafruit_nRF52_Arduino.

Related: #709

@CSC-Sendance
Copy link
Contributor

+1 Would love to see this.

@LokiMetaSmith
Copy link

I'm here looking for this just now :-D

I'm doing an NFC project and it turns out that the Lily T echo includes an NFC antenna on their nfc52840, and I want to go run around with NFC stickers and Boop them with a LoRa radio :-D no clue what it would be used for yet. Maybe a convoluted geocaching? Proof of Attendance? The possibilities are endless!

@hathach
Copy link
Member

hathach commented Nov 29, 2023

not builtable with this core, some updates to nrfx may be needed. I will try to take a look.

Copy link
Member

@hathach hathach left a comment

Choose a reason for hiding this comment

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

the nfc library's util folder is not compatible with nrfx included this core. To proceed ith PR please

  • remove util folder, you can add missing files/driver to core/nordic/nrfx
  • make sure all examples are buildable i.e passed ci

Do you know which nrfx version this util folder is based on ? it may be quicker to migrate if knowning its verion.

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