Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
mryndzionek opened this issue Mar 10, 2024 · 12 comments
Closed

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

mryndzionek opened this issue Mar 10, 2024 · 12 comments
Labels
bug Something isn't working

Comments

@mryndzionek
Copy link

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

@mryndzionek mryndzionek changed the title FS2011 (GD32F150) battery and temperature readouts wrong on main branch FS2011 (GD32F150) battery and temperature readouts wrong on main branch (986d412) Mar 10, 2024
@Gissio Gissio changed the title FS2011 (GD32F150) battery and temperature readouts wrong on main branch (986d412) FS2011 (GD32F150): battery, temperature readings wrong Mar 10, 2024
@Gissio
Copy link
Owner

Gissio commented Mar 10, 2024

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
Copy link
Author

mryndzionek commented Mar 10, 2024

  • ☑ 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
Copy link
Owner

Gissio commented Mar 10, 2024

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
Copy link
Author

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
Copy link
Owner

Gissio commented Mar 13, 2024

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
Copy link
Author

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

@Gissio
Copy link
Owner

Gissio commented Mar 13, 2024

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

@mryndzionek
Copy link
Author

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 😄 .

@Gissio
Copy link
Owner

Gissio commented Mar 13, 2024

I'm not.

@Gissio
Copy link
Owner

Gissio commented Mar 13, 2024

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

@mryndzionek
Copy link
Author

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
Copy link
Owner

Gissio commented Mar 14, 2024 via email

@Gissio Gissio closed this as completed Mar 14, 2024
@Gissio Gissio added duplicate This issue or pull request already exists bug Something isn't working and removed duplicate This issue or pull request already exists labels Mar 14, 2024
@Gissio Gissio changed the title FS2011 (GD32F150): battery, temperature readings wrong FS2011 (GD32F150): Battery, temperature readings wrong Mar 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants