devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] spi: dt-bindings: fsl,dspi: Fix example indentation
@ 2025-05-09 11:21 Krzysztof Kozlowski
  2025-05-09 11:21 ` [PATCH 2/2] spi: dt-bindings: nuvoton,wpcm450-fiu: Drop unrelated nodes from DTS example Krzysztof Kozlowski
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2025-05-09 11:21 UTC (permalink / raw)
  To: Vladimir Oltean, Mark Brown, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Jonathan Neuschäfer, Frank Li, linux-spi, imx,
	devicetree, linux-kernel, openbmc
  Cc: Krzysztof Kozlowski

DTS example in the bindings should be indented with 2- or 4-spaces, so
correct a mixture of different styles to keep consistent 4-spaces.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

---

No functional changes here, but saves some comments during reviews of
new patches built on existing code.
---
 .../devicetree/bindings/spi/fsl,dspi.yaml          | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/Documentation/devicetree/bindings/spi/fsl,dspi.yaml b/Documentation/devicetree/bindings/spi/fsl,dspi.yaml
index 7ca8fceda717..bf9cce53c48d 100644
--- a/Documentation/devicetree/bindings/spi/fsl,dspi.yaml
+++ b/Documentation/devicetree/bindings/spi/fsl,dspi.yaml
@@ -105,12 +105,12 @@ examples:
         big-endian;
 
         flash@0 {
-                compatible = "jedec,spi-nor";
-                reg = <0>;
-                spi-max-frequency = <16000000>;
-                spi-cpol;
-                spi-cpha;
-                spi-cs-setup-delay-ns = <100>;
-                spi-cs-hold-delay-ns = <50>;
+            compatible = "jedec,spi-nor";
+            reg = <0>;
+            spi-max-frequency = <16000000>;
+            spi-cpol;
+            spi-cpha;
+            spi-cs-setup-delay-ns = <100>;
+            spi-cs-hold-delay-ns = <50>;
         };
     };
-- 
2.45.2


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH 2/2] spi: dt-bindings: nuvoton,wpcm450-fiu: Drop unrelated nodes from DTS example
  2025-05-09 11:21 [PATCH 1/2] spi: dt-bindings: fsl,dspi: Fix example indentation Krzysztof Kozlowski
@ 2025-05-09 11:21 ` Krzysztof Kozlowski
  2025-05-09 15:50 ` [PATCH 1/2] spi: dt-bindings: fsl,dspi: Fix example indentation Conor Dooley
  2025-05-11  1:54 ` Mark Brown
  2 siblings, 0 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2025-05-09 11:21 UTC (permalink / raw)
  To: Vladimir Oltean, Mark Brown, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Jonathan Neuschäfer, Frank Li, linux-spi, imx,
	devicetree, linux-kernel, openbmc
  Cc: Krzysztof Kozlowski

Binding example should not contain other nodes, including other
providers like syscon, because this is redundant and only adds
unnecessary bloat.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 .../devicetree/bindings/spi/nuvoton,wpcm450-fiu.yaml         | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/Documentation/devicetree/bindings/spi/nuvoton,wpcm450-fiu.yaml b/Documentation/devicetree/bindings/spi/nuvoton,wpcm450-fiu.yaml
index 4e0d391e1d69..c97bf48b56b4 100644
--- a/Documentation/devicetree/bindings/spi/nuvoton,wpcm450-fiu.yaml
+++ b/Documentation/devicetree/bindings/spi/nuvoton,wpcm450-fiu.yaml
@@ -59,8 +59,3 @@ examples:
         reg = <0>;
       };
     };
-
-    shm: syscon@c8001000 {
-      compatible = "nuvoton,wpcm450-shm", "syscon";
-      reg = <0xc8001000 0x1000>;
-    };
-- 
2.45.2


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/2] spi: dt-bindings: fsl,dspi: Fix example indentation
  2025-05-09 11:21 [PATCH 1/2] spi: dt-bindings: fsl,dspi: Fix example indentation Krzysztof Kozlowski
  2025-05-09 11:21 ` [PATCH 2/2] spi: dt-bindings: nuvoton,wpcm450-fiu: Drop unrelated nodes from DTS example Krzysztof Kozlowski
@ 2025-05-09 15:50 ` Conor Dooley
  2025-05-11  1:54 ` Mark Brown
  2 siblings, 0 replies; 4+ messages in thread
From: Conor Dooley @ 2025-05-09 15:50 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Vladimir Oltean, Mark Brown, Rob Herring, Krzysztof Kozlowski,
	Conor Dooley, Jonathan Neuschäfer, Frank Li, linux-spi, imx,
	devicetree, linux-kernel, openbmc

[-- Attachment #1: Type: text/plain, Size: 409 bytes --]

On Fri, May 09, 2025 at 01:21:31PM +0200, Krzysztof Kozlowski wrote:
> DTS example in the bindings should be indented with 2- or 4-spaces, so
> correct a mixture of different styles to keep consistent 4-spaces.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

For both
Acked-by: Conor Dooley <conor.dooley@microchip.com>

(idc about both appearing in tree with an ack Mark)

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/2] spi: dt-bindings: fsl,dspi: Fix example indentation
  2025-05-09 11:21 [PATCH 1/2] spi: dt-bindings: fsl,dspi: Fix example indentation Krzysztof Kozlowski
  2025-05-09 11:21 ` [PATCH 2/2] spi: dt-bindings: nuvoton,wpcm450-fiu: Drop unrelated nodes from DTS example Krzysztof Kozlowski
  2025-05-09 15:50 ` [PATCH 1/2] spi: dt-bindings: fsl,dspi: Fix example indentation Conor Dooley
@ 2025-05-11  1:54 ` Mark Brown
  2 siblings, 0 replies; 4+ messages in thread
From: Mark Brown @ 2025-05-11  1:54 UTC (permalink / raw)
  To: Vladimir Oltean, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
	Jonathan Neuschäfer, Frank Li, linux-spi, imx, devicetree,
	linux-kernel, openbmc, Krzysztof Kozlowski

On Fri, 09 May 2025 13:21:31 +0200, Krzysztof Kozlowski wrote:
> DTS example in the bindings should be indented with 2- or 4-spaces, so
> correct a mixture of different styles to keep consistent 4-spaces.
> 
> 

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git for-next

Thanks!

[1/2] spi: dt-bindings: fsl,dspi: Fix example indentation
      commit: 846656f278e803cb60161f0cba4ee90a058440cc
[2/2] spi: dt-bindings: nuvoton,wpcm450-fiu: Drop unrelated nodes from DTS example
      commit: a4ca02454821cbc411e0bf16e527d392f188c218

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2025-05-11  1:55 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-05-09 11:21 [PATCH 1/2] spi: dt-bindings: fsl,dspi: Fix example indentation Krzysztof Kozlowski
2025-05-09 11:21 ` [PATCH 2/2] spi: dt-bindings: nuvoton,wpcm450-fiu: Drop unrelated nodes from DTS example Krzysztof Kozlowski
2025-05-09 15:50 ` [PATCH 1/2] spi: dt-bindings: fsl,dspi: Fix example indentation Conor Dooley
2025-05-11  1:54 ` Mark Brown

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).