diff options
| author | Louis Morhet <lmorhet@kalrayinc.com> | 2023-04-04 14:15:14 +0200 |
|---|---|---|
| committer | Benjamin Tissoires <benjamin.tissoires@redhat.com> | 2023-04-13 16:41:37 +0200 |
| commit | e36c31f8cac54d6d900d270078d6b33de74e1b0a (patch) | |
| tree | 99e3ff2f2b4992b022f3eacfd3f1a4182ca8296a | |
| parent | de4664485abbc0529b1eec44d0061bbfe58a28fb (diff) | |
| download | linux-stable-e36c31f8cac54d6d900d270078d6b33de74e1b0a.tar.gz linux-stable-e36c31f8cac54d6d900d270078d6b33de74e1b0a.tar.bz2 linux-stable-e36c31f8cac54d6d900d270078d6b33de74e1b0a.zip | |
HID: mcp2221: fix report layout for gpio get
The documentation of the component (section 3.1.12 GET GPIO VALUES)
describes the hid report structure with two fields per gpio:
its value, followed by its direction.
However, the driver describes it with a wrong order:
direction followed by value.
Fix the structure representing the report answered by the chip to the
GET GPIO VALUES command.
Fixes commit 567b8e9fed8a ("HID: mcp2221: Fix GPIO output handling")
Signed-off-by: Louis Morhet <lmorhet@kalrayinc.com>
Link: https://lore.kernel.org/r/945967fbab56d53f9630ad3844b64734f8c3107e.1680602387.git.lmorhet@kalrayinc.com
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
| -rw-r--r-- | drivers/hid/hid-mcp2221.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/hid/hid-mcp2221.c b/drivers/hid/hid-mcp2221.c index f74a977cf8f8..fa20ed4d395a 100644 --- a/drivers/hid/hid-mcp2221.c +++ b/drivers/hid/hid-mcp2221.c @@ -79,8 +79,8 @@ struct mcp_get_gpio { u8 cmd; u8 dummy; struct { - u8 direction; u8 value; + u8 direction; } gpio[MCP_NGPIO]; } __packed; |