acassis commented on code in PR #15205: URL: https://github.com/apache/nuttx/pull/15205#discussion_r1888631650
########## drivers/input/Kconfig: ########## @@ -65,7 +65,7 @@ endif # FF_AW86225 endif # INPUT_FF config INPUT_MOUSE - bool + bool "Enable mouse support" Review Comment: @terry0012 and @HongChao6 I understand that, but after you let user to see and enable these symbols it is expected to work. So if user enable Mouse or Keyboard they will expect it so work, but it will not, because it depends on some external driver. So, I think it should to become a new session, like Sensors Driver, after you enable it you should be presented with all Sensors drivers. It means all "select" now should become "depends on" -- 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-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
