On Fri, 05 Mar 2021 14:23:42 +0100, Linus Walleij wrote: > This rewrites the Broadcom bluetooth bindings in YAML and > adds a GPIO handle for the BT_RST_N line as used on some > platforms. > > The Ingenic UART binding was using this binding in its > example DTS fragment, however mistakenly using "vcc-supply" > for what is called "vbat-supply". The proper DTS files > and the code in the kernel all use "vbat-supply" so > fix up the example in this patch so we ge a clean > check. > > Cc: devicetree@xxxxxxxxxxxxxxx > Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > --- > ChangeLog v1->v2: > - Fix the error in the Ingenic UART binding example as > part of adding this patch. > --- > .../bindings/net/broadcom-bluetooth.txt | 56 --------- > .../bindings/net/broadcom-bluetooth.yaml | 117 ++++++++++++++++++ > 2 files changed, 117 insertions(+), 56 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/net/broadcom-bluetooth.txt > create mode 100644 Documentation/devicetree/bindings/net/broadcom-bluetooth.yaml > My bot found errors running 'make dt_binding_check' on your patch: yamllint warnings/errors: dtschema/dtc warnings/errors: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/serial/ingenic,uart.example.dt.yaml: bluetooth: 'vcc-supply' does not match any of the regexes: 'pinctrl-[0-9]+' From schema: /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/net/broadcom-bluetooth.yaml See https://patchwork.ozlabs.org/patch/1447828 This check can fail if there are any dependencies. The base for a patch series is generally the most recent rc1. If you already ran 'make dt_binding_check' and didn't see the above error(s), then make sure 'yamllint' is installed and dt-schema is up to date: pip3 install dtschema --upgrade Please check and re-submit.
