| Message ID | 20241108120311.87795-1-charles.goodix@gmail.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | dt-bindings: input: Goodix GT7986U SPI HID Touchscreen | expand |
Hi, On Fri, Nov 8, 2024 at 4:03 AM Charles Wang <charles.goodix@gmail.com> wrote: > > The Goodix GT7986U touch controller report touch data according to the > HID protocol through the SPI bus. However, it is incompatible with > Microsoft's HID-over-SPI protocol. > > The patchset introduces the following two changes: > 1) Add goodix,gt7986u-spifw.yaml. > 2) Drop the 'goodix,hid-report-addr' property. > > Signed-off-by: Charles Wang <charles.goodix@gmail.com> > --- > Changes in v3: > - Split the commit into two patches. LOL, this isn't what I meant. You should have one patch adding the proper bindings, but you need a second patch to _the "driver_. Right now the driver (AKA the file "drivers/hid/hid-goodix-spi.c" in Linux) is still looking for "goodix,hid-report-addr". The driver needs to be updated and that was what I was saying should be patch #2. You'll also need to fix the driver to update the compatible string it's looking for. -Doug
The Goodix GT7986U touch controller report touch data according to the HID protocol through the SPI bus. However, it is incompatible with Microsoft's HID-over-SPI protocol. The patchset introduces the following two changes: 1) Add goodix,gt7986u-spifw.yaml. 2) Drop the 'goodix,hid-report-addr' property. Signed-off-by: Charles Wang <charles.goodix@gmail.com> --- Changes in v3: - Split the commit into two patches. Changes in v2: - Change compatible to 'goodix,gt7986u-spifw'. - Remove 'goodix,hid-report-addr' property. - Change additionalProperties to unevaluatedProperties. - v1: https://lore.kernel.org/all/20241025114642.40793-2-charles.goodix@gmail.com/ --- Charles Wang (2): dt-bindings: input: Goodix GT7986U SPI HID Touchscreen dt-bindings: input: gt7986u-spifw: Remove hid-report-addr property .../bindings/input/goodix,gt7986u-spifw.yaml | 68 +++++++++++++++++++ 1 file changed, 68 insertions(+) create mode 100644 Documentation/devicetree/bindings/input/goodix,gt7986u-spifw.yaml