Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
From:   Sebastian Parschauer <s.parschauer@gmx.de>
Mime-Version: 1.0 (1.0)
Subject: Re: [PATCH] HID: quirks: Always poll Anne Pro 2(C15) Keyboard
Date:   Thu, 6 Jan 2022 22:33:18 +0100
Message-Id: <3DB5A377-5918-4304-BB35-589B594A5E82@gmx.de>
References: <61b44254.1c69fb81.ca496.5d65@mx.google.com>
Cc:     jikos@kernel.org, benjamin.tissoires@redhat.com,
        linux-input@vger.kernel.org, linux-kernel@vger.kernel.org
In-Reply-To: <61b44254.1c69fb81.ca496.5d65@mx.google.com>
To:     kortanzh@gmail.com
Precedence: bulk


> Am 11.12.2021 um 07:16 schrieb kortanzh@gmail.com:
>=20
> =EF=BB=BFFrom: Kortan <kortanzh@gmail.com>
>=20
> According to https://openannepro.github.io/ap2_revisionas, Anne Pro 2
> keyboard has two hardware revisions, C15(04d9:a292) and C18(04d9:a293).
>=20
> Previous patch submitted by Sebastian only fix the C18 revision. This
> patch add missing C15 revision support.
>=20
> Fixes: ca28aff0e1dc ("HID: quirks: Always poll Obins Anne Pro 2 keyboard")=

> Signed-off-by: Kortan <kortanzh@gmail.com>

Thanks for the patch. I was actually the first one reporting the bug to them=
. They replied with something like =E2=80=9EWhy do you do a kernel patch? No=
body reported this bug before. We will fix our firmware.=E2=80=9C That react=
ion actually surprised me. Does this mean they didn=E2=80=99t?

They didn=E2=80=99t say anything about a second hardware revision which migh=
t be affected.=