Skip to content

Fixing RTC Clock: T-Lora Pager actually uses PCF85063 for RTC#7886

Closed
hotellonely wants to merge 2 commits into
meshtastic:masterfrom
hotellonely:master
Closed

Fixing RTC Clock: T-Lora Pager actually uses PCF85063 for RTC#7886
hotellonely wants to merge 2 commits into
meshtastic:masterfrom
hotellonely:master

Conversation

@hotellonely

Copy link
Copy Markdown

PCF85063 has a different RTC masking scheme than the PCF8563. This would fix the RTC clock issue on many devices.

fixes #7885

🤝 Attestations

  • [ ✅] I have tested that my proposed changes behave as described.
  • [ ✅] I have tested that my proposed changes do not cause any obvious regressions on the following devices:
    • [✅] Other (please specify below) T-Lora Pager

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@fifieldt fifieldt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the patch and attention to detail. Unfortunately just adding to the enum does not result in successful detection. There's a need to modify code in src/detect/ScanI2CTwoWire.cpp as well.

@hotellonely

Copy link
Copy Markdown
Author

Thank you for the patch and attention to detail. Unfortunately just adding to the enum does not result in successful detection. There's a need to modify code in src/detect/ScanI2CTwoWire.cpp as well.

I don't know exactly why, but after doing the said changes in the 2 commits you can get correct time display

@mverch67

mverch67 commented Sep 9, 2025

Copy link
Copy Markdown
Collaborator

Supporting PCF85063 would mean to migrate from the no longer maintained library lewisxhe/PCF8563_Library to lewisxhe/SensorLib which implements both RTC clocks with a unique interface. Then there's only work to be done in RTC.cpp besides removing/replacing all lewisxhe/PCF8563_Library in the platformio.ini files.

@mverch67

mverch67 commented Sep 9, 2025

Copy link
Copy Markdown
Collaborator

I don't know exactly why, but after doing the said changes in the 2 commits you can get correct time display

most likely comes from your GPS time

@hotellonely

hotellonely commented Sep 9, 2025 via email

Copy link
Copy Markdown
Author

@WillyJL

WillyJL commented Sep 21, 2025

Copy link
Copy Markdown
Contributor

I have done the actual work of porting to SensorLib for PCF85063 support in #8061, this can be closed

@mverch67 mverch67 closed this Sep 21, 2025
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.

6 participants