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

Jonathan Neuschäfer posted 1 patch 2 years, 7 months ago
Documentation/devicetree/bindings/bus/moxtet.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] dt-bindings: bus: moxtet: Fix reference to SPI peripheral DT properties
Posted by Jonathan Neuschäfer 2 years, 7 months ago
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
Re: [PATCH] dt-bindings: bus: moxtet: Fix reference to SPI peripheral DT properties
Posted by Rob Herring 2 years, 7 months ago
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
Re: [PATCH] dt-bindings: bus: moxtet: Fix reference to SPI peripheral DT properties
Posted by Jonathan Neuschäfer 2 years, 7 months ago
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
Re: [PATCH] dt-bindings: bus: moxtet: Fix reference to SPI peripheral DT properties
Posted by Marek Behún 2 years, 7 months ago
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>