[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <nycvar.YFH.7.76.2409262001220.31206@cbobk.fhfr.pm>
Date: Thu, 26 Sep 2024 20:01:34 +0200 (CEST)
From: Jiri Kosina <jikos@...nel.org>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
cc: Dmitry Torokhov <dmitry.torokhov@...il.com>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Benjamin Tissoires <bentiss@...nel.org>,
Charles Wang <charles.goodix@...il.com>, linux-input@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] dt-bindings: input: Revert "dt-bindings: input:
Goodix SPI HID Touchscreen"
On Wed, 25 Sep 2024, Jiri Kosina wrote:
> > This reverts commit 9184b17fbc23 ("dt-bindings: input: Goodix SPI HID
> > Touchscreen") because it duplicates existing binding leadings to errors:
> >
> > goodix,gt7986u.example.dtb:
> > touchscreen@0: compatible: 'oneOf' conditional failed, one must be fixed:
> > ['goodix,gt7986u'] is too short
> > 'goodix,gt7375p' was expected
> >
> > This was reported on mailing list on 6th of September, but no reaction
> > happened from contributor or maintainer to fix it.
> >
> > Therefore let's drop binding which breaks and duplicates existing one.
> >
> > Fixes: 9184b17fbc23 ("dt-bindings: input: Goodix SPI HID Touchscreen")
> > Reported-by: Rob Herring <robh@...nel.org>
> > Closes: https://lore.kernel.org/all/CAL_Jsq+QfTtRj_JCqXzktQ49H8VUnztVuaBjvvkg3fwEHniUHw@mail.gmail.com/
> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
>
> I've applied both patches now and will expedite them to Linus.
Both are now in Linus' tree.
--
Jiri Kosina
SUSE Labs
Powered by blists - more mailing lists