fdcavalcanti opened a new pull request, #18733:
URL: https://github.com/apache/nuttx/pull/18733
## Summary
<!-- This field should contain a summary of the changes. It will be
pre-filled with the commit's message and descriptions. Adjust it accordingly -->
* arch/risc-v: custom compilation options on some cache sources
This change sets three new compilation flags for avoiding cache miss
when using optimized builds and cache is manipulated.
* arch/risc-v/espressif: fix CMake linking of mbedtls on Wi-Fi support
Fix improper linking against 'arch' when it should link against 'nuttx'
target.
* arch/risc-v: fix linker script manipulation on CMake builds for Espressif
devices
This changes enables the use of the preprocessor function
(nuttx_generate_preprocess_target) so it can properly
process the linker scripts, instead of using the original ones (must process
some `#ifdefs`).
* boards/risc-v: fix LD_SCRIPTS manipulation on CMake builds for Espressif
devices
Changes the global property manipulation of LD_SCRIPTS to APPEND.
This avoids rewriting this global property.
This PR fixes linker script related problems on CMake build for Espressif
devices.
It would manifest on Wi-Fi features and timers, since linker scripts were
not properly processed.
## Impact
<!-- Please fill the following sections with YES/NO and provide a brief
explanation -->
Impact on user: Yes, some applications such as Wi-Fi now work properly with
mbedtls.
<!-- Does it impact user's applications? How? -->
Impact on build: Fix problems during linking stage.
<!-- Does it impact on building NuttX? How? (please describe the required
changes on the build system) -->
Impact on hardware: No.
<!-- Does it impact a specific hardware supported by NuttX? -->
Impact on documentation: No.
<!-- Does it impact the existing documentation? Please provide additional
documentation to reflect that -->
Impact on security: No.
<!-- Does it impact NuttX's security? -->
Impact on compatibility: No.
<!-- Does it impact compatibility between previous and current versions? Is
this a breaking change? -->
## Testing
<!-- Please provide all the testing procedure. Consider that upstream
reviewers should be able to reproduce the same testing performed internally -->
This change can be easily tested using the Wi-Fi features, either station or
AP.
As station, it would fail to connect to any AP that is not `open`. As AP, it
would never work as WPA2 or WPA3, only WEP. This happened due to missing link
of mbedtls.
### Building
<!-- Provide how to build the test for each SoC being tested -->
- `cmake -B build -DBOARD_CONFIG=esp32c6-devkitc:sta_softap -GNinja
-DNXTMPDIR=on`
- `ESPTOOL_PORT=/dev/ttyUSB0 cmake --build -t flash`
### Running
<!-- Provide how to run the test for each SoC being tested -->
Disable wlan0 and enabled wlan1 so we have AP mode working, then, scan for
the AP.
- `ifdown wlan0`
- `ifup wlan1`
### Results
<!-- Provide tests' results and runtime logs -->
Check APs using host PC.
Before changes:
```
$ nmcli -f BSSID,SSID,SECURITY dev wifi list
60:55:F9:F6:06:05 NuttX WEP
```
After changes:
```
$ nmcli -f BSSID,SSID,SECURITY dev wifi list
60:55:F9:F6:06:05 NuttX WPA3
```
--
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]
