SPI update for rc522

[SpaceC]

Is it possible to include an update of:

https://github.com/ondryaso/pi-rc522/blob/master/pirc522/rfid.py

Background:
My distance card to reader is about some 10 mm this causes many "new card" events.
This leads to songs starting over and over again.
This can be avoided using the setting.
"Nur Systembefhle mehrfach ausführen"

But now the following issue ocours.
After sometime, no new card is recognized. WiFi is still there, playback via webfrontend is working, Buttons still working.
But now other song with RFID.

Therefore I started to setup my Box again and during installation I had some time to read....
Found the comment in the link and double checked the datasheet and yes it makes sense.

I stoped the new installation and switched back to the old ones.
Exchanged the file (changed also the comments from C style to pyhton style) and I had no issue again now after 2h testing.

Thanks for that great Project!!

[MiczFlor]

Hi @SpaceC
thanks for the pointer. I am not sure I understand what you suggest to be done. Could you rephrase it for me?

[Menelmacar]

I have the same issue (with rc 522 RFID reader, stops working after 30s - 5 min)
It'S wired as described in the manual.
Version 2.0 rc4
Edition Classic edition (barebones)
on RPI 3b Buster fresh install.
After install I replaced Reader.py with experimental as the manual says.

update:
after updating the
scripts/pirc522/rfid.py with the newest version from this project as in this issue described:
ondryaso/pi-rc522#48

there is no more Blocking of the card and everything works fine.
to save processor time I also added a sleep(1) to the daemon_rfid_reader.py after the subprocess.call(
line.

I never used github. would you merge the new version of pi-rc522 to this projekt to get rid of the blocking of the card?

[s-martin]

@ZyanKLee Should we include https://github.com/ondryaso/pi-rc522 as dependency and remove the sources from our repo?

[ZyanKLee]

@s-martin I'd say this is always preferable as long as there aren't any changes in our local version that were never mainlined into the upstream project.

[s-martin]

I’ll check, if we have any changes on our repo.

[s-martin]

I have just seen that we already install pi-rc522 via requirements.txt

[ZyanKLee]

in that case I will remove scripts/pirc522 and see what breaks :-D

[ZyanKLee]

please test if this still works and solves your issue @Menelmacar and @SpaceC