nuttxpr commented on PR #14262: URL: https://github.com/apache/nuttx/pull/14262#issuecomment-2410761742
[**\[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:** * Lacks a clear explanation of why the change is necessary. Is it a fix, update, or new feature? * Needs more detail on the functional part of the code being changed and how the change works. Be specific about the registers being dumped and the mechanism used. * **Impact:** * While "coredump" is mentioned, it's unclear how this change impacts the user, build process, hardware, documentation, security, or compatibility. Address each impact category with specific details. * **Testing:** * The "Testing" section is entirely empty. Provide: * Details about your build host(s) and target(s). * "Before" and "after" testing logs demonstrating the change's effect. **Recommendations for Improvement:** 1. **Expand the Summary:** Clearly state the problem this PR solves or the enhancement it provides. Explain the technical details of the change (e.g., "This PR adds a new function to `arch/arm/src/xxx/xxx.c` that dumps the contents of ARM Cortex-M NVIC and MPU registers to the console during a coredump."). 2. **Address All Impact Categories:** For each impact category, provide a "YES" or "NO" answer. If "YES", explain the impact in detail. For example: * **Impact on user:** YES. Users will now see NVIC and MPU register values in coredump output, aiding in debugging. * **Impact on documentation:** YES. The coredump documentation needs to be updated to describe the new register dump information. 3. **Provide Thorough Testing Information:** Specify your build host operating system, CPU architecture, compiler, and version. List the target architecture, board, and configuration used for testing. Include relevant "before" and "after" logs to demonstrate the change. By providing the missing information and following the recommendations, you'll make your PR much stronger and easier for the NuttX maintainers to review and merge. -- 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]
