nuttxpr commented on PR #14165: URL: https://github.com/apache/nuttx/pull/14165#issuecomment-2408452338
[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) The provided PR information lacks crucial details to assess its adherence to NuttX requirements. Here's a breakdown: **What's Missing & Why It Matters:** * **Summary:** The summary only mentions warnings related to right shift count and incompatible pointer types in the PCI code. This is insufficient. * **Why is the change necessary?** Are these warnings causing actual issues, or are they false positives? * **What's the solution?** How are these warnings being addressed? A simple cast might hide a deeper problem. * **Issue/PR References:** Are there any related issues in the NuttX or NuttX Apps repositories? Providing links gives context. * **Impact:** Simply stating "PCI" isn't informative enough. * **Functionality:** Does this PR fix a bug, enhance performance, or introduce a new PCI feature? * **User Impact:** Will users need to modify their configurations or code? * **Hardware Impact:** Will specific architectures, boards, or PCI devices be affected? * **Security/Compatibility:** Are there any security implications or backward compatibility concerns? * **Testing:** The testing information is vague. * **Build Hosts:** Specify the operating systems, CPUs, and compiler versions used for building NuttX. * **Target Details:** Provide specific details about the QEMU configurations (e.g., QEMU version, machine type, CPU model) used for armv8a and intel64 testing. * **Testing Logs:** Instead of "your testing logs here," provide relevant snippets of logs demonstrating the warnings before the change and their absence (or expected behavior) after the change. **Verdict:** The PR, in its current form, **does not meet NuttX requirements.** It lacks essential details about the problem, the solution, its impact, and thorough testing evidence. More information is needed to properly review and merge the PR. -- 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]
