On Thu, Aug 10, 2023 at 2:31 AM Krzysztof Kozlowski <krzk@xxxxxxxxxx> wrote: > > On 08/08/2023 09:22, Wenhua Lin wrote: > > Add keypad driver ducumentation. > > 1. Please use scripts/get_maintainers.pl to get a list of necessary > people and lists to CC. It might happen, that command when run on an > older kernel, gives you outdated entries. Therefore please be sure you > base your patches on recent Linux kernel. > > You missed at least DT list (maybe more), so this won't be tested by > automated tooling. Performing review on untested code might be a waste > of time, thus I will skip this patch entirely till you follow the > process allowing the patch to be tested. > > Please kindly resend and include all necessary To/Cc entries. > > > 2. Please use subject prefixes matching the subsystem. You can get them > for example with `git log --oneline -- DIRECTORY_OR_FILE` on the > directory your patch is touching. > > > Limited review follows: > > > > > Signed-off-by: Wenhua Lin <Wenhua.Lin@xxxxxxxxxx> > > --- > > .../bindings/input/sprd-keypad.yaml | 76 +++++++++++++++++++ > > 1 file changed, 76 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/input/sprd-keypad.yaml > > Filename matching compatible, so sprd,block-name-etc.yaml > > > > > diff --git a/Documentation/devicetree/bindings/input/sprd-keypad.yaml b/Documentation/devicetree/bindings/input/sprd-keypad.yaml > > new file mode 100644 > > index 000000000000..51710e1eb389 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/input/sprd-keypad.yaml > > @@ -0,0 +1,76 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +# Copyright 2023 Unisoc Inc. > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/input/sprd-keypad.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Device-Tree bindings for GPIO attached keys > > This should not work and you should see warnings... which means: > > It does not look like you tested the bindings, at least after quick > look. Please run `make dt_binding_check` (see > Documentation/devicetree/bindings/writing-schema.rst for instructions). > Maybe you need to update your dtschema and yamllint. > > so drop "Device-tree bindings". > > Title also says nothing about the hardware - it is entirely unrelated. > Describe the hardware. > > > > + > > +maintainers: > > + - Orson Zhai <orsonzhai@xxxxxxxxx> > > + - Baolin Wang <baolin.wang7@xxxxxxxxx> > > + - Chunyan Zhang <zhang.lyra@xxxxxxxxx> > > + > > +description: | > > + Keypad controller is used to interface a SoC with a matrix-keypad device. > > Mention which SoC. > > > + The keypad controller supports multiple row and column lines. > > + A key can be placed at each intersection of a unique row and a unique column. > > + The keypad controller can sense a key-press and key-release and report the > > + event using a interrupt to the cpu. > > + > > +properties: > > + compatible: > > + const: sprd,sc9860-keypad > > + > > + reg: > > + maxItems: 1 > > OK, so this was for 100% not tested and won't work. No need to waste our > time on this. Test before sending: > > It does not look like you tested the bindings, at least after quick > look. Please run `make dt_binding_check` (see > Documentation/devicetree/bindings/writing-schema.rst for instructions). > Maybe you need to update your dtschema and yamllint. > > Best regards, > Krzysztof > Hi Krzysztof: This is a mistake of ours, sorry for wasting your precious time. Thanks Wenhua.Lin