- 6.0.2.2 Fixed PAULA_8364(AMIGA) annoying noize issue.

Author: 110-kenichi

README.md

@@ -1,4 +1,4 @@
-MAmidiMEmo 6.0.2.1 Itoken (c)2019, 2025 / GPL-2.0
+MAmidiMEmo 6.0.2.2 Itoken (c)2019, 2025 / GPL-2.0
 
 *** What is the MAmidiMEmo? ***
 
@@ -274,6 +274,7 @@ e.g.) YM2151 has 8ch FM sounds, so you can play 8 chords on MIDI 1ch or sharing
    [![Donate](https://img.shields.io/badge/Donate-PayPal-green.svg)](https://www.paypal.com/cgi-bin/webscr?cmd=_s-xclick&hosted_button_id=SNQ9JE3JAQMNQ)
 
 *** Changes
+- 6.0.2.2 Fixed PAULA_8364(AMIGA) annoying noize issue.
 - 6.0.2.1 Fixed PAULA_8364(AMIGA) freezing issue.
 - 6.0.2.0 Improved UART transmission of PAULA_8364(AMIGA).
 - 6.0.1.0 Supported importing *.sf file to PAULA_8364(AMIGA).

src/VSIF/VSIF_AMIGA/audio.c

@@ -18,7 +18,7 @@ void aud_memcpy(volatile struct AudChannel *dest, volatile struct AudChannel *sr
 }
 
 
-void reqPlayPcm(int ch, int id,  UWORD volume, UWORD period)
+void reqPlayPcm(UBYTE ch, UBYTE id,  UWORD volume, UWORD period)
 {
 	if(pcmDataTable[id].dataPtr == 0)
 		return;
@@ -73,13 +73,15 @@ void reqPlayPcm(int ch, int id,  UWORD volume, UWORD period)
 
 /*
 */
-void reqStopPcm(int ch)
+void reqStopPcm(UBYTE ch)
 {
 	// custom->intena = (INTF_AUD0 << ch);
 	// custom->intreq = (INTF_AUD0 << ch); custom->intreq = (INTF_AUD0 << ch);
 
+	custom->dmacon = (DMAF_AUD0 << ch); // Stop DMA for AUD
+
 	ULONG *audl = (ULONG *)&custom->aud[ch];
-	*audl++ = (ULONG)0;
+	*audl++ = (ULONG)StopData;
 	UWORD *aud = (UWORD *)audl;
 	*aud++ = 0;
 	*aud++ = 1;

src/VSIF/VSIF_AMIGA/main.c

@@ -378,28 +378,30 @@ VWritef("Completed serial setting.\n", NULL);
 			case 1:	// Volume
 				{
 					requestSerial(5);
-
+					
 					UWORD vol = *dataBufPtr;
 #ifndef NO_LOOP
 					curPlayData[ch].aud.ac_vol = vol;
 #endif
 					custom->aud[ch].ac_vol = vol;
 #ifdef LOG
-					VWritef("Vol\n", NULL);
+					ULONG arg[] = {vol};
+					VWritef("Vol %N\n", arg);
 #endif
 				}
 				break;
 			case 2:	// Pitch
 				{
 					requestSerial(5);
 
-					UWORD per = ((UWORD)*dataBufPtr++ << 8) + *dataBufPtr;
+					UWORD per = *(UWORD *)dataBufPtr;
 #ifndef NO_LOOP
 					curPlayData[ch].aud.ac_per = per;
 #endif
 					custom->aud[ch].ac_per = per;
 #ifdef LOG
-					VWritef("Pitch\n", NULL);
+					ULONG arg[] = {per};
+					VWritef("Pitch %N\n", arg);
 #endif
 				}
 				break;
@@ -412,7 +414,7 @@ VWritef("Completed serial setting.\n", NULL);
 					// vol 0 - 64;
 					UWORD vol = *dataBufPtr++;
 					// period 0 - 65535
-					UWORD per = ((UWORD)*dataBufPtr++ << 8) + *dataBufPtr;
+					UWORD per = *(UWORD *)dataBufPtr;
 
 					reqPlayPcm(ch, id, vol, per);
 #ifdef LOG
@@ -437,13 +439,10 @@ VWritef("Completed serial setting.\n", NULL);
 					requestSerial(5);
 
 					if(*dataBufPtr == 0)
-		{
+					{
 						//Filter off
 						__asm volatile (
-							"BSET.B #1,0xbfe001\n"
-							:
-							:
-							:
+							"BSET.B #1,0xbfe001\n":::
 							"cc", "memory");
 #ifdef LOG
 						VWritef("Flt OFF\n", NULL);
@@ -452,10 +451,7 @@ VWritef("Completed serial setting.\n", NULL);
 					{
 						//Filter on
 						__asm volatile (
-							"BCLR.B #1,0xbfe001\n"
-							:
-							:
-							:
+							"BCLR.B #1,0xbfe001\n":::
 							"cc", "memory");
 #ifdef LOG
 							VWritef("Flt ON\n", NULL);
@@ -470,8 +466,8 @@ VWritef("Completed serial setting.\n", NULL);
 #endif
 
 					UBYTE id = *dataBufPtr++;
-					UWORD len = ((UWORD)*dataBufPtr++ << 8) + *dataBufPtr++;
-					//UWORD loop = ((UWORD)*dataBufPtr++ << 8) + *dataBufPtr;
+					UWORD len = *(UWORD *)dataBufPtr++;
+					//UWORD loop = *(UWORD *)dataBufPtr++;
 					//if(loop >= len)
 					//	loop = 0xFFFF;
 					UWORD loop = 0;
@@ -532,7 +528,7 @@ VWritef("Completed serial setting.\n", NULL);
 					requestSerial(5);
 
 					UBYTE id = *dataBufPtr++;
-					UWORD loop = ((UWORD)*dataBufPtr++ << 8) + *dataBufPtr;
+					UWORD loop = *(UWORD *)dataBufPtr;
 					UWORD len = pcmDataTable[id].length;
 					if(loop >= len)
 						loop = 0xFFFF;

src/VSIF/VSIF_AMIGA/main.h

@@ -110,7 +110,7 @@ struct AudioMessage
 extern struct PcmData pcmDataTable[256];
 extern volatile struct PlayData curPlayData[4];
 
-void reqPlayPcm(int ch, int id,  UWORD volume, UWORD period);
-void reqStopPcm(int ch);
+void reqPlayPcm(UBYTE ch, UBYTE id,  UWORD volume, UWORD period);
+void reqStopPcm(UBYTE ch);
 void audioTask(void);
 

src/VSIF/VSIF_AMIGA/out/VSIF_AMIGA.exe

@@ -41,7 +41,7 @@ public static class Program
         /// <summary>
         /// 
         /// </summary>
-        public const string FILE_VERSION = "6.0.2.1";
+        public const string FILE_VERSION = "6.0.2.2";
 
         public const string FILE_COPYRIGHT = @"Virtual chiptune sound MIDI module ""MAmidiMEmo"" Version {0}
 Copyright(C) 2019, 2025 Itoken.All rights reserved.";

src/mamidimemo/Program.cs

@@ -551,7 +551,8 @@ private void Paula8364Write(uint unitNumber, PAULA_CMD type, uint address, ushor
                             {
                                 case SoundEngineType.VSIF_AMIGA:
                                     // ch, value
-                                    vsifClient.RawWriteData(new byte[] { (byte)((byte)type | (ch << 4)), (byte)data, 0, 0, 0 }, null);
+                                    vsifClient.RawWriteData(new byte[] { (byte)((byte)type | (ch << 4)),
+                                        (byte)data, 0, 0, 0 }, null);
                                     break;
                             }
                         }
@@ -569,7 +570,8 @@ private void Paula8364Write(uint unitNumber, PAULA_CMD type, uint address, ushor
                             {
                                 case SoundEngineType.VSIF_AMIGA:
                                     // ch, value
-                                    vsifClient.RawWriteData(new byte[] { (byte)((byte)type | (ch << 4)), (byte)(data >> 8), (byte)data, 0, 0 }, null);
+                                    vsifClient.RawWriteData(new byte[] { (byte)((byte)type | (ch << 4)),
+                                        (byte)(data >> 8), (byte)data, 0, 0 }, null);
                                     break;
                             }
                         }