Strooom / PN7150

Library/Driver for NXP PN7150 NFC device
Other
24 stars 25 forks source link

PN7150 EoL #17

Open ab-tools opened 2 months ago

ab-tools commented 2 months ago

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 commented 2 months ago

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 commented 2 months ago

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 commented 2 months ago

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 commented 2 months ago

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 commented 2 months ago

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 commented 2 months ago

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

Strooom commented 1 month ago

Hello Andreas, the PCB has arrived. I cannot start working on it immediately, but in a few weeks I will take a look at the PN7160 and keep you posted on any progress.

P

Op zo 29 sep 2024 om 23:26 schreef ab-tools @.***>:

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

— Reply to this email directly, view it on GitHub https://github.com/Strooom/PN7150/issues/17#issuecomment-2381609877, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD5KOZJXO7AR4Q642TVSH73ZZBV7TAVCNFSM6AAAAABPAYBLOSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBRGYYDSOBXG4 . You are receiving this because you commented.Message ID: @.***>

-- Met vriendelijke groeten

Pascal Roobrouck


Instructables https://www.instructables.com/member/strooom/ | Hackster https://www.hackster.io/strooom

ab-tools commented 1 month ago

Great, thanks a lot, Pascal!

Strooom commented 1 month ago

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 commented 1 month ago

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

Strooom commented 1 month ago

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 commented 1 month ago

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