Get Write opperations working

Did a general rewrite of the cli App class, again.
Finally got the communication to work properly,
and fixed the memory write drivers which were a mess.
Also tuned the i2c timeouts according with the datasheets
and the link speed (at the moment set at 100 kHz)
This is almost ready to merge with main branch.
Only thing left is to test on Windows.

Author: feer9

eeprom_programmer_PC/eeprom-programmer.pro

@@ -20,7 +20,7 @@ DEFINES += QT_DEPRECATED_WARNINGS
 
 # Disable debug messages for release builds
 # evaluate only when "release" is defined of the two options "debug" and "release"
-#CONFIG(release, debug|release):DEFINES += QT_NO_DEBUG_OUTPUT
+CONFIG(release, debug|release):DEFINES += QT_NO_DEBUG_OUTPUT
 DEFINES += QT_NO_DEBUG_OUTPUT
 
 # You can also make your code fail to compile if you use deprecated APIs.

eeprom_programmer_PC/eeprom-programmer.pro.user

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE QtCreatorProject>
-<!-- Written by QtCreator 6.0.2, 2022-03-20T22:37:06. -->
+<!-- Written by QtCreator 6.0.2, 2022-03-23T00:50:34. -->
 <qtcreator>
  <data>
   <variable>EnvironmentId</variable>
@@ -245,7 +245,7 @@
     <value type="QString" key="ProjectExplorer.ProjectConfiguration.DisplayName">eeprom-programmer2</value>
     <value type="QString" key="ProjectExplorer.ProjectConfiguration.Id">Qt4ProjectManager.Qt4RunConfiguration:/home/feer/coding/stm32/STM32CubeIDE/eeprom-programmer/eeprom_programmer_PC/eeprom-programmer.pro</value>
     <value type="QString" key="ProjectExplorer.RunConfiguration.BuildKey">/home/feer/coding/stm32/STM32CubeIDE/eeprom-programmer/eeprom_programmer_PC/eeprom-programmer.pro</value>
-    <value type="QString" key="RunConfiguration.Arguments">X24645 -r</value>
+    <value type="QString" key="RunConfiguration.Arguments">X24645 -w</value>
     <value type="bool" key="RunConfiguration.UseCppDebugger">false</value>
     <value type="bool" key="RunConfiguration.UseCppDebuggerAuto">true</value>
     <value type="bool" key="RunConfiguration.UseLibrarySearchPath">true</value>

eeprom_programmer_PC/src/app.cpp

@@ -1,5 +1,23 @@
-#include "app.h"
+/*
+ *  EEPROM-Programmer - Read and write EEPROM memories.
+ *  Copyright (C) 2022  Fernando Coda <fcoda@pm.me>
+ *
+ *  This program is free software; you can redistribute it and/or modify
+ *  it under the terms of the GNU General Public License as published by
+ *  the Free Software Foundation; either version 2 of the License, or
+ *  (at your option) any later version.
+ *
+ *  This program is distributed in the hope that it will be useful,
+ *  but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *  GNU General Public License for more details.
+ *
+ *  You should have received a copy of the GNU General Public License along
+ *  with this program; if not, write to the Free Software Foundation, Inc.,
+ *  51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+ */
 
+#include "app.h"
 
 
 App::App(int &argc, char **argv, FILE* outStream)
@@ -47,11 +65,11 @@ void App::setSignals()
 void App::setCommandLineOptions(QCommandLineParser& parser)
 {
 	App::setApplicationName("EEPROM Programmer");
-	App::setApplicationVersion("1.0");
+	App::setApplicationVersion("2.0-rc1");
 	parser.setApplicationDescription("Read and write EEPROM memories.");
 
 	parser.addOption({{"h", "help"},
-					 "Displays help on commandline options."});
+					  "Displays help on commandline options."});
 	parser.addVersionOption();
 
 	parser.addOptions({
@@ -62,9 +80,9 @@ void App::setCommandLineOptions(QCommandLineParser& parser)
 			{{"f", "file"},
 							"Read from / write to <file>.", "file"},
 			{{"p", "port"},
-							"Select the serial port to connect.", "port"},
+							"Connect to serial port <port>.", "port"},
 			{{"b", "baudrate"},
-							"Set the desired baudrate.", "baudrate"},
+							"Set the serial port baudrate to <baudrate>.", "baudrate"},
 		});
 
 	parser.addPositionalArgument("target", "24LC16 - X24645 - 24LC64 - 24LC256");
@@ -103,12 +121,12 @@ bool App::configure() {
 	}
 
 	if(parser.isSet("write")) {
-		setOperation(CMD_WRITEMEM);
+		setNextOperation(OP_TX);
 		if(!targetFile.isNull())
 			setInputFilename(targetFile);
 	}
 	else if(parser.isSet("read")) {
-		setOperation(CMD_READMEM);
+		setNextOperation(OP_RX);
 		if(!targetFile.isNull())
 			setOutputFilename(targetFile);
 	}
@@ -153,7 +171,7 @@ void App::setOutputFilename(const QString &newFilename_out)
 	m_filename_out = newFilename_out;
 }
 
-void App::setOperation(int newOperation)
+void App::setNextOperation(operations_e newOperation)
 {
-	m_requestedOperation = newOperation;
+	m_nextOperation = newOperation;
 }

eeprom_programmer_PC/src/app.h

@@ -10,23 +10,28 @@
 #include <QCommandLineParser>
 #include <QFile>
 
-
-
 class App: public MemoryComm
 {
 
 public:
 	App(int &argc, char **argv, FILE* outStream = stdout);
 	~App();
 
-
-	void setOperation(int newOperation);
 	void setOutputFilename(const QString &newFilename_out);
 	void setInputFilename(const QString &newFilename_in);
+	void setNextOperation(operations_e newOperation);
 
 	const QString &getInputFilename() const;
 	const QString &getOutputFilename() const;
 
+	enum app_states_e {
+		ST_DISCONNECTED,
+		ST_INIT,
+		ST_IDLE,
+		ST_PING,
+		ST_WAIT_READMEM,
+		ST_WAIT_WRITEMEM
+	};
 
 private slots:
 	void handleTimeout(void);
@@ -39,7 +44,6 @@ private slots:
 	void printError(pkgdata_t *pkg);
 	bool configure(void);
 
-
 	void printData(void);
 	virtual bool saveData(void);
 	virtual void reconnect();
@@ -49,14 +53,17 @@ private slots:
 	QByteArray m_memBuffer;
 	QTimer m_pingTimer;
 
-	int m_xferState = 0;
+	app_states_e  m_xferState = ST_DISCONNECTED;
 	bool m_connected = false;
-	int m_requestedOperation = 0;
+	// Use two variables so we can change one without affecting
+	// the other (new requests will go to m_nextOperation).
+	operations_e m_currentOperation = OP_NONE;
+	operations_e m_nextOperation    = OP_NONE;
 
-	QString m_filename_in = "mem_in.bin";
+	QString m_filename_in  = "mem_in.bin";
 	QString m_filename_out = "mem_out.bin";
 	void setSignals();
-	void doStuff();
+	bool doSomething();
 };
 
 // m_ = member