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

Fix the temperature ADC_STM32_Die_Temp_Read value getting incorrect for nucleo_h743zi in Zephyr 3.6.0 #72835

Open
ShubhadaHarkut-eaton opened this issue May 16, 2024 · 1 comment
Assignees
Labels
area: ADC Analog-to-Digital Converter (ADC) bug The issue is a bug, or the PR is fixing a bug platform: STM32 ST Micro STM32 priority: low Low impact/importance bug

Comments

@ShubhadaHarkut-eaton
Copy link

ShubhadaHarkut-eaton commented May 16, 2024

nucleo_h743zi: incorrect ADC_STM32_Die_Temp_Read temperature value.
Zephyr 3.2.0

nucleo_h743zi.overlay
/{
zephyr,user {
/* adjust channel number according to pinmux in board.dts */

io-channels = <&adc3 18>;

};
};

...
To Reproduce
Steps to reproduce the behavior:
To Build: west build -b nucleo_h743zi ./ --pristine
To Flash: west flash
4. See error
*** Booting Zephyr OS build v3.6.0
STM32 Die temperature sensor test
Current temperature: -173.3 �°C
Current temperature: -126.2 �°C
Current temperature: -128.2 �°C
Current temperature: -128.8 �°C
Current temperature: -128.8 �°C
Current temperature: -126.9 �°C
Current temperature: -126.2 �°C
Current temperature: -126.5 �°C
Current temperature: -126.9 �°C
Expected behavior
positive value

Impact
High

Logs and console output

STM32 Die temperature sensor test
Current temperature: -173.3 �°C
Current temperature: -126.2 �°C
Current temperature: -128.2 �°C
Current temperature: -128.8 �°C
Current temperature: -128.8 �°C
Current temperature: -126.9 �°C
Current temperature: -126.2 �°C
Current temperature: -126.5 �°C
Current temperature: -126.9 �°C

Environment (please complete the following information):

OS: zephyr-v3.6.0
Toolchain zephyr-sdk-0.16.5
Commit SHA or Version used

@ShubhadaHarkut-eaton ShubhadaHarkut-eaton added the bug The issue is a bug, or the PR is fixing a bug label May 16, 2024
@henrikbrixandersen henrikbrixandersen added the platform: STM32 ST Micro STM32 label May 16, 2024
@erwango erwango added the priority: low Low impact/importance bug label May 16, 2024
@erwango erwango removed their assignment May 16, 2024
@erwango erwango added the area: ADC Analog-to-Digital Converter (ADC) label May 16, 2024
@FRASTM
Copy link
Collaborator

FRASTM commented May 21, 2024

@ShubhadaHarkut-eaton can you please check the boards DTS.
With sha1 adeb19b , we see a correct valu when running
west build -p -b nucleo_h743zi samples/sensor/die_temp_polling/

*** Booting Zephyr OS build v3.6.0-4280-g105fc792675c ***
CPU Die temperature[dietemp]: 28.4 °C

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ADC Analog-to-Digital Converter (ADC) bug The issue is a bug, or the PR is fixing a bug platform: STM32 ST Micro STM32 priority: low Low impact/importance bug
Projects
None yet
Development

No branches or pull requests

5 participants