Assembly Kernel Flash Write support Intel and AMD.

.github/workflows/CheckBuild.yml

@@ -2,6 +2,7 @@
 # Original works by ddub07 <github.com>
 # 03/01/2023 - GampyG28 <github.com> major overhaul, renamed and updated.
 # 04/16/2023 - GampyG28 <github.com> Added Assembly Kernels and Loaders
+# 07/16/2023 - GampyG28 <github.com> Removed C Kernel builds
 #
 name: CheckBuild
 
@@ -25,25 +26,6 @@ jobs:
     - name: Create temporary artifact storage
       run: mkdir TemporaryArtifactStorage
 
-    - name: Build C Kernels
-      run: |
-        cd Kernels
-        #
-        # The following loop is the one location that needs to be updated when adding a new C kernel.
-        #
-        for p in "P01 FF8000" "P10 FFB800" "P12 FF2000"; do
-          pcm=${p%% *}
-          address=${p##* }
-
-          echo Building Kernel ":    ${pcm}"
-          make -f makefile PREFIX=/usr/bin/m68k-linux-gnu- pcm=${pcm} address=${address}
-          echo "    Copy-Item Kernel-${pcm}.bin to TemporaryArtifactStorage"
-          cp Kernel-${pcm}.bin ../TemporaryArtifactStorage/
-          echo "    Cleanup after Kernel-${pcm}"
-          echo
-          make clean
-        done
-
     - name: Build Assembly Kernels and Loaders
       run: |
         cd Kernels
@@ -53,7 +35,7 @@ jobs:
         # 2: Kernel Base Address
         # 3: Loader Base Address or NOLOADER if a loader is not used.
         #
-        for p in "P04 FF9090 FF9890" "P08 FFA800 FFB000" "E54 FF8F50 NOLOADER"; do
+        for p in "P01 FF8000 NOLOADER" "P04 FF9090 FF9890" "P08 FFA800 FFB000" "P10 FFB800 NOLOADER" "P12 FF2000 NOLOADER" "E54 FF8F50 NOLOADER"; do
           pcm="${p%% *}";
           p="${p#* }"
           address="${p%% *}"

Apps/PcmHammer/MainForm.cs

@@ -1161,6 +1161,16 @@ private async void readFullContents_BackgroundThread()
                     string path = "";
                     this.Invoke((MethodInvoker)delegate ()
                     {
+                        this.AddUserMessage($"WARNING: This version uses the new Assembly Kernels, USE AT YOUR OWN RISK!");
+                        string msg = $"WARNING!{Environment.NewLine}This version uses the new Assembly Kernels, USE AT YOUR OWN RISK!" +
+                                     $"{Environment.NewLine}{Environment.NewLine}Are you willing to accept the responsibility ?";
+                        DialogResult warnDialogResult = MessageBox.Show(msg, "Continue?", MessageBoxButtons.YesNo);
+                        if (warnDialogResult == DialogResult.No)
+                        {
+                            this.AddUserMessage("User chose not to proceed.");
+                            return;
+                        }
+
                         path = this.ShowSaveAsDialog();
 
                         if (path == null)
@@ -1353,6 +1363,16 @@ private async void write_BackgroundThread(WriteType writeType, string path = nul
                         this.DisableUserInput();
                         this.cancelButton.Enabled = true;
 
+                        this.AddUserMessage($"WARNING: This version uses the new Assembly Kernels, USE AT YOUR OWN RISK!");
+                        string msg = $"WARNING!{Environment.NewLine}This version uses the new Assembly Kernels, USE AT YOUR OWN RISK!" +
+                                     $"{Environment.NewLine}{Environment.NewLine}Are you willing to accept the responsibility ?";
+                        DialogResult warnDialogResult = MessageBox.Show(msg, "Continue?", MessageBoxButtons.YesNo);
+                        if (warnDialogResult == DialogResult.No)
+                        {
+                            this.AddUserMessage("User chose not to proceed.");
+                            return;
+                        }
+
                         if (string.IsNullOrWhiteSpace(path))
                         {
                             path = this.ShowOpenDialog();
@@ -1496,14 +1516,26 @@ private async void write_BackgroundThread(WriteType writeType, string path = nul
                         }
                     }
 
-                    if (writeType != WriteType.Compare && pcmInfo.HardwareType == PcmType.P04 && pcmInfo.HardwareType == PcmType.P08 && pcmInfo.HardwareType == PcmType.E54)
+                    if (writeType != WriteType.Compare && (pcmInfo.HardwareType == PcmType.P04 || pcmInfo.HardwareType == PcmType.P08))
                     {
                         string msg = $"PCMHammer currently does not support writing to the {pcmInfo.HardwareType.ToString()}";
                         this.AddUserMessage(msg);
                         MessageBox.Show(msg);
                         return;
                     }
 
+                    if (pcmInfo.HardwareType == PcmType.P04 || pcmInfo.HardwareType == PcmType.P08 || pcmInfo.HardwareType == PcmType.E54)
+                    {
+                        string msg = $"WARNING: {pcmInfo.HardwareType.ToString()} Support is still in development.";
+                        this.AddUserMessage(msg);
+                        DialogResult dialogResult = MessageBox.Show(msg, "Continue?", MessageBoxButtons.YesNo);
+                        if (dialogResult == DialogResult.No)
+                        {
+                            this.AddUserMessage("User chose not to proceed.");
+                            return;
+                        }
+                    }
+
                     await this.Vehicle.SuppressChatter();
 
                     if (needUnlock)

Apps/PcmLibrary/Misc/FileValidator.cs

@@ -415,13 +415,6 @@ private PcmType ValidateSignatures()
                         return PcmType.P04;
                 }
 
-                // P08 512Kb
-                this.logger.AddDebugMessage("Trying P08 512Kb");
-                if ((image[0x7FFFC] == 0xA5) && (image[0x7FFFD] == 0x5A) && (image[0x7FFFE] == 0xA5) && (image[0x7FFFF] == 0xA5))
-                {
-                    return PcmType.P08;
-                }
-
                 this.logger.AddDebugMessage("Trying P10 512Kb");
                 if ((image[0x17FFE] == 0x55) && (image[0x17FFF] == 0x55))
                 {
@@ -430,6 +423,13 @@ private PcmType ValidateSignatures()
                         return PcmType.P10;
                     }
                 }
+
+                // P08 512Kb
+                this.logger.AddDebugMessage("Trying P08 512Kb");
+                if ((image[0x7FFFC] == 0xA5) && (image[0x7FFFD] == 0x5A) && (image[0x7FFFE] == 0xA5) && (image[0x7FFFF] == 0xA5))
+                {
+                    return PcmType.P08;
+                }
             }
 
             // 1Mb types

Kernels/BuildAll.cmd

@@ -50,13 +50,14 @@ rem * They would need to be changed below.
   setlocal disabledelayedexpansion
 )
 
+REM   "-pP04 -aFF9090 -lFF9890 -x",
 
 for %%A in (
-  "-pP01 -aFF8000",
+  "-pP01 -aFF8000 -x",
   "-pP04 -aFF9090 -lFF9890 -x",
   "-pP08 -aFFA800 -lFFB000 -x",
-  "-pP10 -aFFB800",
-  "-pP12 -aFF2000",
+  "-pP10 -aFFB800 -x",
+  "-pP12 -aFF2000 -x",
   "-pE54 -aFF8F50 -x"
   ) do call Build.cmd %%~A %*