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/12/09 16:05:57 UTC

[GitHub] [incubator-nuttx-apps] jarivanewijk opened a new pull request #917: timer_gpout example: use /dev/gpio0 and rename to timer_gpio

jarivanewijk opened a new pull request #917:
URL: https://github.com/apache/incubator-nuttx-apps/pull/917


   ## Summary
   https://github.com/apache/incubator-nuttx/pull/4774 changes the naming of GPIO devices. Instead of separate gpout, gpin and gpint devices they are all named /dev/gpioN now. The timer_gpout example seems to be the only application that has explicit references to /dev/gpout0, so I have made changes so that it now uses /dev/gpio0 instead.
   
   I have also renamed the application itself from timer_gpout to timer_gpio, which seems more appropriate now that there is no explicit gpout devices anymore.
   
   ## Impact
   See https://github.com/apache/incubator-nuttx/pull/4774 - everybody and everything that was using /dev/gpoutN devices now needs to use /dev/gpioN. With this PR this will already be fixed for the timer_gpio application, which seems to be the only app to directly use a /dev/gpout device.
   
   ## Testing
   I don't have access to a board/arch that implements both the timer and gpio drivers, so I was not able to test this on real hardware. However, the application builds without issues for a STM32F4Discovery board with timer and gpio drivers enabled.
   
   


-- 
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



[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #917: timer_gpout example: use /dev/gpio0 and rename to timer_gpio

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on pull request #917:
URL: https://github.com/apache/incubator-nuttx-apps/pull/917#issuecomment-990116060


   Yes, but I think that patch will merge soon.:)


-- 
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



[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #917: timer_gpout example: use /dev/gpio0 and rename to timer_gpio

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged pull request #917:
URL: https://github.com/apache/incubator-nuttx-apps/pull/917


   


-- 
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



[GitHub] [incubator-nuttx-apps] jarivanewijk commented on pull request #917: timer_gpout example: use /dev/gpio0 and rename to timer_gpio

Posted by GitBox <gi...@apache.org>.
jarivanewijk commented on pull request #917:
URL: https://github.com/apache/incubator-nuttx-apps/pull/917#issuecomment-990108282


   This probably should not have been merged before https://github.com/apache/incubator-nuttx/pull/4774 ...


-- 
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