integralfx / MemTestHelper

C# WPF to automate HCI MemTest
2.13k stars 214 forks source link

VCCSA/VCCIO voltages table header is shifted #59

Closed KlosiakMK closed 1 year ago

KlosiakMK commented 1 year ago

I think that in the IMC section table header for second column should be "VCCIO/VCCSA (V)" instead of "VCCSA/VCCIO (V)".

integralfx commented 1 year ago

image

If you're referring to this table, this is correct

The ranges apply to both VCCSA and VCCIO

What would swapping it achieve?

KlosiakMK commented 1 year ago

Yes, the ranges are fine but when I was reading the table I was mislead by the header and thought that for example in the first row we have pair VCCSA - VCCIO (1.15 - 1.20) and started to wonder why it is not corresponding with the rule that VCCSA should be 50mV higher than VCCIO. This is why I suggested to change VCCSA/VCCIO to VCCIO/VCCSA but now I think that I have head a bad day that time. Sorry for troubling you.

integralfx commented 1 year ago

the rule that VCCSA should be 50mV higher than VCCIO.

That's not a rule, just a guideline