1

Init battery for ds3231 explicitly, code style fixes

This commit is contained in:
eleksir
2025-02-07 19:14:46 +03:00
parent 24d457f63c
commit 2dc7e9ec79

View File

@@ -1,32 +1,26 @@
/* /* 7 Segment 4 Digits clock with digit point separator.
* 7 Segment 4 Digits clock with digit point separator
* *
* It uses display module with common anode and DS1307 RTC module to enhance * It uses display module with common anode and DS1307/DS3231 RTC module to
* time keeping ability of Atmega 328p based Arduino Nano microcontroller and * enhance time keeping ability of Atmega 328p based Arduino Nano
* 2 buttons for adjusting time. * microcontroller and 2 buttons for adjusting time. */
*/
/* /* uRTCLib
* uRTCLib
* https://github.com/Naguissa/uRTCLib * https://github.com/Naguissa/uRTCLib
* it has support for both DS1307 and DS3231, that i own. * it has support for both DS1307 and DS3231, that I have.
* It is more fat than ds1307-dedicated or ds3231-dedicated lib, but i do not * It is more fat than ds1307-dedicated or ds3231-dedicated lib, but I do not
* want to invent wheel if i have one. * want to re-invent wheel if I have one. */
*/
#include <uRTCLib.h> #include <uRTCLib.h>
/* /* SevSeg
* SevSeg
* https://github.com/DeanIsMe/SevSeg * https://github.com/DeanIsMe/SevSeg
* it has simple api and ideally works in my case * it has simple api and ideally works in my case. */
*/
#include <SevSeg.h> #include <SevSeg.h>
/* disable time compensation /* Disable time compensation.
* as we borrow more accurate chrystal and solder it to rtc breakboard * At last we borrow more accurate chrystal and solder it to rtc breakboard.
*/ */
//#define TIMECOMP //#define TIMECOMP
/* 0.5 second/day compensation flag */ // 0.5 second/day compensation flag.
//#define OFIVECOMP //#define OFIVECOMP
#define cbi(sfr, bit) (_SFR_BYTE(sfr) &= ~_BV(bit)) #define cbi(sfr, bit) (_SFR_BYTE(sfr) &= ~_BV(bit))
@@ -35,7 +29,7 @@
uRTCLib clock; uRTCLib clock;
SevSeg Display; SevSeg Display;
/* push button stuff */ // Push button stuff.
const long initial_delay = 600; // wait a bit before spin numbers const long initial_delay = 600; // wait a bit before spin numbers
const long adjust_interval = 200; // delay between consecutive number increments const long adjust_interval = 200; // delay between consecutive number increments
const long bounce_delay = 20; // contact bounce compensation const long bounce_delay = 20; // contact bounce compensation
@@ -44,118 +38,123 @@ const int min_btn = A1; // put minute adjusting button on A1 lane
volatile bool btn_pushed = false; // is button pushed? volatile bool btn_pushed = false; // is button pushed?
unsigned long previousPushTimer = 0; unsigned long previousPushTimer = 0;
/* dot blinking stuff */ // Dot blinking stuff.
volatile bool dot = true; volatile bool dot = true;
#ifdef TIMECOMP #ifdef TIMECOMP
/* time inaccuracy compensation flag */ // Time inaccuracy compensation flag.
volatile bool compflag1 = true; volatile bool compflag1 = true;
volatile bool compflag2 = false; volatile bool compflag2 = false;
const uint8_t timeSkew = 9; // assume that clock runs faster than should const uint8_t timeSkew = 9; // Assume that clock runs faster than should.
#ifdef OFIVECOMP #ifdef OFIVECOMP
volatile bool timeSkewEven = false; // comp. 1 additional second per 2 days volatile bool timeSkewEven = false; // Comp. 1 additional second per 2 days.
#endif #endif
#endif #endif
/* ambient light compensation aka night mode */ // Ambient light compensation aka night mode.
const int photoPin = A2; const int photoPin = A2;
volatile bool measureAmbLight = true; volatile bool measureAmbLight = true;
unsigned int ambLight = 0; // actual value unsigned int ambLight = 0; // Actual value.
unsigned int ambLight2 = 0; // previous med. value
unsigned int ambLight3 = 0; // pre-previous med. value
unsigned long previousLightTimer = 0; unsigned long previousLightTimer = 0;
const long refresh_interval = 40; // refresh rate 25Hz const long refresh_interval = 40; // Refresh rate 25Hz.
/* time vars */ // Time vars.
volatile uint8_t mysecond; volatile uint8_t mysecond;
volatile uint8_t myminute; volatile uint8_t myminute;
volatile uint8_t myhour; volatile uint8_t myhour;
/* rtc sync flag */ // RTC sync flag.
volatile bool rtcsync = true; volatile bool rtcsync = true;
void setup() void setup()
{ {
// setup display // Setup display.
byte hardwareConfig = COMMON_ANODE; byte hardwareConfig = COMMON_ANODE;
byte numDigits = 4; byte numDigits = 4;
byte digitPins[] = {9, 10, 11, 12}; byte digitPins[] = {9, 10, 11, 12};
byte segmentPins[] = {2, 3, 4, 5, 6, 7, 8, 13}; byte segmentPins[] = {2, 3, 4, 5, 6, 7, 8, 13};
bool resistorsOnSegments = true; // 8 resistors, 330 Ohms each bool resistorsOnSegments = true; // 8 resistors, 330 Ohms each
/* updateWithDelays defaults to false and it is recommended value, /* updateWithDelays defaults to false and it is recommended value,
* but i was unable to make my display to change brightness notably with * but I was unable to make my display to change brightness notably with
* recommended value, so i have to use updateWithDelays=true, * recommended value, so i have to use updateWithDelays=true,
* in this case brightness gets changed. * in this case brightness gets changed. */
*/
bool updateWithDelays = true; bool updateWithDelays = true;
/* without it there is visual bug at 0 o'clock, and in current version /* Without it there is visual bug at 0 o'clock, and in current version
* of SevSeg' setChars() does not dots at all */ * of SevSeg' setChars() does not dots at all. */
bool leadingZeros = false; bool leadingZeros = false;
Display.begin(hardwareConfig, numDigits, digitPins, segmentPins, Display.begin(hardwareConfig, numDigits, digitPins, segmentPins,
resistorsOnSegments, updateWithDelays, leadingZeros); resistorsOnSegments, updateWithDelays, leadingZeros);
Display.setBrightness(100); Display.setBrightness(100);
// setup clock // Init i2c bus and setup clock address.
Wire.begin(); Wire.begin();
clock.set_rtc_address(0x68); clock.set_rtc_address(0x68);
// if rtc returns incorrect time, reset it to some default value // If rtc returns incorrect time, reset it to some default values.
clock.refresh(); clock.refresh();
if (clock.lostPower()) {
clock.set(30, 52, 15, 6, 23, 2, 19); // Saturday, 23 Feb 2019 15:52:30
clock.lostPowerClear();
}
if ((clock.hour() > 23) || (clock.minute() > 59)) { if ((clock.hour() > 23) || (clock.minute() > 59)) {
clock.set(30, 52, 15, 6, 23, 2, 19); // Saturday, 23 Feb 2019 15:52:30 clock.set(30, 52, 15, 6, 23, 2, 19); // Saturday, 23 Feb 2019 15:52:30
} }
// setup control buttons /* In case of ds1307 this will be noop, but ds3231/ds3232 sometimes does
* not enable battery by default. */
clock.enableBattery();
// Setup control buttons.
pinMode(hrs_btn, INPUT_PULLUP); pinMode(hrs_btn, INPUT_PULLUP);
pinMode(min_btn, INPUT_PULLUP); pinMode(min_btn, INPUT_PULLUP);
/* Timer stuff /* Timer stuff.
* timer0 being used in millis() micros() delay() etc, it has 8 bit resolution * timer0 being used in millis() micros() delay() etc, it has 8 bit resolution.
* timer2 being used in analogWrite() tone() notone(), it has 8 bit resolution * timer2 being used in analogWrite() tone() notone(), it has 8 bit resolution.
* timer1 being used in servo lib has 16 bit resolution * timer1 being used in servo lib has 16 bit resolution.
* We'll use timer1, since we have no mechanical parts and don't need servo lib * We'll use timer1, since we have no mechanical parts and don't need servo lib */
*/
// initialize timer1 // Initialize timer1.
noInterrupts(); // disable all interrupts noInterrupts(); // Disable all interrupts.
TCCR1A = 0; TCCR1A = 0;
TCCR1B = 0; TCCR1B = 0;
TCNT1 = 34286; // preload timer 65536-16MHz/256/2Hz TCNT1 = 34286; // Preload timer 65536-16MHz/256/2Hz
TCCR1B |= (1 << CS12); // 256 prescaler TCCR1B |= (1 << CS12); // 256 prescaler
TIMSK1 |= (1 << TOIE1); // enable timer overflow interrupt TIMSK1 |= (1 << TOIE1); // Enable timer overflow interrupt.
interrupts(); // enable all interrupts interrupts(); // Enable all interrupts.
/* increase sampling rate of analogRead() /* Increase sampling rate of analogRead()
* by setting an ADC prescale from 128 to 16, so it takes less time to read * by setting an ADC prescale from 128 to 16, so it takes less time to read
* sensor * sensor. */
*/
sbi(ADCSRA, ADPS2); sbi(ADCSRA, ADPS2);
cbi(ADCSRA, ADPS1); cbi(ADCSRA, ADPS1);
cbi(ADCSRA, ADPS0); cbi(ADCSRA, ADPS0);
// setup ambient brigtness sensor // Setup ambient brigtness sensor.
pinMode(photoPin, INPUT); pinMode(photoPin, INPUT);
ambLight = analogRead(photoPin); ambLight = analogRead(photoPin);
} }
ISR(TIMER1_OVF_vect) ISR(TIMER1_OVF_vect)
{ {
/* Feed clock twice per second */ // Feed clock twice per second.
TCNT1 = 34286; // preload timer TCNT1 = 34286; // Preload timer.
// do not read rtc if time beng adjusted by buttons // Do not read rtc if time beng adjusted by buttons.
(btn_pushed) ? rtcsync : rtcsync = true; (btn_pushed) ? rtcsync : rtcsync = true;
/* Also switch dot */ // Also switch dot.
if (dot) if (dot)
dot = false; dot = false;
else else
dot = true; dot = true;
#ifdef TIMECOMP #ifdef TIMECOMP
/* time compensation magic */ // Time compensation magic.
if ((myhour == 21) and (myminute == 21) and (mysecond == 21) if ((myhour == 21) and (myminute == 21) and (mysecond == 21)
and (compflag2)) { and (compflag2)) {
compflag1 = true; compflag1 = true;
@@ -176,11 +175,9 @@ void loop()
unsigned long currentMillis = millis(); unsigned long currentMillis = millis();
#ifdef TIMECOMP #ifdef TIMECOMP
/* /* Let's implement time compensation for individual rtc module time
* Let's implement time compenstion for individual rtc module time * characteristics. In in my case quartz resonator as inaccurate as
* charachteristics. In in my case quartz resonator as inaccurate as * about +9.5 seconds per 24 hours. */
* about +9.5 seconds per 24 hours
*/
if (compflag1 && compflag2) { if (compflag1 && compflag2) {
#ifdef OFIVECOMP #ifdef OFIVECOMP
if (timeSkewEven) { if (timeSkewEven) {
@@ -221,7 +218,7 @@ void loop()
rtcsync = false; rtcsync = false;
} }
// push button magic // Push button magic.
if ((!btn_pushed) and if ((!btn_pushed) and
((digitalRead(hrs_btn) == LOW) or (digitalRead(min_btn) == LOW))) { ((digitalRead(hrs_btn) == LOW) or (digitalRead(min_btn) == LOW))) {
if (currentMillis - previousPushTimer >= initial_delay) { if (currentMillis - previousPushTimer >= initial_delay) {
@@ -278,15 +275,13 @@ void loop()
(previousLightTimer > currentMillis)) { // millis() reset every ~49 days (previousLightTimer > currentMillis)) { // millis() reset every ~49 days
previousLightTimer = currentMillis; previousLightTimer = currentMillis;
ambLight = analogRead(photoPin); ambLight = analogRead(photoPin);
ambLight = (ambLight + ambLight2 + ambLight3) / 3;
/* Ambient light compensation /* Ambient light compensation.
* ambLight can be in range of 0..1023, if it 910+, it has point to use * ambLight can be in range of 0..1023, if it 910+, it has point to use
* fullbright. * fullbright.
* Use table comparsion (not function of ambLight) because of non-linear * Use table comparsion (not function of ambLight) because of non-linear
* charachteristics of led indicator, that cannot be described via simple * characteristics of led indicator, that cannot be described via simple
* formula. * formula. */
*/
if (ambLight >= 910) { if (ambLight >= 910) {
Display.setBrightness(100); Display.setBrightness(100);
} else if ((ambLight < 900) and (ambLight >= 860)) { } else if ((ambLight < 900) and (ambLight >= 860)) {
@@ -320,9 +315,6 @@ void loop()
} else if (ambLight < 100) { } else if (ambLight < 100) {
Display.setBrightness(-22); Display.setBrightness(-22);
} }
ambLight3 = ambLight2;
ambLight2 = ambLight;
} }
Display.refreshDisplay(); Display.refreshDisplay();