From: George Wright <gw@gwright.org.uk>
Date: Thu, 5 Dec 2024 03:43:14 +0000 (-0800)
Subject: Bodge in a UART logger
X-Git-Url: http://git.gwright.org.uk/?a=commitdiff_plain;h=410b34e6039c94f52728e34870df3e7f7e269ff8;p=WindEmu.git

Bodge in a UART logger
---

diff --git a/WindCore/arm710.h b/WindCore/arm710.h
index 21d01ef..72fab7d 100644
--- a/WindCore/arm710.h
+++ b/WindCore/arm710.h
@@ -1,5 +1,6 @@
 #pragma once
 #include <stdint.h>
+#include <functional>
 #include <optional>
 #include <variant>
 
@@ -118,10 +119,12 @@ public:
 	}
 
 	void setLogger(std::function<void(const char *)> newLogger) { logger = newLogger; }
+	void setSerialOutput(std::function<void(const char *)> newSerialOutput) { serialOutput = newSerialOutput; }
 	uint32_t lastPcExecuted() const { return pcHistory[(pcHistoryIndex - 1) % PcHistoryCount].addr; }
 public:
 	void log(const char *format, ...);
 	void logPcHistory();
+	std::function<void(const char *)> serialOutput;
 private:
 	std::function<void(const char *)> logger;
 
diff --git a/WindCore/sa1100.cpp b/WindCore/sa1100.cpp
index 3e204ac..6de60c6 100644
--- a/WindCore/sa1100.cpp
+++ b/WindCore/sa1100.cpp
@@ -9,7 +9,7 @@ Emulator::Emulator() : EmuBase(true) {
     osTimer = new OsTimer();
 	lcdController = new LCDController();
 	gpioController = new GPIOController();
-	serial3 = new Serial3();
+	serial3 = new Serial3(serialOutput);
 	resetController = new ResetController();
 	intController = new IntController(lcdController, osTimer, gpioController, serial3);
 	powerManager = new PowerManager();
diff --git a/WindCore/sa1100/serial_3.h b/WindCore/sa1100/serial_3.h
index d25d5d1..8335bed 100644
--- a/WindCore/sa1100/serial_3.h
+++ b/WindCore/sa1100/serial_3.h
@@ -34,7 +34,7 @@ class Serial3 {
     
   public:
     // Access
-    
+    Serial3(std::function<void(const char *)>& output) : m_UART(output) {}
     Uart const & get_UART() const { return m_UART; };
     
     inline void reset() { m_UART.reset(); }
diff --git a/WindCore/sa1100/uart.cpp b/WindCore/sa1100/uart.cpp
index 66ab4b4..eb75e79 100644
--- a/WindCore/sa1100/uart.cpp
+++ b/WindCore/sa1100/uart.cpp
@@ -25,6 +25,44 @@ namespace SA1100
     // Becuase of using fifo[0], fifo has to reside in a consecutive memory blocks.
     // This is why I use std::vector rather than std::deque for m_tx_fifo.
     // reinterpret_cast<SerialConsole *>(mp_output_device)->put_char_to_serial_console(m_tx_fifo);
+    std::string output;
+
+      for (std::vector<uint8_t>::iterator iter = m_tx_fifo.begin();
+           iter != m_tx_fifo.end();
+           ++iter)
+      {
+        if ((*iter) & 0x80)
+        {
+          (*iter) = 0x20;
+        }
+      }
+      
+      if (0xd == m_tx_fifo.back())
+      {
+        output.append(reinterpret_cast<char const *>(&(m_tx_fifo[0])), m_tx_fifo.size());
+        
+        m_need_append_more = true;
+      }
+      else
+      {
+        if (true == m_need_append_more)
+        {
+          output.append(reinterpret_cast<char const *>(&(m_tx_fifo[0])), m_tx_fifo.size());
+          
+          m_need_append_more = false;
+        }
+        else
+        {
+          if (false == output.empty())
+          {
+            m_output(output.c_str());
+          }
+          
+          std::string lala = std::string(reinterpret_cast<char const *>(&(m_tx_fifo[0])), m_tx_fifo.size());
+          m_output(lala.c_str());
+        }
+      }
+
   }
 
   inline void Uart::rx_data() {
@@ -34,7 +72,7 @@ namespace SA1100
     // reinterpret_cast<SerialConsole *>(mp_output_device)->get_char_from_serial_console();
   }
   
-  Uart::Uart() {
+  Uart::Uart(std::function<void(const char *)>& output) : m_output(output) {
     init_register();
   }
   
diff --git a/WindCore/sa1100/uart.h b/WindCore/sa1100/uart.h
index 8a83cd8..e6d3f9e 100644
--- a/WindCore/sa1100/uart.h
+++ b/WindCore/sa1100/uart.h
@@ -21,7 +21,7 @@
 #include <cassert>
 #include <deque>
 #include <vector>
-
+#include <functional>
 #include <stdint.h>
 
 namespace SA1100
@@ -95,7 +95,9 @@ class Uart {
     // thus I use std::vector here.
     std::vector<uint8_t> m_tx_fifo;
     std::deque<uint8_t> m_rx_fifo;
-    
+    bool m_need_append_more = false;
+    std::function<void(const char *)>& m_output;
+
     // :NOTE: Wei 2004-Mar-22:
     //
     // For now, this mp_output_device is only used in Uart<Serial_3> to output to SerialConsole.
@@ -118,7 +120,7 @@ class Uart {
     void rx_data();
     
   public:
-    Uart();
+    Uart(std::function<void(const char *)>& output);
     void reset();
     
     inline void register_output_device(void * const output_device) {
diff --git a/WindQt/mainwindow.cpp b/WindQt/mainwindow.cpp
index 3176ea1..f016f4b 100644
--- a/WindQt/mainwindow.cpp
+++ b/WindQt/mainwindow.cpp
@@ -19,6 +19,13 @@ MainWindow::MainWindow(EmuBase *emu, QWidget *parent) :
 		ui->logView->appendPlainText(fullStr);
 	});
 
+	emu->setSerialOutput([&](const char *str) {
+		QString string(str);
+		ui->serialConsole->moveCursor(QTextCursor::End);
+		ui->serialConsole->insertPlainText(str);
+		ui->serialConsole->moveCursor(QTextCursor::End);
+	});
+
     timer = new QTimer(this);
     timer->setInterval(1000/64);
     connect(timer, SIGNAL(timeout()), SLOT(execTimer()));
diff --git a/WindQt/mainwindow.ui b/WindQt/mainwindow.ui
index d176e06..4bb4640 100644
--- a/WindQt/mainwindow.ui
+++ b/WindQt/mainwindow.ui
@@ -15,6 +15,63 @@
   </property>
   <widget class="QWidget" name="centralWidget">
    <layout class="QGridLayout" name="gridLayout">
+    <item row="0" column="0" colspan="6">
+     <widget class="QLabel" name="screen">
+      <property name="focusPolicy">
+       <enum>Qt::ClickFocus</enum>
+      </property>
+      <property name="text">
+       <string/>
+      </property>
+     </widget>
+    </item>
+    <item row="1" column="3">
+     <widget class="QPushButton" name="startButton">
+      <property name="text">
+       <string>Start</string>
+      </property>
+     </widget>
+    </item>
+    <item row="1" column="2">
+     <spacer name="horizontalSpacer">
+      <property name="orientation">
+       <enum>Qt::Horizontal</enum>
+      </property>
+      <property name="sizeHint" stdset="0">
+       <size>
+        <width>40</width>
+        <height>20</height>
+       </size>
+      </property>
+     </spacer>
+    </item>
+    <item row="3" column="0" colspan="5">
+     <widget class="QLabel" name="regsLabel">
+      <property name="font">
+       <font/>
+      </property>
+      <property name="text">
+       <string/>
+      </property>
+     </widget>
+    </item>
+    <item row="2" column="4">
+     <widget class="QPushButton" name="stepTickButton">
+      <property name="text">
+       <string>Step (Tick)</string>
+      </property>
+     </widget>
+    </item>
+    <item row="1" column="4">
+     <widget class="QPushButton" name="stopButton">
+      <property name="enabled">
+       <bool>false</bool>
+      </property>
+      <property name="text">
+       <string>Stop</string>
+      </property>
+     </widget>
+    </item>
     <item row="1" column="0">
      <widget class="QLabel" name="cycleCounter">
       <property name="text">
@@ -22,10 +79,20 @@
       </property>
      </widget>
     </item>
+    <item row="2" column="3">
+     <widget class="QPushButton" name="stepInsnButton">
+      <property name="text">
+       <string>Step (Insn)</string>
+      </property>
+     </widget>
+    </item>
+    <item row="5" column="0" colspan="5">
+     <widget class="QPlainTextEdit" name="logView"/>
+    </item>
     <item row="4" column="0" colspan="5">
      <widget class="QTabWidget" name="tabWidget">
       <property name="currentIndex">
-       <number>2</number>
+       <number>3</number>
       </property>
       <widget class="QWidget" name="tab">
        <attribute name="title">
@@ -104,9 +171,7 @@
         <item row="4" column="0" colspan="13">
          <widget class="QLabel" name="memoryViewLabel">
           <property name="font">
-           <font>
-            <family>Courier New</family>
-           </font>
+           <font/>
           </property>
           <property name="text">
            <string>TextLabel</string>
@@ -183,9 +248,7 @@
         <item>
          <widget class="QLabel" name="codeLabel">
           <property name="font">
-           <font>
-            <family>Courier New</family>
-           </font>
+           <font/>
           </property>
           <property name="text">
            <string>TextLabel</string>
@@ -247,77 +310,18 @@
         </item>
        </layout>
       </widget>
+      <widget class="QWidget" name="tab_4">
+       <attribute name="title">
+        <string>UART</string>
+       </attribute>
+       <layout class="QHBoxLayout" name="horizontalLayout">
+        <item>
+         <widget class="QTextEdit" name="serialConsole"/>
+        </item>
+       </layout>
+      </widget>
      </widget>
     </item>
-    <item row="3" column="0" colspan="5">
-     <widget class="QLabel" name="regsLabel">
-      <property name="font">
-       <font>
-        <family>Courier New</family>
-       </font>
-      </property>
-      <property name="text">
-       <string/>
-      </property>
-     </widget>
-    </item>
-    <item row="1" column="3">
-     <widget class="QPushButton" name="startButton">
-      <property name="text">
-       <string>Start</string>
-      </property>
-     </widget>
-    </item>
-    <item row="2" column="4">
-     <widget class="QPushButton" name="stepTickButton">
-      <property name="text">
-       <string>Step (Tick)</string>
-      </property>
-     </widget>
-    </item>
-    <item row="0" column="0" colspan="6">
-     <widget class="QLabel" name="screen">
-      <property name="focusPolicy">
-       <enum>Qt::ClickFocus</enum>
-      </property>
-      <property name="text">
-       <string/>
-      </property>
-     </widget>
-    </item>
-    <item row="2" column="3">
-     <widget class="QPushButton" name="stepInsnButton">
-      <property name="text">
-       <string>Step (Insn)</string>
-      </property>
-     </widget>
-    </item>
-    <item row="1" column="2">
-     <spacer name="horizontalSpacer">
-      <property name="orientation">
-       <enum>Qt::Horizontal</enum>
-      </property>
-      <property name="sizeHint" stdset="0">
-       <size>
-        <width>40</width>
-        <height>20</height>
-       </size>
-      </property>
-     </spacer>
-    </item>
-    <item row="1" column="4">
-     <widget class="QPushButton" name="stopButton">
-      <property name="enabled">
-       <bool>false</bool>
-      </property>
-      <property name="text">
-       <string>Stop</string>
-      </property>
-     </widget>
-    </item>
-    <item row="5" column="0" colspan="5">
-     <widget class="QPlainTextEdit" name="logView"/>
-    </item>
    </layout>
   </widget>
  </widget>