Re: [PATCH] dt-bindings: yamllint: Require a space after a comment '#'

From: Rob Herring
Date: Wed Mar 08 2023 - 13:12:03 EST



On Fri, 03 Mar 2023 15:42:23 -0600, Rob Herring wrote:
> Enable yamllint to check the prefered commenting style of requiring a
> space after a comment character '#'. Fix the cases in the tree which
> have a warning with this enabled. Most cases just need a space after the
> '#'. A couple of cases with comments which were not intended to be
> comments are revealed. Those were in ti,sa2ul.yaml, ti,cal.yaml, and
> brcm,bcmgenet.yaml.
>
> Signed-off-by: Rob Herring <robh@xxxxxxxxxx>
> ---
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@xxxxxxxxxx>
> Cc: Stephen Boyd <sboyd@xxxxxxxxxx>
> Cc: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
> Cc: "David S. Miller" <davem@xxxxxxxxxxxxx>
> Cc: Rob Clark <robdclark@xxxxxxxxx>
> Cc: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
> Cc: Sean Paul <sean@xxxxxxxxxx>
> Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Cc: Marc Zyngier <maz@xxxxxxxxxx>
> Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>
> Cc: Eric Dumazet <edumazet@xxxxxxxxxx>
> Cc: Jakub Kicinski <kuba@xxxxxxxxxx>
> Cc: Paolo Abeni <pabeni@xxxxxxxxxx>
> Cc: Andrew Lunn <andrew@xxxxxxx>
> Cc: Heiner Kallweit <hkallweit1@xxxxxxxxx>
> Cc: Vinod Koul <vkoul@xxxxxxxxxx>
> Cc: Kishon Vijay Abraham I <kishon@xxxxxxxxxx>
> Cc: Mark Brown <broonie@xxxxxxxxxx>
> Cc: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
> Cc: linux-clk@xxxxxxxxxxxxxxx
> Cc: linux-crypto@xxxxxxxxxxxxxxx
> Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx
> Cc: freedreno@xxxxxxxxxxxxxxxxxxxxx
> Cc: linux-media@xxxxxxxxxxxxxxx
> Cc: netdev@xxxxxxxxxxxxxxx
> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> Cc: linux-phy@xxxxxxxxxxxxxxxxxxx
> Cc: linux-gpio@xxxxxxxxxxxxxxx
> Cc: alsa-devel@xxxxxxxxxxxxxxxx
> Cc: linux-riscv@xxxxxxxxxxxxxxxxxxx
> Cc: linux-spi@xxxxxxxxxxxxxxx
> ---
> Documentation/devicetree/bindings/.yamllint | 2 +-
> .../bindings/clock/qcom,a53pll.yaml | 4 ++--
> .../devicetree/bindings/crypto/ti,sa2ul.yaml | 4 ++--
> .../bindings/display/msm/qcom,mdp5.yaml | 2 +-
> .../interrupt-controller/arm,gic.yaml | 4 ++--
> .../loongson,pch-msi.yaml | 2 +-
> .../bindings/media/renesas,vin.yaml | 4 ++--
> .../devicetree/bindings/media/ti,cal.yaml | 4 ++--
> .../bindings/net/brcm,bcmgenet.yaml | 2 --
> .../bindings/net/cortina,gemini-ethernet.yaml | 6 ++---
> .../devicetree/bindings/net/mdio-gpio.yaml | 4 ++--
> .../phy/marvell,armada-cp110-utmi-phy.yaml | 2 +-
> .../bindings/phy/phy-stm32-usbphyc.yaml | 2 +-
> .../phy/qcom,sc7180-qmp-usb3-dp-phy.yaml | 2 +-
> .../bindings/pinctrl/pinctrl-mt8192.yaml | 2 +-
> .../regulator/nxp,pca9450-regulator.yaml | 8 +++----
> .../regulator/rohm,bd71828-regulator.yaml | 20 ++++++++--------
> .../regulator/rohm,bd71837-regulator.yaml | 6 ++---
> .../regulator/rohm,bd71847-regulator.yaml | 6 ++---
> .../bindings/soc/renesas/renesas.yaml | 2 +-
> .../devicetree/bindings/soc/ti/ti,pruss.yaml | 2 +-
> .../bindings/sound/amlogic,axg-tdm-iface.yaml | 2 +-
> .../bindings/sound/qcom,lpass-rx-macro.yaml | 4 ++--
> .../bindings/sound/qcom,lpass-tx-macro.yaml | 4 ++--
> .../bindings/sound/qcom,lpass-va-macro.yaml | 4 ++--
> .../sound/qcom,q6dsp-lpass-ports.yaml | 2 +-
> .../bindings/sound/simple-card.yaml | 24 +++++++++----------
> .../bindings/spi/microchip,mpfs-spi.yaml | 2 +-
> 28 files changed, 65 insertions(+), 67 deletions(-)
>

Applied, thanks!