Fishwaldo commented on PR #19878: URL: https://github.com/apache/nuttx/pull/19878#issuecomment-5315982285
Thanks — you are right, the new statics only had a `Name:` line. Every function this commit adds now carries Description, Input Parameters and Returned Value, matching the existing headers on `gpio_pin_register()` and friends in the same file. Two of them are worth calling out, since the answer is a deliberate "none": `gpio_procfs_add()` and `gpio_procfs_remove()` return void because a pin that cannot be listed in /proc is still a working pin, so a failure there must not fail the registration. That is now stated in the header rather than left implicit. I am applying the same fix to the sibling procfs entries in #19871, #19876 and #19879, which had the same gap. -- 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]
