Skip to content

Commit ebdb5bd

Browse files
wygodadclaude
andcommitted
1594EMS1 comment: state the curve-table facts, keep the slider mapping as a hypothesis
The screen shows six sliders; the seven-byte table in the EC and the unlanded top values are the facts, and the six-sliders-at-base-plus-one mapping is the best fit pending a second capture with all six sliders set to distinct values. Comment only, shipped bytes unchanged. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 3d0d45a commit ebdb5bd

1 file changed

Lines changed: 10 additions & 7 deletions

File tree

Core/Devices.cs

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -744,13 +744,16 @@ private static string FwPrefix(string firmware)
744744
// 7 C cooler on slower fans, Extreme unlocks +34% (4472 MHz), recipes read back intact
745745
// each phase. 0xD6 flips to 03 in Extreme by itself - FIFTH board for the #52
746746
// observation. RPM at 0xC9/0xCB, single-byte divisors alive (86/87/A0 = ~2700-2840 rpm).
747-
// NO FanCurve ON PURPOSE (issue #129): this family runs a SEVEN-slot speed table at the
748-
// same bases (CPU 0x72-0x78, GPU 0x8A-0x90, six temp thresholds between the slots -
749-
// exactly MControlCenter's model, which reports working fan control on this firmware).
750-
// The owner's screenshot shows seven sliders per fan, the first pinned at 0% and the
751-
// stock top at 130% (0x82), and his dump matches that screen byte for byte. Our 6-point
752-
// reader/writer would misalign this table, so no fan-table writes until the app grows a
753-
// per-model 7-slot format with the 130% range (research list).
747+
// NO FanCurve ON PURPOSE (issue #129). Hard facts: the EC holds SEVEN plausible curve
748+
// bytes at 0x72-0x78 (00, then the four values the owner typed, then the factory 130
749+
// 130; GPU mirror at 0x8A-0x90), MSI Center's screen shows SIX sliders ending at 130%,
750+
// and the values he typed into the top sliders did not land (both EC and his re-opened
751+
// screen read 130, not the instructed 65/75). Best-fit mapping: the six sliders write
752+
// 0x73-0x78 with 0x72 as a fixed leading 00 - one byte above every other family -
753+
// which also matches MControlCenter's seven-speeds-plus-six-temps model of this region
754+
// (their fan control works on this firmware). Unconfirmed until a second capture with
755+
// all six sliders set to distinct values; meanwhile our 6-point writer at base 0x72
756+
// would land one byte off, so no fan-table writes here. Research: roadmap #107/TODO 32.
754757
new() { Name = "MSI Prestige 16 Studio A13VE / Summit E16 Flip A13VFT", FirmwarePrefixes = new[] { "1594EMS1" }, Tier = Tier.Tested,
755758
CpuRpmAddr = 0xC9, GpuRpmAddr = 0xCB, Recipes = StdRecipes(0xD2, 0xD4, 0xEB),
756759
Credit = "Flo827", CreditUrl = "https://github.com/wygodad/ghostdeck/issues/127" },

0 commit comments

Comments
 (0)