Received-SPF: pass (google.com: domain of linux-kernel+bounces-129672-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Date: Wed, 3 Apr 2024 13:36:19 +0200 (CEST)
From: Jiri Kosina <jikos@kernel.org>
To: Ivan Gorinov <linux-kernel@altimeter.info>
cc: linux-input@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3] Input: WinWing Orion2 throttle support
In-Reply-To: <20240216215447.426ca3e7@lxl-9drxzh3.na.joby.aero>
Message-ID: <nycvar.YFH.7.76.2404031336070.20263@cbobk.fhfr.pm>
References: <20240216215447.426ca3e7@lxl-9drxzh3.na.joby.aero>
User-Agent: Alpine 2.21 (LSU 202 2017-01-01)
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII

On Fri, 16 Feb 2024, Ivan Gorinov wrote:

> WinWing Orion2 throttle works with Linux out of box,
> but the kernel sees only 16 of 47 buttons on the throttle base.
> 
> This module enables all buttons, and also adds LED controls.
> 
> Button numbers 0 .. 63 on Orion2 are reserved for throttle grip;
> the throttle base buttons have numbers 64 .. 110.
> 
> Linux kernel HID subsystem only supports up to 80 buttons.
> 
> Remap throttle base buttons to numbers 32 .. 78,
> reserving only numbers 0 .. 31 for buttons on the grip handle.
> 
> Changes since v2:
>    - Fixed automatic line wraps added by mail client
> 
> Changes since v1:
>    - Fixed formatting of descriptor byte array;
>    - Using product codes of Winwing grips in config.
> 
> Signed-off-by: Ivan Gorinov <ivan.gorinov@jobyaviation.com>

Applied, thanks.

-- 
Jiri Kosina
SUSE Labs