ElectronicCats / ElectronicCats-PN7150

Arduino library for I2C access to the PN7150 RFID/Near Field Communication chip
MIT License
34 stars 15 forks source link

Issue with OM5578/PN7150ARDM #28

Closed m1ckey closed 2 years ago

m1ckey commented 3 years ago

I would like to use this library, but cannot get the example "DetectTags" to work.

My hardware is OM5578/PN7150ARDM and Arduino Mega R3 2560. I modified in the DetectTags.ino the pin definitions (IRQ=8, VEN=7). If I start the program it fails in Electroniccats_PN7150::ConfigureSettings: https://github.com/ElectronicCats/ElectronicCats-PN7150/blob/b0297f30d0117d373a0399a2c9f43245de281490/src/Electroniccats_PN7150.cpp#L1274 (The rxBuffer is all zeros)

If I simply skip Electroniccats_PN7150::ConfigureSettings in the example code, the PN7150 behaves very weird. (It successfully reads the first card, but cannot read another card)

I am not sure how to debug this further and would appreciate any input.

Michael

PS: The output from Electroniccats_PN7150::connectNCI:

0xROM_CODE_V: 10
FW_MAJOR_NO: 1
0xFW_MINOR_NO: a0
gNfcController_generation: 2
wilson-elechouse commented 2 years ago

I tested the code with Arduino Miro. I got the same error: 0xROM_CODE_V: 10 FW_MAJOR_NO: 1 0xFW_MINOR_NO: a0 gNfcController_generation: 2

jazhe commented 2 years ago

Hello @m1ckey and @wilson-elechouse, thank you for reporting this! Do you have any updates? We are trying to replicate your issue in order to give you a better response and support, please bear with us in this process and we will keep you updated soon! Thanks again for your patience.

m1ckey commented 2 years ago

Thank you for reaching out.
Sorry, I do not have any updates. I ended up using other hardware.

jazhe commented 2 years ago

Hello @m1ckey! Thanks for the information, would you like to receive an update on our findings? If not, please let us know if we can close this issue now. Also, thanks for reporting this issue, feedback like this helps us improve!

Thank you for reaching out. Sorry, I do not have any updates. I ended up using other hardware.

m1ckey commented 2 years ago

If not, please let us know if we can close this issue now.

wilson-elechouse commented 2 years ago

Hi, I would like to receive the updating. We are still waiting for the problem to be resolved. Thanks

On Fri, Jun 24, 2022, 02:17 Jaz, @.***> wrote:

Hello @m1ckey https://github.com/m1ckey! Thanks for the information, would you like to receive an update on our findings? If not, please let us know if we can close this issue now. Also, thanks for reporting this issue, feedback like this helps us improve!

Thank you for reaching out. Sorry, I do not have any updates. I ended up using other hardware.

— Reply to this email directly, view it on GitHub https://github.com/ElectronicCats/ElectronicCats-PN7150/issues/28#issuecomment-1164726337, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABB5QDETZPG5K6Q55ZJHR2DVQSS3XANCNFSM5DQX47HA . You are receiving this because you were mentioned.Message ID: @.***>

jazhe commented 2 years ago

Hello @wilson-elechouse!

Thanks for your patience. Unfortunately, the issue will persist due to incompatibility with AVR boards (Uno, Mega, Micro, etc), the library was not created for 8-bit microcontrollers. Apologies for the inconvenience caused.

Hope you have a nice day!

wilson-elechouse commented 4 months ago

Will this library supports PN7160?

jazhe commented 4 months ago

Hello @wilson-elechouse, we have not been able to validate the compatibility because we do not have the chip, however, we are planning to do the necessary tests in the near future. If you would like these tests to be speeded up, you can become a sponsor so that we can purchase the necessary hardware or contribute to this project, everything is appreciated 😸

wilson-elechouse commented 4 months ago

We could ship you sample boards of PN7160 and PN7161, if you need them

Jaz @.***> 于2024年6月15日周六 03:57写道:

Hello @wilson-elechouse https://github.com/wilson-elechouse, we have not been able to validate the compatibility because we do not have the chip, however, we are planning to do the necessary tests in the near future. If you would like these tests to be speeded up, you can become a sponsor https://github.com/sponsors/ElectronicCats or contribute https://github.com/ElectronicCats/electroniccats-cla/blob/main/electroniccats-contribution-manual.md to this project, everything is appreciated 😸

— Reply to this email directly, view it on GitHub https://github.com/ElectronicCats/ElectronicCats-PN7150/issues/28#issuecomment-2168672248, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABB5QDD62T6DYDF3MFSEJIDZHNDKPAVCNFSM6AAAAABJIFV4LWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRYGY3TEMRUHA . You are receiving this because you were mentioned.Message ID: @.***>

jazhe commented 4 months ago

Awesome! @wilson-elechouse please email us to start the process, we appreciate it a lot!