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

backup-switchover-mode value definitions improvement #1861

Closed
wants to merge 1 commit into from
Closed

backup-switchover-mode value definitions improvement #1861

wants to merge 1 commit into from

Conversation

tiagofreire-pt
Copy link

For the RV3028 RTC, the definitions for its backup-switchover-mode overlay where not intelligible neither complete/exhaustive.

Accordingly to the raspberrypi/linux#2912 (comment), these should be correct.

@tiagofreire-pt
Copy link
Author

/boot/config.txt should be:

Configurating example, for rv3028, on a Uputronics GPS Extension HAT:

# For GPS Expansion Board from Uputronics
dtparam=i2c_arm=on
dtoverlay=i2c-rtc,rv3028,backup-switchover-mode=3
dtoverlay=pps-gpio,gpiopin=18
init_uart_baud=115200```

@tiagofreire-pt
Copy link
Author

From my tests (sudo i2cget -y 1 0x52 0x37):

Default from factory: 0x10

Mode 0: 0x10

Mode 1: 0x14

Mode 2: 0x18

Mode 3: 0x1c

@tiagofreire-pt
Copy link
Author

Mode 3: 0x1c is consistent with the manufacturer configuration script: http://store.uputronics.com/files/configure-rv3028.sh

@pelwell
Copy link
Contributor

pelwell commented Jan 22, 2024

The README content looks reasonable, but it needs to patched at source - in our Linux repo: https://github.com/raspberrypi/linux

@tiagofreire-pt
Copy link
Author

tiagofreire-pt commented Jan 23, 2024

The README content looks reasonable, but it needs to patched at source - in our Linux repo: https://github.com/raspberrypi/linux

Done, as requested: raspberrypi/linux#5884

Thanks.

@tiagofreire-pt tiagofreire-pt deleted the patch-1 branch January 23, 2024 09:48
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.

2 participants