| Message ID | 20230118180144.364756-2-krzysztof.kozlowski@linaro.org (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [1/2] dt-bindings: dma: drop unneeded quotes | expand |
| Context | Check | Description |
|---|---|---|
| conchuod/cover_letter | success | Single patches do not need cover letters |
| conchuod/tree_selection | success | Guessed tree name to be for-next |
| conchuod/fixes_present | success | Fixes tag not required for -next series |
| conchuod/maintainers_pattern | success | MAINTAINERS pattern errors before the patch: 13 and now 13 |
| conchuod/verify_signedoff | success | Signed-off-by tag matches author and committer |
| conchuod/kdoc | success | Errors and warnings before: 0 this patch: 0 |
| conchuod/module_param | success | Was 0 now: 0 |
| conchuod/build_rv64_gcc_allmodconfig | success | Errors and warnings before: 0 this patch: 0 |
| conchuod/alphanumeric_selects | success | Out of order selects before the patch: 57 and now 57 |
| conchuod/build_rv32_defconfig | success | Build OK |
| conchuod/dtb_warn_rv64 | success | Errors and warnings before: 2 this patch: 2 |
| conchuod/header_inline | success | No static functions without inline keyword in header files |
| conchuod/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 50 lines checked |
| conchuod/source_inline | success | Was 0 now: 0 |
| conchuod/build_rv64_nommu_k210_defconfig | success | Build OK |
| conchuod/verify_fixes | success | No Fixes tag |
| conchuod/build_rv64_nommu_virt_defconfig | success | Build OK |
On Wed, Jan 18, 2023 at 07:01:44PM +0100, Krzysztof Kozlowski wrote: > Cleanup examples: > - use 4-space indentation (for cases when it is neither 4 not 2 space), > - use lowercase hex. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../bindings/dma/snps,dw-axi-dmac.yaml | 36 +++++++++---------- > .../bindings/dma/stericsson,dma40.yaml | 4 +-- > 2 files changed, 20 insertions(+), 20 deletions(-) > > diff --git a/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml b/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml > index 2bedab1f74e0..d34d0fa62ab5 100644 > --- a/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml > +++ b/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml > @@ -113,21 +113,21 @@ additionalProperties: false > > examples: > - | > - #include <dt-bindings/interrupt-controller/arm-gic.h> > - #include <dt-bindings/interrupt-controller/irq.h> > - /* example with snps,dw-axi-dmac */ > - dmac: dma-controller@80000 { > - compatible = "snps,axi-dma-1.01a"; > - reg = <0x80000 0x400>; > - clocks = <&core_clk>, <&cfgr_clk>; > - clock-names = "core-clk", "cfgr-clk"; > - interrupt-parent = <&intc>; > - interrupts = <27>; > - #dma-cells = <1>; > - dma-channels = <4>; > - snps,dma-masters = <2>; > - snps,data-width = <3>; > - snps,block-size = <4096 4096 4096 4096>; > - snps,priority = <0 1 2 3>; > - snps,axi-max-burst-len = <16>; > - }; > + #include <dt-bindings/interrupt-controller/arm-gic.h> > + #include <dt-bindings/interrupt-controller/irq.h> > + /* example with snps,dw-axi-dmac */ > + dmac: dma-controller@80000 { Drop unused labels while you are here. > + compatible = "snps,axi-dma-1.01a"; > + reg = <0x80000 0x400>; > + clocks = <&core_clk>, <&cfgr_clk>; > + clock-names = "core-clk", "cfgr-clk"; > + interrupt-parent = <&intc>; > + interrupts = <27>; > + #dma-cells = <1>; > + dma-channels = <4>; > + snps,dma-masters = <2>; > + snps,data-width = <3>; > + snps,block-size = <4096 4096 4096 4096>; > + snps,priority = <0 1 2 3>; > + snps,axi-max-burst-len = <16>; > + }; > diff --git a/Documentation/devicetree/bindings/dma/stericsson,dma40.yaml b/Documentation/devicetree/bindings/dma/stericsson,dma40.yaml > index 664ee61a00d8..57395a810719 100644 > --- a/Documentation/devicetree/bindings/dma/stericsson,dma40.yaml > +++ b/Documentation/devicetree/bindings/dma/stericsson,dma40.yaml > @@ -147,9 +147,9 @@ examples: > #include <dt-bindings/interrupt-controller/irq.h> > #include <dt-bindings/interrupt-controller/arm-gic.h> > #include <dt-bindings/mfd/dbx500-prcmu.h> > - dma-controller@801C0000 { > + dma-controller@801c0000 { > compatible = "stericsson,db8500-dma40", "stericsson,dma40"; > - reg = <0x801C0000 0x1000>, <0x40010000 0x800>; > + reg = <0x801c0000 0x1000>, <0x40010000 0x800>; > reg-names = "base", "lcpa"; > interrupts = <GIC_SPI 25 IRQ_TYPE_LEVEL_HIGH>; > #dma-cells = <3>; Indentation?
On 23/01/2023 21:20, Rob Herring wrote: > On Wed, Jan 18, 2023 at 07:01:44PM +0100, Krzysztof Kozlowski wrote: >> Cleanup examples: >> - use 4-space indentation (for cases when it is neither 4 not 2 space), >> - use lowercase hex. >> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >> --- >> .../bindings/dma/snps,dw-axi-dmac.yaml | 36 +++++++++---------- >> .../bindings/dma/stericsson,dma40.yaml | 4 +-- >> 2 files changed, 20 insertions(+), 20 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml b/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml >> index 2bedab1f74e0..d34d0fa62ab5 100644 >> --- a/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml >> +++ b/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml >> @@ -113,21 +113,21 @@ additionalProperties: false >> >> examples: >> - | >> - #include <dt-bindings/interrupt-controller/arm-gic.h> >> - #include <dt-bindings/interrupt-controller/irq.h> >> - /* example with snps,dw-axi-dmac */ >> - dmac: dma-controller@80000 { >> - compatible = "snps,axi-dma-1.01a"; >> - reg = <0x80000 0x400>; >> - clocks = <&core_clk>, <&cfgr_clk>; >> - clock-names = "core-clk", "cfgr-clk"; >> - interrupt-parent = <&intc>; >> - interrupts = <27>; >> - #dma-cells = <1>; >> - dma-channels = <4>; >> - snps,dma-masters = <2>; >> - snps,data-width = <3>; >> - snps,block-size = <4096 4096 4096 4096>; >> - snps,priority = <0 1 2 3>; >> - snps,axi-max-burst-len = <16>; >> - }; >> + #include <dt-bindings/interrupt-controller/arm-gic.h> >> + #include <dt-bindings/interrupt-controller/irq.h> >> + /* example with snps,dw-axi-dmac */ >> + dmac: dma-controller@80000 { > > Drop unused labels while you are here. Ack > >> + compatible = "snps,axi-dma-1.01a"; >> + reg = <0x80000 0x400>; >> + clocks = <&core_clk>, <&cfgr_clk>; >> + clock-names = "core-clk", "cfgr-clk"; >> + interrupt-parent = <&intc>; >> + interrupts = <27>; >> + #dma-cells = <1>; >> + dma-channels = <4>; >> + snps,dma-masters = <2>; >> + snps,data-width = <3>; >> + snps,block-size = <4096 4096 4096 4096>; >> + snps,priority = <0 1 2 3>; >> + snps,axi-max-burst-len = <16>; >> + }; >> diff --git a/Documentation/devicetree/bindings/dma/stericsson,dma40.yaml b/Documentation/devicetree/bindings/dma/stericsson,dma40.yaml >> index 664ee61a00d8..57395a810719 100644 >> --- a/Documentation/devicetree/bindings/dma/stericsson,dma40.yaml >> +++ b/Documentation/devicetree/bindings/dma/stericsson,dma40.yaml >> @@ -147,9 +147,9 @@ examples: >> #include <dt-bindings/interrupt-controller/irq.h> >> #include <dt-bindings/interrupt-controller/arm-gic.h> >> #include <dt-bindings/mfd/dbx500-prcmu.h> >> - dma-controller@801C0000 { >> + dma-controller@801c0000 { >> compatible = "stericsson,db8500-dma40", "stericsson,dma40"; >> - reg = <0x801C0000 0x1000>, <0x40010000 0x800>; >> + reg = <0x801c0000 0x1000>, <0x40010000 0x800>; >> reg-names = "base", "lcpa"; >> interrupts = <GIC_SPI 25 IRQ_TYPE_LEVEL_HIGH>; >> #dma-cells = <3>; > > Indentation? Ack Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml b/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml index 2bedab1f74e0..d34d0fa62ab5 100644 --- a/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml +++ b/Documentation/devicetree/bindings/dma/snps,dw-axi-dmac.yaml @@ -113,21 +113,21 @@ additionalProperties: false examples: - | - #include <dt-bindings/interrupt-controller/arm-gic.h> - #include <dt-bindings/interrupt-controller/irq.h> - /* example with snps,dw-axi-dmac */ - dmac: dma-controller@80000 { - compatible = "snps,axi-dma-1.01a"; - reg = <0x80000 0x400>; - clocks = <&core_clk>, <&cfgr_clk>; - clock-names = "core-clk", "cfgr-clk"; - interrupt-parent = <&intc>; - interrupts = <27>; - #dma-cells = <1>; - dma-channels = <4>; - snps,dma-masters = <2>; - snps,data-width = <3>; - snps,block-size = <4096 4096 4096 4096>; - snps,priority = <0 1 2 3>; - snps,axi-max-burst-len = <16>; - }; + #include <dt-bindings/interrupt-controller/arm-gic.h> + #include <dt-bindings/interrupt-controller/irq.h> + /* example with snps,dw-axi-dmac */ + dmac: dma-controller@80000 { + compatible = "snps,axi-dma-1.01a"; + reg = <0x80000 0x400>; + clocks = <&core_clk>, <&cfgr_clk>; + clock-names = "core-clk", "cfgr-clk"; + interrupt-parent = <&intc>; + interrupts = <27>; + #dma-cells = <1>; + dma-channels = <4>; + snps,dma-masters = <2>; + snps,data-width = <3>; + snps,block-size = <4096 4096 4096 4096>; + snps,priority = <0 1 2 3>; + snps,axi-max-burst-len = <16>; + }; diff --git a/Documentation/devicetree/bindings/dma/stericsson,dma40.yaml b/Documentation/devicetree/bindings/dma/stericsson,dma40.yaml index 664ee61a00d8..57395a810719 100644 --- a/Documentation/devicetree/bindings/dma/stericsson,dma40.yaml +++ b/Documentation/devicetree/bindings/dma/stericsson,dma40.yaml @@ -147,9 +147,9 @@ examples: #include <dt-bindings/interrupt-controller/irq.h> #include <dt-bindings/interrupt-controller/arm-gic.h> #include <dt-bindings/mfd/dbx500-prcmu.h> - dma-controller@801C0000 { + dma-controller@801c0000 { compatible = "stericsson,db8500-dma40", "stericsson,dma40"; - reg = <0x801C0000 0x1000>, <0x40010000 0x800>; + reg = <0x801c0000 0x1000>, <0x40010000 0x800>; reg-names = "base", "lcpa"; interrupts = <GIC_SPI 25 IRQ_TYPE_LEVEL_HIGH>; #dma-cells = <3>;
Cleanup examples: - use 4-space indentation (for cases when it is neither 4 not 2 space), - use lowercase hex. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- .../bindings/dma/snps,dw-axi-dmac.yaml | 36 +++++++++---------- .../bindings/dma/stericsson,dma40.yaml | 4 +-- 2 files changed, 20 insertions(+), 20 deletions(-)