From 6ee7d2f247afe4ac30c0a42a58806956a0bedc69 Mon Sep 17 00:00:00 2001
From: Uri Shaked
Date: Thu, 9 Jul 2020 16:09:44 +0300
Subject: fix(usart): TXC interrupt triggered incorrectly

close #51
---
 src/peripherals/usart.spec.ts | 20 +++++++++++++++++---
 1 file changed, 17 insertions(+), 3 deletions(-)

(limited to 'src/peripherals/usart.spec.ts')

diff --git a/src/peripherals/usart.spec.ts b/src/peripherals/usart.spec.ts
index cda141d..2f976ec 100644
--- a/src/peripherals/usart.spec.ts
+++ b/src/peripherals/usart.spec.ts
@@ -23,6 +23,10 @@ const TXCIE = 0x40;
 const TXC = 0x40;
 const UDRE = 0x20;
 
+// Interrupt address
+const PC_INT_UDRE = 0x26;
+const PC_INT_TXC = 0x28;
+
 describe('USART', () => {
   it('should correctly calculate the baudRate from UBRR', () => {
     const cpu = new CPU(new Uint16Array(1024));
@@ -103,23 +107,33 @@ describe('USART', () => {
       cpu.writeData(0xc6, 0x61);
       cpu.data[SREG] = 0x80; // SREG: I-------
       usart.tick();
-      expect(cpu.pc).toEqual(0x26);
+      expect(cpu.pc).toEqual(PC_INT_UDRE);
       expect(cpu.cycles).toEqual(2);
       expect(cpu.data[UCSR0A] & UDRE).toEqual(0);
     });
 
-    it('should trigger data tx complete interrupt if TXCIE is set', () => {
+    it('should trigger data TX Complete interrupt if TXCIE is set', () => {
       const cpu = new CPU(new Uint16Array(1024));
       const usart = new AVRUSART(cpu, usart0Config, FREQ_16MHZ);
       cpu.writeData(UCSR0B, TXCIE | TXEN);
       cpu.writeData(UDR0, 0x61);
       cpu.data[SREG] = 0x80; // SREG: I-------
       usart.tick();
-      expect(cpu.pc).toEqual(0x28);
+      expect(cpu.pc).toEqual(PC_INT_TXC);
       expect(cpu.cycles).toEqual(2);
       expect(cpu.data[UCSR0A] & TXC).toEqual(0);
     });
 
+    it('should not trigger data TX Complete interrupt if UDR was not written to', () => {
+      const cpu = new CPU(new Uint16Array(1024));
+      const usart = new AVRUSART(cpu, usart0Config, FREQ_16MHZ);
+      cpu.writeData(UCSR0B, TXCIE | TXEN);
+      cpu.data[SREG] = 0x80; // SREG: I-------
+      usart.tick();
+      expect(cpu.pc).toEqual(0);
+      expect(cpu.cycles).toEqual(0);
+    });
+
     it('should not trigger any interrupt if interrupts are disabled', () => {
       const cpu = new CPU(new Uint16Array(1024));
       const usart = new AVRUSART(cpu, usart0Config, FREQ_16MHZ);
-- 
cgit v1.2.3