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

Leonardo Pin & Self Test for RC522 #2

Merged
merged 3 commits into from
Dec 15, 2013
Merged

Leonardo Pin & Self Test for RC522 #2

merged 3 commits into from
Dec 15, 2013

Conversation

cniesen
Copy link
Collaborator

@cniesen cniesen commented Dec 15, 2013

Hi, I added pin instructions for Leonardo and code to run the digital self test on the RC522. Feel free to take whatever you like. No hard feelings if you don't want any of it. Best wishes, Claus.

@ljos
Copy link
Owner

ljos commented Dec 15, 2013

Cool! First time I wrote the response here I was going to suggest you to rather send this to upstream, but it seems to me now that we have diverged completely. I was going to ensure that my changes where good and then send a PR to upstream, but it seems they have already received a PR from someone else and accepted.

It is going to be too much work to get my branch compatible with their repo now, so this branch is here to stay.

Thanks for the changes!

ljos added a commit that referenced this pull request Dec 15, 2013
Leonardo Pin & Self Test for RC522
@ljos ljos merged commit 93d9b16 into ljos:master Dec 15, 2013
@ljos
Copy link
Owner

ljos commented Dec 15, 2013

Also added you as a collaborator if you want to do other changes later.

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.

2 participants