ec/system76/ec: Remove unused EC RAM fields

These fields were originally added for compatibility with the
proprietary ITE EC firmware, but the System76 EC firmware does not use
them. Take the opportunity to document most of the fields as well.

Change-Id: I5581437c67ec67705ce16ba20254183a0261fd83
Signed-off-by: Jeremy Soller <jeremy@system76.com>
Signed-off-by: Tim Crawford <tcrawford@system76.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/49129
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>

src/ec/system76/ec/acpi/ec_ram.asl

@@ -3,173 +3,52 @@
 OperationRegion (ERAM, EmbeddedControl, Zero, 0xFF)
 Field (ERAM, ByteAcc, Lock, Preserve)
 {
-	NMSG,   8,
-	SLED,   4,
-	Offset (0x02),
-	MODE,   1,
-	FAN0,   1,
-	TME0,   1,
-	TME1,   1,
-	FAN1,   1,
-		,   2,
 	Offset (0x03),
-	LSTE,   1,
+	LSTE, 1,	// Lid is open
-	LSW0,   1,
+	, 1,
-	LWKE,   1,
+	LWKE, 1,	// Lid wake
-	WAKF,   1,
+	, 5,
-		,   2,
+	Offset (0x07),
-	PWKE,   1,
+	TMP1, 8,	// CPU temperature
-	MWKE,   1,
-	AC0,    8,
-	PSV,    8,
-	CRT,    8,
-	TMP,    8,
-	AC1,    8,
-	BBST,   8,
-	Offset (0x0B),
-	Offset (0x0C),
-	Offset (0x0D),
-	Offset (0x0E),
-	SLPT,   8,
-	SWEJ,   1,
-	SWCH,   1,
 	Offset (0x10),
-	ADP,    1,
+	ADP, 1,		// AC adapter connected
-	AFLT,   1,
+	, 1,
-	BAT0,   1,
+	BAT0, 1,	// Battery connected
-	BAT1,   1,
+	, 5,
-		,   3,
+	WFNO, 8,	// Wake cause (not implemented)
-	PWOF,   1,
+	Offset (0x16),
-	WFNO,   8,
+	BDC0, 32,	// Battery design capacity
-	BPU0,   32,
+	BFC0, 32,	// Battery full capacity
-	BDC0,   32,
+	Offset (0x22),
-	BFC0,   32,
+	BDV0, 32,	// Battery design voltage
-	BTC0,   32,
+	BST0, 32,	// Battery status
-	BDV0,   32,
+	BPR0, 32,	// Battery current
-	BST0,   32,
+	BRC0, 32,	// Battery remaining capacity
-	BPR0,   32,
+	BPV0, 32,	// Battery voltage
-	BRC0,   32,
+	Offset (0x3A),
-	BPV0,   32,
-	BTP0,   16,
-	BRS0,   16,
 	BCW0, 32,
 	BCL0, 32,
-	BCG0,   32,
-	BG20,   32,
-	BMO0,   64,
-	BIF0,   64,
-	BSN0,   32,
-	BTY0,   64,
-	Offset (0x67),
 	Offset (0x68),
-	ECOS,   8,
+	ECOS, 8,	// Detected OS, 0 = no ACPI, 1 = ACPI but no driver, 2 = ACPI with driver
-	LNXD,   8,
+	Offset (0xC8),
-	ECPS,   8,
-	Offset (0x6C),
-	BTMP,   16,
-	EVTN,   8,
-	Offset (0x72),
-	PRCL,   8,
-	PRC0,   8,
-	PRC1,   8,
-	PRCM,   8,
-	PRIN,   8,
-	PSTE,   8,
-	PCAD,   8,
-	PEWL,   8,
-	PWRL,   8,
-	PECD,   8,
-	PEHI,   8,
-	PECI,   8,
-	PEPL,   8,
-	PEPM,   8,
-	PWFC,   8,
-	PECC,   8,
-	PDT0,   8,
-	PDT1,   8,
-	PDT2,   8,
-	PDT3,   8,
-	PRFC,   8,
-	PRS0,   8,
-	PRS1,   8,
-	PRS2,   8,
-	PRS3,   8,
-	PRS4,   8,
-	PRCS,   8,
-	PEC0,   8,
-	PEC1,   8,
-	PEC2,   8,
-	PEC3,   8,
-	CMDR,   8,
-	CVRT,   8,
-	GTVR,   8,
-	FANT,   8,
-	SKNT,   8,
-	AMBT,   8,
-	MCRT,   8,
-	DIM0,   8,
-	DIM1,   8,
-	PMAX,   8,
-	PPDT,   8,
-	PECH,   8,
-	PMDT,   8,
-	TSD0,   8,
-	TSD1,   8,
-	TSD2,   8,
-	TSD3,   8,
-	CPUP,   16,
-	MCHP,   16,
-	SYSP,   16,
-	CPAP,   16,
-	MCAP,   16,
-	SYAP,   16,
-	CFSP,   16,
-	CPUE,   16,
-	Offset (0xC6),
-	Offset (0xC7),
-	VGAT,   8,
 	OEM1, 8,
 	OEM2, 8,
 	OEM3, 16,
-	OEM4,   8,
+	OEM4, 8,	// Extra SCI data
-	Offset (0xCE),
+	Offset (0xCD),
-	DUT1,   8,
+	TMP2, 8,	// GPU temperature
-	DUT2,   8,
+	DUT1, 8,	// Fan 1 duty
-	RPM1,   16,
+	DUT2, 8,	// Fan 2 duty
-	RPM2,   16,
+	RPM1, 16,	// Fan 1 RPM
-	RPM4,   16,
+	RPM2, 16,	// Fan 2 RPM
-	Offset (0xD7),
+	Offset (0xD9),
-	DTHL,   8,
+	AIRP, 8,	// Airplane mode LED
-	DTBP,   8,
+	WINF, 8,	// Enable ACPI brightness controls
-	AIRP,   8,
-	WINF,   8,
-	RINF,   8,
-	Offset (0xDD),
-	INF2,   8,
-	MUTE,   1,
-	Offset (0xE0),
-	RPM3,   16,
-	ECKS,   8,
-	Offset (0xE4),
-		,   4,
-	XTUF,   1,
-	EP12,   1,
-	Offset (0xE5),
-	INF3,   8,
-	Offset (0xE7),
-	GFOF,   8,
-	Offset (0xE9),
-	KPCR,   1,
-	Offset (0xEA),
-	Offset (0xF0),
-	PL1T,   16,
-	PL2T,   16,
-	TAUT,   8,
 	Offset (0xF8),
 	FCMD, 8,
 	FDAT, 8,
 	FBUF, 8,
 	FBF1, 8,
 	FBF2, 8,
-	FBF3,   8
+	FBF3, 8,
 }