diff mbox series

dt-bindings: bus: moxtet: Fix reference to SPI peripheral DT properties

Message ID 20230129123553.1284503-1-j.neuschaefer@gmx.net
State Changes Requested, archived
Headers show
Series dt-bindings: bus: moxtet: Fix reference to SPI peripheral DT properties | expand

Checks

Context Check Description
robh/checkpatch warning total: 0 errors, 4 warnings, 8 lines checked
robh/patch-applied success

Commit Message

J. Neuschäfer Jan. 29, 2023, 12:35 p.m. UTC
spi-bus.txt has been converted to YAML and the peripheral properties
have been moved to spi-controller.yaml.

Fixes: 0a1b929356830 ("spi: Add YAML schemas for the generic SPI options")
Fixes: 8762b07c95c18 ("spi: dt-bindings: add schema listing peripheral-specific properties")
Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
---
 Documentation/devicetree/bindings/bus/moxtet.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
2.39.0

Comments

Marek Behún Jan. 29, 2023, 12:57 p.m. UTC | #1
On Sun, 29 Jan 2023 13:35:53 +0100
Jonathan Neuschäfer <j.neuschaefer@gmx.net> wrote:

> spi-bus.txt has been converted to YAML and the peripheral properties
> have been moved to spi-controller.yaml.
> 
> Fixes: 0a1b929356830 ("spi: Add YAML schemas for the generic SPI options")
> Fixes: 8762b07c95c18 ("spi: dt-bindings: add schema listing peripheral-specific properties")
> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
> ---
>  Documentation/devicetree/bindings/bus/moxtet.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/bus/moxtet.txt b/Documentation/devicetree/bindings/bus/moxtet.txt
> index fb50fc8653368..d6bf929c00c4e 100644
> --- a/Documentation/devicetree/bindings/bus/moxtet.txt
> +++ b/Documentation/devicetree/bindings/bus/moxtet.txt
> @@ -11,7 +11,7 @@ Required properties:
>   - #interrupt-cells	: Has to be 1
> 
>  For other required and optional properties of SPI slave nodes please refer to
> -../spi/spi-bus.txt.
> +../spi/spi-peripheral-props.yaml.
> 
>  Required properties of subnodes:
>   - reg			: Should be position on the Moxtet bus (how many Moxtet
> --
> 2.39.0
> 

Acked-by: Marek Behún <kabel@kernel.org>
Rob Herring Jan. 30, 2023, 10:16 p.m. UTC | #2
On Sun, Jan 29, 2023 at 01:35:53PM +0100, Jonathan Neuschäfer wrote:
> spi-bus.txt has been converted to YAML and the peripheral properties
> have been moved to spi-controller.yaml.
> 
> Fixes: 0a1b929356830 ("spi: Add YAML schemas for the generic SPI options")
> Fixes: 8762b07c95c18 ("spi: dt-bindings: add schema listing peripheral-specific properties")
> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
> ---
>  Documentation/devicetree/bindings/bus/moxtet.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

I'm not really interested in fixes to .txt bindings. Fix it by 
converting to schema if you want to. By my count, there's 37 other 
cases, why just this one?

Rob
J. Neuschäfer Jan. 31, 2023, 8:52 p.m. UTC | #3
On Mon, Jan 30, 2023 at 04:16:26PM -0600, Rob Herring wrote:
> On Sun, Jan 29, 2023 at 01:35:53PM +0100, Jonathan Neuschäfer wrote:
> > spi-bus.txt has been converted to YAML and the peripheral properties
> > have been moved to spi-controller.yaml.
> > 
> > Fixes: 0a1b929356830 ("spi: Add YAML schemas for the generic SPI options")
> > Fixes: 8762b07c95c18 ("spi: dt-bindings: add schema listing peripheral-specific properties")
> > Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
> > ---
> >  Documentation/devicetree/bindings/bus/moxtet.txt | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> I'm not really interested in fixes to .txt bindings. Fix it by 
> converting to schema if you want to.

Fair enough.

> By my count, there's 37 other cases, why just this one?

Mainly because a patch to a large number of files seemed harder to
coordinate well.


Jonathan
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/bus/moxtet.txt b/Documentation/devicetree/bindings/bus/moxtet.txt
index fb50fc8653368..d6bf929c00c4e 100644
--- a/Documentation/devicetree/bindings/bus/moxtet.txt
+++ b/Documentation/devicetree/bindings/bus/moxtet.txt
@@ -11,7 +11,7 @@  Required properties:
  - #interrupt-cells	: Has to be 1

 For other required and optional properties of SPI slave nodes please refer to
-../spi/spi-bus.txt.
+../spi/spi-peripheral-props.yaml.

 Required properties of subnodes:
  - reg			: Should be position on the Moxtet bus (how many Moxtet