FS2011 (GD32F150): Battery, temperature readings wrong

Gissio / radpro

Custom firmware for Geiger counters/radiation meters (FS2011, Bosean FS-600, FS-1000, FS-5000, FNIRSI GC-01)

MIT License

154 stars 20 forks source link

FS2011 (GD32F150): Battery, temperature readings wrong #21

Closed mryndzionek closed 5 months ago

mryndzionek commented 5 months ago

The temperature shows >300C and the battery icon shows "charging" all the time.

Gissio commented 5 months ago

Another regression... could you also please check these?

Dead-time measurement works (takes some hours).
Random generator works.
Display contrast/brightness settings work.
Device clock works.
radpro-tool works (over SWD).
Data logging works (read datalog with radpro-tool).

mryndzionek commented 5 months ago

☑ Display contrast/brightness settings - works
☑ Device clock - works
☒ Random generator - doesn't seem to be working
☑ radpro-tool works (over SWD) (reading temperature, etc.)
☒ Data logging (read datalog with radpro-tool) - doesn't work - error AP fault at address: 0x20000c84

Gissio commented 5 months ago

Can you try again and see if the ADC works?

radpro-fs2011-gd32f150c8-2.0beta17-test1-install.bin.zip

The statistics view will not display the dead-time, but rather two hexadecimal numbers separated by ":". Both should change. Let me know if they do not change, if only one changes (which one) or if both change. Also, if both change, let me know which one changes faster.

pd: each number represents the count of a 16-bit timer. They should chain to form a 32-bit timer. If the timers do not work, there is no dead-time measurement nor random number generation.

mryndzionek commented 5 months ago

Can you try again and see if the ADC works?

radpro-fs2011-gd32f150c8-2.0beta17-test1-install.bin.zip

The statistics view will not display the dead-time, but rather two hexadecimal numbers separated by ":". Both should change. Let me know if they do not change, if only one changes (which one) or if both change. Also, if both change, let me know which one changes faster.

pd: each number represents the count of a 16-bit timer. They should chain to form a 32-bit timer. If the timers do not work, there is no dead-time measurement nor random number generation.

ADC still doesn't work and both dead-time values are always zero.

Gissio commented 5 months ago

I just released Rad Pro 2.0beta18. Can you go over this checklist?

The battery indicator works correctly.
The temperature indicator in the statistics menu shows reasonable values.
The random generator and dead-time compensation work.
The serial port communication works.
Data logging works and you can download it with radpro-tool.

mryndzionek commented 5 months ago

Random generator seems to be working. Dead time shows some (I think sane) values decreasing with time. The rest without changes.

Gissio commented 5 months ago

So the ADC is still not working...? I'll keep you updated.

mryndzionek commented 5 months ago

So the ADC is still not working...? I'll keep you updated.

Yeah, the temp. shows 378.7C and the battery icon is "charging". Don't feel pressured. I'll wait :smile: .

Gissio commented 5 months ago

I'm not.

Gissio commented 5 months ago

Ok. I think I found the error. Can you try again? Try beta19.

mryndzionek commented 5 months ago

Ok. I think I found the error. Can you try again? Try beta19.

Yes, now it works. The temperature is a little bit high (37.8C), but yeah, the internal sensor is probably not that accurate.

Gissio commented 5 months ago

Excellent! Remember device temperature is MCU temperature, not ambient temperature, so the value looks great.

Ok. I think I found the error. Can you try again? Try beta19

Yes, now it works. The temperature is a little bit high (37.8C), but yeah, the internal sensor is probably not that accurate.