| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <ZvTpG1cTWM3gY_vk@ux-UP-WHL01> Date: Thu, 26 Sep 2024 12:54:51 +0800 From: Charles Wang <charles.goodix@...il.com> To: Rob Herring <robh@...nel.org> Cc: dmitry.torokhov@...il.com, dianders@...omium.org, dan.carpenter@...aro.org, conor@...nel.org, krzk+dt@...nel.org, jikos@...nel.org, bentiss@...nel.org, hbarnor@...omium.org, linux-input@...r.kernel.org, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org, conor.dooley@...rochip.com Subject: Re: [PATCH v7 2/2] dt-bindings: input: Goodix SPI HID Touchscreen Hi Rob, Very sorry for the late response. A fiexd version with compatible "goodix,gt7986u-spi" has been submitted. On Wed, Sep 25, 2024 at 08:51:14PM -0500, Rob Herring wrote: > On Wed, Sep 25, 2024 at 4:46 PM Dmitry Torokhov > <dmitry.torokhov@...il.com> wrote: > > > > On Wed, Sep 25, 2024 at 12:40:56PM -0500, Rob Herring wrote: > > > On Tue, Sep 10, 2024 at 5:41 PM Rob Herring <robh@...nel.org> wrote: > > > > > > > > On Fri, Sep 6, 2024 at 3:28 PM Rob Herring <robh@...nel.org> wrote: > > > > > > > > > > On Tue, Aug 13, 2024 at 9:45 PM Charles Wang <charles.goodix@...il.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. > > > > > > > > > > > > Reviewed-by: Conor Dooley <conor.dooley@...rochip.com> > > > > > > Signed-off-by: Charles Wang <charles.goodix@...il.com> > > > > > > --- > > > > > > .../bindings/input/goodix,gt7986u.yaml | 71 +++++++++++++++++++ > > > > > > 1 file changed, 71 insertions(+) > > > > > > create mode 100644 Documentation/devicetree/bindings/input/goodix,gt7986u.yaml > > > > > > > > > > > > diff --git a/Documentation/devicetree/bindings/input/goodix,gt7986u.yaml b/Documentation/devicetree/bindings/input/goodix,gt7986u.yaml > > > > > > new file mode 100644 > > > > > > index 000000000..a7d42a5d6 > > > > > > --- /dev/null > > > > > > +++ b/Documentation/devicetree/bindings/input/goodix,gt7986u.yaml > > > > > > @@ -0,0 +1,71 @@ > > > > > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > > > > > +%YAML 1.2 > > > > > > +--- > > > > > > +$id: http://devicetree.org/schemas/input/goodix,gt7986u.yaml# > > > > > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > > > > > + > > > > > > +title: GOODIX GT7986U SPI HID Touchscreen > > > > > > + > > > > > > +maintainers: > > > > > > + - Charles Wang <charles.goodix@...il.com> > > > > > > + > > > > > > +description: Supports the Goodix GT7986U touchscreen. > > > > > > + This touch controller reports data packaged according to the HID protocol, > > > > > > + but is incompatible with Microsoft's HID-over-SPI protocol. > > > > > > + > > > > > > +allOf: > > > > > > + - $ref: /schemas/spi/spi-peripheral-props.yaml# > > > > > > + > > > > > > +properties: > > > > > > + compatible: > > > > > > + enum: > > > > > > + - goodix,gt7986u > > > > > > > > > > This is already documented in goodix,gt7375p.yaml. Now linux-next has warnings: > > > > > > > > > > /builds/robherring/linux-dt/Documentation/devicetree/bindings/input/goodix,gt7986u.example.dtb: > > > > > touchscreen@0: compatible: 'oneOf' conditional failed, one must be > > > > > fixed: > > > > > ['goodix,gt7986u'] is too short > > > > > 'goodix,gt7375p' was expected > > > > > from schema $id: > > > > > http://devicetree.org/schemas/input/goodix,gt7375p.yaml# > > > > > /builds/robherring/linux-dt/Documentation/devicetree/bindings/input/goodix,gt7986u.example.dtb: > > > > > touchscreen@0: reg:0:0: 0 is not one of [93, 20] > > > > > from schema $id: > > > > > http://devicetree.org/schemas/input/goodix,gt7375p.yaml# > > > > > /builds/robherring/linux-dt/Documentation/devicetree/bindings/input/goodix,gt7986u.example.dtb: > > > > > touchscreen@0: 'vdd-supply' is a required property > > > > > from schema $id: > > > > > http://devicetree.org/schemas/input/goodix,gt7375p.yaml# > > > > > /builds/robherring/linux-dt/Documentation/devicetree/bindings/input/goodix,gt7986u.example.dtb: > > > > > touchscreen@0: 'goodix,hid-report-addr', 'spi-max-frequency' do not > > > > > match any of the regexes: 'pinctrl-[0-9]+' > > > > > from schema $id: > > > > > http://devicetree.org/schemas/input/goodix,gt7375p.yaml# > > > > > > > > > > Please sort this out and send a fix. > > > > > > > > I should add that it is intermittent whether you see this error or > > > > not. The tools select a single schema based on the compatible string > > > > and it is undefined which of the 2 schemas you will get. > > > > > > Still an issue and no response. Please fix or revert the series. > > > > I see that Krzysztof sent a revert, but what a proper fix would be? > > Apparently Goodix is using the same product ID gt7986u for both I2C and > > SPI parts, and covering them in one binding is not really easy (well, I > > guess it is possible with a big ugly "if"). Do we just slap "-spi" > > suffix on the compatible, so it becomes "goodix,gt7986u-spi" and go on > > on our merry way? Is there a better option for such products that > > support multiple interfaces/transports? > > Generally we just have 1 compatible and 1 schema for both. The schema > can reference spi-peripheral-props.yaml unconditionally. If someone > wants to put SPI properties in a node for I2C connected device, then > that would be allowed, but who cares. > > There's been some discussion about having some sort of pseudo property > similar to $nodename to define the bus for a node and then we could > use that. > Charles
Powered by blists - more mailing lists