search

masagrator / Status-Monitor-Overlay

Monitor many stats of Nintendo Switch hardware
GNU General Public License v2.0
496 stars 32 forks source link

the real RAM freq is not reading correctly above 1600, ver 1.0.0 #67

Closed WE1ZARD closed 10 months ago

WE1ZARD commented 10 months ago

the real RAM freq is not reading correctly above 1600, (target 2502 shows -1787.-3/-1787.-4) tested both with rc4 and b6 and the reading is fine in sys-clk overlays.

AMS 1.6.2 FW16.03/FW 17.0

masagrator commented 10 months ago

It's definitely not "above 1600" as it works fine at 1996. Can you provide more specific value above which this happens? And please provide a screenshot / pic.

masagrator commented 10 months ago

Ah, okay. I see the issue in code. It's for anything above int32_max.

masagrator commented 10 months ago

For tests Status-Monitor-Overlay.zip If I will find anybody else who will test it before you that will confirm issue is fixed, I will close this and upload that as shadow update.

WE1ZARD commented 10 months ago

the issues seems fixed with the test ovl, now it read 2502 correctly, i will test other freq and feedback here.

masagrator commented 10 months ago

Since everything else follow the same pattern, next issue will come if we will get anything to work at >4.2 GHz which I bet will never happen. :P But still there is one thing I need to fix. Thanks for making this issue, this bug shouldn't happen to begin with if I would think of that solution first when trying to fix why deltas are not converting to signed integers.

masagrator commented 10 months ago

Uploaded shadow update fixing your issue + one more that would come out because of that fix eventually.