Rev.G. Add SRAM device and update documentation

robsonsmartins · Dec 19, 2023 · fc4fc95 · fc4fc95
1 parent dd391a7
commit fc4fc95
Show file tree

Hide file tree

Showing 11 changed files with 1,609 additions and 25 deletions.
diff --git a/docs/devices_sram.graphml b/docs/devices_sram.graphml
diff --git a/docs/img/devices_sram.png b/docs/img/devices_sram.png
diff --git a/docs/img/devices_sram_resetbus.png b/docs/img/devices_sram_resetbus.png
diff --git a/docs/img/devices_sram_test1.png b/docs/img/devices_sram_test1.png
diff --git a/docs/img/devices_sram_test2.png b/docs/img/devices_sram_test2.png
diff --git a/docs/specs.odt b/docs/specs.odt
diff --git a/docs/specs.pdf b/docs/specs.pdf
diff --git a/software/usbflashprog/backend/devices/parallel/dummy.hpp b/software/usbflashprog/backend/devices/parallel/dummy.hpp
@@ -45,19 +45,27 @@ class Dummy : public Device {
     explicit Dummy(QObject *parent = nullptr);
     /** @brief Destructor. */
     virtual ~Dummy();
-
+    /* Reimplemented */
     virtual void setSize(uint32_t value);
-
+    /* Reimplemented */
     virtual bool getId(TDeviceID &result);
+    /* Reimplemented */
     virtual bool read(QByteArray &buffer);
+    /* Reimplemented */
     virtual bool program(const QByteArray &buffer, bool verify = false);
+    /* Reimplemented */
     virtual bool verify(const QByteArray &buffer);
+    /* Reimplemented */
     virtual bool erase(bool check = false);
+    /* Reimplemented */
     virtual bool blankCheck();
+    /* Reimplemented */
     virtual bool unprotect();
 
   protected:
+    /* @brief The internal buffer. */
     QByteArray buffer_;
+    /* @brief Indicates if is write protected. */
     bool protected_;
 };
 

diff --git a/software/usbflashprog/backend/devices/parallel/sram.cpp b/software/usbflashprog/backend/devices/parallel/sram.cpp
@@ -45,24 +45,25 @@ bool SRAM::program(const QByteArray &buffer, bool verify) {
         runner_.vddSet(info_.voltage.vddProgram);
         runner_.vddCtrl();
         runner_.setCE();
-        if (!doPatternTest_(current, total)) {
-            error = true;
-        }
-        if (!doRandomTest_(current, total)) {
+        runner_.usDelay(30);  // 30 uS
+        if (!doPatternTest_(current, total) || !doRandomTest_(current, total)) {
             error = true;
         }
         resetBus_();
         if (!error) {
             error = runner_.hasError();
             if (error) {
+                // fail (error)
                 emit onProgress(total, total, true, false);
             }
         }
         runner_.close();
     } else {
+        // fail (no open port)
         emit onProgress(0, total, true, false);
     }
     if (!error) {
+        // success (done)
         emit onProgress(total, total, true);
     }
     return !error;
@@ -71,26 +72,18 @@ bool SRAM::program(const QByteArray &buffer, bool verify) {
 bool SRAM::doPatternTest_(uint32_t &current, uint32_t total) {
     QByteArray buffer = generatePatternData_();
     runner_.addrClr();
-    if (!program_(buffer, current, total)) {
-        return false;
-    }
+    if (!program_(buffer, current, total)) return false;
     runner_.addrClr();
-    if (!verify_(buffer, current, total)) {
-        return false;
-    }
+    if (!verify_(buffer, current, total)) return false;
     return !runner_.hasError();
 }
 
 bool SRAM::doRandomTest_(uint32_t &current, uint32_t total) {
     QByteArray buffer = generateRandomData_();
     runner_.addrClr();
-    if (!program_(buffer, current, total)) {
-        return false;
-    }
+    if (!program_(buffer, current, total)) return false;
     runner_.addrClr();
-    if (!verify_(buffer, current, total)) {
-        return false;
-    }
+    if (!verify_(buffer, current, total)) return false;
     return !runner_.hasError();
 }
 
@@ -107,15 +100,16 @@ bool SRAM::resetBus_() {
     runner_.vppOnOE(false);
     runner_.vppOnWE(false);
     runner_.dataClr();
+    runner_.usDelay(25);  // 25 uS
     return !runner_.hasError();
 }
 
 bool SRAM::write_(uint8_t data) {
     runner_.dataSet(data);
     runner_.setWE(true);
-    runner_.usDelay(twp_);
+    runner_.usDelay(twp_);  // tWP uS
     runner_.setWE(false);
-    runner_.usDelay(twc_);
+    runner_.usDelay(twc_);  // tWC uS
     return !runner_.hasError();
 }
 

diff --git a/software/usbflashprog/main/mainwindow.cpp b/software/usbflashprog/main/mainwindow.cpp
@@ -528,12 +528,15 @@ void MainWindow::createDevice_() {
     if (device_) {
         device_->disconnect();
         delete device_;
+        device_ = nullptr;
+    }
+    createDeviceIfSRAM_(ui_->btnProgDevice->text());
+    if (!device_) {
+        device_ = new Dummy(this);
+        device_->setSize(2 * 1024);
+        ui_->actionDoProgram->setText(tr("Program"));
+        ui_->btnProgram->setToolTip(ui_->actionDoProgram->text());
     }
-
-    // createDeviceIfSRAM_(ui_->btnProgDevice->text());
-    device_ = new Dummy(this);
-    device_->setSize(2 * 1024);
-
     connect(device_, &Device::onProgress, this, &MainWindow::onActionProgress);
 }
 

diff --git a/software/usbflashprog/main/mainwindow.ui b/software/usbflashprog/main/mainwindow.ui
@@ -2980,6 +2980,7 @@
      <addaction name="actionEPROMElectricalErasable"/>
      <addaction name="actionEEPROM"/>
      <addaction name="actionFlash"/>
+     <addaction name="actionDummy"/>
     </widget>
     <addaction name="menuParallelMemory"/>
     <addaction name="separator"/>
@@ -3438,6 +3439,17 @@
     <string notr="true"/>
    </property>
   </action>
+  <action name="actionDummy">
+   <property name="text">
+    <string notr="true">Dummy</string>
+   </property>
+   <property name="iconText">
+    <string notr="true">Dummy</string>
+   </property>
+   <property name="toolTip">
+    <string notr="true">Dummy</string>
+   </property>
+  </action>
  </widget>
  <tabstops>
   <tabstop>btnSave</tabstop>