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

PN7150 EoL #17

Open
ab-tools opened this issue Sep 28, 2024 · 12 comments
Open

PN7150 EoL #17

ab-tools opened this issue Sep 28, 2024 · 12 comments

Comments

@ab-tools
Copy link

Hello Pascal,

first thanks for creating this great library!

The PN7150, however, is EoL for a while and not recommended for new designs by NXP.

Any chance you could support the successor PN7160 with your library as well?

Best regards and thanks
Andreas

@Strooom
Copy link
Owner

Strooom commented Sep 29, 2024

I don't know what are the differences between PN7150 and PN7160, but looking at the partnumber it seems like it's minor changes. So there is a good chance the library already works as it is today.

This library was developed for a product which is still being sold, so in case the 7150 is EOL, I will have to update it for this product when we replace it with the 7160.

@ab-tools
Copy link
Author

First thanks a lot for your quick reply, Pascal!

I did give your library a quick try with a PN7160 and while it is discovered on the I2C bus (on the expected address), it seems not to react upon any command being sent.

Looking at the "PN7150 to PN7160 Migration Guidelines" from NXP
https://www.nxp.com/docs/en/application-note/AN14285.pdf
they mention "PN7150 NFC Controller supports NCI 1.0 while PN7160 NFC Controller supports NCI 2.0, which is not fully backward compatible". I can't really judge what changed here.

They also mention that there would be SW6705 - PN7160 NXP-NCI MCUXpresso Example Project which provides reference source code for PN7160 NFC Controller.

I took a quick look, but having a very hard time adapting anything from their into your project.

Any advice would be highly appreciated.

Best regards and thanks
Andreas

@Strooom
Copy link
Owner

Strooom commented Sep 29, 2024

Ok, that is useful input : so it would need to modify the NCI (basically the commands / responses between MCU and PN71X0).
I will take a look at that AN to see how big the differences are.
I remember the example project being a huge amount of code, so that was the main reason to implement something simpler.

I don't have any PN7160 at this time, so it will take some time to fix this. Not sure if you have a deadline or so. My paid work always comes before the things I do for free.

@ab-tools
Copy link
Author

Really appreciating your quick response and support, Pascal!

This is not extremely urgent, but you know, would still be happy to get it working after all. ;-)

I figure it's much easier to investigate with something to actually test. I see you're located in Belgium - if you like, I could send you a test PCB with the PN7160 and I2C exposed:
Just send me an e-mail to info@ab-smarthouse.com with your shipment address.

Thanks again
Andreas

@Strooom
Copy link
Owner

Strooom commented Sep 29, 2024

I was trying to get the NCI 2.0 spec, but now I remember the NFC forum does not provide them for free...
I will take a look at the example project to see if it contains enough info..

@ab-tools
Copy link
Author

Thanks, Pascal, and the PCB is already on the way to you! :-)

@Strooom
Copy link
Owner

Strooom commented Oct 2, 2024 via email

@ab-tools
Copy link
Author

ab-tools commented Oct 2, 2024

Great, thanks a lot, Pascal!

@Strooom
Copy link
Owner

Strooom commented Oct 8, 2024

I've managed to get a copy of NCI V2.0 spec, so now I can put it side-by-side with V1.0 to see where it differs.

@ab-tools
Copy link
Author

ab-tools commented Oct 8, 2024

That's really great to hear - thanks for the update, Pascal!

@Strooom
Copy link
Owner

Strooom commented Oct 18, 2024

Hi Andreas,
I've found there are differences in the first 2 messages which are used to initialize the so called NFC Device (CORE_RST_XXX and CORE_INIT_XXX), the payload of these messages has changed, So my next step is to adapt the NCI for this and then I am going to try on your hardware. I briefly scanned the remainder of the messages and they look the same..
So with a bit of luck that could be a quick solution to upgrade to the NCI2.0 and hence PN7160.

@ab-tools
Copy link
Author

That's great to hear, much appreciating your feedback, Pascal! :-)

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

No branches or pull requests

2 participants