Re: [PR] boards/stm32f411-minimum: add support for /dev/gpio* [nuttx]

2024-04-22 Thread via GitHub
acassis merged PR #11798: URL: https://github.com/apache/nuttx/pull/11798 -- 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:

Re: [PR] boards/stm32f411-minimum: add support for /dev/gpio* [nuttx]

2024-04-22 Thread via GitHub
mlyszczek commented on PR #11798: URL: https://github.com/apache/nuttx/pull/11798#issuecomment-2068763897 Yay, finally a success. Sorry it took so long. -- 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

Re: [PR] boards/stm32f411-minimum: add support for /dev/gpio* [nuttx]

2024-03-24 Thread via GitHub
acassis commented on PR #11798: URL: https://github.com/apache/nuttx/pull/11798#issuecomment-2016947650 @mlyszczek the error seems to be in your Kconfig: Kconfig Configuration Error: boards/arm/stm32/stm32f411-minimum/Kconfig:83: warning: style: quotes recommended around

Re: [PR] boards/stm32f411-minimum: add support for /dev/gpio* [nuttx]

2024-03-24 Thread via GitHub
mlyszczek commented on PR #11798: URL: https://github.com/apache/nuttx/pull/11798#issuecomment-2016723005 let's try another rebase, and hope it works :) -- 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

Re: [PR] boards/stm32f411-minimum: add support for /dev/gpio* [nuttx]

2024-03-10 Thread via GitHub
acassis commented on PR #11798: URL: https://github.com/apache/nuttx/pull/11798#issuecomment-1987257422 Please rebase to fix the CI issue -- 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

Re: [PR] boards/stm32f411-minimum: add support for /dev/gpio* [nuttx]

2024-02-28 Thread via GitHub
mlyszczek commented on PR #11798: URL: https://github.com/apache/nuttx/pull/11798#issuecomment-1969226469 Yes, in stm32 datasheet they refer to pins as PA, PB etc. but board refers to them as A, B... I'm pretty sure it's to save space on PCB, as they would not fit PA, PB... Here is

Re: [PR] boards/stm32f411-minimum: add support for /dev/gpio* [nuttx]

2024-02-28 Thread via GitHub
acassis commented on PR #11798: URL: https://github.com/apache/nuttx/pull/11798#issuecomment-1969204177 > Hmm... you think? I named them exactly as they are printed on the PCB. Hmm, ok! In the datasheet they are referenced as PA*, PB*, PC*, etc -- This is an automated message from

Re: [PR] boards/stm32f411-minimum: add support for /dev/gpio* [nuttx]

2024-02-27 Thread via GitHub
mlyszczek commented on PR #11798: URL: https://github.com/apache/nuttx/pull/11798#issuecomment-1967748237 Hmm... you think? I named them exactly as they are printed on the PCB. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] boards/stm32f411-minimum: add support for /dev/gpio* [nuttx]

2024-02-27 Thread via GitHub
acassis commented on PR #11798: URL: https://github.com/apache/nuttx/pull/11798#issuecomment-1967737360 @mlyszczek I suggest you to use pin PAx, PBx, PCx, etc, because if someone see /dev/gpio_a0 could be induced to think it is an analog, because this is the naming used for example in

Re: [PR] boards/stm32f411-minimum: add support for /dev/gpio* [nuttx]

2024-02-27 Thread via GitHub
mlyszczek commented on PR #11798: URL: https://github.com/apache/nuttx/pull/11798#issuecomment-1967562274 For reference - it looks like it ``` ┌── enable gpio subsystem ──┐ │

[PR] boards/stm32f411-minimum: add support for /dev/gpio* [nuttx]

2024-02-27 Thread via GitHub
mlyszczek opened a new pull request, #11798: URL: https://github.com/apache/nuttx/pull/11798 ## Summary This patch allows user to simply select which GPIOs he wants to use and access them via /dev/gpio_a0 or similar device with standard gpio driver. By default gpio names match pin