casaroli opened a new pull request, #20678: URL: https://github.com/apache/nuttx/pull/20678
## Summary `esp_temperature_sensor_initialize()` calls `kmm_free()`, but `arch/xtensa/src/common/espressif/esp_temperature_sensor.c` does not include `nuttx/kmalloc.h`. The declaration used to come in through `stdio.h`, which no longer includes it (#20565), so `esp32s3-ws-lcd128:imu-qmi8658` fails to build: ``` common/espressif/esp_temperature_sensor.c:871:7: error: implicit declaration of function 'kmm_free' [-Wimplicit-function-declaration] ``` This stops the `xtensa-03` CI job (found on apache/nuttx-apps#3850). The risc-v copy of the file already includes `nuttx/kmalloc.h`. Two commits: - Fix the old nxstyle error in the file (one line indented by eight spaces instead of six), so that checkpatch passes. - Include `nuttx/kmalloc.h`. ## Impact Build fix only. No change in behaviour. ## Testing `esp32s3-ws-lcd128:imu-qmi8658` with xtensa-esp-elf 14.2.0, `-Werror`, apps master 467e22088 + apache/nuttx-apps#3850: - master b3418f076a5: fails with the error above. - with this PR: builds. `tools/checkpatch.sh -c -u -m -g` passes. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
