lkml.org 
[lkml]   [2019]   [Dec]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] dt-bindings: stm32: convert mlahb to json-schema
On Thu, Nov 28, 2019 at 04:46:03PM +0100, Arnaud Pouliquen wrote:
> Convert the ML-AHB bus bindings to DT schema format using json-schema
>
> Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@st.com>
> ---
> Notice that this patch requests an update of the simple-bus schema to add
> the support of the "dma-ranges" property.
> A Pull request has been sent in parallel to the dt-schema github repo:
> https://github.com/devicetree-org/dt-schema/pull/30
>
> To remind the topic around the use of "dma-ranges" please
> refer to following discussion: https://lkml.org/lkml/2019/4/3/1261
> ---
> .../devicetree/bindings/arm/stm32/mlahb.txt | 37 ----------
> .../bindings/arm/stm32/st,mlahb.yaml | 69 +++++++++++++++++++
> 2 files changed, 69 insertions(+), 37 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/arm/stm32/mlahb.txt
> create mode 100644 Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml
>
> diff --git a/Documentation/devicetree/bindings/arm/stm32/mlahb.txt b/Documentation/devicetree/bindings/arm/stm32/mlahb.txt
> deleted file mode 100644
> index 25307aa1eb9b..000000000000
> --- a/Documentation/devicetree/bindings/arm/stm32/mlahb.txt
> +++ /dev/null
> @@ -1,37 +0,0 @@
> -ML-AHB interconnect bindings
> -
> -These bindings describe the STM32 SoCs ML-AHB interconnect bus which connects
> -a Cortex-M subsystem with dedicated memories.
> -The MCU SRAM and RETRAM memory parts can be accessed through different addresses
> -(see "RAM aliases" in [1]) using different buses (see [2]) : balancing the
> -Cortex-M firmware accesses among those ports allows to tune the system
> -performance.
> -
> -[1]: https://www.st.com/resource/en/reference_manual/dm00327659.pdf
> -[2]: https://wiki.st.com/stm32mpu/wiki/STM32MP15_RAM_mapping
> -
> -Required properties:
> -- compatible: should be "simple-bus"
> -- dma-ranges: describes memory addresses translation between the local CPU and
> - the remote Cortex-M processor. Each memory region, is declared with
> - 3 parameters:
> - - param 1: device base address (Cortex-M processor address)
> - - param 2: physical base address (local CPU address)
> - - param 3: size of the memory region.
> -
> -The Cortex-M remote processor accessed via the mlahb interconnect is described
> -by a child node.
> -
> -Example:
> -mlahb {
> - compatible = "simple-bus";
> - #address-cells = <1>;
> - #size-cells = <1>;
> - dma-ranges = <0x00000000 0x38000000 0x10000>,
> - <0x10000000 0x10000000 0x60000>,
> - <0x30000000 0x30000000 0x60000>;
> -
> - m4_rproc: m4@10000000 {
> - ...
> - };
> -};
> diff --git a/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml b/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml
> new file mode 100644
> index 000000000000..8ad3f7c7f9ab
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/stm32/st,mlahb.yaml
> @@ -0,0 +1,69 @@
> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: "http://devicetree.org/schemas/arm/stm32/st,mlahb.yaml#"
> +$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +
> +title: STMicroelectronics STM32 ML-AHB interconnect bindings
> +
> +maintainers:
> + - Fabien Dessenne <fabien.dessenne@st.com>
> + - Arnaud Pouliquen <arnaud.pouliquen@st.com>
> +
> +description: |
> + These bindings describe the STM32 SoCs ML-AHB interconnect bus which connects
> + a Cortex-M subsystem with dedicated memories. The MCU SRAM and RETRAM memory
> + parts can be accessed through different addresses (see "RAM aliases" in [1])
> + using different buses (see [2]): balancing the Cortex-M firmware accesses
> + among those ports allows to tune the system performance.
> + [1]: https://www.st.com/resource/en/reference_manual/dm00327659.pdf
> + [2]: https://wiki.st.com/stm32mpu/wiki/STM32MP15_RAM_mapping
> +
> +allOf:
> + - $ref: /schemas/simple-bus.yaml#
> +
> +properties:
> + compatible:
> + contains:
> + enum:
> + - st,mlahb
> +
> + dma-ranges:
> + description: |
> + Describe memory addresses translation between the local CPU and the
> + remote Cortex-M processor. Each memory region, is declared with
> + 3 parameters:
> + - param 1: device base address (Cortex-M processor address)
> + - param 2: physical base address (local CPU address)
> + - param 3: size of the memory region.
> + maxItems: 3
> +
> + '#address-cells':
> + const: 1
> +
> + '#size-cells':
> + const: 1
> +
> +required:
> + - compatible
> + - '#address-cells'
> + - '#size-cells'
> + - dma-ranges
> +
> +examples:
> + - |
> + mlahb: ahb {
> + compatible = "st,mlahb", "simple-bus";
> + #address-cells = <1>;
> + #size-cells = <1>;
> + reg = <0x10000000 0x40000>;
> + dma-ranges = <0x00000000 0x38000000 0x10000>,
> + <0x10000000 0x10000000 0x60000>,
> + <0x30000000 0x30000000 0x60000>;
> +

Fails to build:

builds/robherring/linux-dt-review/Documentation/devicetree/bindings/arm/stm32/st,mlahb.example.dt.yaml:
ahb: 'ranges' is a required property

Run 'make dt_binding_check'

> + m4_rproc: m4@10000000 {
> + reg = <0x10000000 0x40000>;
> + };
> + };
> +
> +...
> --
> 2.17.1
>

\
 
 \ /
  Last update: 2019-12-13 22:39    [W:2.109 / U:0.008 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site