Skip to content

Conversation

@jimforr
Copy link
Contributor

@jimforr jimforr commented Nov 5, 2025

Description

Add prefix esp32_camera for sensor function name, to avoid same name as functions in esp_cam_sensor.
For example, sc030iot_detect is a same name function as it in esp_cam_sensor. Then once the linker finds sc030iot_detect in one translation unit, it won't even look into any other translation for this function. Another translation unit where this function is defined won't be considered by the linker for linking.

Testing

Tested locally on IDF release/v5.5.

Checklist

Before submitting a Pull Request, please ensure the following:

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

@jimforr
Copy link
Contributor Author

jimforr commented Nov 6, 2025

can‘t pass CI checks (latest IDF version) because of following breaking change:
espressif/esp-idf@d0e24e4

@jimforr jimforr marked this pull request as ready for review November 6, 2025 07:11
@me-no-dev me-no-dev merged commit 00c3042 into espressif:master Nov 6, 2025
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants