seniordesign-firmware Changeset - a2269c9fe536

Changeset - a2269c9fe536

Parent rev.

Child rev.

[Not reviewed]

Merge default

0 7 0

kripperger@CL-SEC241-09.cedarville.edu - 12 years ago 2013-01-21 19:41:29
kripperger@CL-SEC241-09.cedarville.edu

merge

7 files changed with 67 insertions and 101 deletions:

master/master/lib/led.c

master/master/lib/looptime.c

master/master/lib/serial.c

master/master/lib/serial.h

master/master/lib/serparser.c

master/master/lib/serparser.h

master/master/lib/slavesensors.c

0 comments (0 inline, 0 general)

master/master/lib/led.c

➞

Show inline comments

@@ @@ -10,12 +10,13 @@ @@
+ *
  */
 #include "../config.h"
 #include <avr/io.h>
 #include <util/delay.h>
 #include <stdbool.h>
 #include "led.h"
 // Configure port direction and initial state of LEDs
 void led_setup()
+{
 	for(int i=0; i<NUM_LEDS; i++)  {
@@ @@ -83,7 +84,22 @@ void led_spin() { @@
 		led_on(LED_ACT3);
 		led_off(LED_ACT1);
 		led_off(LED_ACT2);
 		led_off(LED_ACT0);
+	}
 	ctr = (ctr + 1) % 4;
+}
 bool pulseUp = true;
 void led_pulsate() {
 	if(OCR0B >= 240 && pulseUp)
 		pulseUp = false;
 	else if(OCR0B <= 12 && !pulseUp);
 		pulseUp = true;
 	if(pulseUp)
 		OCR0B+=5;
 	else
 		OCR0B-=5;
+}
@@ \ No newline at end of file @@

master/master/lib/looptime.c

➞

Show inline comments

@@ @@ -19,13 +19,15 @@ @@
 volatile uint32_t millis = 0; // Milliseconds since initialization
 void time_setup()
+{
 	// Generic microcontroller config options
 	OCR0A = 173; // Approx 172.7969 ticks per ms with 64 prescaler
 	OCR0B = 15;
 	OCR0B = 20; // Power LED
 	TCCR0A |= (1 << WGM01) | (1 << WGM00) | // Count until OCR0A, then overflow (wgm02 in the next line specifies this as well)
 			  (1<< COM0B1); // Non-inverting PWM on OC0B
 	TCCR0B |= (1 << CS01) | (1 << CS00) | (1 << WGM02); // clock div by 64
 	TIFR0 |= ( 1 << TOV0 ); // clear pending interrupts
 	TIMSK0 |= (1 << TOIE0); // enable overflow interrupt

master/master/lib/serial.c

➞

Show inline comments

@@ @@ -133,32 +133,26 @@ void serial1_sendString(const char* stri @@
+	{
 		serial1_sendChar(*stringPtr);
 		stringPtr++;
+	}
+}
-void serial_sendCommand( char moduleID, char sensorID, char* data )
 void serial_sendCommand(char* data )
+{
 	char checkSum = 0x00; //initialize checksum
 	serial0_sendChar('['); //bracket indicates start of command
 	serial0_sendChar(moduleID);
 	checkSum+=moduleID;
 	serial0_sendChar(sensorID);
 	checkSum+=sensorID;
 	// send data, null-terminated
 	while(*data != 0x00)
+	{
 		serial0_sendChar(*data);
 		checkSum += *data;
 		data++;
+	}
 	serial0_sendChar(']'); //bracket indicates end of command
 	serial0_sendChar(checkSum); // checksum moved behind bracket to solve bug FS#29
+}
 void serial_sendResponseData()
+{

master/master/lib/serial.h

➞

Show inline comments

@@ @@ -29,13 +29,13 @@ void serial0_sendString(const char* stri @@
 void serial1_sendString(const char* stringPtr);
 unsigned char serial0_readChar();
 uint8_t serial0_hasChar();
 char* serial0_readLine();
-void serial_sendCommand( char moduleID, char sensorID, char* data );
 void serial_sendCommand( char* data );
 void serial_sendResponseData();
 void serial1_ion();
 void serial1_ioff();
 #endif /* SERIAL_H_ */
@@ \ No newline at end of file @@

master/master/lib/serparser.c

➞

Show inline comments

@@ @@ -87,13 +87,13 @@ ISR(USART0_RX_vect) @@
 	//serial0_sendString((debugBuff));
+}
-//#define DEBUG
 #define DEBUG
 // Parse data from circular buffer
 int serparser_parse(void)
+{
 	char byte;
@@ @@ -108,44 +108,24 @@ int serparser_parse(void) @@
+		{
 			if(byte == '[') // Start of frame; keep parsing
+			{
 				#ifdef DEBUG
 				serial0_sendString("start\r\n");
 				#endif
-				setParserState(STATE_GETID);
+				setParserState(STATE_GETDATATYPE);
+			}
 			else // Not start of frame, reset
+			{
 				#ifdef DEBUG
 				serial0_sendString("invalid\r\n");
 				#endif
 				setParserState(STATE_RESET);
 				return PARSERESULT_NODATA; // no valid start bit; better luck next time. run the function the next time around the main loop.
+			}
+		}
 		// Get destination module ID
 		else if(parserState == STATE_GETID)
+		{
 			if(byte == MODULE_ID) // Message intended for this module; keep parsing
+			{
 				#ifdef DEBUG
 				serial0_sendString("dest\r\n");
 				#endif
 				checksumCalc += byte;
 				setParserState(STATE_GETDATATYPE);
+			}
 			else // Transmission is intended for another module; reset
+			{
 				#ifdef DEBUG
 				serial0_sendString("bad dest\r\n");
 				#endif
 				setParserState(STATE_RESET);
+			}
+		}
 		// Get payload type ID
 		else if(parserState == STATE_GETDATATYPE)
+		{
 			#ifdef DEBUG
 			serial0_sendString("type\r\n");
 			#endif

master/master/lib/serparser.h

➞

Show inline comments

@@ @@ -23,13 +23,12 @@ enum parseResults @@
 };
 // Parser states
 enum parseStates
+{
 	STATE_RESET = 0,
 	STATE_GETID,
 	STATE_GETDATATYPE,
 	STATE_GETDATA,
 	STATE_GETCHECKSUM,
 };

master/master/lib/slavesensors.c

➞

Show inline comments

@@ @@ -27,51 +27,24 @@ uint8_t currentSlaveSensor = 0; @@
 uint16_t slaves[MAX_SLAVES][MAX_SLAVE_SENSORS];
 bool requesting = false;
 void slavesensors_setup()
+{
 	// Empty array
 	for(int i=0; i<MAX_SLAVES; i++)
+	{
 		for(int j=0; j<MAX_SLAVE_SENSORS; j++)
+		{
 			slaves[i][j] = NONE;
+		}
+	}
 	// Slave Configuration
 	// slave 0
 	slaves[0][0] = BOARDTEMP;
 	// slave 1
 	slaves[1][0] = BOARDTEMP;
 	slaves[1][1] = HUMIDITY;
 	slaves[1][2] = TEMPERATURE;
 	slaves[1][3] = PRESSURE;
 	slaves[1][4] = AMBIENTLIGHT;
 	// slave 2
 	slaves[2][0] = BOARDTEMP;
 	slaves[2][1] = GEIGER;
 	// slave 3
 	slaves[3][0] = BOARDTEMP;
 	slaves[3][1] = CAMERA;
+}
 #define DEBUG_NETWORKSCAN
 char* bufPtr = 0x00;
 char debugBuf[64];
 char slaveAddressLow[6][9];
 char slaveAddressHigh[6][9];
 uint8_t loggerIndex = 255;
 uint8_t nodeCount = 0;
 void slavesensors_network_scan() {
 	serial0_ioff();
 	int atOK;
@@ @@ -86,13 +59,13 @@ void slavesensors_network_scan() { @@
 	atOK = slavesensors_enterAT();
 	char nameString[20] = "NONE";
 	char slaveNames[6][16]; // Hold 16-char addresses of max 6 nodes, we only need them for debug so they are local
-	uint8_t nodeCount = 0;
+	</i>
 	// wait for OK
 	//todo
 	if(atOK == 0)
+	{
 		led_on(LED_CYCLE);
@@ @@ -182,18 +155,46 @@ void slavesensors_network_scan() { @@
 		if(strcmp(slaveNames[i], XBEE_LOGDEST_NAME) == 0)
+		{
 			loggerIndex = i;
+		}
+	}
-	slavesensors_loggerdest();
+	slavesensors_selectlogger();
 	serial0_ion();
+}
 void slavesensors_loggerdest()
 void slavesensors_selectnode(uint8_t nodeIndex)
+{
 	char tmpBuf[23];
 	// If we can get into AT mode
 	if(slavesensors_enterAT() == 0) {
 		snprintf(tmpBuf, 23, "ATDH %s%c",slaveAddressHigh[nodeIndex], 0x0D);
 		serial0_sendString(tmpBuf);
 		if(xbeeIsOk() != 0) {
 			led_errorcode(ERROR_NOXBEE);
 			return;
+		}
 		snprintf(tmpBuf, 23, "ATDL %s%c",slaveAddressLow[nodeIndex], 0x0D);
 		serial0_sendString(tmpBuf);
 		if(xbeeIsOk() != 0) {
 			led_errorcode(ERROR_NOXBEE);
 			return;
+		}
 		slavesensors_exitAT();
+	}
 	return;
+}
 void slavesensors_selectlogger()
+{
 	if(loggerIndex != 255) {
 		slavesensors_selectnode(loggerIndex);
+	}
+}
@@ @@ -241,40 +242,12 @@ int xbeeIsOk() @@
+	{
 		led_errorcode(ERROR_NOXBEE);
 		return 1;
+	}
+}
 void slavesensors_selectnode(uint8_t nodeIndex)
+{
 	char tmpBuf[23];
 	// If we can get into AT mode
 	if(slavesensors_enterAT() == 0) {
 		snprintf(tmpBuf, 23, "ATDH %s%c",slaveAddressHigh[nodeIndex], 0x0D);
 		serial0_sendString(tmpBuf);
 		if(xbeeIsOk() != 0) {
 			led_errorcode(ERROR_NOXBEE);
 			return;
+		}
 		snprintf(tmpBuf, 23, "ATDL %s%c",slaveAddressLow[nodeIndex], 0x0D);
 		serial0_sendString(tmpBuf);
 		if(xbeeIsOk() != 0) {
 			led_errorcode(ERROR_NOXBEE);
 			return;
+		}
 		slavesensors_exitAT();
+	}
 	return;
+}
 bool slavesensors_isrequesting()
+{
 	return requesting;
+}
 void slavesensors_startprocess()
@@ @@ -283,15 +256,21 @@ void slavesensors_startprocess() @@
 	slavesensors_request();
+}
 // TODO: inline. static.
 void slavesensors_request()
+{
 	serial_sendCommand(currentSlave + 0x30, slaves[currentSlave][currentSlaveSensor] + 0x30, "");
 	//slavesensors_selectnode(currentSlave);
 	serial_sendCommand("@"); // Request data!
+}
 int numValues = 0; // number of values that the slave is about to send (testing)
 // TODO: needs to skip logger!
 void slavesensors_process(uint8_t parseResult)
+{
 	if(!requesting) {
 		// we got a command when we didn't request anything. probably skip it.
 		return;
+	}
@@ @@ -307,32 +286,28 @@ void slavesensors_process(uint8_t parseR @@
 		// We got some data, let's handle it
 		// ASCII payload
 		uint8_t len = getPayloadLength();
 		uint8_t* load = getPayload();
 		uint8_t type = getPayloadType();
-		// TODO: Check if type matches. if it doesn't, we have a problem...
 		uint16_t parsedVal = atoi(load);
 		if(slaves[currentSlave][currentSlaveSensor] == BOARDTEMP) {
 			sensordata_setBoardTemp(currentSlave, parsedVal);
+		}
 		else {
 			sensordata_set(slaves[currentSlave][currentSlaveSensor], parsedVal);
+		}
 		// Store data in structure
 		//sensordata_set(slaves[currentSlave][currentSlaveSensor], parsedVal);
 		// If we finished all sensors for all slaves
-		if(slaves[currentSlave+1][0] == NONE && slaves[currentSlave][currentSlaveSensor+1] == NONE) // If next sensor is none and finished all slaves, reset
+		if(currentSlave >= nodeCount && currentSlaveSensor >= numValues)
+		{
 			currentSlave = 0;
 			currentSlaveSensor = 0;
 			requesting = false;
+		}
 		// If we finished up one slave, go to the next
-		else if(slaves[currentSlave][currentSlaveSensor+1] == NONE)
+		else if(currentSlaveSensor >= numValues)
+		{
 			currentSlave++;
 			currentSlaveSensor = 0;
 			requesting = true;
 			slavesensors_request();
+		}

0 comments (0 inline, 0 general)