* [PATCH 1/2] dt-bindings: mmc: drop unneeded quotes
@ 2023-01-20 8:57 Krzysztof Kozlowski
2023-01-20 8:57 ` [PATCH 2/2] dt-bindings: mmc: correct pwrseq node names Krzysztof Kozlowski
` (5 more replies)
0 siblings, 6 replies; 12+ messages in thread
From: Krzysztof Kozlowski @ 2023-01-20 8:57 UTC (permalink / raw)
To: Ulf Hansson, Rob Herring, Krzysztof Kozlowski, Chen-Yu Tsai,
Jernej Skrabec, Samuel Holland, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
Lars Povlsen, Steen Hegelund, Daniel Machon, UNGLinuxDriver,
Thierry Reding, Jonathan Hunter, Andreas Färber,
Manivannan Sadhasivam, Heiko Stuebner, Alim Akhtar, Tony Huang,
Li-hao Kuo, Maxime Ripard, Adrian Hunter, Masahiro Yamada,
Markus Pargmann, Wolfram Sang, Jaehoon Chung, linux-mmc,
devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-amlogic, linux-tegra, linux-actions, linux-rockchip,
linux-samsung-soc
Cc: Krzysztof Kozlowski
Cleanup by removing unneeded quotes from refs and redundant blank lines.
No functional impact except adjusting to preferred coding style.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
.../bindings/mmc/allwinner,sun4i-a10-mmc.yaml | 2 +-
.../bindings/mmc/amlogic,meson-mx-sdhc.yaml | 2 +-
.../devicetree/bindings/mmc/arasan,sdhci.yaml | 6 ++--
.../devicetree/bindings/mmc/cdns,sdhci.yaml | 22 ++++++------
.../bindings/mmc/fsl-imx-esdhc.yaml | 4 +--
.../devicetree/bindings/mmc/fsl-imx-mmc.yaml | 2 +-
.../mmc/microchip,dw-sparx5-sdhci.yaml | 4 +--
.../devicetree/bindings/mmc/mmc-spi-slot.yaml | 2 +-
.../devicetree/bindings/mmc/mxs-mmc.yaml | 2 +-
.../bindings/mmc/nvidia,tegra20-sdhci.yaml | 36 +++++++++----------
.../devicetree/bindings/mmc/owl-mmc.yaml | 2 +-
.../bindings/mmc/renesas,mmcif.yaml | 2 +-
.../devicetree/bindings/mmc/renesas,sdhi.yaml | 6 ++--
.../bindings/mmc/rockchip-dw-mshc.yaml | 2 +-
.../bindings/mmc/samsung,exynos-dw-mshc.yaml | 2 +-
.../devicetree/bindings/mmc/sunplus,mmc.yaml | 2 +-
.../bindings/mmc/synopsys-dw-mshc-common.yaml | 2 +-
17 files changed, 50 insertions(+), 50 deletions(-)
diff --git a/Documentation/devicetree/bindings/mmc/allwinner,sun4i-a10-mmc.yaml b/Documentation/devicetree/bindings/mmc/allwinner,sun4i-a10-mmc.yaml
index 02ecc93417ef..0ccd632d5620 100644
--- a/Documentation/devicetree/bindings/mmc/allwinner,sun4i-a10-mmc.yaml
+++ b/Documentation/devicetree/bindings/mmc/allwinner,sun4i-a10-mmc.yaml
@@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
title: Allwinner A10 MMC Controller
allOf:
- - $ref: "mmc-controller.yaml"
+ - $ref: mmc-controller.yaml
maintainers:
- Chen-Yu Tsai <wens@csie.org>
diff --git a/Documentation/devicetree/bindings/mmc/amlogic,meson-mx-sdhc.yaml b/Documentation/devicetree/bindings/mmc/amlogic,meson-mx-sdhc.yaml
index 1c391bec43dc..1a6cda82f296 100644
--- a/Documentation/devicetree/bindings/mmc/amlogic,meson-mx-sdhc.yaml
+++ b/Documentation/devicetree/bindings/mmc/amlogic,meson-mx-sdhc.yaml
@@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
title: Amlogic Meson SDHC controller
allOf:
- - $ref: "mmc-controller.yaml"
+ - $ref: mmc-controller.yaml
maintainers:
- Martin Blumenstingl <martin.blumenstingl@googlemail.com>
diff --git a/Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml b/Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml
index 4053de758db6..8296c34cfa00 100644
--- a/Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml
+++ b/Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml
@@ -1,8 +1,8 @@
# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
%YAML 1.2
---
-$id: "http://devicetree.org/schemas/mmc/arasan,sdhci.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/mmc/arasan,sdhci.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
title: Arasan SDHCI Controller
@@ -10,7 +10,7 @@ maintainers:
- Adrian Hunter <adrian.hunter@intel.com>
allOf:
- - $ref: "mmc-controller.yaml#"
+ - $ref: mmc-controller.yaml#
- if:
properties:
compatible:
diff --git a/Documentation/devicetree/bindings/mmc/cdns,sdhci.yaml b/Documentation/devicetree/bindings/mmc/cdns,sdhci.yaml
index 8b1a0fdcb5e3..d3dce4d6c168 100644
--- a/Documentation/devicetree/bindings/mmc/cdns,sdhci.yaml
+++ b/Documentation/devicetree/bindings/mmc/cdns,sdhci.yaml
@@ -36,43 +36,43 @@ properties:
cdns,phy-input-delay-sd-highspeed:
description: Value of the delay in the input path for SD high-speed timing
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
minimum: 0
maximum: 0x1f
cdns,phy-input-delay-legacy:
description: Value of the delay in the input path for legacy timing
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
minimum: 0
maximum: 0x1f
cdns,phy-input-delay-sd-uhs-sdr12:
description: Value of the delay in the input path for SD UHS SDR12 timing
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
minimum: 0
maximum: 0x1f
cdns,phy-input-delay-sd-uhs-sdr25:
description: Value of the delay in the input path for SD UHS SDR25 timing
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
minimum: 0
maximum: 0x1f
cdns,phy-input-delay-sd-uhs-sdr50:
description: Value of the delay in the input path for SD UHS SDR50 timing
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
minimum: 0
maximum: 0x1f
cdns,phy-input-delay-sd-uhs-ddr50:
description: Value of the delay in the input path for SD UHS DDR50 timing
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
minimum: 0
maximum: 0x1f
cdns,phy-input-delay-mmc-highspeed:
description: Value of the delay in the input path for MMC high-speed timing
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
minimum: 0
maximum: 0x1f
@@ -83,7 +83,7 @@ properties:
# Each delay property represents the fraction of the clock period.
# The approximate delay value will be
# (<delay property value>/128)*sdmclk_clock_period.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
minimum: 0
maximum: 0x1f
@@ -91,7 +91,7 @@ properties:
description: |
Value of the delay introduced on the sdclk output for all modes except
HS200, HS400 and HS400_ES.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
minimum: 0
maximum: 0x7f
@@ -99,7 +99,7 @@ properties:
description: |
Value of the delay introduced on the sdclk output for HS200, HS400 and
HS400_ES speed modes.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
minimum: 0
maximum: 0x7f
@@ -107,7 +107,7 @@ properties:
description: |
Value of the delay introduced on the dat_strobe input used in
HS400 / HS400_ES speed modes.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
minimum: 0
maximum: 0x7f
diff --git a/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml b/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
index b3289593d5f1..269e0f421407 100644
--- a/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
+++ b/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
@@ -10,7 +10,7 @@ maintainers:
- Shawn Guo <shawnguo@kernel.org>
allOf:
- - $ref: "mmc-controller.yaml"
+ - $ref: mmc-controller.yaml
description: |
The Enhanced Secure Digital Host Controller on Freescale i.MX family
@@ -112,7 +112,7 @@ properties:
default: 0
voltage-ranges:
- $ref: '/schemas/types.yaml#/definitions/uint32-matrix'
+ $ref: /schemas/types.yaml#/definitions/uint32-matrix
description: |
Specify the voltage range in case there are software transparent level
shifters on the outputs of the controller. Two cells are required, first
diff --git a/Documentation/devicetree/bindings/mmc/fsl-imx-mmc.yaml b/Documentation/devicetree/bindings/mmc/fsl-imx-mmc.yaml
index ffa162722b8e..221f5bc047bd 100644
--- a/Documentation/devicetree/bindings/mmc/fsl-imx-mmc.yaml
+++ b/Documentation/devicetree/bindings/mmc/fsl-imx-mmc.yaml
@@ -10,7 +10,7 @@ maintainers:
- Markus Pargmann <mpa@pengutronix.de>
allOf:
- - $ref: "mmc-controller.yaml"
+ - $ref: mmc-controller.yaml
properties:
compatible:
diff --git a/Documentation/devicetree/bindings/mmc/microchip,dw-sparx5-sdhci.yaml b/Documentation/devicetree/bindings/mmc/microchip,dw-sparx5-sdhci.yaml
index fa6cfe092fc9..1f63faf17743 100644
--- a/Documentation/devicetree/bindings/mmc/microchip,dw-sparx5-sdhci.yaml
+++ b/Documentation/devicetree/bindings/mmc/microchip,dw-sparx5-sdhci.yaml
@@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
title: Microchip Sparx5 Mobile Storage Host Controller
allOf:
- - $ref: "mmc-controller.yaml"
+ - $ref: mmc-controller.yaml
maintainers:
- Lars Povlsen <lars.povlsen@microchip.com>
@@ -35,7 +35,7 @@ properties:
microchip,clock-delay:
description: Delay clock to card to meet setup time requirements.
Each step increase by 1.25ns.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
minimum: 1
maximum: 15
diff --git a/Documentation/devicetree/bindings/mmc/mmc-spi-slot.yaml b/Documentation/devicetree/bindings/mmc/mmc-spi-slot.yaml
index c0662ce9946d..36acc40c7d18 100644
--- a/Documentation/devicetree/bindings/mmc/mmc-spi-slot.yaml
+++ b/Documentation/devicetree/bindings/mmc/mmc-spi-slot.yaml
@@ -10,7 +10,7 @@ maintainers:
- Ulf Hansson <ulf.hansson@linaro.org>
allOf:
- - $ref: "mmc-controller.yaml"
+ - $ref: mmc-controller.yaml
- $ref: /schemas/spi/spi-peripheral-props.yaml
description: |
diff --git a/Documentation/devicetree/bindings/mmc/mxs-mmc.yaml b/Documentation/devicetree/bindings/mmc/mxs-mmc.yaml
index bec8f8c71ff2..32e512a68ed6 100644
--- a/Documentation/devicetree/bindings/mmc/mxs-mmc.yaml
+++ b/Documentation/devicetree/bindings/mmc/mxs-mmc.yaml
@@ -17,7 +17,7 @@ description: |
and the properties used by the mxsmmc driver.
allOf:
- - $ref: "mmc-controller.yaml"
+ - $ref: mmc-controller.yaml
properties:
compatible:
diff --git a/Documentation/devicetree/bindings/mmc/nvidia,tegra20-sdhci.yaml b/Documentation/devicetree/bindings/mmc/nvidia,tegra20-sdhci.yaml
index fe0270207622..285057523ead 100644
--- a/Documentation/devicetree/bindings/mmc/nvidia,tegra20-sdhci.yaml
+++ b/Documentation/devicetree/bindings/mmc/nvidia,tegra20-sdhci.yaml
@@ -83,7 +83,7 @@ properties:
maxItems: 1
operating-points-v2:
- $ref: "/schemas/types.yaml#/definitions/phandle"
+ $ref: /schemas/types.yaml#/definitions/phandle
power-domains:
items:
@@ -100,53 +100,53 @@ properties:
The DQS trim values are only used on controllers which support HS400
timing. Only SDMMC4 on Tegra210 and Tegra186 supports HS400.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
nvidia,default-trim:
description: Specify the default outbound clock trimmer value.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
nvidia,dqs-trim:
description: Specify DQS trim value for HS400 timing.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
nvidia,pad-autocal-pull-down-offset-1v8:
description: Specify drive strength calibration offsets for 1.8 V
signaling modes.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
nvidia,pad-autocal-pull-down-offset-1v8-timeout:
description: Specify drive strength used as a fallback in case the
automatic calibration times out on a 1.8 V signaling mode.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
nvidia,pad-autocal-pull-down-offset-3v3:
description: Specify drive strength calibration offsets for 3.3 V
signaling modes.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
nvidia,pad-autocal-pull-down-offset-3v3-timeout:
description: Specify drive strength used as a fallback in case the
automatic calibration times out on a 3.3 V signaling mode.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
nvidia,pad-autocal-pull-down-offset-sdr104:
description: Specify drive strength calibration offsets for SDR104 mode.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
nvidia,pad-autocal-pull-down-offset-hs400:
description: Specify drive strength calibration offsets for HS400 mode.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
nvidia,pad-autocal-pull-up-offset-1v8:
description: Specify drive strength calibration offsets for 1.8 V
signaling modes.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
nvidia,pad-autocal-pull-up-offset-1v8-timeout:
description: Specify drive strength used as a fallback in case the
automatic calibration times out on a 1.8 V signaling mode.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
nvidia,pad-autocal-pull-up-offset-3v3:
description: Specify drive strength calibration offsets for 3.3 V
@@ -158,25 +158,25 @@ properties:
refer to the reference manual of the SoC for correct values. The SDR104
and HS400 timing specific values are used in corresponding modes if
specified.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
nvidia,pad-autocal-pull-up-offset-3v3-timeout:
description: Specify drive strength used as a fallback in case the
automatic calibration times out on a 3.3 V signaling mode.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
nvidia,pad-autocal-pull-up-offset-sdr104:
description: Specify drive strength calibration offsets for SDR104 mode.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
nvidia,pad-autocal-pull-up-offset-hs400:
description: Specify drive strength calibration offsets for HS400 mode.
- $ref: "/schemas/types.yaml#/definitions/uint32"
+ $ref: /schemas/types.yaml#/definitions/uint32
nvidia,only-1-8v:
description: The presence of this property indicates that the controller
operates at a 1.8 V fixed I/O voltage.
- $ref: "/schemas/types.yaml#/definitions/flag"
+ $ref: /schemas/types.yaml#/definitions/flag
required:
- compatible
@@ -187,7 +187,7 @@ required:
- reset-names
allOf:
- - $ref: "mmc-controller.yaml"
+ - $ref: mmc-controller.yaml
- if:
properties:
compatible:
diff --git a/Documentation/devicetree/bindings/mmc/owl-mmc.yaml b/Documentation/devicetree/bindings/mmc/owl-mmc.yaml
index b0d81ebe0f6e..1b7d88ed3799 100644
--- a/Documentation/devicetree/bindings/mmc/owl-mmc.yaml
+++ b/Documentation/devicetree/bindings/mmc/owl-mmc.yaml
@@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
title: Actions Semi Owl SoCs SD/MMC/SDIO controller
allOf:
- - $ref: "mmc-controller.yaml"
+ - $ref: mmc-controller.yaml
maintainers:
- Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
diff --git a/Documentation/devicetree/bindings/mmc/renesas,mmcif.yaml b/Documentation/devicetree/bindings/mmc/renesas,mmcif.yaml
index c36ba561c387..024313b79ec9 100644
--- a/Documentation/devicetree/bindings/mmc/renesas,mmcif.yaml
+++ b/Documentation/devicetree/bindings/mmc/renesas,mmcif.yaml
@@ -10,7 +10,7 @@ maintainers:
- Wolfram Sang <wsa+renesas@sang-engineering.com>
allOf:
- - $ref: "mmc-controller.yaml"
+ - $ref: mmc-controller.yaml
properties:
compatible:
diff --git a/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml b/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml
index fa089cbb155b..7756a8687eaf 100644
--- a/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml
+++ b/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml
@@ -1,8 +1,8 @@
# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
%YAML 1.2
---
-$id: "http://devicetree.org/schemas/mmc/renesas,sdhi.yaml#"
-$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+$id: http://devicetree.org/schemas/mmc/renesas,sdhi.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
title: Renesas SDHI SD/MMC controller
@@ -112,7 +112,7 @@ properties:
max-frequency: true
allOf:
- - $ref: "mmc-controller.yaml"
+ - $ref: mmc-controller.yaml
- if:
properties:
diff --git a/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml b/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml
index c7e14b7dba9e..67d7223f74da 100644
--- a/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml
+++ b/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml
@@ -14,7 +14,7 @@ description:
file and the Rockchip specific extensions.
allOf:
- - $ref: "synopsys-dw-mshc-common.yaml#"
+ - $ref: synopsys-dw-mshc-common.yaml#
maintainers:
- Heiko Stuebner <heiko@sntech.de>
diff --git a/Documentation/devicetree/bindings/mmc/samsung,exynos-dw-mshc.yaml b/Documentation/devicetree/bindings/mmc/samsung,exynos-dw-mshc.yaml
index fdaa18481aa0..6ee78a38bd74 100644
--- a/Documentation/devicetree/bindings/mmc/samsung,exynos-dw-mshc.yaml
+++ b/Documentation/devicetree/bindings/mmc/samsung,exynos-dw-mshc.yaml
@@ -112,7 +112,7 @@ required:
- samsung,dw-mshc-sdr-timing
allOf:
- - $ref: "synopsys-dw-mshc-common.yaml#"
+ - $ref: synopsys-dw-mshc-common.yaml#
- if:
properties:
compatible:
diff --git a/Documentation/devicetree/bindings/mmc/sunplus,mmc.yaml b/Documentation/devicetree/bindings/mmc/sunplus,mmc.yaml
index 23aa8e6b2d70..611687166735 100644
--- a/Documentation/devicetree/bindings/mmc/sunplus,mmc.yaml
+++ b/Documentation/devicetree/bindings/mmc/sunplus,mmc.yaml
@@ -12,7 +12,7 @@ maintainers:
- Li-hao Kuo <lhjeff911@gmail.com>
allOf:
- - $ref: "mmc-controller.yaml"
+ - $ref: mmc-controller.yaml
properties:
compatible:
diff --git a/Documentation/devicetree/bindings/mmc/synopsys-dw-mshc-common.yaml b/Documentation/devicetree/bindings/mmc/synopsys-dw-mshc-common.yaml
index 8dfad89c78a7..6f11b2adf103 100644
--- a/Documentation/devicetree/bindings/mmc/synopsys-dw-mshc-common.yaml
+++ b/Documentation/devicetree/bindings/mmc/synopsys-dw-mshc-common.yaml
@@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
title: Synopsys Designware Mobile Storage Host Controller Common Properties
allOf:
- - $ref: "mmc-controller.yaml#"
+ - $ref: mmc-controller.yaml#
maintainers:
- Ulf Hansson <ulf.hansson@linaro.org>
--
2.34.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/2] dt-bindings: mmc: correct pwrseq node names
2023-01-20 8:57 [PATCH 1/2] dt-bindings: mmc: drop unneeded quotes Krzysztof Kozlowski
@ 2023-01-20 8:57 ` Krzysztof Kozlowski
2023-01-20 9:38 ` Heiko Stübner
` (2 more replies)
2023-01-20 9:12 ` [PATCH 1/2] dt-bindings: mmc: drop unneeded quotes Wolfram Sang
` (4 subsequent siblings)
5 siblings, 3 replies; 12+ messages in thread
From: Krzysztof Kozlowski @ 2023-01-20 8:57 UTC (permalink / raw)
To: Ulf Hansson, Rob Herring, Krzysztof Kozlowski, Chen-Yu Tsai,
Jernej Skrabec, Samuel Holland, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
Lars Povlsen, Steen Hegelund, Daniel Machon, UNGLinuxDriver,
Thierry Reding, Jonathan Hunter, Andreas Färber,
Manivannan Sadhasivam, Heiko Stuebner, Alim Akhtar, Tony Huang,
Li-hao Kuo, Maxime Ripard, Adrian Hunter, Masahiro Yamada,
Markus Pargmann, Wolfram Sang, Jaehoon Chung, linux-mmc,
devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-amlogic, linux-tegra, linux-actions, linux-rockchip,
linux-samsung-soc
Cc: Krzysztof Kozlowski
Node names should be generic and should not contain underscores.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml | 2 +-
Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml | 2 +-
Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml
index 911a5996e099..588be73168fa 100644
--- a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml
+++ b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml
@@ -41,7 +41,7 @@ additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
- sdhci0_pwrseq {
+ pwrseq {
compatible = "mmc-pwrseq-emmc";
reset-gpios = <&gpio1 12 GPIO_ACTIVE_LOW>;
};
diff --git a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml
index 3397dbff88c2..b35e00e8c65e 100644
--- a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml
+++ b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml
@@ -35,7 +35,7 @@ additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
- wifi_pwrseq: wifi_pwrseq {
+ pwrseq {
compatible = "mmc-pwrseq-sd8787";
powerdown-gpios = <&twl_gpio 0 GPIO_ACTIVE_LOW>;
reset-gpios = <&twl_gpio 1 GPIO_ACTIVE_LOW>;
diff --git a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
index 64e3644eefeb..00feaafc1063 100644
--- a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
+++ b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
@@ -55,7 +55,7 @@ additionalProperties: false
examples:
- |
#include <dt-bindings/gpio/gpio.h>
- sdhci0_pwrseq {
+ pwrseq {
compatible = "mmc-pwrseq-simple";
reset-gpios = <&gpio1 12 GPIO_ACTIVE_LOW>;
clocks = <&clk_32768_ck>;
--
2.34.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] dt-bindings: mmc: drop unneeded quotes
2023-01-20 8:57 [PATCH 1/2] dt-bindings: mmc: drop unneeded quotes Krzysztof Kozlowski
2023-01-20 8:57 ` [PATCH 2/2] dt-bindings: mmc: correct pwrseq node names Krzysztof Kozlowski
@ 2023-01-20 9:12 ` Wolfram Sang
2023-01-20 9:30 ` Heiko Stübner
` (3 subsequent siblings)
5 siblings, 0 replies; 12+ messages in thread
From: Wolfram Sang @ 2023-01-20 9:12 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Ulf Hansson, Rob Herring, Krzysztof Kozlowski, Chen-Yu Tsai,
Jernej Skrabec, Samuel Holland, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
Lars Povlsen, Steen Hegelund, Daniel Machon, UNGLinuxDriver,
Thierry Reding, Jonathan Hunter, Andreas Färber,
Manivannan Sadhasivam, Heiko Stuebner, Alim Akhtar, Tony Huang,
Li-hao Kuo, Maxime Ripard, Adrian Hunter, Masahiro Yamada,
Markus Pargmann, Jaehoon Chung, linux-mmc, devicetree,
linux-arm-kernel, linux-sunxi, linux-kernel, linux-amlogic,
linux-tegra, linux-actions, linux-rockchip, linux-samsung-soc
[-- Attachment #1: Type: text/plain, Size: 381 bytes --]
On Fri, Jan 20, 2023 at 09:57:21AM +0100, Krzysztof Kozlowski wrote:
> Cleanup by removing unneeded quotes from refs and redundant blank lines.
> No functional impact except adjusting to preferred coding style.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com> # for Renesas controllers
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] dt-bindings: mmc: drop unneeded quotes
2023-01-20 8:57 [PATCH 1/2] dt-bindings: mmc: drop unneeded quotes Krzysztof Kozlowski
2023-01-20 8:57 ` [PATCH 2/2] dt-bindings: mmc: correct pwrseq node names Krzysztof Kozlowski
2023-01-20 9:12 ` [PATCH 1/2] dt-bindings: mmc: drop unneeded quotes Wolfram Sang
@ 2023-01-20 9:30 ` Heiko Stübner
2023-01-20 9:31 ` Neil Armstrong
` (2 subsequent siblings)
5 siblings, 0 replies; 12+ messages in thread
From: Heiko Stübner @ 2023-01-20 9:30 UTC (permalink / raw)
To: Ulf Hansson, Rob Herring, Krzysztof Kozlowski, Chen-Yu Tsai,
Jernej Skrabec, Samuel Holland, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
Lars Povlsen, Steen Hegelund, Daniel Machon, UNGLinuxDriver,
Thierry Reding, Jonathan Hunter, Andreas Färber,
Manivannan Sadhasivam, Alim Akhtar, Tony Huang, Li-hao Kuo,
Maxime Ripard, Adrian Hunter, Masahiro Yamada, Markus Pargmann,
Wolfram Sang, Jaehoon Chung, linux-mmc, devicetree,
linux-arm-kernel, linux-sunxi, linux-kernel, linux-amlogic,
linux-tegra, linux-actions, linux-rockchip, linux-samsung-soc,
Krzysztof Kozlowski
Cc: Krzysztof Kozlowski
Am Freitag, 20. Januar 2023, 09:57:21 CET schrieb Krzysztof Kozlowski:
> Cleanup by removing unneeded quotes from refs and redundant blank lines.
> No functional impact except adjusting to preferred coding style.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> diff --git a/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml b/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml
> index c7e14b7dba9e..67d7223f74da 100644
> --- a/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml
> +++ b/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml
> @@ -14,7 +14,7 @@ description:
> file and the Rockchip specific extensions.
>
> allOf:
> - - $ref: "synopsys-dw-mshc-common.yaml#"
> + - $ref: synopsys-dw-mshc-common.yaml#
>
> maintainers:
> - Heiko Stuebner <heiko@sntech.de>
For the Rockchip part
Reviewed-by: Heiko Stuebner <heiko@sntech.de>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] dt-bindings: mmc: drop unneeded quotes
2023-01-20 8:57 [PATCH 1/2] dt-bindings: mmc: drop unneeded quotes Krzysztof Kozlowski
` (2 preceding siblings ...)
2023-01-20 9:30 ` Heiko Stübner
@ 2023-01-20 9:31 ` Neil Armstrong
2023-01-23 22:04 ` Rob Herring
2023-01-24 12:57 ` Ulf Hansson
5 siblings, 0 replies; 12+ messages in thread
From: Neil Armstrong @ 2023-01-20 9:31 UTC (permalink / raw)
To: Krzysztof Kozlowski, Ulf Hansson, Rob Herring,
Krzysztof Kozlowski, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Kevin Hilman, Jerome Brunet, Martin Blumenstingl,
Shawn Guo, Sascha Hauer, Pengutronix Kernel Team, Fabio Estevam,
NXP Linux Team, Lars Povlsen, Steen Hegelund, Daniel Machon,
UNGLinuxDriver, Thierry Reding, Jonathan Hunter,
Andreas Färber, Manivannan Sadhasivam, Heiko Stuebner,
Alim Akhtar, Tony Huang, Li-hao Kuo, Maxime Ripard,
Adrian Hunter, Masahiro Yamada, Markus Pargmann, Wolfram Sang,
Jaehoon Chung, linux-mmc, devicetree, linux-arm-kernel,
linux-sunxi, linux-kernel, linux-amlogic, linux-tegra,
linux-actions, linux-rockchip, linux-samsung-soc
On 20/01/2023 09:57, Krzysztof Kozlowski wrote:
> Cleanup by removing unneeded quotes from refs and redundant blank lines.
> No functional impact except adjusting to preferred coding style.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> .../bindings/mmc/allwinner,sun4i-a10-mmc.yaml | 2 +-
> .../bindings/mmc/amlogic,meson-mx-sdhc.yaml | 2 +-
For Amlogic part:
Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
> .../devicetree/bindings/mmc/arasan,sdhci.yaml | 6 ++--
> .../devicetree/bindings/mmc/cdns,sdhci.yaml | 22 ++++++------
> .../bindings/mmc/fsl-imx-esdhc.yaml | 4 +--
> .../devicetree/bindings/mmc/fsl-imx-mmc.yaml | 2 +-
> .../mmc/microchip,dw-sparx5-sdhci.yaml | 4 +--
> .../devicetree/bindings/mmc/mmc-spi-slot.yaml | 2 +-
> .../devicetree/bindings/mmc/mxs-mmc.yaml | 2 +-
> .../bindings/mmc/nvidia,tegra20-sdhci.yaml | 36 +++++++++----------
> .../devicetree/bindings/mmc/owl-mmc.yaml | 2 +-
> .../bindings/mmc/renesas,mmcif.yaml | 2 +-
> .../devicetree/bindings/mmc/renesas,sdhi.yaml | 6 ++--
> .../bindings/mmc/rockchip-dw-mshc.yaml | 2 +-
> .../bindings/mmc/samsung,exynos-dw-mshc.yaml | 2 +-
> .../devicetree/bindings/mmc/sunplus,mmc.yaml | 2 +-
> .../bindings/mmc/synopsys-dw-mshc-common.yaml | 2 +-
> 17 files changed, 50 insertions(+), 50 deletions(-)
>
<snip>
> diff --git a/Documentation/devicetree/bindings/mmc/amlogic,meson-mx-sdhc.yaml b/Documentation/devicetree/bindings/mmc/amlogic,meson-mx-sdhc.yaml
> index 1c391bec43dc..1a6cda82f296 100644
> --- a/Documentation/devicetree/bindings/mmc/amlogic,meson-mx-sdhc.yaml
> +++ b/Documentation/devicetree/bindings/mmc/amlogic,meson-mx-sdhc.yaml
> @@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
> title: Amlogic Meson SDHC controller
>
> allOf:
> - - $ref: "mmc-controller.yaml"
> + - $ref: mmc-controller.yaml
>
> maintainers:
> - Martin Blumenstingl <martin.blumenstingl@googlemail.com>
<snip>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] dt-bindings: mmc: correct pwrseq node names
2023-01-20 8:57 ` [PATCH 2/2] dt-bindings: mmc: correct pwrseq node names Krzysztof Kozlowski
@ 2023-01-20 9:38 ` Heiko Stübner
2023-01-24 8:17 ` Krzysztof Kozlowski
2023-01-23 22:04 ` Rob Herring
2023-01-24 12:57 ` Ulf Hansson
2 siblings, 1 reply; 12+ messages in thread
From: Heiko Stübner @ 2023-01-20 9:38 UTC (permalink / raw)
To: Ulf Hansson, Rob Herring, Krzysztof Kozlowski, Chen-Yu Tsai,
Jernej Skrabec, Samuel Holland, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
Lars Povlsen, Steen Hegelund, Daniel Machon, UNGLinuxDriver,
Thierry Reding, Jonathan Hunter, Andreas Färber,
Manivannan Sadhasivam, Alim Akhtar, Tony Huang, Li-hao Kuo,
Maxime Ripard, Adrian Hunter, Masahiro Yamada, Markus Pargmann,
Wolfram Sang, Jaehoon Chung, linux-mmc, devicetree,
linux-arm-kernel, linux-sunxi, linux-kernel, linux-amlogic,
linux-tegra, linux-actions, linux-rockchip, linux-samsung-soc,
Krzysztof Kozlowski
Cc: Krzysztof Kozlowski
Am Freitag, 20. Januar 2023, 09:57:22 CET schrieb Krzysztof Kozlowski:
> Node names should be generic and should not contain underscores.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml | 2 +-
> Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml | 2 +-
> Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml
> index 911a5996e099..588be73168fa 100644
> --- a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml
> +++ b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml
> @@ -41,7 +41,7 @@ additionalProperties: false
> examples:
> - |
> #include <dt-bindings/gpio/gpio.h>
> - sdhci0_pwrseq {
> + pwrseq {
[applicable for all 3 examples]
hmm, power-sequences are not necessarily tied to an address and I guess
it will be very much a common case to have multiple ones on a system.
So might it be better to follow other patterns (like leds) to number them
or suggest a "foo-pwrseq" / "pwrseq-foo"?
Heiko
> compatible = "mmc-pwrseq-emmc";
> reset-gpios = <&gpio1 12 GPIO_ACTIVE_LOW>;
> };
> diff --git a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml
> index 3397dbff88c2..b35e00e8c65e 100644
> --- a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml
> +++ b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml
> @@ -35,7 +35,7 @@ additionalProperties: false
> examples:
> - |
> #include <dt-bindings/gpio/gpio.h>
> - wifi_pwrseq: wifi_pwrseq {
> + pwrseq {
> compatible = "mmc-pwrseq-sd8787";
> powerdown-gpios = <&twl_gpio 0 GPIO_ACTIVE_LOW>;
> reset-gpios = <&twl_gpio 1 GPIO_ACTIVE_LOW>;
> diff --git a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
> index 64e3644eefeb..00feaafc1063 100644
> --- a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
> +++ b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
> @@ -55,7 +55,7 @@ additionalProperties: false
> examples:
> - |
> #include <dt-bindings/gpio/gpio.h>
> - sdhci0_pwrseq {
> + pwrseq {
> compatible = "mmc-pwrseq-simple";
> reset-gpios = <&gpio1 12 GPIO_ACTIVE_LOW>;
> clocks = <&clk_32768_ck>;
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] dt-bindings: mmc: drop unneeded quotes
2023-01-20 8:57 [PATCH 1/2] dt-bindings: mmc: drop unneeded quotes Krzysztof Kozlowski
` (3 preceding siblings ...)
2023-01-20 9:31 ` Neil Armstrong
@ 2023-01-23 22:04 ` Rob Herring
2023-01-24 12:57 ` Ulf Hansson
5 siblings, 0 replies; 12+ messages in thread
From: Rob Herring @ 2023-01-23 22:04 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Steen Hegelund, Jonathan Hunter, Daniel Machon, Masahiro Yamada,
linux-arm-kernel, Adrian Hunter, ,
Martin Blumenstingl, Neil Armstrong, Krzysztof Kozlowski,
Samuel Holland, Thierry Reding, Shawn Guo, Markus Pargmann,
UNGLinuxDriver, linux-tegra, NXP Linux Team,
Manivannan Sadhasivam, Wolfram Sang, Jaehoon Chung, Lars Povlsen,
Jernej Skrabec, linux-rockchip@lists.infradead.org ,
Tony Huang, Kevin Hilman, Jerome Brunet, devicetree, Rob Herring,
Fabio Estevam, Pengutronix Kernel Team, linux-actions,
Maxime Ripard, linux-sunxi, linux-kernel, linux-mmc, Li-hao Kuo,
linux-amlogic, Ulf Hansson, linux-samsung-soc, Sascha Hauer,
Chen-Yu Tsai, Andreas Färber, Alim Akhtar, Heiko Stuebner
On Fri, 20 Jan 2023 09:57:21 +0100, Krzysztof Kozlowski wrote:
> Cleanup by removing unneeded quotes from refs and redundant blank lines.
> No functional impact except adjusting to preferred coding style.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> .../bindings/mmc/allwinner,sun4i-a10-mmc.yaml | 2 +-
> .../bindings/mmc/amlogic,meson-mx-sdhc.yaml | 2 +-
> .../devicetree/bindings/mmc/arasan,sdhci.yaml | 6 ++--
> .../devicetree/bindings/mmc/cdns,sdhci.yaml | 22 ++++++------
> .../bindings/mmc/fsl-imx-esdhc.yaml | 4 +--
> .../devicetree/bindings/mmc/fsl-imx-mmc.yaml | 2 +-
> .../mmc/microchip,dw-sparx5-sdhci.yaml | 4 +--
> .../devicetree/bindings/mmc/mmc-spi-slot.yaml | 2 +-
> .../devicetree/bindings/mmc/mxs-mmc.yaml | 2 +-
> .../bindings/mmc/nvidia,tegra20-sdhci.yaml | 36 +++++++++----------
> .../devicetree/bindings/mmc/owl-mmc.yaml | 2 +-
> .../bindings/mmc/renesas,mmcif.yaml | 2 +-
> .../devicetree/bindings/mmc/renesas,sdhi.yaml | 6 ++--
> .../bindings/mmc/rockchip-dw-mshc.yaml | 2 +-
> .../bindings/mmc/samsung,exynos-dw-mshc.yaml | 2 +-
> .../devicetree/bindings/mmc/sunplus,mmc.yaml | 2 +-
> .../bindings/mmc/synopsys-dw-mshc-common.yaml | 2 +-
> 17 files changed, 50 insertions(+), 50 deletions(-)
>
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] dt-bindings: mmc: correct pwrseq node names
2023-01-20 8:57 ` [PATCH 2/2] dt-bindings: mmc: correct pwrseq node names Krzysztof Kozlowski
2023-01-20 9:38 ` Heiko Stübner
@ 2023-01-23 22:04 ` Rob Herring
2023-01-24 12:57 ` Ulf Hansson
2 siblings, 0 replies; 12+ messages in thread
From: Rob Herring @ 2023-01-23 22:04 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Kevin Hilman, Krzysztof Kozlowski, Pengutronix Kernel Team,
linux-rockchip, linux-tegra, Neil Armstrong, Andreas Färber,
Tony Huang, Maxime Ripard, Jernej Skrabec, Jonathan Hunter,
Thierry Reding, Shawn Guo, linux-arm-kernel, linux-sunxi,
Adrian Hunter, Alim Akhtar, linux-samsung-soc, Li-hao Kuo,
Sascha Hauer, Fabio Estevam, Martin Blumenstingl, Jerome Brunet,
Manivannan Sadhasivam, ,
Steen Hegelund, Ulf Hansson, Lars Povlsen, Daniel Machon,
Jaehoon Chung, Wolfram Sang, devicetree, Chen-Yu Tsai,
Rob Herring, Heiko Stuebner, linux-actions, Samuel Holland,
linux-amlogic, Masahiro Yamada, Markus Pargmann, linux-kernel,
NXP Linux Team
On Fri, 20 Jan 2023 09:57:22 +0100, Krzysztof Kozlowski wrote:
> Node names should be generic and should not contain underscores.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml | 2 +-
> Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml | 2 +-
> Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
>
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] dt-bindings: mmc: correct pwrseq node names
2023-01-20 9:38 ` Heiko Stübner
@ 2023-01-24 8:17 ` Krzysztof Kozlowski
2023-01-24 11:41 ` Heiko Stübner
0 siblings, 1 reply; 12+ messages in thread
From: Krzysztof Kozlowski @ 2023-01-24 8:17 UTC (permalink / raw)
To: Heiko Stübner, Ulf Hansson, Rob Herring,
Krzysztof Kozlowski, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Neil Armstrong, Kevin Hilman, Jerome Brunet,
Martin Blumenstingl, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
Lars Povlsen, Steen Hegelund, Daniel Machon, UNGLinuxDriver,
Thierry Reding, Jonathan Hunter, Andreas Färber,
Manivannan Sadhasivam, Alim Akhtar, Tony Huang, Li-hao Kuo,
Maxime Ripard, Adrian Hunter, Masahiro Yamada, Markus Pargmann,
Wolfram Sang, Jaehoon Chung, linux-mmc, devicetree,
linux-arm-kernel, linux-sunxi, linux-kernel, linux-amlogic,
linux-tegra, linux-actions, linux-rockchip, linux-samsung-soc
On 20/01/2023 10:38, Heiko Stübner wrote:
> Am Freitag, 20. Januar 2023, 09:57:22 CET schrieb Krzysztof Kozlowski:
>> Node names should be generic and should not contain underscores.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>> ---
>> Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml | 2 +-
>> Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml | 2 +-
>> Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml | 2 +-
>> 3 files changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml
>> index 911a5996e099..588be73168fa 100644
>> --- a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml
>> +++ b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml
>> @@ -41,7 +41,7 @@ additionalProperties: false
>> examples:
>> - |
>> #include <dt-bindings/gpio/gpio.h>
>> - sdhci0_pwrseq {
>> + pwrseq {
>
> [applicable for all 3 examples]
>
> hmm, power-sequences are not necessarily tied to an address and I guess
> it will be very much a common case to have multiple ones on a system.
>
> So might it be better to follow other patterns (like leds) to number them
> or suggest a "foo-pwrseq" / "pwrseq-foo"?
In such cases one can add number or descriptive suffix (pwrseq-0,
pwrseq-foo), just like we do for regulators. However the examples have
here only one node and in such case "pwrseq" is enough.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] dt-bindings: mmc: correct pwrseq node names
2023-01-24 8:17 ` Krzysztof Kozlowski
@ 2023-01-24 11:41 ` Heiko Stübner
0 siblings, 0 replies; 12+ messages in thread
From: Heiko Stübner @ 2023-01-24 11:41 UTC (permalink / raw)
To: Ulf Hansson, Rob Herring, Krzysztof Kozlowski, Chen-Yu Tsai,
Jernej Skrabec, Samuel Holland, Neil Armstrong, Kevin Hilman,
Jerome Brunet, Martin Blumenstingl, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
Lars Povlsen, Steen Hegelund, Daniel Machon, UNGLinuxDriver,
Thierry Reding, Jonathan Hunter, Andreas Färber,
Manivannan Sadhasivam, Alim Akhtar, Tony Huang, Li-hao Kuo,
Maxime Ripard, Adrian Hunter, Masahiro Yamada, Markus Pargmann,
Wolfram Sang, Jaehoon Chung, linux-mmc, devicetree,
linux-arm-kernel, linux-sunxi, linux-kernel, linux-amlogic,
linux-tegra, linux-actions, linux-rockchip, linux-samsung-soc,
Krzysztof Kozlowski
Am Dienstag, 24. Januar 2023, 09:17:12 CET schrieb Krzysztof Kozlowski:
> On 20/01/2023 10:38, Heiko Stübner wrote:
> > Am Freitag, 20. Januar 2023, 09:57:22 CET schrieb Krzysztof Kozlowski:
> >> Node names should be generic and should not contain underscores.
> >>
> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> >> ---
> >> Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml | 2 +-
> >> Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml | 2 +-
> >> Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml | 2 +-
> >> 3 files changed, 3 insertions(+), 3 deletions(-)
> >>
> >> diff --git a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml
> >> index 911a5996e099..588be73168fa 100644
> >> --- a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml
> >> +++ b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml
> >> @@ -41,7 +41,7 @@ additionalProperties: false
> >> examples:
> >> - |
> >> #include <dt-bindings/gpio/gpio.h>
> >> - sdhci0_pwrseq {
> >> + pwrseq {
> >
> > [applicable for all 3 examples]
> >
> > hmm, power-sequences are not necessarily tied to an address and I guess
> > it will be very much a common case to have multiple ones on a system.
> >
> > So might it be better to follow other patterns (like leds) to number them
> > or suggest a "foo-pwrseq" / "pwrseq-foo"?
>
> In such cases one can add number or descriptive suffix (pwrseq-0,
> pwrseq-foo), just like we do for regulators. However the examples have
> here only one node and in such case "pwrseq" is enough.
ok, when that is already part of the schema then all is fine :-)
Reviewed-by: Heiko Stuebner <heiko@sntech.de>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] dt-bindings: mmc: drop unneeded quotes
2023-01-20 8:57 [PATCH 1/2] dt-bindings: mmc: drop unneeded quotes Krzysztof Kozlowski
` (4 preceding siblings ...)
2023-01-23 22:04 ` Rob Herring
@ 2023-01-24 12:57 ` Ulf Hansson
5 siblings, 0 replies; 12+ messages in thread
From: Ulf Hansson @ 2023-01-24 12:57 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, Krzysztof Kozlowski, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Neil Armstrong, Kevin Hilman, Jerome Brunet,
Martin Blumenstingl, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
Lars Povlsen, Steen Hegelund, Daniel Machon, UNGLinuxDriver,
Thierry Reding, Jonathan Hunter, Andreas Färber,
Manivannan Sadhasivam, Heiko Stuebner, Alim Akhtar, Tony Huang,
Li-hao Kuo, Maxime Ripard, Adrian Hunter, Masahiro Yamada,
Markus Pargmann, Wolfram Sang, Jaehoon Chung, linux-mmc,
devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-amlogic, linux-tegra, linux-actions, linux-rockchip,
linux-samsung-soc
On Fri, 20 Jan 2023 at 09:57, Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> Cleanup by removing unneeded quotes from refs and redundant blank lines.
> No functional impact except adjusting to preferred coding style.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Applied for next, thanks!
Kind regards
Uffe
> ---
> .../bindings/mmc/allwinner,sun4i-a10-mmc.yaml | 2 +-
> .../bindings/mmc/amlogic,meson-mx-sdhc.yaml | 2 +-
> .../devicetree/bindings/mmc/arasan,sdhci.yaml | 6 ++--
> .../devicetree/bindings/mmc/cdns,sdhci.yaml | 22 ++++++------
> .../bindings/mmc/fsl-imx-esdhc.yaml | 4 +--
> .../devicetree/bindings/mmc/fsl-imx-mmc.yaml | 2 +-
> .../mmc/microchip,dw-sparx5-sdhci.yaml | 4 +--
> .../devicetree/bindings/mmc/mmc-spi-slot.yaml | 2 +-
> .../devicetree/bindings/mmc/mxs-mmc.yaml | 2 +-
> .../bindings/mmc/nvidia,tegra20-sdhci.yaml | 36 +++++++++----------
> .../devicetree/bindings/mmc/owl-mmc.yaml | 2 +-
> .../bindings/mmc/renesas,mmcif.yaml | 2 +-
> .../devicetree/bindings/mmc/renesas,sdhi.yaml | 6 ++--
> .../bindings/mmc/rockchip-dw-mshc.yaml | 2 +-
> .../bindings/mmc/samsung,exynos-dw-mshc.yaml | 2 +-
> .../devicetree/bindings/mmc/sunplus,mmc.yaml | 2 +-
> .../bindings/mmc/synopsys-dw-mshc-common.yaml | 2 +-
> 17 files changed, 50 insertions(+), 50 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mmc/allwinner,sun4i-a10-mmc.yaml b/Documentation/devicetree/bindings/mmc/allwinner,sun4i-a10-mmc.yaml
> index 02ecc93417ef..0ccd632d5620 100644
> --- a/Documentation/devicetree/bindings/mmc/allwinner,sun4i-a10-mmc.yaml
> +++ b/Documentation/devicetree/bindings/mmc/allwinner,sun4i-a10-mmc.yaml
> @@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
> title: Allwinner A10 MMC Controller
>
> allOf:
> - - $ref: "mmc-controller.yaml"
> + - $ref: mmc-controller.yaml
>
> maintainers:
> - Chen-Yu Tsai <wens@csie.org>
> diff --git a/Documentation/devicetree/bindings/mmc/amlogic,meson-mx-sdhc.yaml b/Documentation/devicetree/bindings/mmc/amlogic,meson-mx-sdhc.yaml
> index 1c391bec43dc..1a6cda82f296 100644
> --- a/Documentation/devicetree/bindings/mmc/amlogic,meson-mx-sdhc.yaml
> +++ b/Documentation/devicetree/bindings/mmc/amlogic,meson-mx-sdhc.yaml
> @@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
> title: Amlogic Meson SDHC controller
>
> allOf:
> - - $ref: "mmc-controller.yaml"
> + - $ref: mmc-controller.yaml
>
> maintainers:
> - Martin Blumenstingl <martin.blumenstingl@googlemail.com>
> diff --git a/Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml b/Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml
> index 4053de758db6..8296c34cfa00 100644
> --- a/Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml
> +++ b/Documentation/devicetree/bindings/mmc/arasan,sdhci.yaml
> @@ -1,8 +1,8 @@
> # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> %YAML 1.2
> ---
> -$id: "http://devicetree.org/schemas/mmc/arasan,sdhci.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/mmc/arasan,sdhci.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Arasan SDHCI Controller
>
> @@ -10,7 +10,7 @@ maintainers:
> - Adrian Hunter <adrian.hunter@intel.com>
>
> allOf:
> - - $ref: "mmc-controller.yaml#"
> + - $ref: mmc-controller.yaml#
> - if:
> properties:
> compatible:
> diff --git a/Documentation/devicetree/bindings/mmc/cdns,sdhci.yaml b/Documentation/devicetree/bindings/mmc/cdns,sdhci.yaml
> index 8b1a0fdcb5e3..d3dce4d6c168 100644
> --- a/Documentation/devicetree/bindings/mmc/cdns,sdhci.yaml
> +++ b/Documentation/devicetree/bindings/mmc/cdns,sdhci.yaml
> @@ -36,43 +36,43 @@ properties:
>
> cdns,phy-input-delay-sd-highspeed:
> description: Value of the delay in the input path for SD high-speed timing
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
> minimum: 0
> maximum: 0x1f
>
> cdns,phy-input-delay-legacy:
> description: Value of the delay in the input path for legacy timing
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
> minimum: 0
> maximum: 0x1f
>
> cdns,phy-input-delay-sd-uhs-sdr12:
> description: Value of the delay in the input path for SD UHS SDR12 timing
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
> minimum: 0
> maximum: 0x1f
>
> cdns,phy-input-delay-sd-uhs-sdr25:
> description: Value of the delay in the input path for SD UHS SDR25 timing
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
> minimum: 0
> maximum: 0x1f
>
> cdns,phy-input-delay-sd-uhs-sdr50:
> description: Value of the delay in the input path for SD UHS SDR50 timing
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
> minimum: 0
> maximum: 0x1f
>
> cdns,phy-input-delay-sd-uhs-ddr50:
> description: Value of the delay in the input path for SD UHS DDR50 timing
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
> minimum: 0
> maximum: 0x1f
>
> cdns,phy-input-delay-mmc-highspeed:
> description: Value of the delay in the input path for MMC high-speed timing
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
> minimum: 0
> maximum: 0x1f
>
> @@ -83,7 +83,7 @@ properties:
> # Each delay property represents the fraction of the clock period.
> # The approximate delay value will be
> # (<delay property value>/128)*sdmclk_clock_period.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
> minimum: 0
> maximum: 0x1f
>
> @@ -91,7 +91,7 @@ properties:
> description: |
> Value of the delay introduced on the sdclk output for all modes except
> HS200, HS400 and HS400_ES.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
> minimum: 0
> maximum: 0x7f
>
> @@ -99,7 +99,7 @@ properties:
> description: |
> Value of the delay introduced on the sdclk output for HS200, HS400 and
> HS400_ES speed modes.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
> minimum: 0
> maximum: 0x7f
>
> @@ -107,7 +107,7 @@ properties:
> description: |
> Value of the delay introduced on the dat_strobe input used in
> HS400 / HS400_ES speed modes.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
> minimum: 0
> maximum: 0x7f
>
> diff --git a/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml b/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
> index b3289593d5f1..269e0f421407 100644
> --- a/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
> +++ b/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.yaml
> @@ -10,7 +10,7 @@ maintainers:
> - Shawn Guo <shawnguo@kernel.org>
>
> allOf:
> - - $ref: "mmc-controller.yaml"
> + - $ref: mmc-controller.yaml
>
> description: |
> The Enhanced Secure Digital Host Controller on Freescale i.MX family
> @@ -112,7 +112,7 @@ properties:
> default: 0
>
> voltage-ranges:
> - $ref: '/schemas/types.yaml#/definitions/uint32-matrix'
> + $ref: /schemas/types.yaml#/definitions/uint32-matrix
> description: |
> Specify the voltage range in case there are software transparent level
> shifters on the outputs of the controller. Two cells are required, first
> diff --git a/Documentation/devicetree/bindings/mmc/fsl-imx-mmc.yaml b/Documentation/devicetree/bindings/mmc/fsl-imx-mmc.yaml
> index ffa162722b8e..221f5bc047bd 100644
> --- a/Documentation/devicetree/bindings/mmc/fsl-imx-mmc.yaml
> +++ b/Documentation/devicetree/bindings/mmc/fsl-imx-mmc.yaml
> @@ -10,7 +10,7 @@ maintainers:
> - Markus Pargmann <mpa@pengutronix.de>
>
> allOf:
> - - $ref: "mmc-controller.yaml"
> + - $ref: mmc-controller.yaml
>
> properties:
> compatible:
> diff --git a/Documentation/devicetree/bindings/mmc/microchip,dw-sparx5-sdhci.yaml b/Documentation/devicetree/bindings/mmc/microchip,dw-sparx5-sdhci.yaml
> index fa6cfe092fc9..1f63faf17743 100644
> --- a/Documentation/devicetree/bindings/mmc/microchip,dw-sparx5-sdhci.yaml
> +++ b/Documentation/devicetree/bindings/mmc/microchip,dw-sparx5-sdhci.yaml
> @@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
> title: Microchip Sparx5 Mobile Storage Host Controller
>
> allOf:
> - - $ref: "mmc-controller.yaml"
> + - $ref: mmc-controller.yaml
>
> maintainers:
> - Lars Povlsen <lars.povlsen@microchip.com>
> @@ -35,7 +35,7 @@ properties:
> microchip,clock-delay:
> description: Delay clock to card to meet setup time requirements.
> Each step increase by 1.25ns.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
> minimum: 1
> maximum: 15
>
> diff --git a/Documentation/devicetree/bindings/mmc/mmc-spi-slot.yaml b/Documentation/devicetree/bindings/mmc/mmc-spi-slot.yaml
> index c0662ce9946d..36acc40c7d18 100644
> --- a/Documentation/devicetree/bindings/mmc/mmc-spi-slot.yaml
> +++ b/Documentation/devicetree/bindings/mmc/mmc-spi-slot.yaml
> @@ -10,7 +10,7 @@ maintainers:
> - Ulf Hansson <ulf.hansson@linaro.org>
>
> allOf:
> - - $ref: "mmc-controller.yaml"
> + - $ref: mmc-controller.yaml
> - $ref: /schemas/spi/spi-peripheral-props.yaml
>
> description: |
> diff --git a/Documentation/devicetree/bindings/mmc/mxs-mmc.yaml b/Documentation/devicetree/bindings/mmc/mxs-mmc.yaml
> index bec8f8c71ff2..32e512a68ed6 100644
> --- a/Documentation/devicetree/bindings/mmc/mxs-mmc.yaml
> +++ b/Documentation/devicetree/bindings/mmc/mxs-mmc.yaml
> @@ -17,7 +17,7 @@ description: |
> and the properties used by the mxsmmc driver.
>
> allOf:
> - - $ref: "mmc-controller.yaml"
> + - $ref: mmc-controller.yaml
>
> properties:
> compatible:
> diff --git a/Documentation/devicetree/bindings/mmc/nvidia,tegra20-sdhci.yaml b/Documentation/devicetree/bindings/mmc/nvidia,tegra20-sdhci.yaml
> index fe0270207622..285057523ead 100644
> --- a/Documentation/devicetree/bindings/mmc/nvidia,tegra20-sdhci.yaml
> +++ b/Documentation/devicetree/bindings/mmc/nvidia,tegra20-sdhci.yaml
> @@ -83,7 +83,7 @@ properties:
> maxItems: 1
>
> operating-points-v2:
> - $ref: "/schemas/types.yaml#/definitions/phandle"
> + $ref: /schemas/types.yaml#/definitions/phandle
>
> power-domains:
> items:
> @@ -100,53 +100,53 @@ properties:
>
> The DQS trim values are only used on controllers which support HS400
> timing. Only SDMMC4 on Tegra210 and Tegra186 supports HS400.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
>
> nvidia,default-trim:
> description: Specify the default outbound clock trimmer value.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
>
> nvidia,dqs-trim:
> description: Specify DQS trim value for HS400 timing.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
>
> nvidia,pad-autocal-pull-down-offset-1v8:
> description: Specify drive strength calibration offsets for 1.8 V
> signaling modes.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
>
> nvidia,pad-autocal-pull-down-offset-1v8-timeout:
> description: Specify drive strength used as a fallback in case the
> automatic calibration times out on a 1.8 V signaling mode.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
>
> nvidia,pad-autocal-pull-down-offset-3v3:
> description: Specify drive strength calibration offsets for 3.3 V
> signaling modes.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
>
> nvidia,pad-autocal-pull-down-offset-3v3-timeout:
> description: Specify drive strength used as a fallback in case the
> automatic calibration times out on a 3.3 V signaling mode.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
>
> nvidia,pad-autocal-pull-down-offset-sdr104:
> description: Specify drive strength calibration offsets for SDR104 mode.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
>
> nvidia,pad-autocal-pull-down-offset-hs400:
> description: Specify drive strength calibration offsets for HS400 mode.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
>
> nvidia,pad-autocal-pull-up-offset-1v8:
> description: Specify drive strength calibration offsets for 1.8 V
> signaling modes.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
>
> nvidia,pad-autocal-pull-up-offset-1v8-timeout:
> description: Specify drive strength used as a fallback in case the
> automatic calibration times out on a 1.8 V signaling mode.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
>
> nvidia,pad-autocal-pull-up-offset-3v3:
> description: Specify drive strength calibration offsets for 3.3 V
> @@ -158,25 +158,25 @@ properties:
> refer to the reference manual of the SoC for correct values. The SDR104
> and HS400 timing specific values are used in corresponding modes if
> specified.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
>
> nvidia,pad-autocal-pull-up-offset-3v3-timeout:
> description: Specify drive strength used as a fallback in case the
> automatic calibration times out on a 3.3 V signaling mode.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
>
> nvidia,pad-autocal-pull-up-offset-sdr104:
> description: Specify drive strength calibration offsets for SDR104 mode.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
>
> nvidia,pad-autocal-pull-up-offset-hs400:
> description: Specify drive strength calibration offsets for HS400 mode.
> - $ref: "/schemas/types.yaml#/definitions/uint32"
> + $ref: /schemas/types.yaml#/definitions/uint32
>
> nvidia,only-1-8v:
> description: The presence of this property indicates that the controller
> operates at a 1.8 V fixed I/O voltage.
> - $ref: "/schemas/types.yaml#/definitions/flag"
> + $ref: /schemas/types.yaml#/definitions/flag
>
> required:
> - compatible
> @@ -187,7 +187,7 @@ required:
> - reset-names
>
> allOf:
> - - $ref: "mmc-controller.yaml"
> + - $ref: mmc-controller.yaml
> - if:
> properties:
> compatible:
> diff --git a/Documentation/devicetree/bindings/mmc/owl-mmc.yaml b/Documentation/devicetree/bindings/mmc/owl-mmc.yaml
> index b0d81ebe0f6e..1b7d88ed3799 100644
> --- a/Documentation/devicetree/bindings/mmc/owl-mmc.yaml
> +++ b/Documentation/devicetree/bindings/mmc/owl-mmc.yaml
> @@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
> title: Actions Semi Owl SoCs SD/MMC/SDIO controller
>
> allOf:
> - - $ref: "mmc-controller.yaml"
> + - $ref: mmc-controller.yaml
>
> maintainers:
> - Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> diff --git a/Documentation/devicetree/bindings/mmc/renesas,mmcif.yaml b/Documentation/devicetree/bindings/mmc/renesas,mmcif.yaml
> index c36ba561c387..024313b79ec9 100644
> --- a/Documentation/devicetree/bindings/mmc/renesas,mmcif.yaml
> +++ b/Documentation/devicetree/bindings/mmc/renesas,mmcif.yaml
> @@ -10,7 +10,7 @@ maintainers:
> - Wolfram Sang <wsa+renesas@sang-engineering.com>
>
> allOf:
> - - $ref: "mmc-controller.yaml"
> + - $ref: mmc-controller.yaml
>
> properties:
> compatible:
> diff --git a/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml b/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml
> index fa089cbb155b..7756a8687eaf 100644
> --- a/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml
> +++ b/Documentation/devicetree/bindings/mmc/renesas,sdhi.yaml
> @@ -1,8 +1,8 @@
> # SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> %YAML 1.2
> ---
> -$id: "http://devicetree.org/schemas/mmc/renesas,sdhi.yaml#"
> -$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +$id: http://devicetree.org/schemas/mmc/renesas,sdhi.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>
> title: Renesas SDHI SD/MMC controller
>
> @@ -112,7 +112,7 @@ properties:
> max-frequency: true
>
> allOf:
> - - $ref: "mmc-controller.yaml"
> + - $ref: mmc-controller.yaml
>
> - if:
> properties:
> diff --git a/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml b/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml
> index c7e14b7dba9e..67d7223f74da 100644
> --- a/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml
> +++ b/Documentation/devicetree/bindings/mmc/rockchip-dw-mshc.yaml
> @@ -14,7 +14,7 @@ description:
> file and the Rockchip specific extensions.
>
> allOf:
> - - $ref: "synopsys-dw-mshc-common.yaml#"
> + - $ref: synopsys-dw-mshc-common.yaml#
>
> maintainers:
> - Heiko Stuebner <heiko@sntech.de>
> diff --git a/Documentation/devicetree/bindings/mmc/samsung,exynos-dw-mshc.yaml b/Documentation/devicetree/bindings/mmc/samsung,exynos-dw-mshc.yaml
> index fdaa18481aa0..6ee78a38bd74 100644
> --- a/Documentation/devicetree/bindings/mmc/samsung,exynos-dw-mshc.yaml
> +++ b/Documentation/devicetree/bindings/mmc/samsung,exynos-dw-mshc.yaml
> @@ -112,7 +112,7 @@ required:
> - samsung,dw-mshc-sdr-timing
>
> allOf:
> - - $ref: "synopsys-dw-mshc-common.yaml#"
> + - $ref: synopsys-dw-mshc-common.yaml#
> - if:
> properties:
> compatible:
> diff --git a/Documentation/devicetree/bindings/mmc/sunplus,mmc.yaml b/Documentation/devicetree/bindings/mmc/sunplus,mmc.yaml
> index 23aa8e6b2d70..611687166735 100644
> --- a/Documentation/devicetree/bindings/mmc/sunplus,mmc.yaml
> +++ b/Documentation/devicetree/bindings/mmc/sunplus,mmc.yaml
> @@ -12,7 +12,7 @@ maintainers:
> - Li-hao Kuo <lhjeff911@gmail.com>
>
> allOf:
> - - $ref: "mmc-controller.yaml"
> + - $ref: mmc-controller.yaml
>
> properties:
> compatible:
> diff --git a/Documentation/devicetree/bindings/mmc/synopsys-dw-mshc-common.yaml b/Documentation/devicetree/bindings/mmc/synopsys-dw-mshc-common.yaml
> index 8dfad89c78a7..6f11b2adf103 100644
> --- a/Documentation/devicetree/bindings/mmc/synopsys-dw-mshc-common.yaml
> +++ b/Documentation/devicetree/bindings/mmc/synopsys-dw-mshc-common.yaml
> @@ -7,7 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml#
> title: Synopsys Designware Mobile Storage Host Controller Common Properties
>
> allOf:
> - - $ref: "mmc-controller.yaml#"
> + - $ref: mmc-controller.yaml#
>
> maintainers:
> - Ulf Hansson <ulf.hansson@linaro.org>
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] dt-bindings: mmc: correct pwrseq node names
2023-01-20 8:57 ` [PATCH 2/2] dt-bindings: mmc: correct pwrseq node names Krzysztof Kozlowski
2023-01-20 9:38 ` Heiko Stübner
2023-01-23 22:04 ` Rob Herring
@ 2023-01-24 12:57 ` Ulf Hansson
2 siblings, 0 replies; 12+ messages in thread
From: Ulf Hansson @ 2023-01-24 12:57 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, Krzysztof Kozlowski, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland, Neil Armstrong, Kevin Hilman, Jerome Brunet,
Martin Blumenstingl, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team,
Lars Povlsen, Steen Hegelund, Daniel Machon, UNGLinuxDriver,
Thierry Reding, Jonathan Hunter, Andreas Färber,
Manivannan Sadhasivam, Heiko Stuebner, Alim Akhtar, Tony Huang,
Li-hao Kuo, Maxime Ripard, Adrian Hunter, Masahiro Yamada,
Markus Pargmann, Wolfram Sang, Jaehoon Chung, linux-mmc,
devicetree, linux-arm-kernel, linux-sunxi, linux-kernel,
linux-amlogic, linux-tegra, linux-actions, linux-rockchip,
linux-samsung-soc
On Fri, 20 Jan 2023 at 09:57, Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> Node names should be generic and should not contain underscores.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Applied for next, thanks!
Kind regards
Uffe
> ---
> Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml | 2 +-
> Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml | 2 +-
> Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml
> index 911a5996e099..588be73168fa 100644
> --- a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml
> +++ b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-emmc.yaml
> @@ -41,7 +41,7 @@ additionalProperties: false
> examples:
> - |
> #include <dt-bindings/gpio/gpio.h>
> - sdhci0_pwrseq {
> + pwrseq {
> compatible = "mmc-pwrseq-emmc";
> reset-gpios = <&gpio1 12 GPIO_ACTIVE_LOW>;
> };
> diff --git a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml
> index 3397dbff88c2..b35e00e8c65e 100644
> --- a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml
> +++ b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-sd8787.yaml
> @@ -35,7 +35,7 @@ additionalProperties: false
> examples:
> - |
> #include <dt-bindings/gpio/gpio.h>
> - wifi_pwrseq: wifi_pwrseq {
> + pwrseq {
> compatible = "mmc-pwrseq-sd8787";
> powerdown-gpios = <&twl_gpio 0 GPIO_ACTIVE_LOW>;
> reset-gpios = <&twl_gpio 1 GPIO_ACTIVE_LOW>;
> diff --git a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
> index 64e3644eefeb..00feaafc1063 100644
> --- a/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
> +++ b/Documentation/devicetree/bindings/mmc/mmc-pwrseq-simple.yaml
> @@ -55,7 +55,7 @@ additionalProperties: false
> examples:
> - |
> #include <dt-bindings/gpio/gpio.h>
> - sdhci0_pwrseq {
> + pwrseq {
> compatible = "mmc-pwrseq-simple";
> reset-gpios = <&gpio1 12 GPIO_ACTIVE_LOW>;
> clocks = <&clk_32768_ck>;
> --
> 2.34.1
>
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2023-01-24 12:57 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-20 8:57 [PATCH 1/2] dt-bindings: mmc: drop unneeded quotes Krzysztof Kozlowski
2023-01-20 8:57 ` [PATCH 2/2] dt-bindings: mmc: correct pwrseq node names Krzysztof Kozlowski
2023-01-20 9:38 ` Heiko Stübner
2023-01-24 8:17 ` Krzysztof Kozlowski
2023-01-24 11:41 ` Heiko Stübner
2023-01-23 22:04 ` Rob Herring
2023-01-24 12:57 ` Ulf Hansson
2023-01-20 9:12 ` [PATCH 1/2] dt-bindings: mmc: drop unneeded quotes Wolfram Sang
2023-01-20 9:30 ` Heiko Stübner
2023-01-20 9:31 ` Neil Armstrong
2023-01-23 22:04 ` Rob Herring
2023-01-24 12:57 ` Ulf Hansson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).