Add ability to correct 0.5 second per day

+ Added ability to fix timw skew of 1 second per two
   days
* Removed text of license as it shipped as file
* Fixed some grammar mistekes
* compflag now boolean
* Coding style fixes
* Tinker a bit values for brightness, again. Make display
   more dusk.

--HG--
branch : phtoresistor

.gitignore

@@ -1,6 +0,0 @@
-.vscode/*
-*.swp
-*.bak
-*.old
-*.orig
-*.clang-format

.hgignore

@@ -0,0 +1,6 @@
+^\.vscode
+\.swp$
+\.bak$
+\.old$
+\.orig$
+\.clang-format

.hgtags

@@ -0,0 +1,4 @@
+ee4999ebdd2d1b54512facafdd03c8a530ea4bfa v1.0
+1167bd4b622768ed17e116a4cbac3db90031e6f8 v1.1
+109112211e454e608911949914eaa2722fbfed7d v1.2
+20da600eb0ccd51e73722f3bdfa5833df4dab826 v1.3

README.md

@@ -12,6 +12,8 @@
 * usb phone charger as power supply
 * 2 push buttons
 * 1 on/off switch
+* 1 photoresistor
+* 1 resistor to make secund shoulder of photosensitive "bridge", 10 kOhms, 0.25 watts
 * Gainta G521G enclosure (92x66.5x28мм)
 * 5m power cord
 
@@ -24,12 +26,3 @@
 ### Additional Notes:
 
 My RTC module is not very accurate due to bad quartz chrystal calibration, looks like it is not exactly 32768 Hz and clock runs faster than it should be, giving about 5-8 seconds runaway per day. So i have to implement time correction in software (temperature changes during day/month/year is not very big - about 10 degrees Celsius, so we can use software compensation without thermal compensation).
-
-This project split into 2 branches:
-
-* **default** that contains generic clock without any additional elements
-* **photoresistor** that contains additional hardware - photocell bridge - and software algorithm that compensates ambient light sensed by photocell basing on LED display charachteristics.
-
-### Stable tags for branches:
-v1.6 - default
-v1.7 - photoresistor

simpleclock.ino

@@ -20,25 +20,25 @@ const long adjust_interval = 200; // delay between consecutive number increments
 const long bounce_delay = 20;     // contact bounce compensation
 const int hrs_btn = A0;           // put hour adjusting button on A0 lane
 const int min_btn = A1;           // put minute adjusting button on A1 lane
-volatile bool btn_pushed = false; // is button pushed?
+bool btn_pushed = false;          // is button pushed?
+unsigned long currentPushTimer;
 unsigned long previousPushTimer = 0;
+uint8_t minutes;
+uint8_t hours;
 
 /* dot blinking stuff */
-volatile bool dot = true;
+unsigned long currentMillis;
+unsigned long previousMillis = 0;
+const long interval = 500;
+bool dot = true;
 
 /* time inaccuracy compensation flag */
-volatile bool compflag1 = true;
+bool compflag = true;
-volatile bool compflag2 = false;
+const uint8_t timeSkew = 6; // assume that clock runs faster than should
-const uint8_t timeSkew = 8;          // assume that clock runs faster than should
+bool timeSkewEven = false;  // comp. 1 additional second per 2 days
-volatile bool timeSkewEven = false;  // comp. 1 additional second per 2 days
 
-/* time vars */
+/* ambient light compensation aka night mode */
-volatile uint8_t mysecond;
+const int photoPin = A2;
-volatile uint8_t myminute;
-volatile uint8_t myhour;
-
-/* rtc sync flag */
-volatile bool rtcsync = true;
 
 void setup()
 {
@@ -48,7 +48,12 @@ void setup()
 	byte digitPins[] = {9, 10, 11, 12};
 	byte segmentPins[] = {2, 3, 4, 5, 6, 7, 8, 13};
 	bool resistorsOnSegments = true; // 8 resistors, 330 Ohms each
-	bool updateWithDelays = false;   // Default. Recommended
+	/* updateWithDelays defaults to false and it is recommended value,
+	 * but i was unable to make my display to change brightness with
+	 * recommended value, so i have to use updateWithDelays=true,
+	 * in this case brightness gets changed.
+	 */
+	bool updateWithDelays = true;
 
 	/* without it there is visual bug at 0 o'clock, and in current version
 	 * of SevSeg' setChars() does not dots at all */
@@ -75,83 +80,25 @@ void setup()
 	pinMode(hrs_btn, INPUT_PULLUP);
 	pinMode(min_btn, INPUT_PULLUP);
 
-/* Timer stuff
+	// setup ambient brigtness sensor
- * timer0 being used in millis() micros() delay() etc, it has 8 bit resolution
+	pinMode(photoPin, INPUT);
- * timer2 being used in analogWrite() tone() notone(), it has 8 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
- */
-	// initialize timer1
-	noInterrupts();           // disable all interrupts
-	TCCR1A = 0;
-	TCCR1B = 0;
-
-	TCNT1 = 34286;            // preload timer 65536-16MHz/256/2Hz
-	TCCR1B |= (1 << CS12);    // 256 prescaler 
-	TIMSK1 |= (1 << TOIE1);   // enable timer overflow interrupt
-	interrupts();             // enable all interrupts
-}
-
-ISR(TIMER1_OVF_vect)
-{
-/* Feed clock twice per second */
-	TCNT1 = 34286;            // preload timer
-	// do not read rtc if time beng adjusted by buttons
-	(btn_pushed) ? rtcsync : rtcsync = true;
-
-/* Also switch dot */
-	if (dot)
-		dot = false;
-	else
-		dot = true;
-
-/* time compensation magic */
-	if ((myhour == 21) and (myminute == 21) and (mysecond == 21)
-	    and (compflag2)) {
-		compflag1 = true;
-		(timeSkewEven) ? timeSkewEven = false : timeSkewEven;
-	}
-
-	if ((myhour == 21) and (myminute == 21) and (mysecond == 23)) {
-		compflag1 = false;
-		compflag2 = true;
-	}
 }
 
 void loop()
 {
-	unsigned long currentMillis = millis();
+	currentMillis = millis();
+	currentPushTimer = currentMillis;
+	char str[4];
+	float d;
+	unsigned int ambLight = analogRead(photoPin);
 
-/*
+	clock.getTime();
- * Let's implement time compenstion for individual rtc module time
- * charachteristics. In in my case quartz resonator as inaccurate as
- * about +8.5 seconds per 24 hours
- */
-	if (compflag1 && compflag2) {
-		if (timeSkewEven) {
-			clock.fillByHMS(myhour, myminute, (mysecond - timeSkew - 1));
-		} else {
-			clock.fillByHMS(myhour, myminute, (mysecond - timeSkew));
-		}
-
-		clock.setTime();
-		compflag2 = false;
-	}
-
-	if (rtcsync) {
-		clock.getTime();
-		mysecond = clock.second;
-		myminute = clock.minute;
-		myhour = clock.hour;
-		updateTimeStr(myhour, myminute);
-		rtcsync = false;
-	}
 
 	// push button magic
 	if ((!btn_pushed) and
 	    ((digitalRead(hrs_btn) == LOW) or (digitalRead(min_btn) == LOW))) {
-		if (currentMillis - previousPushTimer >= initial_delay) {
+		if (currentPushTimer - previousPushTimer >= initial_delay) {
-			previousPushTimer = currentMillis;
+			previousPushTimer = currentPushTimer;
 			btn_pushed = true;
 		}
 	}
@@ -159,47 +106,115 @@ void loop()
 	if (btn_pushed and
 	    (digitalRead(hrs_btn) != LOW) and (digitalRead(min_btn) != LOW)) {
 		// compensate possible contact bounce with delay
-		if (currentMillis - previousPushTimer >= bounce_delay) {
+		if (currentPushTimer - previousPushTimer >= bounce_delay) {
 			btn_pushed = false;
 		}
 	}
 
 	if (btn_pushed) {
-		if (currentMillis - previousPushTimer >= adjust_interval) {
+		if (currentPushTimer - previousPushTimer >= adjust_interval) {
-			previousPushTimer = currentMillis;
+			previousPushTimer = currentPushTimer;
 
 			if (digitalRead(hrs_btn) == LOW) {
-				myhour++;
+				hours = clock.hour + 1;
-				(myhour > 23) ? myhour = 0 : myhour;
+				(hours > 23) ? hours = 0 : hours;
-				clock.fillByHMS(myhour, myminute, 0);
+				clock.fillByHMS(hours, clock.minute, 0);
 				clock.setTime();
 			}
 
 			if (digitalRead(min_btn) == LOW) {
-				myminute++;
+				minutes = clock.minute + 1;
-				(myminute > 59) ? myminute = 0 : myminute;
+				(minutes > 59) ? minutes = 0 : minutes;
-				clock.fillByHMS(myhour, myminute, 0);
+				clock.fillByHMS(clock.hour, minutes, 0);
 				clock.setTime();
 			}
 		}
+	}
 
-		updateTimeStr(myhour, myminute);
+	if (currentMillis - previousMillis >= interval) {
+		previousMillis = currentMillis;
+
+		if (dot)
+			dot = false;
+		else
+			dot = true;
+	}
+
+	/*
+	 * Let's implement time compenstion for individual rtc module time
+	 * charachteristics. In in my case quartz resonator as inaccurate as
+	 * +/- about 6.5 seconds per 24 hours, maybe its require more EMI
+	 * shielding?
+	 */
+
+	if ((clock.hour == 21) and (clock.minute == 21) and
+	    (clock.second == 21) and (compflag)) {
+
+		if (timeSkewEven) {
+			timeSkewEven = false;
+			clock.fillByHMS(clock.hour, clock.minute,
+			                (clock.second - timeSkew - 1));
+		} else {
+			timeSkewEven = true;
+			clock.fillByHMS(clock.hour, clock.minute,
+			                (clock.second - timeSkew));
+		}
+
+		clock.setTime();
+		compflag = false;
+	}
+
+	if ((clock.hour == 21) and (clock.minute == 21) and
+	    (clock.second == 23) and (!compflag)) {
+		compflag = true;
+	}
+
+	// make blinking dot
+	if (clock.hour > 0) {
+		if (dot) {
+			Display.setNumber((clock.hour * 100 + clock.minute), 2);
+		} else {
+			Display.setNumber((clock.hour * 100 + clock.minute));
+		}
+	} else {
+		/* 0 o'clock is an exception, we don't want to see "00.00", we
+		   want see "0.00" or "0 00", so we have to use this "hack" */
+		if (dot) {
+			d = float(clock.minute) / 100;
+			Display.setNumber(d, 2);
+		} else {
+			sprintf(str, "%2d%02d", clock.hour, clock.minute);
+			Display.setChars(str);
+		}
+	}
+
+	/* ambient light compensation
+	 * ambLight can be in range of 0..1023, if it 800, it has point to use
+	 * fullbright
+	 */
+	if (ambLight >= 940) {
+		Display.setBrightness(100);
+	} else if ((ambLight < 900) and (ambLight >= 840)) {
+		Display.setBrightness(75);
+	} else if ((ambLight < 800) and (ambLight >= 735)) {
+		Display.setBrightness(40);
+	} else if ((ambLight < 700) and (ambLight >= 635)) {
+		Display.setBrightness(15);
+	} else if ((ambLight < 600) and (ambLight >= 530)) {
+		Display.setBrightness(12);
+	} else if ((ambLight < 500) and (ambLight >= 430)) {
+		Display.setBrightness(10);
+	} else if ((ambLight < 400) and (ambLight >= 325)) {
+		Display.setBrightness(8);
+	} else if ((ambLight < 300) and (ambLight >= 220)) {
+		Display.setBrightness(6);
+	} else if ((ambLight < 200) and (ambLight >= 110)) {
+		Display.setBrightness(4);
+	} else if (ambLight < 100) {
+		Display.setBrightness(3);
 	}
 
 	Display.refreshDisplay();
 }
 
-void updateTimeStr(uint8_t h, uint8_t m)
-{
-	if (dot) {
-		char str[5];
-		sprintf(str, "%2d.%02d", h, m);
-		Display.setChars(str);
-	} else {
-		char str[4];
-		sprintf(str, "%2d%02d", h, m);
-		Display.setChars(str);
-	}
-}
-
 /* vim: set filetype=cpp: */