You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2021/10/01 13:28:17 UTC

[GitHub] [incubator-nuttx] acassis commented on pull request #4635: Userled driver: Implement getall by checking actual pin state

acassis commented on pull request #4635:
URL: https://github.com/apache/incubator-nuttx/pull/4635#issuecomment-932228429


   I agree with this modification, but why don't call it just "userled_ledget" instead of "userled_ledgetall"? Notice that userled_ledset is able to set all LEDs, but we don't call it userled_ledsetall. Just to keep things uniform/equivalent.


-- 
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: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org