/* This is called when a command is received over serial from TunerStudio / Megatune It parses the command and calls the relevant function A detailed description of each call can be found at: http://www.msextra.com/doc/ms1extra/COM_RS232.htm */ //#include "comms.h" //#include "globals.h" //#include "storage.h" void command() { switch (Serial.read()) { case 'A': // send 22 bytes of realtime values sendValues(22); break; case 'B': // Burn current values to eeprom writeConfig(); break; case 'C': // test communications. This is used by Tunerstudio to see whether there is an ECU on a given serial port testComm(); break; case 'P': // set the current page //A 2nd byte of data is required after the 'P' specifying the new page number. //This loop should never need to run as the byte should already be in the buffer, but is here just in case while (Serial.available() == 0) { } currentPage = Serial.read(); break; case 'R': // send 39 bytes of realtime values sendValues(39); break; case 'S': // send code version Serial.print(signature); break; case 'Q': // send code version Serial.print(signature); //Serial.write("Speeduino_0_2"); break; case 'V': // send VE table and constants sendPage(); break; case 'W': // receive new VE or constant at 'W'++ byte offset; while (Serial.available() == 0) { } offset = Serial.read(); while (Serial.available() == 0) { } receiveValue(offset, Serial.read()); break; case 't': // receive new Calibration info. Command structure: "t", . This is an MS2/Extra command, NOT part of MS1 spec byte tableID; //byte canID; //The first 2 bytes sent represent the canID and tableID while (Serial.available() == 0) { } tableID = Serial.read(); //Not currently used for anything receiveCalibration(tableID); //Receive new values and store in memory writeCalibration(); //Store received values in EEPROM break; case 'Z': //Totally non-standard testing function. Will be removed once calibration testing is completed. This function takes 1.5kb of program space! :S Serial.println("Coolant"); for(int x=0; x> 8) & 0xFF); response[27] = lowByte(currentStatus.freeRAM); response[28] = currentStatus.batCorrection; //Battery voltage correction (%) Serial.write(response, (size_t)packetSize); //Serial.flush(); return; } void receiveValue(byte offset, byte newValue) { byte* pnt_configPage; switch (currentPage) { case vePage: pnt_configPage = (byte *)&configPage1; //Setup a pointer to the relevant config page if (offset < 64) //New value is part of the fuel map { fuelTable.values[7-offset/8][offset%8] = newValue; return; } else if (offset < 80) //New value is one of the X or Y axis bins { //Check whether this is on the X (RPM) or Y (MAP/TPS) axis if (offset < 72) { //X Axis fuelTable.axisX[(offset-64)] = (int(newValue) * 100); //The RPM values sent by megasquirt are divided by 100, need to multiple it back by 100 to make it correct } else { //Y Axis offset = 7-(offset-72); //Need to do a translation to flip the order (Due to us using (0,0) in the top left rather than bottom right fuelTable.axisY[offset] = int(newValue); } return; } else //New value is one of the remaining config items { //For some reason, TunerStudio is sending offsets greater than the maximum page size. I'm not sure if it's their bug or mine, but the fix is to only update the config page if the offset is less than the maximum size if( offset < page_size) { *(pnt_configPage + byte(offset - 80)) = newValue; //Need to subtract 80 because the map and bins (Which make up 80 bytes) aren't part of the config pages } return; } break; case ignPage: //Ignition settings page (Page 2) pnt_configPage = (byte *)&configPage2; if (offset < 64) //New value is part of the ignition map { ignitionTable.values[7-offset/8][offset%8] = newValue; return; } else if (offset < 80) //New value is one of the X or Y axis bins { //Check whether this is on the X (RPM) or Y (MAP/TPS) axis if (offset < 72) { //X Axis ignitionTable.axisX[(offset-64)] = int(newValue) * int(100); //The RPM values sent by megasquirt are divided by 100, need to multiple it back by 100 to make it correct } else { //Y Axis offset = 7-(offset-72); //Need to do a translation to flip the order ignitionTable.axisY[offset] = int(newValue); } return; } else //New value is one of the remaining config items { //For some reason, TunerStudio is sending offsets greater than the maximum page size. I'm not sure if it's their bug or mine, but the fix is to only update the config page if the offset is less than the maximum size if( offset < page_size) { *(pnt_configPage + byte(offset - 80)) = newValue; //Need to subtract 80 because the map and bins (Which make up 80 bytes) aren't part of the config pages } return; } break; case afrPage: //Air/Fuel ratio target settings page (Page 3) pnt_configPage = (byte *)&configPage3; if (offset < 64) //New value is part of the afr map { afrTable.values[7-offset/8][offset%8] = newValue; return; } else if (offset < 80) //New value is one of the X or Y axis bins { //Check whether this is on the X (RPM) or Y (MAP/TPS) axis if (offset < 72) { //X Axis afrTable.axisX[(offset-64)] = int(newValue) * int(100); //The RPM values sent by megasquirt are divided by 100, need to multiply it back by 100 to make it correct } else { //Y Axis offset = 7-(offset-72); //Need to do a translation to flip the order afrTable.axisY[offset] = int(newValue); } return; } else //New value is one of the remaining config items { //For some reason, TunerStudio is sending offsets greater than the maximum page size. I'm not sure if it's their bug or mine, but the fix is to only update the config page if the offset is less than the maximum size if( offset < page_size) { *(pnt_configPage + byte(offset - 80)) = newValue; //Need to subtract 80 because the map and bins (Which make up 80 bytes) aren't part of the config pages } return; } break; default: break; } } /* sendPage() packs the data within the current page (As set with the 'P' command) into a buffer and sends it. Note that some translation of the data is required to lay it out in the way Megasqurit / TunerStudio expect it */ void sendPage() { byte response[page_size]; byte offset; byte* pnt_configPage; switch (currentPage) { case vePage: //Need to perform a translation of the values[MAP/TPS][RPM] into the MS expected format //MS format has origin (0,0) in the bottom left corner, we use the top left for efficiency reasons for(byte x=0;x<64;x++) { response[x] = fuelTable.values[7-x/8][x%8]; } //This is slightly non-intuitive, but essentially just flips the table vertically (IE top line becomes the bottom line etc). Columns are unchanged for(byte x=64;x<72;x++) { response[x] = byte(fuelTable.axisX[(x-64)] / 100); } //RPM Bins for VE table (Need to be dvidied by 100) for(byte y=72;y<80;y++) { response[y] = byte(fuelTable.axisY[7-(y-72)]); } //MAP or TPS bins for VE table //All other bytes can simply be copied from the config table pnt_configPage = (byte *)&configPage1; //Create a pointer to Page 1 in memory offset = 80; //Offset is based on the amount already copied above (table + bins) for(byte x=offset; x 255) { tempValue = 255; } // Cap the maximum value to prevent overflow when converting to byte if (tempValue < 0) { tempValue = 0; } pnt_TargetTable[(x / 2)] = (byte)tempValue; int y = EEPROM_CALIBRATION_O2 + counter; every2nd = false; analogWrite(13, (counter % 50) ); counter++; } else { every2nd = true; } } } /* Send 256 tooth log entries * if useChar is true, the values are sent as chars to be printed out by a terminal emulator * if useChar is false, the values are sent as a 2 byte integer which is readable by TunerStudios tooth logger */ void sendToothLog(bool useChar) { //We need 256 records to send to TunerStudio. If there aren't that many in the buffer (Buffer is 512 long) then we just return and wait for the next call if (toothHistoryIndex < 256) { return; } //Don't believe this is the best way to go. Just display whatever is in the buffer int tempToothHistory[512]; //Create a temporary array that will contain a copy of what is in the main toothHistory array //Copy the working history into the temporary buffer array. This is done so that, if the history loops whilst the values are being sent over serial, it doesn't affect the values memcpy( (void*)tempToothHistory, (void*)toothHistory, sizeof(tempToothHistory) ); toothHistoryIndex = 0; //Reset the history index //Loop only needs to go to 256 (Even though the buffer is 512 long) as we only ever send 256 entries at a time if (useChar) { for(int x=0; x<256; x++) { Serial.println(tempToothHistory[x]); } } else { for(int x=0; x<256; x++) { Serial.write(highByte(tempToothHistory[x])); Serial.write(lowByte(tempToothHistory[x])); } } Serial.flush(); } void testComm() { Serial.write(1); return; }