Clean up SGB commands

1 files changed, 74 insertions, 44 deletions

diff --git a/src/engine/home.asm b/src/engine/home.asm
index 799d476..ebffefa 100755
--- a/src/engine/home.asm
+++ b/src/engine/home.asm
@@ -628,14 +628,18 @@ Func_04a2: ; 04a2 (0:04a2)
 	cp CONSOLE_SGB
 	ret nz
 	call EnableLCD                ;
-	ld hl, SGB_ATTR_BLK_04bf      ; send SGB data
+	ld hl, AttrBlkPacket_04bf      ; send SGB data
 	call SendSGB                  ;
 	call DisableLCD               ;
 	ret
 
-SGB_ATTR_BLK_04bf: ; 04bf (0:04bf)
+AttrBlkPacket_04bf: ; 04bf (0:04bf)
 	sgb ATTR_BLK, 1 ; sgb_command, length
-	db $01,$03,$00,$00,$00,$13,$11,$00,$00,$00,$00,$00,$00,$00,$00
+	db 1 ; number of data sets
+	; Control Code,  Color Palette Designation, X1, Y1, X2, Y2
+	db ATTR_BLK_CTRL_INSIDE + ATTR_BLK_CTRL_LINE, 0 << 0 + 0 << 2, 0, 0, 19, 17 ; data set 1
+	ds 6 ; data set 2
+	ds 2 ; data set 3
 
 ; returns vBGMapTiles + BG_MAP_WIDTH * c + b in de.
 ; used to map coordinates at bc to a BGMap0 address.
@@ -1667,81 +1671,101 @@ RST28: ; 09e9 (0:09e9)
 
 ; setup SNES memory $810-$867 and palette
 InitSGB: ; 0a0d (0:0a0d)
-	ld hl, SGB_MASK_EN_ON
+	ld hl, MaskEnPacket_Freeze
 	call SendSGB
-	ld hl, SGB_DATA_SND_0a50
+	ld hl, DataSndPacket_0a50
 	call SendSGB
-	ld hl, SGB_DATA_SND_0a60
+	ld hl, DataSndPacket_0a60
 	call SendSGB
-	ld hl, SGB_DATA_SND_0a70
+	ld hl, DataSndPacket_0a70
 	call SendSGB
-	ld hl, SGB_DATA_SND_0a80
+	ld hl, DataSndPacket_0a80
 	call SendSGB
-	ld hl, SGB_DATA_SND_0a90
+	ld hl, DataSndPacket_0a90
 	call SendSGB
-	ld hl, SGB_DATA_SND_0aa0
+	ld hl, DataSndPacket_0aa0
 	call SendSGB
-	ld hl, SGB_DATA_SND_0ab0
+	ld hl, DataSndPacket_0ab0
 	call SendSGB
-	ld hl, SGB_DATA_SND_0ac0
+	ld hl, DataSndPacket_0ac0
 	call SendSGB
-	ld hl, SGB_PAL01
+	ld hl, Pal01Packet
 	call SendSGB
-	ld hl, SGB_MASK_EN_OFF
+	ld hl, MaskEnPacket_Cancel
 	call SendSGB
 	ret
 
-SGB_DATA_SND_0a50: ; 0a50 (0:0a50)
+DataSndPacket_0a50: ; 0a50 (0:0a50)
 	sgb DATA_SND, 1 ; sgb_command, length
 	db $5d,$08,$00,$0b,$8c,$d0,$f4,$60,$00,$00,$00,$00,$00,$00,$00
 
-SGB_DATA_SND_0a60: ; 0a60 (0:0a60)
+DataSndPacket_0a60: ; 0a60 (0:0a60)
 	sgb DATA_SND, 1 ; sgb_command, length
 	db $52,$08,$00,$0b,$a9,$e7,$9f,$01,$c0,$7e,$e8,$e8,$e8,$e8,$e0
 
-SGB_DATA_SND_0a70: ; 0a70 (0:0a70)
+DataSndPacket_0a70: ; 0a70 (0:0a70)
 	sgb DATA_SND, 1 ; sgb_command, length
 	db $47,$08,$00,$0b,$c4,$d0,$16,$a5,$cb,$c9,$05,$d0,$10,$a2,$28
 
-SGB_DATA_SND_0a80: ; 0a80 (0:0a80)
+DataSndPacket_0a80: ; 0a80 (0:0a80)
 	sgb DATA_SND, 1 ; sgb_command, length
 	db $3c,$08,$00,$0b,$f0,$12,$a5,$c9,$c9,$c8,$d0,$1c,$a5,$ca,$c9
 
-SGB_DATA_SND_0a90: ; 0a90 (0:0a90)
+DataSndPacket_0a90: ; 0a90 (0:0a90)
 	sgb DATA_SND, 1 ; sgb_command, length
 	db $31,$08,$00,$0b,$0c,$a5,$ca,$c9,$7e,$d0,$06,$a5,$cb,$c9,$7e
 
-SGB_DATA_SND_0aa0: ; 0aa0 (0:0aa0)
+DataSndPacket_0aa0: ; 0aa0 (0:0aa0)
 	sgb DATA_SND, 1 ; sgb_command, length
 	db $26,$08,$00,$0b,$39,$cd,$48,$0c,$d0,$34,$a5,$c9,$c9,$80,$d0
 
-SGB_DATA_SND_0ab0: ; 0ab0 (0:0ab0)
+DataSndPacket_0ab0: ; 0ab0 (0:0ab0)
 	sgb DATA_SND, 1 ; sgb_command, length
 	db $1b,$08,$00,$0b,$ea,$ea,$ea,$ea,$ea,$a9,$01,$cd,$4f,$0c,$d0
 
-SGB_DATA_SND_0ac0: ; 0ac0 (0:0ac0)
+DataSndPacket_0ac0: ; 0ac0 (0:0ac0)
 	sgb DATA_SND, 1 ; sgb_command, length
 	db $10,$08,$00,$0b,$4c,$20,$08,$ea,$ea,$ea,$ea,$ea,$60,$ea,$ea
 
-SGB_MASK_EN_ON: ; 0ad0 (0:0ad0)
+MaskEnPacket_Freeze: ; 0ad0 (0:0ad0)
 	sgb MASK_EN, 1 ; sgb_command, length
-	db $01,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00
+	db MASK_EN_FREEZE_SCREEN
+	ds $0e
 
-SGB_MASK_EN_OFF: ; 0ae0 (0:0ae0)
+MaskEnPacket_Cancel: ; 0ae0 (0:0ae0)
 	sgb MASK_EN, 1 ; sgb_command, length
-	db $00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00
+	db MASK_EN_CANCEL_MASK
+	ds $0e
 
-SGB_PAL01: ; 0af0 (0:0af0)
+Pal01Packet: ; 0af0 (0:0af0)
 	sgb PAL01, 1 ; sgb_command, length
-	db $9c,$63,$94,$42,$08,$21,$00,$00,$1f,$00,$0f,$00,$07,$00,$00
-
-SGB_PAL23: ; 0b00 (0:0b00)
+	rgb 28, 28, 24
+	rgb 20, 20, 16
+	rgb 8, 8, 8
+	rgb 0, 0, 0
+	rgb 31, 0, 0
+	rgb 15, 0, 0
+	rgb 7, 0, 0
+	db $00
+
+Pal23Packet: ; 0b00 (0:0b00)
 	sgb PAL23, 1 ; sgb_command, length
-	db $e0,$03,$e0,$01,$e0,$00,$00,$00,$00,$7c,$00,$3c,$00,$1c,$00
-
-SGB_ATTR_BLK_0b10: ; 0b10 (0:0b10)
+	rgb 0, 31, 0
+	rgb 0, 15, 0
+	rgb 0, 7, 0
+	rgb 0, 0, 0
+	rgb 0, 0, 31
+	rgb 0, 0, 15
+	rgb 0, 0, 7
+	db $00
+
+AttrBlkPacket_0b10: ; 0b10 (0:0b10)
 	sgb ATTR_BLK, 1 ; sgb_command, length
-	db $01,$03,$09,$05,$05,$0a,$0a,$00,$00,$00,$00,$00,$00,$00,$00
+	db 1 ; number of data sets
+	; Control Code,  Color Palette Designation, X1, Y1, X2, Y2
+	db ATTR_BLK_CTRL_INSIDE + ATTR_BLK_CTRL_LINE, 1 << 0 + 2 << 2, 5, 5, 10, 10 ; data set 1
+	ds 6 ; data set 2
+	ds 2 ; data set 3
 
 ; send SGB command
 SendSGB: ; 0b20 (0:0b20)
@@ -1789,7 +1813,7 @@ SendSGB: ; 0b20 (0:0b20)
 DetectSGB: ; 0b59 (0:0b59)
 	ld bc, 60
 	call Wait
-	ld hl, SGB_MLT_REQ_2
+	ld hl, MltReq2Packet
 	call SendSGB
 	ld a, [rJOYP]
 	and $3
@@ -1818,23 +1842,25 @@ DetectSGB: ; 0b59 (0:0b59)
 	and $3
 	cp $3
 	jr nz, .sgb
-	ld hl, SGB_MLT_REQ_1
+	ld hl, MltReq1Packet
 	call SendSGB
 	or a
 	ret
 .sgb
-	ld hl, SGB_MLT_REQ_1
+	ld hl, MltReq1Packet
 	call SendSGB
 	scf
 	ret
 
-SGB_MLT_REQ_1: ; 0bab (0:0bab)
+MltReq1Packet: ; 0bab (0:0bab)
 	sgb MLT_REQ, 1 ; sgb_command, length
-	db $00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00
+	db MLT_REQ_1_PLAYER
+	ds $0e
 
-SGB_MLT_REQ_2: ; 0bbb (0:0bbb)
+MltReq2Packet: ; 0bbb (0:0bbb)
 	sgb MLT_REQ, 1 ; sgb_command, length
-	db $01,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00
+	db MLT_REQ_2_PLAYERS
+	ds $0e
 
 Func_0bcb: ; 0bcb (0:0bcb)
 	di
@@ -4177,7 +4203,7 @@ ColorizeTextBoxSGB
 	push bc
 	push de
 	ld hl, $cae0
-	ld de, SGB_ATTR_BLK_1f4f
+	ld de, AttrBlkPacket_1f4f
 	ld c, $10
 .copy_sgb_command_loop
 	ld a, [de]
@@ -4210,9 +4236,13 @@ ColorizeTextBoxSGB
 	call SendSGB
 	ret
 
-SGB_ATTR_BLK_1f4f: ; 1f4f (0:1f4f)
+AttrBlkPacket_1f4f: ; 1f4f (0:1f4f)
 	sgb ATTR_BLK, 1 ; sgb_command, length
-	db $01,$03,$04,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00,$00
+	db 1 ; number of data sets
+	; Control Code,  Color Palette Designation, X1, Y1, X2, Y2
+	db ATTR_BLK_CTRL_INSIDE + ATTR_BLK_CTRL_LINE, 0 << 0 + 1 << 2, 0, 0, 0, 0 ; data set 1
+	ds 6 ; data set 2
+	ds 2 ; data set 3
 
 Func_1f5f: ; 1f5f (0:1f5f)
 	push de