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

Update MFRC522.cpp #11073

Closed
wants to merge 1 commit into from
Closed

Update MFRC522.cpp #11073

wants to merge 1 commit into from

Conversation

rapi3
Copy link

@rapi3 rapi3 commented Feb 21, 2021

Description: Extend range for MFRC522 RFID Reader. Set RFCfgReg to 48db gain

Related issue (if applicable): fixes #

Checklist:

  • The pull request is done against the latest development branch
  • Only relevant files were touched
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • The code change is tested and works on Tasmota core ESP8266 V.2.7.4.9
  • The code change is tested and works on Tasmota core ESP32 V.1.0.5-rc6
  • I accept the CLA.

NOTE: The code change must pass CI tests. Your PR cannot be merged unless tests pass

@Jason2866
Copy link
Collaborator

Increasing sensitivity normally results in decreasing accuracy (in decoding).
Have you tested how this change affects near range detection?

@rapi3
Copy link
Author

rapi3 commented Feb 22, 2021

yes I have and I am testing in the last days:

  • without this change the range it is very low on my reader ~0.5-1cm, I almost have to touch the cards to antenna and I don't think it is possible to have the reader in a case and use it for a real project attached to a door without RFCfgReg increased.
  • with change RFCfgReg increased the range it is ~3 cm and I don't see any problem.
    It will be nice in the future to have availability SetOptions for this parameter, bu that will require a lot of work from you.

p.s.
I just found another project that have the option to change this parameter like SetOptions in Tasmota and I also don't see any problems with accuracy as I am also testing:
https://github.com/esprfid/esp-rfid

@arendst arendst self-assigned this Feb 24, 2021
arendst added a commit that referenced this pull request Feb 27, 2021
Add command ``Sensor80 1 <0..7>`` to control MFRC522 RFID antenna gain from 18dB (0) to 48dB (7) (#11073)
@arendst
Copy link
Owner

arendst commented Feb 27, 2021

Implemented using command Sensor80 1 <0..7> where 0 to 7 see below:

image

@arendst arendst closed this Feb 27, 2021
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.

3 participants