NuttX PR comments

Results 61 comments of


                                            NuttX PR

can: Add more critical section to fix the race condition

[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) This pull request appears incomplete and lacks the necessary information to determine if it meets NuttX requirements. **Here's what's missing:** * **Summary:** * **Why is...

nuttx/arch: Remove GCCVER and add compilation options directly.

[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) This PR appears to be **missing critical information**. Here's a breakdown based on the provided template: **Summary:** * **What change is necessary**: While it mentions...

mmcsd: move block driver register to probe stage in order to read ext_csd reg

[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) **No, this PR does not meet the NuttX Requirements.** **Missing Information:** * **Summary:** * A clear explanation of *why* this change is necessary is missing....

drivers/mmcsd: Rename mmcsd_read_csd to mmcsd_read_extcsd

[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) **No, this PR does not fully meet the NuttX requirements.** **Missing Information:** * **Summary:** * **Why** is this change necessary? What is the problem with...

nuttx/libxx: Simplify compilation warnings for passing CXXFLAGS.

[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) **No, this PR does not meet the NuttX requirements.** **Missing Information:** * **Summary:** * **Why** is this change necessary? What problem does it solve? *...

Suspend and dump all CPU registers from assert

[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) This PR summary doesn't fully meet the requirements. Here's why and how to improve it: **What's Missing/Needs Improvement** * **Summary:** * **Why is this change...

Allow each chip to use its own timer

[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) The PR as described **does not meet** the NuttX requirements. Here's why: * **Insufficient Testing Information:** * While the PR mentions testing with QEMU ARM64,...

Add support for smp_call from interrupt context

[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) The PR appears to meet the NuttX requirements, but with a caveat: **Strengths:** * **Clear Summary:** Explains the "why," "what," and "how" of the changes....

arm/stm32f401rc-rs485: Add support to WS2812 addressable LED

[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) The PR summary is missing some key information: * **What functional part of the code is being changed?** (e.g., specific driver, board configuration, etc.) *...

sim_uart: rm LF to CRLF convertion

[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) **No, this PR does not meet the NuttX Requirements.** **Missing Information:** * **Summary:** * Why is the change necessary? What problem does it solve? *...