drivers/usbhost: Let the HID keyboard pick its interrupt pipe.

HIDKBD_NOGETREPORT reads keyboard reports with DRVR_ASYNCH(), and that
macro is only defined when USBHOST_ASYNCH is set.  The option selected
neither, so turning it on by itself fails at the call site with no hint
that a second option was meant to come with it.

Select it.  Every in-tree configuration that sets NOGETREPORT already
resolves USBHOST_ASYNCH: ci20:jumbo and sama5d3-xplained:bluetooth
through USBHOST_HUB, and the two linum-stm32h753bi configurations by
setting it directly.  No existing build changes.

The two that set it directly no longer can, since a selected symbol is
no longer settable, so savedefconfig drops the line.  Their defconfigs
are normalized here to keep them canonical.

Assisted-by: Claude:claude-opus-5
Signed-off-by: Justin Hammond <justin@dynam.ac>
This commit is contained in:
Justin Hammond
2026-08-09 20:57:41 +08:00
committed by Xiang Xiao
parent a023f61a38
commit b8e26b127e
3 changed files with 5 additions and 2 deletions
@@ -91,7 +91,6 @@ CONFIG_UINPUT_KEYBOARD=y
CONFIG_UINPUT_KEYBOARD_BUFNUMBER=128
CONFIG_USART1_SERIAL_CONSOLE=y
CONFIG_USBHOST=y
CONFIG_USBHOST_ASYNCH=y
CONFIG_USBHOST_HIDKBD=y
CONFIG_USEC_PER_TICK=1000
CONFIG_VIDEO_FB=y
@@ -99,7 +99,6 @@ CONFIG_USART1_RXBUFSIZE=4096
CONFIG_USART1_SERIAL_CONSOLE=y
CONFIG_USART1_TXBUFSIZE=1024
CONFIG_USBHOST=y
CONFIG_USBHOST_ASYNCH=y
CONFIG_USBHOST_HIDKBD=y
CONFIG_USEC_PER_TICK=1000
CONFIG_VIDEO_FB=y
+5
View File
@@ -442,9 +442,14 @@ config HIDKBD_REPORT_MODIFIERS
config HIDKBD_NOGETREPORT
bool "Use Interrupt pipe to get keys"
default n
select USBHOST_ASYNCH
---help---
Use the INTERRUPT IN pipe to get keyboard input reports.
This reads the pipe with DRVR_ASYNCH(), which only exists when
USBHOST_ASYNCH is set, so it is selected here rather than left
for a configuration to discover as a link failure.
endif
config USBHOST_HIDMOUSE