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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/03/2023 23:42, 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(-)

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> # drm/msm
(and other Qualcom-specific schemas)

--
With best wishes
Dmitry




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux