qt: chip selected command changed to use async read/write

Author: bbogush

qt/main_window.cpp

@@ -265,8 +265,13 @@ void MainWindow::slotProgWrite()
         qInfo() << "Data has been successfully written";
 }
 
+void MainWindow::selectChipCb()
+{
+    qInfo() << "Chip has been selected successfully";
+}
+
 void MainWindow::slotSelectChip(int selectedChipNum)
 {
-    if (!prog->selectChip(selectedChipNum))
-        qInfo() << "Chip successfully set";
+    prog->selectChip(std::bind(&MainWindow::selectChipCb, this),
+        selectedChipNum);
 }

qt/main_window.h

@@ -28,6 +28,7 @@ class MainWindow : public QMainWindow
     void insertBufferRow(quint8 *readBuf, quint32 size, quint32 rowNum,
         quint32 address);
     void readChipIdCb(ChipId id);
+    void selectChipCb();
 
 public slots:
     void slotFileOpen();

qt/programmer.cpp

@@ -488,27 +488,51 @@ int Programmer::writeChip(uint8_t *buf, uint32_t addr, uint32_t len)
     return 0;
 }
 
-int Programmer::selectChip(uint32_t chipNum)
+void Programmer::readRespSelectChipCb(int status)
 {
-    RespHeader resp;
-    Cmd cmd = { .code = CMD_NAND_SELECT };
-    SelectCmd selectCmd = { .cmd = cmd, .chipNum = chipNum };
+    uint size;
+    RespHeader *header;
 
-    if (sendCmd(&selectCmd.cmd, sizeof(SelectCmd)))
-        return -1;
+    if (status == SerialPortReader::READ_ERROR)
+        return;
 
-    if (readRespHead(&resp))
-        return -1;
+    size = readData.size();
+    if (size < sizeof(RespHeader))
+    {
+        qCritical() << "Header size of chip ID response is wrong:" << size;
+        return;
+    }
 
-    switch (resp.code)
+    header = (RespHeader *)readData.data();
+    switch (header->code)
     {
     case RESP_STATUS:
-        return handleStatus(&resp);
+        if (header->info == STATUS_OK)
+            selectChipCb();
+        else
+            qCritical() << "Programmer error: failed to select chip";
+        break;
     default:
-        return handleWrongResp(resp.code);
+        handleWrongResp(header->code);
+        break;
     }
+}
 
-    return 0;
+void Programmer::selectChip(std::function<void(void)> callback,
+    uint32_t chipNum)
+{
+    Cmd cmd = { .code = CMD_NAND_SELECT };
+    SelectCmd selectCmd = { .cmd = cmd, .chipNum = chipNum };
+
+    readData.clear();
+    serialPortReader->read(std::bind(&Programmer::readRespSelectChipCb, this,
+        std::placeholders::_1), &readData);
+
+    selectChipCb = callback;
+    writeData.clear();
+    writeData.append((const char *)&selectCmd, sizeof(selectCmd));
+    serialPortWriter->write(std::bind(&Programmer::sendCmdCb, this,
+        std::placeholders::_1), &writeData);
 }
 
 

qt/programmer.h

@@ -117,12 +117,14 @@ class Programmer : public QObject
     QSerialPort serialPort;
     bool isConn;
     std::function<void(ChipId)> readChipIdCb;
+    std::function<void(void)> selectChipCb;
 
     int sendCmd(Cmd *cmd, size_t size);
     void sendCmdCb(int status);
     int readRespHead(RespHeader *respHead);
     int readRespBadBlockAddress(RespBadBlock *badBlock);
     void readRespChipIdCb(int status);
+    void readRespSelectChipCb(int status);
     int handleStatus(RespHeader *respHead);
     int handleWrongResp(uint8_t code);
 public:
@@ -140,7 +142,7 @@ class Programmer : public QObject
     int eraseChip(uint32_t addr, uint32_t len);
     int readChip(uint8_t *buf, uint32_t addr, uint32_t len);
     int writeChip(uint8_t *buf, uint32_t addr, uint32_t len);
-    int selectChip(uint32_t chipNum);
+    void selectChip(std::function<void(void)> callback, uint32_t chipNum);
 };
 
 #endif // PROGRAMMER_H