Skip to content

drivers: Added support pn532 driver for NFC communication#2692

Closed
berbecche wants to merge 8 commits intoRIOT-OS:masterfrom
berbecche:RIOT-NFC
Closed

drivers: Added support pn532 driver for NFC communication#2692
berbecche wants to merge 8 commits intoRIOT-OS:masterfrom
berbecche:RIOT-NFC

Conversation

@berbecche
Copy link
Copy Markdown

This library is for pn532's chips via SPI communication.
Library is developed for your udoo board, you should be able to interactively use any Adafruit PN532 Shield right connected to ICSP header and CS on pin 23. It should work with any SAM3X8E mcu.
It should also work with other boards connected to this device but I haven't tried it out yet due to the lack of time.

@miri64 miri64 added the Type: new feature The issue requests / The PR implemements a new feature for RIOT label Mar 24, 2015
@OlegHahm OlegHahm force-pushed the master branch 3 times, most recently from 9f184dd to 45554bf Compare March 31, 2015 13:01
@OlegHahm
Copy link
Copy Markdown
Member

Does anyone plan to continue to work on this?

@thomaseichinger
Copy link
Copy Markdown
Member

Will take a look at it.

@thomaseichinger thomaseichinger added the CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable label Aug 24, 2015
@thomaseichinger
Copy link
Copy Markdown
Member

@berbecche I'm sorry I missed this one. Could you elaborate why this should only be usable with the sam3x CPU's? Maybe we can get it working with all of RIOT's supported MCUs.

@berbecche
Copy link
Copy Markdown
Author

Hi. I'm pretty sure it works with sam3x CPU's because I tested it. I'm not
able to assure this driver works with all RIOT MCUs and I haven't.
Unfortunately, right now, I haven't time to spend to make it works for all
RIOT MCUs.

Il 24/ago/2015 21:31, "Thomas Eichinger" [email protected] ha
scritto:

@berbecche I'm sorry I missed this one. Could you elaborate why this
should only be usable with the sam3x CPU's? Maybe we can get it working
with all of RIOT's supported MCUs.


Reply to this email directly or view it on GitHub.

@thomaseichinger
Copy link
Copy Markdown
Member

Ok, I will take a look to see what has to be done.

@thomaseichinger
Copy link
Copy Markdown
Member

@berbecche I discussed this with other maintainers offline. I will take a look what has to be done to get your code usable by more RIOT supported boards and will open PR against your repo. This is of high value for us so we would then merge it, after some testing, and open an issue to fix styling and debugging messages as a newbie task.
Are you ok if we proceed like this?

@berbecche
Copy link
Copy Markdown
Author

Of course. Whatever to make more usable for RIOT.
Regards.

2015-09-01 17:51 GMT+02:00 Thomas Eichinger [email protected]:

@berbecche https://github.com/berbecche I discussed this with other
maintainers offline. I will take a look what has to be done to get your
code usable by more RIOT supported boards and will open PR against your
repo. This is of high value for us so we would then merge it, after some
testing, and open an issue to fix styling and debugging messages as a
newbie task.
Are you ok if we proceed like this?


Reply to this email directly or view it on GitHub
#2692 (comment).

Dott. Simone Pecora
Cell. 340 108 6698
Linkedin: Simone Pecora

@OlegHahm OlegHahm modified the milestone: Release NEXT MAJOR Sep 2, 2015
@thomaseichinger
Copy link
Copy Markdown
Member

@berbecche Please consider berbecche#1

@OlegHahm OlegHahm modified the milestone: Release 2015.12 Nov 28, 2015
@OlegHahm OlegHahm modified the milestone: Release 2016.03 Dec 8, 2015
@OlegHahm
Copy link
Copy Markdown
Member

@thomaseichinger, would you be willing to adopt this apparently abandoned PR?

@berbecche
Copy link
Copy Markdown
Author

I'm so sorry about it, but I'm too busy this period.

2016-03-11 10:43 GMT+01:00 Oleg Hahm [email protected]:

@thomaseichinger https://github.com/thomaseichinger, would you be
willing to adopt this apparently abandoned PR?


Reply to this email directly or view it on GitHub
#2692 (comment).

Dott. Simone Pecora
Cell. 340 108 6698
Linkedin: Simone Pecora

@miri64 miri64 added the Area: drivers Area: Device drivers label Mar 18, 2016
@thomaseichinger
Copy link
Copy Markdown
Member

@OlegHahm I can, I actually have a branch with some fixes already laying around. Will pick it up as I find time.

@OlegHahm OlegHahm modified the milestones: Release 2016.07, Release 2016.04 Mar 28, 2016
@OlegHahm
Copy link
Copy Markdown
Member

Thanks, @thomaseichinger. Won't make it for the release though.

@kYc0o
Copy link
Copy Markdown
Contributor

kYc0o commented Jul 25, 2016

Feature freeze -> postponed.

@kYc0o kYc0o modified the milestones: Release 2016.10, Release 2016.07 Jul 25, 2016
@PeterKietzmann
Copy link
Copy Markdown
Member

#5573 and #2692 implement a driver for the same device. One is based on SPI, the other on I2C. Who is willing to merge both into one implementation :-)?

@OlegHahm
Copy link
Copy Markdown
Member

@thomaseichinger, could you bring your fixes upstream please?

@lebrush, would you be willing to see if you can merge the SPI support into your PR?

@lebrush
Copy link
Copy Markdown
Member

lebrush commented Oct 14, 2016

@OlegHahm i'll try to find some time to work on SPI for #5573. Works start to pile and I can't give a concrete date.

@miri64
Copy link
Copy Markdown
Member

miri64 commented Oct 31, 2016

Postponed due to feature freeze

@miri64 miri64 modified the milestones: Release 2017.01, Release 2016.10 Oct 31, 2016
@PeterKietzmann
Copy link
Copy Markdown
Member

Closing this one in favour of #5573

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: drivers Area: Device drivers CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable Type: new feature The issue requests / The PR implemements a new feature for RIOT

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants