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

Improve paragraph in READE.md #65

Closed
Koepel opened this issue Mar 12, 2019 · 1 comment
Closed

Improve paragraph in READE.md #65

Koepel opened this issue Mar 12, 2019 · 1 comment

Comments

@Koepel
Copy link

Koepel commented Mar 12, 2019

Would it be possible to improve this paragraph in README.md:

DS3232RTC also implements functions to support the additional features of the DS3232 and DS3231. The DS3231 has the same features as the DS3232 except: (1) Battery-backed SRAM, (2) Battery-backed 32kHz output (BB32kHz bit in Control/Status register 0x0F), and (3) Adjustable temperature sensor sample rate (CRATE1:0 bits in the Control/Status register).

I had to read it a couple of times to understand that the DS3232 has RAM and the DS3231 does not have RAM. It can be confusing that "DS323RTC" is the library and "DS3232" is the chip.

Proposal:

The DS3232RTC library also implements functions to support the additional features of the DS3232 and DS3231. The DS3232 has the same features as the DS3231, but the DS3232 has also these extra features:

  1. Battery-backed SRAM
  2. Battery-backed 32kHz output (BB32kHz bit in Control/Status register 0x0F)
  3. Adjustable temperature sensor sample rate (CRATE1:0 bits in the Control/Status register).
@JChristensen
Copy link
Owner

Done. Good suggestion, thank you!

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

No branches or pull requests

2 participants