lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite for Android: free password hash cracker in your pocket
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Sat, 8 Jun 2024 15:14:10 +0200
From: Stanislav Jakubek <stano.jakubek@...il.com>
To: Artur Weber <aweber.kernel@...il.com>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>,
	Florian Fainelli <florian.fainelli@...adcom.com>,
	Ray Jui <rjui@...adcom.com>, Scott Branden <sbranden@...adcom.com>,
	Broadcom internal kernel review list <bcm-kernel-feedback-list@...adcom.com>,
	~postmarketos/upstreaming@...ts.sr.ht, devicetree@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] ARM: dts: bcm-mobile: Split out nodes used by both
 BCM21664 and BCM23550

Hi Artur, some notes below.

On Wed, Jun 05, 2024 at 08:40:13AM +0200, Artur Weber wrote:
> The BCM21664 is nearly identical in terms of register layout to the
> BCM23550. Move the shared nodes into a new file, bcm21664-common.dtsi,
> and make both bcm21664.dtsi and bcm23550.dtsi include it. This new
> common file is based on the former bcm23550.dtsi file, and inherits
> its licensing.
> 
> Signed-off-by: Artur Weber <aweber.kernel@...il.com>
> ---
> As for other uses - the BCM21654 (not in mainline) appears to be fairly
> similar in a few ways to the BCM21664 as well; the BCM21855/BCM11531 has
> a similar layout of registers, but there are much more differences.
> We could potentially further split more nodes into a single common
> Broadcom Kona DTSI, then make sub-DTSIs to just change the compatibles,
> but that's a task for another patch.
> ---
>  .../{bcm23550.dtsi => bcm21664-common.dtsi}        | 109 +----
>  arch/arm/boot/dts/broadcom/bcm21664.dtsi           | 388 ++---------------
>  arch/arm/boot/dts/broadcom/bcm23550.dtsi           | 482 +++------------------
>  3 files changed, 131 insertions(+), 848 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/broadcom/bcm23550.dtsi b/arch/arm/boot/dts/broadcom/bcm21664-common.dtsi
> similarity index 68%
> copy from arch/arm/boot/dts/broadcom/bcm23550.dtsi
> copy to arch/arm/boot/dts/broadcom/bcm21664-common.dtsi
> index 50ebe93d6bd0..44978a459d32 100644
> --- a/arch/arm/boot/dts/broadcom/bcm23550.dtsi
> +++ b/arch/arm/boot/dts/broadcom/bcm21664-common.dtsi
> @@ -1,36 +1,13 @@
> +// SPDX-License-Identifier: BSD-3-Clause
>  /*
> - *  BSD LICENSE
> + * Common device tree for components shared between the BCM21664 and BCM23550
> + * SoCs.
>   *
> - *  Copyright(c) 2016 Broadcom.  All rights reserved.
> - *
> - *  Redistribution and use in source and binary forms, with or without
> - *  modification, are permitted provided that the following conditions
> - *  are met:
> - *
> - *    * Redistributions of source code must retain the above copyright
> - *      notice, this list of conditions and the following disclaimer.
> - *    * Redistributions in binary form must reproduce the above copyright
> - *      notice, this list of conditions and the following disclaimer in
> - *      the documentation and/or other materials provided with the
> - *      distribution.
> - *    * Neither the name of Broadcom Corporation nor the names of its
> - *      contributors may be used to endorse or promote products derived
> - *      from this software without specific prior written permission.
> - *
> - *  THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
> - *  "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
> - *  LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
> - *  A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
> - *  OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
> - *  SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
> - *  LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
> - *  DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
> - *  THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
> - *  (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
> - * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
> + * Copyright (C) 2016 Broadcom
>   */
>  
> -/* BCM23550 and BCM21664 have almost identical clocks */
> +/dts-v1/;
> +
>  #include <dt-bindings/clock/bcm21664.h>
>  #include <dt-bindings/interrupt-controller/arm-gic.h>
>  #include <dt-bindings/interrupt-controller/irq.h>
> @@ -38,58 +15,16 @@
>  / {
>  	#address-cells = <1>;
>  	#size-cells = <1>;
> -	model = "BCM23550 SoC";
> -	compatible = "brcm,bcm23550";
> -	interrupt-parent = <&gic>;
> -
> -	cpus {
> -		#address-cells = <1>;
> -		#size-cells = <0>;
> -
> -		cpu0: cpu@0 {
> -			device_type = "cpu";
> -			compatible = "arm,cortex-a7";
> -			reg = <0>;
> -			clock-frequency = <1000000000>;
> -		};
> -
> -		cpu1: cpu@1 {
> -			device_type = "cpu";
> -			compatible = "arm,cortex-a7";
> -			enable-method = "brcm,bcm23550";
> -			secondary-boot-reg = <0x35004178>;
> -			reg = <1>;
> -			clock-frequency = <1000000000>;
> -		};
> -
> -		cpu2: cpu@2 {
> -			device_type = "cpu";
> -			compatible = "arm,cortex-a7";
> -			enable-method = "brcm,bcm23550";
> -			secondary-boot-reg = <0x35004178>;
> -			reg = <2>;
> -			clock-frequency = <1000000000>;
> -		};
> -
> -		cpu3: cpu@3 {
> -			device_type = "cpu";
> -			compatible = "arm,cortex-a7";
> -			enable-method = "brcm,bcm23550";
> -			secondary-boot-reg = <0x35004178>;
> -			reg = <3>;
> -			clock-frequency = <1000000000>;
> -		};
> -	};
>  
>  	/* Hub bus */
> -	hub@...00000 {
> +	hub: hub-bus@...00000 {
>  		compatible = "simple-bus";
>  		ranges = <0 0x34000000 0x102f83ac>;
>  		#address-cells = <1>;
>  		#size-cells = <1>;
>  
>  		smc@...00 {
> -			compatible = "brcm,bcm23550-smc", "brcm,kona-smc";
> +			compatible = "brcm,bcm21664-smc", "brcm,kona-smc";
>  			reg = <0x0004e000 0x400>; /* 1 KiB in SRAM */
>  		};
>  
> @@ -99,7 +34,7 @@ resetmgr: reset-controller@...1f00 {
>  		};
>  
>  		gpio: gpio@...3000 {
> -			compatible = "brcm,bcm23550-gpio", "brcm,kona-gpio";
> +			compatible = "brcm,bcm21664-gpio", "brcm,kona-gpio";
>  			reg = <0x01003000 0x524>;
>  			interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>,
>  				     <GIC_SPI 115 IRQ_TYPE_LEVEL_HIGH>,
> @@ -120,7 +55,7 @@ timer@...6000 {
>  	};
>  
>  	/* Slaves bus */
> -	slaves@...00000 {
> +	slaves: slaves-bus@...00000 {
>  		compatible = "simple-bus";
>  		ranges = <0 0x3e000000 0x0001c070>;
>  		#address-cells = <1>;
> @@ -157,7 +92,7 @@ uartb3: serial@...0 {
>  		};
>  
>  		bsc1: i2c@...00 {
> -			compatible = "brcm,bcm23550-i2c", "brcm,kona-i2c";
> +			compatible = "brcm,bcm21664-i2c", "brcm,kona-i2c";
>  			reg = <0x00016000 0x70>;
>  			interrupts = <GIC_SPI 103 IRQ_TYPE_LEVEL_HIGH>;
>  			#address-cells = <1>;
> @@ -167,7 +102,7 @@ bsc1: i2c@...00 {
>  		};
>  
>  		bsc2: i2c@...00 {
> -			compatible = "brcm,bcm23550-i2c", "brcm,kona-i2c";
> +			compatible = "brcm,bcm21664-i2c", "brcm,kona-i2c";
>  			reg = <0x00017000 0x70>;
>  			interrupts = <GIC_SPI 102 IRQ_TYPE_LEVEL_HIGH>;
>  			#address-cells = <1>;
> @@ -177,7 +112,7 @@ bsc2: i2c@...00 {
>  		};
>  
>  		bsc3: i2c@...00 {
> -			compatible = "brcm,bcm23550-i2c", "brcm,kona-i2c";
> +			compatible = "brcm,bcm21664-i2c", "brcm,kona-i2c";
>  			reg = <0x00018000 0x70>;
>  			interrupts = <GIC_SPI 169 IRQ_TYPE_LEVEL_HIGH>;
>  			#address-cells = <1>;
> @@ -187,7 +122,7 @@ bsc3: i2c@...00 {
>  		};
>  
>  		bsc4: i2c@...00 {
> -			compatible = "brcm,bcm23550-i2c", "brcm,kona-i2c";
> +			compatible = "brcm,bcm21664-i2c", "brcm,kona-i2c";
>  			reg = <0x0001c000 0x70>;
>  			interrupts = <GIC_SPI 170 IRQ_TYPE_LEVEL_HIGH>;
>  			#address-cells = <1>;
> @@ -198,7 +133,7 @@ bsc4: i2c@...00 {
>  	};
>  
>  	/* Apps bus */
> -	apps@...00000 {
> +	apps: apps-bus@...00000 {
>  		compatible = "simple-bus";
>  		ranges = <0 0x3e300000 0x01b77000>;
>  		#address-cells = <1>;
> @@ -253,20 +188,6 @@ sdio4: mmc@...000 {
>  			clocks = <&master_ccu BCM21664_MASTER_CCU_SDIO4>;
>  			status = "disabled";
>  		};
> -
> -		cdc: cdc@...e000 {
> -			compatible = "brcm,bcm23550-cdc";
> -			reg = <0x01b0e000 0x78>;
> -		};
> -
> -		gic: interrupt-controller@...1000 {
> -			compatible = "arm,cortex-a9-gic";
> -			#interrupt-cells = <3>;
> -			#address-cells = <0>;
> -			interrupt-controller;
> -			reg = <0x01b21000 0x1000>,
> -			      <0x01b22000 0x1000>;
> -		};
>  	};
>  
>  	clocks {
> diff --git a/arch/arm/boot/dts/broadcom/bcm21664.dtsi b/arch/arm/boot/dts/broadcom/bcm21664.dtsi
> dissimilarity index 93%
> index c1ad5123bad4..a868f09e8930 100644
> --- a/arch/arm/boot/dts/broadcom/bcm21664.dtsi
> +++ b/arch/arm/boot/dts/broadcom/bcm21664.dtsi
> @@ -1,341 +1,47 @@
> -// SPDX-License-Identifier: GPL-2.0-only
> -// Copyright (C) 2014 Broadcom Corporation
> -
> -#include <dt-bindings/clock/bcm21664.h>
> -#include <dt-bindings/interrupt-controller/arm-gic.h>
> -#include <dt-bindings/interrupt-controller/irq.h>
> -
> -/ {
> -	#address-cells = <1>;
> -	#size-cells = <1>;
> -	model = "BCM21664 SoC";
> -	compatible = "brcm,bcm21664";
> -	interrupt-parent = <&gic>;
> -
> -	cpus {
> -		#address-cells = <1>;
> -		#size-cells = <0>;
> -
> -		cpu0: cpu@0 {
> -			device_type = "cpu";
> -			compatible = "arm,cortex-a9";
> -			reg = <0>;
> -		};
> -
> -		cpu1: cpu@1 {
> -			device_type = "cpu";
> -			compatible = "arm,cortex-a9";
> -			enable-method = "brcm,bcm11351-cpu-method";
> -			secondary-boot-reg = <0x35004178>;
> -			reg = <1>;
> -		};
> -	};
> -
> -	gic: interrupt-controller@...00100 {
> -		compatible = "arm,cortex-a9-gic";
> -		#interrupt-cells = <3>;
> -		#address-cells = <0>;
> -		interrupt-controller;
> -		reg = <0x3ff01000 0x1000>,
> -		      <0x3ff00100 0x100>;
> -	};
> -
> -	smc@...4e000 {
> -		compatible = "brcm,bcm21664-smc", "brcm,kona-smc";
> -		reg = <0x3404e000 0x400>; /* 1 KiB in SRAM */
> -	};
> -
> -	uartb: serial@...00000 {
> -		compatible = "brcm,bcm21664-dw-apb-uart", "snps,dw-apb-uart";
> -		reg = <0x3e000000 0x118>;
> -		clocks = <&slave_ccu BCM21664_SLAVE_CCU_UARTB>;
> -		interrupts = <GIC_SPI 67 IRQ_TYPE_LEVEL_HIGH>;
> -		reg-shift = <2>;
> -		reg-io-width = <4>;
> -		status = "disabled";
> -	};
> -
> -	uartb2: serial@...01000 {
> -		compatible = "brcm,bcm21664-dw-apb-uart", "snps,dw-apb-uart";
> -		reg = <0x3e001000 0x118>;
> -		clocks = <&slave_ccu BCM21664_SLAVE_CCU_UARTB2>;
> -		interrupts = <GIC_SPI 66 IRQ_TYPE_LEVEL_HIGH>;
> -		reg-shift = <2>;
> -		reg-io-width = <4>;
> -		status = "disabled";
> -	};
> -
> -	uartb3: serial@...02000 {
> -		compatible = "brcm,bcm21664-dw-apb-uart", "snps,dw-apb-uart";
> -		reg = <0x3e002000 0x118>;
> -		clocks = <&slave_ccu BCM21664_SLAVE_CCU_UARTB3>;
> -		interrupts = <GIC_SPI 65 IRQ_TYPE_LEVEL_HIGH>;
> -		reg-shift = <2>;
> -		reg-io-width = <4>;
> -		status = "disabled";
> -	};
> -
> -	L2: cache-controller@...20000 {
> -		compatible = "arm,pl310-cache";
> -		reg = <0x3ff20000 0x1000>;
> -		cache-unified;
> -		cache-level = <2>;
> -	};
> -
> -	brcm,resetmgr@...01f00 {
> -		compatible = "brcm,bcm21664-resetmgr";
> -		reg = <0x35001f00 0x24>;
> -	};
> -
> -	timer@...06000 {
> -		compatible = "brcm,kona-timer";
> -		reg = <0x35006000 0x1c>;
> -		interrupts = <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>;
> -		clocks = <&aon_ccu BCM21664_AON_CCU_HUB_TIMER>;
> -	};
> -
> -	gpio: gpio@...03000 {
> -		compatible = "brcm,bcm21664-gpio", "brcm,kona-gpio";
> -		reg = <0x35003000 0x524>;
> -		interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>,
> -			     <GIC_SPI 115 IRQ_TYPE_LEVEL_HIGH>,
> -			     <GIC_SPI 114 IRQ_TYPE_LEVEL_HIGH>,
> -			     <GIC_SPI 113 IRQ_TYPE_LEVEL_HIGH>;
> -		#gpio-cells = <2>;
> -		#interrupt-cells = <2>;
> -		gpio-controller;
> -		interrupt-controller;
> -	};
> -
> -	sdio1: mmc@...80000 {
> -		compatible = "brcm,kona-sdhci";
> -		reg = <0x3f180000 0x801c>;
> -		interrupts = <GIC_SPI 77 IRQ_TYPE_LEVEL_HIGH>;
> -		clocks = <&master_ccu BCM21664_MASTER_CCU_SDIO1>;
> -		status = "disabled";
> -	};
> -
> -	sdio2: mmc@...90000 {
> -		compatible = "brcm,kona-sdhci";
> -		reg = <0x3f190000 0x801c>;
> -		interrupts = <GIC_SPI 76 IRQ_TYPE_LEVEL_HIGH>;
> -		clocks = <&master_ccu BCM21664_MASTER_CCU_SDIO2>;
> -		status = "disabled";
> -	};
> -
> -	sdio3: mmc@...a0000 {
> -		compatible = "brcm,kona-sdhci";
> -		reg = <0x3f1a0000 0x801c>;
> -		interrupts = <GIC_SPI 74 IRQ_TYPE_LEVEL_HIGH>;
> -		clocks = <&master_ccu BCM21664_MASTER_CCU_SDIO3>;
> -		status = "disabled";
> -	};
> -
> -	sdio4: mmc@...b0000 {
> -		compatible = "brcm,kona-sdhci";
> -		reg = <0x3f1b0000 0x801c>;
> -		interrupts = <GIC_SPI 73 IRQ_TYPE_LEVEL_HIGH>;
> -		clocks = <&master_ccu BCM21664_MASTER_CCU_SDIO4>;
> -		status = "disabled";
> -	};
> -
> -	bsc1: i2c@...16000 {
> -		compatible = "brcm,bcm21664-i2c", "brcm,kona-i2c";
> -		reg = <0x3e016000 0x70>;
> -		interrupts = <GIC_SPI 103 IRQ_TYPE_LEVEL_HIGH>;
> -		#address-cells = <1>;
> -		#size-cells = <0>;
> -		clocks = <&slave_ccu BCM21664_SLAVE_CCU_BSC1>;
> -		status = "disabled";
> -	};
> -
> -	bsc2: i2c@...17000 {
> -		compatible = "brcm,bcm21664-i2c", "brcm,kona-i2c";
> -		reg = <0x3e017000 0x70>;
> -		interrupts = <GIC_SPI 102 IRQ_TYPE_LEVEL_HIGH>;
> -		#address-cells = <1>;
> -		#size-cells = <0>;
> -		clocks = <&slave_ccu BCM21664_SLAVE_CCU_BSC2>;
> -		status = "disabled";
> -	};
> -
> -	bsc3: i2c@...18000 {
> -		compatible = "brcm,bcm21664-i2c", "brcm,kona-i2c";
> -		reg = <0x3e018000 0x70>;
> -		interrupts = <GIC_SPI 169 IRQ_TYPE_LEVEL_HIGH>;
> -		#address-cells = <1>;
> -		#size-cells = <0>;
> -		clocks = <&slave_ccu BCM21664_SLAVE_CCU_BSC3>;
> -		status = "disabled";
> -	};
> -
> -	bsc4: i2c@...1c000 {
> -		compatible = "brcm,bcm21664-i2c", "brcm,kona-i2c";
> -		reg = <0x3e01c000 0x70>;
> -		interrupts = <GIC_SPI 170 IRQ_TYPE_LEVEL_HIGH>;
> -		#address-cells = <1>;
> -		#size-cells = <0>;
> -		clocks = <&slave_ccu BCM21664_SLAVE_CCU_BSC4>;
> -		status = "disabled";
> -	};
> -
> -	clocks {
> -		#address-cells = <1>;
> -		#size-cells = <1>;
> -		ranges;
> -
> -		/*
> -		 * Fixed clocks are defined before CCUs whose
> -		 * clocks may depend on them.
> -		 */
> -
> -		ref_32k_clk: ref_32k {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <32768>;
> -		};
> -
> -		bbl_32k_clk: bbl_32k {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <32768>;
> -		};
> -
> -		ref_13m_clk: ref_13m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <13000000>;
> -		};
> -
> -		var_13m_clk: var_13m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <13000000>;
> -		};
> -
> -		dft_19_5m_clk: dft_19_5m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <19500000>;
> -		};
> -
> -		ref_crystal_clk: ref_crystal {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <26000000>;
> -		};
> -
> -		ref_52m_clk: ref_52m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <52000000>;
> -		};
> -
> -		var_52m_clk: var_52m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <52000000>;
> -		};
> -
> -		usb_otg_ahb_clk: usb_otg_ahb {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <52000000>;
> -		};
> -
> -		ref_96m_clk: ref_96m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <96000000>;
> -		};
> -
> -		var_96m_clk: var_96m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <96000000>;
> -		};
> -
> -		ref_104m_clk: ref_104m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <104000000>;
> -		};
> -
> -		var_104m_clk: var_104m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <104000000>;
> -		};
> -
> -		ref_156m_clk: ref_156m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <156000000>;
> -		};
> -
> -		var_156m_clk: var_156m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <156000000>;
> -		};
> -
> -		root_ccu: root_ccu@...01000 {
> -			compatible = "brcm,bcm21664-root-ccu";
> -			reg = <0x35001000 0x0f00>;
> -			#clock-cells = <1>;
> -			clock-output-names = "frac_1m";
> -		};
> -
> -		aon_ccu: aon_ccu@...02000 {
> -			compatible = "brcm,bcm21664-aon-ccu";
> -			reg = <0x35002000 0x0f00>;
> -			#clock-cells = <1>;
> -			clock-output-names = "hub_timer";
> -		};
> -
> -		master_ccu: master_ccu@...01000 {
> -			compatible = "brcm,bcm21664-master-ccu";
> -			reg = <0x3f001000 0x0f00>;
> -			#clock-cells = <1>;
> -			clock-output-names = "sdio1",
> -					     "sdio2",
> -					     "sdio3",
> -					     "sdio4",
> -					     "sdio1_sleep",
> -					     "sdio2_sleep",
> -					     "sdio3_sleep",
> -					     "sdio4_sleep";
> -		};
> -
> -		slave_ccu: slave_ccu@...11000 {
> -			compatible = "brcm,bcm21664-slave-ccu";
> -			reg = <0x3e011000 0x0f00>;
> -			#clock-cells = <1>;
> -			clock-output-names = "uartb",
> -					     "uartb2",
> -					     "uartb3",
> -					     "bsc1",
> -					     "bsc2",
> -					     "bsc3",
> -					     "bsc4";
> -		};
> -	};
> -
> -	usbotg: usb@...20000 {
> -		compatible = "snps,dwc2";
> -		reg = <0x3f120000 0x10000>;
> -		interrupts = <GIC_SPI 47 IRQ_TYPE_LEVEL_HIGH>;
> -		clocks = <&usb_otg_ahb_clk>;
> -		clock-names = "otg";
> -		phys = <&usbphy>;
> -		phy-names = "usb2-phy";
> -		status = "disabled";
> -	};
> -
> -	usbphy: usb-phy@...30000 {
> -		compatible = "brcm,kona-usb2-phy";
> -		reg = <0x3f130000 0x28>;
> -		#phy-cells = <0>;
> -		status = "disabled";
> -	};
> -};
> +// SPDX-License-Identifier: GPL-2.0-only
> +// Copyright (C) 2014 Broadcom Corporation
> +
> +#include "bcm21664-common.dtsi"
> +
> +/ {
> +	model = "BCM21664 SoC";
> +	compatible = "brcm,bcm21664";

Aren't model and compatible board-specific?

> +	interrupt-parent = <&gic>;
> +
> +	cpus {
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +
> +		cpu0: cpu@0 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a9";
> +			reg = <0>;
> +		};
> +
> +		cpu1: cpu@1 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a9";
> +			enable-method = "brcm,bcm11351-cpu-method";
> +			secondary-boot-reg = <0x35004178>;
> +			reg = <1>;
> +		};
> +	};
> +
> +	apps: apps-bus@...00000 {

You can just reference this by the label, i.e.

&apps {
	gic: ...
	L2: ...
};

> +		gic: interrupt-controller@...0100 {
> +			compatible = "arm,cortex-a9-gic";
> +			#interrupt-cells = <3>;
> +			#address-cells = <0>;
> +			interrupt-controller;
> +			reg = <0x01c01000 0x1000>,
> +			      <0x01c00100 0x100>;
> +		};
> +
> +		L2: cache-controller@...0000 {
> +			compatible = "arm,pl310-cache";
> +			reg = <0x01c20000 0x1000>;
> +			cache-unified;
> +			cache-level = <2>;
> +		};
> +	};
> +};
> diff --git a/arch/arm/boot/dts/broadcom/bcm23550.dtsi b/arch/arm/boot/dts/broadcom/bcm23550.dtsi
> dissimilarity index 88%
> index 50ebe93d6bd0..bb5ae970816b 100644
> --- a/arch/arm/boot/dts/broadcom/bcm23550.dtsi
> +++ b/arch/arm/boot/dts/broadcom/bcm23550.dtsi
> @@ -1,413 +1,69 @@
> -/*
> - *  BSD LICENSE
> - *
> - *  Copyright(c) 2016 Broadcom.  All rights reserved.
> - *
> - *  Redistribution and use in source and binary forms, with or without
> - *  modification, are permitted provided that the following conditions
> - *  are met:
> - *
> - *    * Redistributions of source code must retain the above copyright
> - *      notice, this list of conditions and the following disclaimer.
> - *    * Redistributions in binary form must reproduce the above copyright
> - *      notice, this list of conditions and the following disclaimer in
> - *      the documentation and/or other materials provided with the
> - *      distribution.
> - *    * Neither the name of Broadcom Corporation nor the names of its
> - *      contributors may be used to endorse or promote products derived
> - *      from this software without specific prior written permission.
> - *
> - *  THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
> - *  "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
> - *  LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
> - *  A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
> - *  OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
> - *  SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
> - *  LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
> - *  DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
> - *  THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
> - *  (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
> - * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
> - */
> -
> -/* BCM23550 and BCM21664 have almost identical clocks */
> -#include <dt-bindings/clock/bcm21664.h>
> -#include <dt-bindings/interrupt-controller/arm-gic.h>
> -#include <dt-bindings/interrupt-controller/irq.h>
> -
> -/ {
> -	#address-cells = <1>;
> -	#size-cells = <1>;
> -	model = "BCM23550 SoC";
> -	compatible = "brcm,bcm23550";
> -	interrupt-parent = <&gic>;
> -
> -	cpus {
> -		#address-cells = <1>;
> -		#size-cells = <0>;
> -
> -		cpu0: cpu@0 {
> -			device_type = "cpu";
> -			compatible = "arm,cortex-a7";
> -			reg = <0>;
> -			clock-frequency = <1000000000>;
> -		};
> -
> -		cpu1: cpu@1 {
> -			device_type = "cpu";
> -			compatible = "arm,cortex-a7";
> -			enable-method = "brcm,bcm23550";
> -			secondary-boot-reg = <0x35004178>;
> -			reg = <1>;
> -			clock-frequency = <1000000000>;
> -		};
> -
> -		cpu2: cpu@2 {
> -			device_type = "cpu";
> -			compatible = "arm,cortex-a7";
> -			enable-method = "brcm,bcm23550";
> -			secondary-boot-reg = <0x35004178>;
> -			reg = <2>;
> -			clock-frequency = <1000000000>;
> -		};
> -
> -		cpu3: cpu@3 {
> -			device_type = "cpu";
> -			compatible = "arm,cortex-a7";
> -			enable-method = "brcm,bcm23550";
> -			secondary-boot-reg = <0x35004178>;
> -			reg = <3>;
> -			clock-frequency = <1000000000>;
> -		};
> -	};
> -
> -	/* Hub bus */
> -	hub@...00000 {
> -		compatible = "simple-bus";
> -		ranges = <0 0x34000000 0x102f83ac>;
> -		#address-cells = <1>;
> -		#size-cells = <1>;
> -
> -		smc@...00 {
> -			compatible = "brcm,bcm23550-smc", "brcm,kona-smc";
> -			reg = <0x0004e000 0x400>; /* 1 KiB in SRAM */
> -		};
> -
> -		resetmgr: reset-controller@...1f00 {
> -			compatible = "brcm,bcm21664-resetmgr";
> -			reg = <0x01001f00 0x24>;
> -		};
> -
> -		gpio: gpio@...3000 {
> -			compatible = "brcm,bcm23550-gpio", "brcm,kona-gpio";
> -			reg = <0x01003000 0x524>;
> -			interrupts = <GIC_SPI 106 IRQ_TYPE_LEVEL_HIGH>,
> -				     <GIC_SPI 115 IRQ_TYPE_LEVEL_HIGH>,
> -				     <GIC_SPI 114 IRQ_TYPE_LEVEL_HIGH>,
> -				     <GIC_SPI 113 IRQ_TYPE_LEVEL_HIGH>;
> -			#gpio-cells = <2>;
> -			#interrupt-cells = <2>;
> -			gpio-controller;
> -			interrupt-controller;
> -		};
> -
> -		timer@...6000 {
> -			compatible = "brcm,kona-timer";
> -			reg = <0x01006000 0x1c>;
> -			interrupts = <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>;
> -			clocks = <&aon_ccu BCM21664_AON_CCU_HUB_TIMER>;
> -		};
> -	};
> -
> -	/* Slaves bus */
> -	slaves@...00000 {
> -		compatible = "simple-bus";
> -		ranges = <0 0x3e000000 0x0001c070>;
> -		#address-cells = <1>;
> -		#size-cells = <1>;
> -
> -		uartb: serial@0 {
> -			compatible = "snps,dw-apb-uart";
> -			reg = <0x00000000 0x118>;
> -			clocks = <&slave_ccu BCM21664_SLAVE_CCU_UARTB>;
> -			interrupts = <GIC_SPI 67 IRQ_TYPE_LEVEL_HIGH>;
> -			reg-shift = <2>;
> -			reg-io-width = <4>;
> -			status = "disabled";
> -		};
> -
> -		uartb2: serial@...0 {
> -			compatible = "snps,dw-apb-uart";
> -			reg = <0x00001000 0x118>;
> -			clocks = <&slave_ccu BCM21664_SLAVE_CCU_UARTB2>;
> -			interrupts = <GIC_SPI 66 IRQ_TYPE_LEVEL_HIGH>;
> -			reg-shift = <2>;
> -			reg-io-width = <4>;
> -			status = "disabled";
> -		};
> -
> -		uartb3: serial@...0 {
> -			compatible = "snps,dw-apb-uart";
> -			reg = <0x00002000 0x118>;
> -			clocks = <&slave_ccu BCM21664_SLAVE_CCU_UARTB3>;
> -			interrupts = <GIC_SPI 65 IRQ_TYPE_LEVEL_HIGH>;
> -			reg-shift = <2>;
> -			reg-io-width = <4>;
> -			status = "disabled";
> -		};
> -
> -		bsc1: i2c@...00 {
> -			compatible = "brcm,bcm23550-i2c", "brcm,kona-i2c";
> -			reg = <0x00016000 0x70>;
> -			interrupts = <GIC_SPI 103 IRQ_TYPE_LEVEL_HIGH>;
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			clocks = <&slave_ccu BCM21664_SLAVE_CCU_BSC1>;
> -			status = "disabled";
> -		};
> -
> -		bsc2: i2c@...00 {
> -			compatible = "brcm,bcm23550-i2c", "brcm,kona-i2c";
> -			reg = <0x00017000 0x70>;
> -			interrupts = <GIC_SPI 102 IRQ_TYPE_LEVEL_HIGH>;
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			clocks = <&slave_ccu BCM21664_SLAVE_CCU_BSC2>;
> -			status = "disabled";
> -		};
> -
> -		bsc3: i2c@...00 {
> -			compatible = "brcm,bcm23550-i2c", "brcm,kona-i2c";
> -			reg = <0x00018000 0x70>;
> -			interrupts = <GIC_SPI 169 IRQ_TYPE_LEVEL_HIGH>;
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			clocks = <&slave_ccu BCM21664_SLAVE_CCU_BSC3>;
> -			status = "disabled";
> -		};
> -
> -		bsc4: i2c@...00 {
> -			compatible = "brcm,bcm23550-i2c", "brcm,kona-i2c";
> -			reg = <0x0001c000 0x70>;
> -			interrupts = <GIC_SPI 170 IRQ_TYPE_LEVEL_HIGH>;
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			clocks = <&slave_ccu BCM21664_SLAVE_CCU_BSC4>;
> -			status = "disabled";
> -		};
> -	};
> -
> -	/* Apps bus */
> -	apps@...00000 {
> -		compatible = "simple-bus";
> -		ranges = <0 0x3e300000 0x01b77000>;
> -		#address-cells = <1>;
> -		#size-cells = <1>;
> -
> -		usbotg: usb@...000 {
> -			compatible = "snps,dwc2";
> -			reg = <0x00e20000 0x10000>;
> -			interrupts = <GIC_SPI 47 IRQ_TYPE_LEVEL_HIGH>;
> -			clocks = <&usb_otg_ahb_clk>;
> -			clock-names = "otg";
> -			phys = <&usbphy>;
> -			phy-names = "usb2-phy";
> -			status = "disabled";
> -		};
> -
> -		usbphy: usb-phy@...000 {
> -			compatible = "brcm,kona-usb2-phy";
> -			reg = <0x00e30000 0x28>;
> -			#phy-cells = <0>;
> -			status = "disabled";
> -		};
> -
> -		sdio1: mmc@...000 {
> -			compatible = "brcm,kona-sdhci";
> -			reg = <0x00e80000 0x801c>;
> -			interrupts = <GIC_SPI 77 IRQ_TYPE_LEVEL_HIGH>;
> -			clocks = <&master_ccu BCM21664_MASTER_CCU_SDIO1>;
> -			status = "disabled";
> -		};
> -
> -		sdio2: mmc@...000 {
> -			compatible = "brcm,kona-sdhci";
> -			reg = <0x00e90000 0x801c>;
> -			interrupts = <GIC_SPI 76 IRQ_TYPE_LEVEL_HIGH>;
> -			clocks = <&master_ccu BCM21664_MASTER_CCU_SDIO2>;
> -			status = "disabled";
> -		};
> -
> -		sdio3: mmc@...000 {
> -			compatible = "brcm,kona-sdhci";
> -			reg = <0x00ea0000 0x801c>;
> -			interrupts = <GIC_SPI 74 IRQ_TYPE_LEVEL_HIGH>;
> -			clocks = <&master_ccu BCM21664_MASTER_CCU_SDIO3>;
> -			status = "disabled";
> -		};
> -
> -		sdio4: mmc@...000 {
> -			compatible = "brcm,kona-sdhci";
> -			reg = <0x00eb0000 0x801c>;
> -			interrupts = <GIC_SPI 73 IRQ_TYPE_LEVEL_HIGH>;
> -			clocks = <&master_ccu BCM21664_MASTER_CCU_SDIO4>;
> -			status = "disabled";
> -		};
> -
> -		cdc: cdc@...e000 {
> -			compatible = "brcm,bcm23550-cdc";
> -			reg = <0x01b0e000 0x78>;
> -		};
> -
> -		gic: interrupt-controller@...1000 {
> -			compatible = "arm,cortex-a9-gic";
> -			#interrupt-cells = <3>;
> -			#address-cells = <0>;
> -			interrupt-controller;
> -			reg = <0x01b21000 0x1000>,
> -			      <0x01b22000 0x1000>;
> -		};
> -	};
> -
> -	clocks {
> -		#address-cells = <1>;
> -		#size-cells = <1>;
> -		ranges;
> -
> -		/*
> -		 * Fixed clocks are defined before CCUs whose
> -		 * clocks may depend on them.
> -		 */
> -
> -		ref_32k_clk: ref_32k {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <32768>;
> -		};
> -
> -		bbl_32k_clk: bbl_32k {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <32768>;
> -		};
> -
> -		ref_13m_clk: ref_13m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <13000000>;
> -		};
> -
> -		var_13m_clk: var_13m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <13000000>;
> -		};
> -
> -		dft_19_5m_clk: dft_19_5m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <19500000>;
> -		};
> -
> -		ref_crystal_clk: ref_crystal {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <26000000>;
> -		};
> -
> -		ref_52m_clk: ref_52m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <52000000>;
> -		};
> -
> -		var_52m_clk: var_52m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <52000000>;
> -		};
> -
> -		usb_otg_ahb_clk: usb_otg_ahb {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <52000000>;
> -		};
> -
> -		ref_96m_clk: ref_96m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <96000000>;
> -		};
> -
> -		var_96m_clk: var_96m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <96000000>;
> -		};
> -
> -		ref_104m_clk: ref_104m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <104000000>;
> -		};
> -
> -		var_104m_clk: var_104m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <104000000>;
> -		};
> -
> -		ref_156m_clk: ref_156m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <156000000>;
> -		};
> -
> -		var_156m_clk: var_156m {
> -			#clock-cells = <0>;
> -			compatible = "fixed-clock";
> -			clock-frequency = <156000000>;
> -		};
> -
> -		root_ccu: root_ccu@...01000 {
> -			compatible = "brcm,bcm21664-root-ccu";
> -			reg = <0x35001000 0x0f00>;
> -			#clock-cells = <1>;
> -			clock-output-names = "frac_1m";
> -		};
> -
> -		aon_ccu: aon_ccu@...02000 {
> -			compatible = "brcm,bcm21664-aon-ccu";
> -			reg = <0x35002000 0x0f00>;
> -			#clock-cells = <1>;
> -			clock-output-names = "hub_timer";
> -		};
> -
> -		slave_ccu: slave_ccu@...11000 {
> -			compatible = "brcm,bcm21664-slave-ccu";
> -			reg = <0x3e011000 0x0f00>;
> -			#clock-cells = <1>;
> -			clock-output-names = "uartb",
> -					     "uartb2",
> -					     "uartb3",
> -					     "bsc1",
> -					     "bsc2",
> -					     "bsc3",
> -					     "bsc4";
> -		};
> -
> -		master_ccu: master_ccu@...01000 {
> -			compatible = "brcm,bcm21664-master-ccu";
> -			reg = <0x3f001000 0x0f00>;
> -			#clock-cells = <1>;
> -			clock-output-names = "sdio1",
> -					     "sdio2",
> -					     "sdio3",
> -					     "sdio4",
> -					     "sdio1_sleep",
> -					     "sdio2_sleep",
> -					     "sdio3_sleep",
> -					     "sdio4_sleep";
> -		};
> -	};
> -};
> +// SPDX-License-Identifier: BSD-3-Clause
> +/*
> + * Device tree for the BCM23550 SoC.
> + *
> + * Copyright (C) 2016 Broadcom
> + */
> +
> +#include "bcm21664-common.dtsi"
> +
> +/ {
> +	model = "BCM23550 SoC";
> +	compatible = "brcm,bcm23550";

Same here.

> +	interrupt-parent = <&gic>;
> +
> +	cpus {
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +
> +		cpu0: cpu@0 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a7";
> +			reg = <0>;
> +			clock-frequency = <1000000000>;
> +		};
> +
> +		cpu1: cpu@1 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a7";
> +			enable-method = "brcm,bcm23550";
> +			secondary-boot-reg = <0x35004178>;
> +			reg = <1>;
> +			clock-frequency = <1000000000>;
> +		};
> +
> +		cpu2: cpu@2 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a7";
> +			enable-method = "brcm,bcm23550";
> +			secondary-boot-reg = <0x35004178>;
> +			reg = <2>;
> +			clock-frequency = <1000000000>;
> +		};
> +
> +		cpu3: cpu@3 {
> +			device_type = "cpu";
> +			compatible = "arm,cortex-a7";
> +			enable-method = "brcm,bcm23550";
> +			secondary-boot-reg = <0x35004178>;
> +			reg = <3>;
> +			clock-frequency = <1000000000>;
> +		};
> +	};
> +
> +	apps: apps-bus@...00000 {

Same here.

Regards,
Stanislav

> +		cdc: cdc@...e000 {
> +			compatible = "brcm,bcm23550-cdc";
> +			reg = <0x01b0e000 0x78>;
> +		};
> +
> +		gic: interrupt-controller@...1000 {
> +			compatible = "arm,cortex-a9-gic";
> +			#interrupt-cells = <3>;
> +			#address-cells = <0>;
> +			interrupt-controller;
> +			reg = <0x01b21000 0x1000>,
> +			      <0x01b22000 0x1000>;
> +		};
> +	};
> +};
> 
> -- 
> 2.45.1
> 

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ