X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=arducontrol%2Fmonitor.c;h=99f0d4598d27f40f4cf05d156ddf89e5070c131c;hb=HEAD;hp=9b5154dddfcf1d2209b3253436ac9b0ee2289421;hpb=3fb9da57664c39827f687d664343488b0136f4e1;p=model-railway-devices.git

diff --git a/arducontrol/monitor.c b/arducontrol/monitor.c
index 9b5154d..99f0d45 100644
--- a/arducontrol/monitor.c
+++ b/arducontrol/monitor.c
@@ -6,19 +6,19 @@
 
 static uint16_t track_current_samples[16] = { 0 };
 static uint8_t track_current_head = 0;
-static volatile uint16_t track_current_sum = 0;
+static uint16_t track_current_sum = 0;
+static uint16_t track_current_peak = 0;
 static uint16_t overcurrent_limit = 9707;
 static uint8_t overcurrent_sent = 0;
 
 static uint16_t input_voltage_samples[16] = { 0 };
 static uint8_t input_voltage_head = 0;
-static volatile uint16_t input_voltage_sum = 0;
+static uint16_t input_voltage_sum = 0;
 
 static volatile uint8_t adc_state = 0;
 static volatile uint16_t adc_value = 0;
 
-static uint16_t track_current_milliamps(void);
-static uint16_t input_voltage_millivolts(void);
+static uint16_t current_milliamps(uint16_t);
 
 void monitor_init(void)
 {
@@ -40,6 +40,9 @@ void monitor_check(void)
 			track_current_sum += value;
 			track_current_head = (i+1)&15;
 
+			if(track_current_sum>track_current_peak)
+				track_current_peak = track_current_sum;
+
 			if(track_current_sum>overcurrent_limit)
 			{
 				output_set_power(0);
@@ -74,15 +77,19 @@ uint8_t monitor_command(const uint8_t *cmd_buf, uint8_t cmd_length)
 {
 	if(cmd_buf[0]==READ_TRACK_CURRENT)
 	{
-		uint8_t reply[3];
-
 		if(cmd_length!=1)
 			return LENGTH_ERROR;
 
-		uint16_t value = track_current_milliamps();
+		uint16_t value = monitor_track_current();
+		uint16_t peak = current_milliamps(track_current_peak);
+		track_current_peak = 0;
+
+		uint8_t reply[5];
 		reply[0] = TRACK_CURRENT;
 		reply[1] = value>>8;
 		reply[2] = value;
+		reply[3] = peak>>8;
+		reply[4] = peak;
 		interface_send(reply, sizeof(reply));
 	}
 	else if(cmd_buf[0]==SET_OVERCURRENT_LIMIT)
@@ -101,12 +108,11 @@ uint8_t monitor_command(const uint8_t *cmd_buf, uint8_t cmd_length)
 	}
 	else if(cmd_buf[0]==READ_INPUT_VOLTAGE)
 	{
-		uint8_t reply[3];
-
 		if(cmd_length!=1)
 			return LENGTH_ERROR;
 
-		uint16_t value = input_voltage_millivolts();
+		uint16_t value = monitor_input_voltage();
+		uint8_t reply[3];
 		reply[0] = INPUT_VOLTAGE;
 		reply[1] = value>>8;
 		reply[2] = value;
@@ -118,10 +124,8 @@ uint8_t monitor_command(const uint8_t *cmd_buf, uint8_t cmd_length)
 	return COMMAND_OK;
 }
 
-static uint16_t track_current_milliamps(void)
+static uint16_t current_milliamps(uint16_t value)
 {
-	uint16_t value = track_current_sum;
-
 	// Convert to milliamps: (v/16*5/1024-2.5)*1000/0.185
 	if(value<8192)  // Ignore negative current readings
 		return 0;
@@ -135,7 +139,12 @@ static uint16_t track_current_milliamps(void)
 	}
 }
 
-static uint16_t input_voltage_millivolts(void)
+uint16_t monitor_track_current(void)
+{
+	return current_milliamps(track_current_sum);
+}
+
+uint16_t monitor_input_voltage(void)
 {
 	uint16_t value = input_voltage_sum;