| Message ID | 20220908131548.48120-10-jeff@labundy.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Additional fixes for Azoteq IQS7222A/B/C | expand |
On Thu, Sep 08, 2022 at 08:15:46AM -0500, Jeff LaBundy wrote: > The 'linux,code' property has been made optional in the driver; > update the binding to reflect this change. But with an old kernel, it would still be required. So still required in the DT. > > Fixes: 44dc42d254bf ("dt-bindings: input: Add bindings for Azoteq IQS7222A/B/C") > Signed-off-by: Jeff LaBundy <jeff@labundy.com> > --- > Documentation/devicetree/bindings/input/azoteq,iqs7222.yaml | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/Documentation/devicetree/bindings/input/azoteq,iqs7222.yaml b/Documentation/devicetree/bindings/input/azoteq,iqs7222.yaml > index 785fb9e83354..913fd2da9862 100644 > --- a/Documentation/devicetree/bindings/input/azoteq,iqs7222.yaml > +++ b/Documentation/devicetree/bindings/input/azoteq,iqs7222.yaml > @@ -473,9 +473,6 @@ patternProperties: > Specifies whether the event is to be interpreted as a key (1) > or a switch (5). > > - required: > - - linux,code > - > additionalProperties: false > > dependencies: > @@ -620,9 +617,6 @@ patternProperties: > GPIO, they must all be of the same type (proximity, touch or > slider gesture). > > - required: > - - linux,code > - > additionalProperties: false > > required: > -- > 2.25.1 > >
Hi Rob, Thank you for taking a look. On Tue, Sep 13, 2022 at 06:42:49AM -0500, Rob Herring wrote: > On Thu, Sep 08, 2022 at 08:15:46AM -0500, Jeff LaBundy wrote: > > The 'linux,code' property has been made optional in the driver; > > update the binding to reflect this change. > > But with an old kernel, it would still be required. So still required in > the DT. For v2 of this series, the corresponding driver change will be absorbed in a patch intended for -stable; the refactoring just worked out that way. In that case, even older kernels would drop this requirement, so I will keep the patch. In case I misunderstood your comment, please let me know. > > > > > Fixes: 44dc42d254bf ("dt-bindings: input: Add bindings for Azoteq IQS7222A/B/C") > > Signed-off-by: Jeff LaBundy <jeff@labundy.com> > > --- > > Documentation/devicetree/bindings/input/azoteq,iqs7222.yaml | 6 ------ > > 1 file changed, 6 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/input/azoteq,iqs7222.yaml b/Documentation/devicetree/bindings/input/azoteq,iqs7222.yaml > > index 785fb9e83354..913fd2da9862 100644 > > --- a/Documentation/devicetree/bindings/input/azoteq,iqs7222.yaml > > +++ b/Documentation/devicetree/bindings/input/azoteq,iqs7222.yaml > > @@ -473,9 +473,6 @@ patternProperties: > > Specifies whether the event is to be interpreted as a key (1) > > or a switch (5). > > > > - required: > > - - linux,code > > - > > additionalProperties: false > > > > dependencies: > > @@ -620,9 +617,6 @@ patternProperties: > > GPIO, they must all be of the same type (proximity, touch or > > slider gesture). > > > > - required: > > - - linux,code > > - > > additionalProperties: false > > > > required: > > -- > > 2.25.1 > > > > Kind regards, Jeff LaBundy
diff --git a/Documentation/devicetree/bindings/input/azoteq,iqs7222.yaml b/Documentation/devicetree/bindings/input/azoteq,iqs7222.yaml index 785fb9e83354..913fd2da9862 100644 --- a/Documentation/devicetree/bindings/input/azoteq,iqs7222.yaml +++ b/Documentation/devicetree/bindings/input/azoteq,iqs7222.yaml @@ -473,9 +473,6 @@ patternProperties: Specifies whether the event is to be interpreted as a key (1) or a switch (5). - required: - - linux,code - additionalProperties: false dependencies: @@ -620,9 +617,6 @@ patternProperties: GPIO, they must all be of the same type (proximity, touch or slider gesture). - required: - - linux,code - additionalProperties: false required:
The 'linux,code' property has been made optional in the driver; update the binding to reflect this change. Fixes: 44dc42d254bf ("dt-bindings: input: Add bindings for Azoteq IQS7222A/B/C") Signed-off-by: Jeff LaBundy <jeff@labundy.com> --- Documentation/devicetree/bindings/input/azoteq,iqs7222.yaml | 6 ------ 1 file changed, 6 deletions(-)