From 4e49d0d0e6c16d0ffabcd3f63432def96241e522 Mon Sep 17 00:00:00 2001 From: "Dr. Christian Giessen" Date: Wed, 22 Apr 2026 20:36:54 +0000 Subject: [PATCH] Fix reg1/reg2 tables: CNTR is REG2[14], STACKA is REG1[15] Register tables were wrong per the official core register table (docs/opcode_mnemonics.txt Table 9-5): - reg1[15]: was CNTR, should be STACKA - reg2[12]: was STACKA, should be RES (reserved) - reg2[13]: was LPCSTACKA, should be RES (reserved) - reg2[14]: was RES, should be CNTR - reg2[15]: was RES, should be LPSTACKA This fixes CNTR loads showing as 'RES = ...' in disassembly. Verified: isa_test.bin now shows CNTR = 0x000A, iir.bin shows CNTR = 0x0100 and CNTR = 0x0002. --- r2plugin/asm_adsp219x.c | 4 ++-- r2plugin/asm_adsp219x.so | Bin 28512 -> 28512 bytes 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/r2plugin/asm_adsp219x.c b/r2plugin/asm_adsp219x.c index db0b481..d67d4b0 100644 --- a/r2plugin/asm_adsp219x.c +++ b/r2plugin/asm_adsp219x.c @@ -19,10 +19,10 @@ static const char *reg0[] = { "AX0", "AX1", "MX0", "MX1", "AY0", "AY1", "MY0", "MY1", "MR2", "SR2", "AR", "SI", "MR1", "SR1", "MR0", "SR0" }; static const char *reg1[] = -{ "I0", "I1", "I2", "I3", "M0", "M1", "M2", "M3", "L0", "L1", "L2", "L3", "IMASK", "IRPTL", "ICNTL", "CNTR" }; +{ "I0", "I1", "I2", "I3", "M0", "M1", "M2", "M3", "L0", "L1", "L2", "L3", "IMASK", "IRPTL", "ICNTL", "STACKA" }; static const char *reg2[] = -{ "I4", "I5", "I6", "I7", "M4", "M5", "M6", "M7", "L4", "L5", "L6", "L7", "STACKA", "LPCSTACKA", "RES", "RES" }; +{ "I4", "I5", "I6", "I7", "M4", "M5", "M6", "M7", "L4", "L5", "L6", "L7", "RES", "RES", "CNTR", "LPSTACKA" }; static const char *reg3[] = { "ASTAT", "MSTAT", "SSTAT", "LPSTACKP", "CCODE", "SE", "SB", "PX", "DMPG1", "DMPG2", "IOPG", "IJPG", "RES", "RES", "RES", "STACKP" }; diff --git a/r2plugin/asm_adsp219x.so b/r2plugin/asm_adsp219x.so index 883de27c527fe7f440a6e17efcde40030f19d9ff..8632862fac89515f77baee7541716917eddfbd29 100755 GIT binary patch delta 1127 zcmZA1PiPZC6bA52LuyilZ0O3eD%2vF9z?xq5d#vMMA&K@6cd`(lophxVg#|EmHtsN zV5tdDu;3x7hk^&8%k)r5(FPQHT+oB@&?a~h@zRq7l<~#9QJjV0<(uC!kll~1ZL``o z%R9aeuj|jXQK-HR=+RdAJK-fO>W7xszufy){`%{hb2N8%xg!=n_K~OBe!5vdB-^jB ze*U_B$_Hu93#T-#IRgW!)6i_daW)+P?1@#i?tfC3nMI9o| zD>?$*0mB?6Eh#zakvfX5 zL0vaknKyKgpS|JYaHfsj;h7^Jy7q+6UyQq~0XQ2u1NS2#*oyQd({&-9>QI^$Nq%6y_ zL|N9ONLdzQg|e)GOl=d(7`;I#e>peg8fdjm$}3j6NYT J-a8p|{{|+R<`Vz_ delta 1108 zcmZ9MQD{<87{@(xiavQI;bI9U!E8^hmtsUKS?NYYG=$BiBy6mP1`-4Jv}qf~U@+|? zVh|+tuzCsIVNhVGFk+8JPwQdoV-UrjLMTq(y5CuK4t)22{@?GMdpX=cr*ptM2dvn# z@7hd%#%^i9?(~EVbFA>1J$F@erME9mFWmcbwLZ}PHd4J1e)XLv`hJ?(IK-{D**Je^ z&DmK1&TBfPG#iV64`+8Z4TdzEGnFu2N*^+fK}plyH-bhttI#kskw%H;6&iy>q?1I8 z3Y`X@Q+FqcmKB5KLV-4_u)obj7~ E1FB=?YXATM