dt-bindings: yamllint: Require a space after a comment '#'

Enable yamllint to check the preferred 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.

Acked-by: Jakub Kicinski <kuba@kernel.org>
Acked-by: Mark Brown <broonie@kernel.org>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> # drm/msm
Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
Reviewed-by: Stephen Boyd <sboyd@kernel.org>
Link: https://lore.kernel.org/r/20230303214223.49451-1-robh@kernel.org
Signed-off-by: Rob Herring <robh@kernel.org>
This commit is contained in:
Rob Herring 2023-03-03 15:42:23 -06:00
parent 20a72af11f
commit d0dcd0ce84
28 changed files with 65 additions and 67 deletions

View File

@ -19,7 +19,7 @@ rules:
colons: {max-spaces-before: 0, max-spaces-after: 1}
commas: {min-spaces-after: 1, max-spaces-after: 1}
comments:
require-starting-space: false
require-starting-space: true
min-spaces-from-content: 1
comments-indentation: disable
document-start:

View File

@ -45,14 +45,14 @@ required:
additionalProperties: false
examples:
#Example 1 - A53 PLL found on MSM8916 devices
# Example 1 - A53 PLL found on MSM8916 devices
- |
a53pll: clock@b016000 {
compatible = "qcom,msm8916-a53pll";
reg = <0xb016000 0x40>;
#clock-cells = <0>;
};
#Example 2 - A53 PLL found on IPQ6018 devices
# Example 2 - A53 PLL found on IPQ6018 devices
- |
a53pll_ipq: clock-controller@b116000 {
compatible = "qcom,ipq6018-a53pll";

View File

@ -26,8 +26,8 @@ properties:
dmas:
items:
- description: TX DMA Channel
- description: RX DMA Channel #1
- description: RX DMA Channel #2
- description: 'RX DMA Channel #1'
- description: 'RX DMA Channel #2'
dma-names:
items:

View File

@ -61,7 +61,7 @@ properties:
- const: lut
- const: tbu
- const: tbu_rt
#MSM8996 has additional iommu clock
# MSM8996 has additional iommu clock
- items:
- const: iface
- const: bus

View File

@ -133,8 +133,8 @@ properties:
- items: # for "arm,cortex-a9-gic"
- const: PERIPHCLK
- const: PERIPHCLKEN
- const: clk # for "arm,gic-400" and "nvidia,tegra210"
- const: gclk #for "arm,pl390"
- const: clk # for "arm,gic-400" and "nvidia,tegra210"
- const: gclk # for "arm,pl390"
power-domains:
maxItems: 1

View File

@ -46,7 +46,7 @@ required:
- loongson,msi-base-vec
- loongson,msi-num-vecs
additionalProperties: true #fixme
additionalProperties: true # fixme
examples:
- |

View File

@ -69,7 +69,7 @@ properties:
resets:
maxItems: 1
#The per-board settings for Gen2 and RZ/G1 platforms:
# The per-board settings for Gen2 and RZ/G1 platforms:
port:
$ref: /schemas/graph.yaml#/$defs/port-base
unevaluatedProperties: false
@ -108,7 +108,7 @@ properties:
data-active: true
#The per-board settings for Gen3 and RZ/G2 platforms:
# The per-board settings for Gen3 and RZ/G2 platforms:
renesas,id:
description: VIN channel number
$ref: /schemas/types.yaml#/definitions/uint32

View File

@ -75,7 +75,7 @@ properties:
port@0:
$ref: /schemas/graph.yaml#/$defs/port-base
unevaluatedProperties: false
description: CSI2 Port #0
description: 'CSI2 Port #0'
properties:
endpoint:
@ -93,7 +93,7 @@ properties:
port@1:
$ref: /schemas/graph.yaml#/$defs/port-base
unevaluatedProperties: false
description: CSI2 Port #1
description: 'CSI2 Port #1'
properties:
endpoint:

View File

@ -73,8 +73,6 @@ allOf:
unevaluatedProperties: false
examples:
#include <dt-bindings/interrupt-controller/arm-gic.h>
- |
ethernet@f0b60000 {
phy-mode = "internal";

View File

@ -31,9 +31,9 @@ properties:
ranges: true
#The subnodes represents the two ethernet ports in this device.
#They are not independent of each other since they share resources
#in the parent node, and are thus children.
# The subnodes represents the two ethernet ports in this device.
# They are not independent of each other since they share resources
# in the parent node, and are thus children.
patternProperties:
"^ethernet-port@[0-9]+$":
type: object

View File

@ -33,8 +33,8 @@ properties:
- description: MDIO
- description: MDO
#Note: Each gpio-mdio bus should have an alias correctly numbered in "aliases"
#node.
# Note: Each gpio-mdio bus should have an alias correctly numbered in "aliases"
# node.
additionalProperties:
type: object

View File

@ -41,7 +41,7 @@ properties:
Phandle to the system controller node
$ref: /schemas/types.yaml#/definitions/phandle
#Required child nodes:
# Required child nodes:
patternProperties:
"^usb-phy@[0|1]$":

View File

@ -55,7 +55,7 @@ properties:
description: number of clock cells for ck_usbo_48m consumer
const: 0
#Required child nodes:
# Required child nodes:
patternProperties:
"^usb-phy@[0|1]$":

View File

@ -83,7 +83,7 @@ properties:
description:
Phandle to a regulator supply to any specific refclk pll block.
#Required nodes:
# Required nodes:
patternProperties:
"^usb3-phy@[0-9a-f]+$":
type: object

View File

@ -51,7 +51,7 @@ properties:
description: The interrupt outputs to sysirq.
maxItems: 1
#PIN CONFIGURATION NODES
# PIN CONFIGURATION NODES
patternProperties:
'-pins$':
type: object

View File

@ -17,10 +17,10 @@ description: |
Datasheet is available at
https://www.nxp.com/docs/en/data-sheet/PCA9450DS.pdf
#The valid names for PCA9450 regulator nodes are:
#BUCK1, BUCK2, BUCK3, BUCK4, BUCK5, BUCK6,
#LDO1, LDO2, LDO3, LDO4, LDO5
#Note: Buck3 removed on PCA9450B and connect with Buck1 on PCA9450C.
# The valid names for PCA9450 regulator nodes are:
# BUCK1, BUCK2, BUCK3, BUCK4, BUCK5, BUCK6,
# LDO1, LDO2, LDO3, LDO4, LDO5
# Note: Buck3 removed on PCA9450B and connect with Buck1 on PCA9450C.
properties:
compatible:

View File

@ -82,20 +82,20 @@ patternProperties:
# Supported default DVS states:
# buck | run | idle | suspend | lpsr
#--------------------------------------------------------------
# --------------------------------------------------------------
# 1, 2, 6, and 7 | supported | supported | supported (*)
#--------------------------------------------------------------
# --------------------------------------------------------------
# 3, 4, and 5 | supported (**)
#--------------------------------------------------------------
# --------------------------------------------------------------
#
#(*) LPSR and SUSPEND states use same voltage but both states have own
# enable /
# disable settings. Voltage 0 can be specified for a state to make
# regulator disabled on that state.
# (*) LPSR and SUSPEND states use same voltage but both states have own
# enable /
# disable settings. Voltage 0 can be specified for a state to make
# regulator disabled on that state.
#
#(**) All states use same voltage but have own enable / disable
# settings. Voltage 0 can be specified for a state to make
# regulator disabled on that state.
# (**) All states use same voltage but have own enable / disable
# settings. Voltage 0 can be specified for a state to make
# regulator disabled on that state.
required:
- regulator-name

View File

@ -23,9 +23,9 @@ description: |
if they are disabled at startup the voltage monitoring for LDO5/LDO6 will
cause PMIC to reset.
#The valid names for BD71837 regulator nodes are:
#BUCK1, BUCK2, BUCK3, BUCK4, BUCK5, BUCK6, BUCK7, BUCK8
#LDO1, LDO2, LDO3, LDO4, LDO5, LDO6, LDO7
# The valid names for BD71837 regulator nodes are:
# BUCK1, BUCK2, BUCK3, BUCK4, BUCK5, BUCK6, BUCK7, BUCK8
# LDO1, LDO2, LDO3, LDO4, LDO5, LDO6, LDO7
patternProperties:
"^LDO[1-7]$":

View File

@ -22,9 +22,9 @@ description: |
not be disabled by driver at startup. If BUCK5 is disabled at startup the
voltage monitoring for LDO5/LDO6 can cause PMIC to reset.
#The valid names for BD71847 regulator nodes are:
#BUCK1, BUCK2, BUCK3, BUCK4, BUCK5, BUCK6
#LDO1, LDO2, LDO3, LDO4, LDO5, LDO6
# The valid names for BD71847 regulator nodes are:
# BUCK1, BUCK2, BUCK3, BUCK4, BUCK5, BUCK6
# LDO1, LDO2, LDO3, LDO4, LDO5, LDO6
patternProperties:
"^LDO[1-6]$":

View File

@ -111,7 +111,7 @@ properties:
- description: RZ/G1C (R8A77470)
items:
- enum:
- iwave,g23s #iWave Systems RZ/G1C Single Board Computer (iW-RainboW-G23S)
- iwave,g23s # iWave Systems RZ/G1C Single Board Computer (iW-RainboW-G23S)
- const: renesas,r8a77470
- description: RZ/G2M (R8A774A1)

View File

@ -313,7 +313,7 @@ additionalProperties: false
# Due to inability of correctly verifying sub-nodes with an @address through
# the "required" list, the required sub-nodes below are commented out for now.
#required:
# required:
# - memories
# - interrupt-controller
# - pru

View File

@ -24,7 +24,7 @@ properties:
items:
- description: Bit clock
- description: Sample clock
- description: Master clock #optional
- description: Master clock # optional
clock-names:
minItems: 2

View File

@ -34,13 +34,13 @@ properties:
clock-names:
oneOf:
- items: #for ADSP based platforms
- items: # for ADSP based platforms
- const: mclk
- const: npl
- const: macro
- const: dcodec
- const: fsgen
- items: #for ADSP bypass based platforms
- items: # for ADSP bypass based platforms
- const: mclk
- const: npl
- const: fsgen

View File

@ -36,13 +36,13 @@ properties:
clock-names:
oneOf:
- items: #for ADSP based platforms
- items: # for ADSP based platforms
- const: mclk
- const: npl
- const: macro
- const: dcodec
- const: fsgen
- items: #for ADSP bypass based platforms
- items: # for ADSP bypass based platforms
- const: mclk
- const: npl
- const: fsgen

View File

@ -34,11 +34,11 @@ properties:
clock-names:
oneOf:
- items: #for ADSP based platforms
- items: # for ADSP based platforms
- const: mclk
- const: macro
- const: dcodec
- items: #for ADSP bypass based platforms
- items: # for ADSP bypass based platforms
- const: mclk
clock-output-names:

View File

@ -26,7 +26,7 @@ properties:
'#size-cells':
const: 0
#Digital Audio Interfaces
# Digital Audio Interfaces
patternProperties:
'^dai@[0-9]+$':
type: object

View File

@ -262,9 +262,9 @@ required:
additionalProperties: false
examples:
#--------------------
# --------------------
# single DAI link
#--------------------
# --------------------
- |
sound {
compatible = "simple-audio-card";
@ -291,9 +291,9 @@ examples:
};
};
#--------------------
# --------------------
# Multi DAI links
#--------------------
# --------------------
- |
sound {
compatible = "simple-audio-card";
@ -334,10 +334,10 @@ examples:
};
};
#--------------------
# --------------------
# route audio from IMX6 SSI2 through TLV320DAC3100 codec
# through TPA6130A2 amplifier to headphones:
#--------------------
# --------------------
- |
sound {
compatible = "simple-audio-card";
@ -359,9 +359,9 @@ examples:
};
};
#--------------------
# --------------------
# Sampling Rate Conversion
#--------------------
# --------------------
- |
sound {
compatible = "simple-audio-card";
@ -387,9 +387,9 @@ examples:
};
};
#--------------------
# --------------------
# 2 CPU 1 Codec (Mixing)
#--------------------
# --------------------
- |
sound {
compatible = "simple-audio-card";
@ -424,7 +424,7 @@ examples:
};
};
#--------------------
# --------------------
# Multi DAI links with DPCM:
#
# CPU0 ------ ak4613
@ -433,7 +433,7 @@ examples:
# CPU3 --/ /* DPCM 5ch/6ch */
# CPU4 --/ /* DPCM 7ch/8ch */
# CPU5 ------ PCM3168A-c
#--------------------
# --------------------
- |
sound {
compatible = "simple-audio-card";

View File

@ -22,7 +22,7 @@ properties:
- items:
- const: microchip,mpfs-qspi
- const: microchip,coreqspi-rtl-v2
- const: microchip,coreqspi-rtl-v2 #FPGA QSPI
- const: microchip,coreqspi-rtl-v2 # FPGA QSPI
- const: microchip,mpfs-spi
reg: