From 52bc2b14c99cc1ba196ac6796791c5c0bdbe3d23 Mon Sep 17 00:00:00 2001
From: Gunnar Skjold <gunnar@origin.no>
Date: Sun, 23 Feb 2020 20:31:10 +0100
Subject: [PATCH] Some changes after testing

---
 src/AmsToMqttBridge.ino | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/AmsToMqttBridge.ino b/src/AmsToMqttBridge.ino
index a1be1c67..50aa7835 100644
--- a/src/AmsToMqttBridge.ino
+++ b/src/AmsToMqttBridge.ino
@@ -234,7 +234,7 @@ void loop() {
 		else					  led_off();
 
 	}
-	if(now - lastRead > 150) {
+	if(now - lastRead > 100) {
 		yield();
 		readHanPort();
 		lastRead = now;
@@ -332,6 +332,11 @@ void mqttMessageReceived(String &topic, String &payload)
 
 void readHanPort() {
 	if (hanReader.read()) {
+		// Empty serial buffer. For some reason this seems to make a difference. Some garbage on the wire after package?
+		while(hanSerial->available()) {
+			hanSerial->read();
+		}
+
 		lastSuccessfulRead = millis();
 
 		if(config.getMeterType() > 0) {
@@ -534,9 +539,10 @@ void sendMqttData(String data)
 }
 
 void rgb_led(int color, int mode) {
-// Activate red and green LEDs
+// Activate red and green LEDs if RGB LED is present (HAS_RGB_LED=1)
+// If no RGB LED present (HAS_RGB_LED=0 or not defined), all output goes to ESP onboard LED
 // color: 1=red, 2=green, 3=yellow
-// mode: 0=OFF, 1=ON, 2=Short blink
+// mode: 0=OFF, 1=ON, >=2 -> Short blink(s), number of blinks: (mode - 1)
 #ifndef  HAS_RGB_LED
 #define LEDPIN_RGB_RED LED_PIN
 #define LEDPIN_RGB_GREEN LED_PIN