[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aJ6XuVZbJo8uUbIw@xhacker>
Date: Fri, 15 Aug 2025 10:13:13 +0800
From: Jisheng Zhang <jszhang@...nel.org>
To: "Rob Herring (Arm)" <robh@...nel.org>
Cc: Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] dt-bindings: arm: Convert marvell,berlin to DT schema
On Wed, Aug 06, 2025 at 04:27:32PM -0500, Rob Herring (Arm) wrote:
> Signed-off-by: Rob Herring (Arm) <robh@...nel.org>
Reviewed-by: Jisheng Zhang <jszhang@...nel.org>
BTW: can you please directly take this patch in dt pr? It's likely no other
patches for next window.
Thanks
> ---
> .../bindings/arm/marvell,berlin.yaml | 45 ++++++++++
> .../devicetree/bindings/arm/syna.txt | 89 -------------------
> 2 files changed, 45 insertions(+), 89 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/arm/marvell,berlin.yaml
> delete mode 100644 Documentation/devicetree/bindings/arm/syna.txt
>
> diff --git a/Documentation/devicetree/bindings/arm/marvell,berlin.yaml b/Documentation/devicetree/bindings/arm/marvell,berlin.yaml
> new file mode 100644
> index 000000000000..4e8442980dcb
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/marvell,berlin.yaml
> @@ -0,0 +1,45 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/arm/marvell,berlin.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Synaptics/Marvell Berlin SoC
> +
> +maintainers:
> + - Jisheng Zhang <jszhang@...nel.org>
> +
> +description:
> + According to https://www.synaptics.com/company/news/conexant-marvell
> + Synaptics has acquired the Multimedia Solutions Business of Marvell, so
> + Berlin SoCs are now Synaptics' SoCs.
> +
> +properties:
> + $nodename:
> + const: '/'
> + compatible:
> + oneOf:
> + - items:
> + - enum:
> + - sony,nsz-gs7
> + - const: marvell,berlin2
> + - const: marvell,berlin
> + - items:
> + - enum:
> + - google,chromecast
> + - valve,steamlink
> + - const: marvell,berlin2cd
> + - const: marvell,berlin
> + - items:
> + - enum:
> + - marvell,berlin2q-dmp
> + - const: marvell,berlin2q
> + - const: marvell,berlin
> + - items:
> + - enum:
> + - marvell,berlin4ct-dmp
> + - marvell,berlin4ct-stb
> + - const: marvell,berlin4ct
> + - const: marvell,berlin
> +
> +additionalProperties: true
> diff --git a/Documentation/devicetree/bindings/arm/syna.txt b/Documentation/devicetree/bindings/arm/syna.txt
> deleted file mode 100644
> index f53c430f648c..000000000000
> --- a/Documentation/devicetree/bindings/arm/syna.txt
> +++ /dev/null
> @@ -1,89 +0,0 @@
> -Synaptics SoC Device Tree Bindings
> -
> -According to https://www.synaptics.com/company/news/conexant-marvell
> -Synaptics has acquired the Multimedia Solutions Business of Marvell, so
> -berlin SoCs are now Synaptics' SoCs now.
> -
> ----------------------------------------------------------------
> -
> -Boards with a SoC of the Marvell Berlin family, e.g. Armada 1500
> -shall have the following properties:
> -
> -* Required root node properties:
> -compatible: must contain "marvell,berlin"
> -
> -In addition, the above compatible shall be extended with the specific
> -SoC and board used. Currently known SoC compatibles are:
> - "marvell,berlin2" for Marvell Armada 1500 (BG2, 88DE3100),
> - "marvell,berlin2cd" for Marvell Armada 1500-mini (BG2CD, 88DE3005)
> - "marvell,berlin2ct" for Marvell Armada ? (BG2CT, 88DE????)
> - "marvell,berlin2q" for Marvell Armada 1500-pro (BG2Q, 88DE3114)
> - "marvell,berlin3" for Marvell Armada ? (BG3, 88DE????)
> -
> -* Example:
> -
> -/ {
> - model = "Sony NSZ-GS7";
> - compatible = "sony,nsz-gs7", "marvell,berlin2", "marvell,berlin";
> -
> - ...
> -}
> -
> -* Marvell Berlin CPU control bindings
> -
> -CPU control register allows various operations on CPUs, like resetting them
> -independently.
> -
> -Required properties:
> -- compatible: should be "marvell,berlin-cpu-ctrl"
> -- reg: address and length of the register set
> -
> -Example:
> -
> -cpu-ctrl@...d0000 {
> - compatible = "marvell,berlin-cpu-ctrl";
> - reg = <0xf7dd0000 0x10000>;
> -};
> -
> -* Marvell Berlin2 chip control binding
> -
> -Marvell Berlin SoCs have a chip control register set providing several
> -individual registers dealing with pinmux, padmux, clock, reset, and secondary
> -CPU boot address. Unfortunately, the individual registers are spread among the
> -chip control registers, so there should be a single DT node only providing the
> -different functions which are described below.
> -
> -Required properties:
> -- compatible:
> - * the first and second values must be:
> - "simple-mfd", "syscon"
> -- reg: address and length of following register sets for
> - BG2/BG2CD: chip control register set
> - BG2Q: chip control register set and cpu pll registers
> -
> -* Marvell Berlin2 system control binding
> -
> -Marvell Berlin SoCs have a system control register set providing several
> -individual registers dealing with pinmux, padmux, and reset.
> -
> -Required properties:
> -- compatible:
> - * the first and second values must be:
> - "simple-mfd", "syscon"
> -- reg: address and length of the system control register set
> -
> -Example:
> -
> -chip: chip-control@...000 {
> - compatible = "simple-mfd", "syscon";
> - reg = <0xea0000 0x400>;
> -
> - /* sub-device nodes */
> -};
> -
> -sysctrl: system-controller@...0 {
> - compatible = "simple-mfd", "syscon";
> - reg = <0xd000 0x100>;
> -
> - /* sub-device nodes */
> -};
> --
> 2.47.2
>
Powered by blists - more mailing lists