Skip to content

Conversation

@kuba2k2
Copy link

@kuba2k2 kuba2k2 commented Jun 1, 2025

This PR uses FreeRTOS v9.0.0 on LibreTiny, making it compatible with both BK7231 and RTL8710BN (and also LN882).

Until LibreTiny migrates to v9.0.0 by default, this PlatformIO option is needed.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR restores support for the LibreTiny RTL8710BN board by configuring the project to use FreeRTOS v9.0.0 for platforms where it is not the default.

  • Updated platformio.ini to set custom_versions.freertos = 9.0.0 for RTL8710BN
  • Enabled RTL8710BN board testing in the CI workflow

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
platformio.ini Adds FreeRTOS v9.0.0 configuration and accompanying comment
.github/workflows/ci.yml Uncomments and enables the RTL8710BN test environment configuration
Comments suppressed due to low confidence (1)

.github/workflows/ci.yml:83

  • [nitpick] Verify that enabling the RTL8710BN target in the CI job aligns with the overall testing strategy and that its dependency configurations are up to date.
- env: ci-arduino-libretiny

-Wno-missing-field-initializers
; use FreeRTOS v9.0.0 for RTL8710BN
; (BK7231 already uses it)
custom_versions.freertos = 9.0.0
Copy link

Copilot AI Jun 1, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] The FreeRTOS version is configured in two different sections; if supported by the configuration, consider consolidating this setting to reduce duplication.

Suggested change
custom_versions.freertos = 9.0.0
custom_versions.freertos = ${env.custom_versions.freertos}

Copilot uses AI. Check for mistakes.
@kuba2k2
Copy link
Author

kuba2k2 commented Jun 1, 2025

To be clear: I have quickly verified that it still works on real hardware.

@mathieucarbou mathieucarbou merged commit 4ffd517 into ESP32Async:main Jun 1, 2025
22 of 28 checks passed
@mathieucarbou
Copy link
Member

mathieucarbou commented Jun 1, 2025

To be clear: I have quickly verified that it still works on real hardware.

Thanks!
I will issue a release now so that you are able to finish the other PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants