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: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Wed, 20 Jan 2021 15:00:00 -0800
From:   Ray Jui <ray.jui@...adcom.com>
To:     Scott Branden <scott.branden@...adcom.com>,
        Rob Herring <robh+dt@...nel.org>, Ray Jui <rjui@...adcom.com>,
        Scott Branden <sbranden@...adcom.com>
Cc:     BCM Kernel Feedback <bcm-kernel-feedback-list@...adcom.com>,
        devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arm64: dts: stingray: remove sata



On 1/20/2021 10:51 AM, Scott Branden wrote:
> Remove sata from stingray as it is unsupported.
> 
> Signed-off-by: Scott Branden <scott.branden@...adcom.com>
> ---
>  .../dts/broadcom/stingray/bcm958742-base.dtsi |  64 ----
>  .../dts/broadcom/stingray/stingray-sata.dtsi  | 278 ------------------
>  .../boot/dts/broadcom/stingray/stingray.dtsi  |   7 -
>  3 files changed, 349 deletions(-)
>  delete mode 100644 arch/arm64/boot/dts/broadcom/stingray/stingray-sata.dtsi
> 
> diff --git a/arch/arm64/boot/dts/broadcom/stingray/bcm958742-base.dtsi b/arch/arm64/boot/dts/broadcom/stingray/bcm958742-base.dtsi
> index 43aa5e9c0020..8fe7325cfbb2 100644
> --- a/arch/arm64/boot/dts/broadcom/stingray/bcm958742-base.dtsi
> +++ b/arch/arm64/boot/dts/broadcom/stingray/bcm958742-base.dtsi
> @@ -56,70 +56,6 @@
>  	};
>  };
>  
> -&sata0 {
> -	status = "okay";
> -};
> -
> -&sata_phy0{
> -	status = "okay";
> -};
> -
> -&sata1 {
> -	status = "okay";
> -};
> -
> -&sata_phy1{
> -	status = "okay";
> -};
> -
> -&sata2 {
> -	status = "okay";
> -};
> -
> -&sata_phy2{
> -	status = "okay";
> -};
> -
> -&sata3 {
> -	status = "okay";
> -};
> -
> -&sata_phy3{
> -	status = "okay";
> -};
> -
> -&sata4 {
> -	status = "okay";
> -};
> -
> -&sata_phy4{
> -	status = "okay";
> -};
> -
> -&sata5 {
> -	status = "okay";
> -};
> -
> -&sata_phy5{
> -	status = "okay";
> -};
> -
> -&sata6 {
> -	status = "okay";
> -};
> -
> -&sata_phy6{
> -	status = "okay";
> -};
> -
> -&sata7 {
> -	status = "okay";
> -};
> -
> -&sata_phy7{
> -	status = "okay";
> -};
> -
>  &pwm {
>  	status = "okay";
>  };
> diff --git a/arch/arm64/boot/dts/broadcom/stingray/stingray-sata.dtsi b/arch/arm64/boot/dts/broadcom/stingray/stingray-sata.dtsi
> deleted file mode 100644
> index 8c68e0c26f1b..000000000000
> --- a/arch/arm64/boot/dts/broadcom/stingray/stingray-sata.dtsi
> +++ /dev/null
> @@ -1,278 +0,0 @@
> -/*
> - *  BSD LICENSE
> - *
> - *  Copyright(c) 2016-2017 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 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.
> - */
> -
> -	sata {
> -		compatible = "simple-bus";
> -		#address-cells = <1>;
> -		#size-cells = <1>;
> -		ranges = <0x0 0x0 0x67d00000 0x00800000>;
> -
> -		sata0: ahci@0 {
> -			compatible = "brcm,iproc-ahci", "generic-ahci";
> -			reg = <0x00000000 0x1000>;
> -			reg-names = "ahci";
> -			interrupts = <GIC_SPI 321 IRQ_TYPE_LEVEL_HIGH>;
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			status = "disabled";
> -
> -			sata0_port0: sata-port@0 {
> -				reg = <0>;
> -				phys = <&sata0_phy0>;
> -				phy-names = "sata-phy";
> -			};
> -		};
> -
> -		sata_phy0: sata_phy@...0 {
> -			compatible = "brcm,iproc-sr-sata-phy";
> -			reg = <0x00002100 0x1000>;
> -			reg-names = "phy";
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			status = "disabled";
> -
> -			sata0_phy0: sata-phy@0 {
> -				reg = <0>;
> -				#phy-cells = <0>;
> -			};
> -		};
> -
> -		sata1: ahci@...00 {
> -			compatible = "brcm,iproc-ahci", "generic-ahci";
> -			reg = <0x00010000 0x1000>;
> -			reg-names = "ahci";
> -			interrupts = <GIC_SPI 323 IRQ_TYPE_LEVEL_HIGH>;
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			status = "disabled";
> -
> -			sata1_port0: sata-port@0 {
> -				reg = <0>;
> -				phys = <&sata1_phy0>;
> -				phy-names = "sata-phy";
> -			};
> -		};
> -
> -		sata_phy1: sata_phy@...00 {
> -			compatible = "brcm,iproc-sr-sata-phy";
> -			reg = <0x00012100 0x1000>;
> -			reg-names = "phy";
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			status = "disabled";
> -
> -			sata1_phy0: sata-phy@0 {
> -				reg = <0>;
> -				#phy-cells = <0>;
> -			};
> -		};
> -
> -		sata2: ahci@...00 {
> -			compatible = "brcm,iproc-ahci", "generic-ahci";
> -			reg = <0x00020000 0x1000>;
> -			reg-names = "ahci";
> -			interrupts = <GIC_SPI 325 IRQ_TYPE_LEVEL_HIGH>;
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			status = "disabled";
> -
> -			sata2_port0: sata-port@0 {
> -				reg = <0>;
> -				phys = <&sata2_phy0>;
> -				phy-names = "sata-phy";
> -			};
> -		};
> -
> -		sata_phy2: sata_phy@...00 {
> -			compatible = "brcm,iproc-sr-sata-phy";
> -			reg = <0x00022100 0x1000>;
> -			reg-names = "phy";
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			status = "disabled";
> -
> -			sata2_phy0: sata-phy@0 {
> -				reg = <0>;
> -				#phy-cells = <0>;
> -			};
> -		};
> -
> -		sata3: ahci@...00 {
> -			compatible = "brcm,iproc-ahci", "generic-ahci";
> -			reg = <0x00030000 0x1000>;
> -			reg-names = "ahci";
> -			interrupts = <GIC_SPI 327 IRQ_TYPE_LEVEL_HIGH>;
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			status = "disabled";
> -
> -			sata3_port0: sata-port@0 {
> -				reg = <0>;
> -				phys = <&sata3_phy0>;
> -				phy-names = "sata-phy";
> -			};
> -		};
> -
> -		sata_phy3: sata_phy@...00 {
> -			compatible = "brcm,iproc-sr-sata-phy";
> -			reg = <0x00032100 0x1000>;
> -			reg-names = "phy";
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			status = "disabled";
> -
> -			sata3_phy0: sata-phy@0 {
> -				reg = <0>;
> -				#phy-cells = <0>;
> -			};
> -		};
> -
> -		sata4: ahci@...000 {
> -			compatible = "brcm,iproc-ahci", "generic-ahci";
> -			reg = <0x00100000 0x1000>;
> -			reg-names = "ahci";
> -			interrupts = <GIC_SPI 329 IRQ_TYPE_LEVEL_HIGH>;
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			status = "disabled";
> -
> -			sata4_port0: sata-port@0 {
> -				reg = <0>;
> -				phys = <&sata4_phy0>;
> -				phy-names = "sata-phy";
> -			};
> -		};
> -
> -		sata_phy4: sata_phy@...100 {
> -			compatible = "brcm,iproc-sr-sata-phy";
> -			reg = <0x00102100 0x1000>;
> -			reg-names = "phy";
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			status = "disabled";
> -
> -			sata4_phy0: sata-phy@0 {
> -				reg = <0>;
> -				#phy-cells = <0>;
> -			};
> -		};
> -
> -		sata5: ahci@...000 {
> -			compatible = "brcm,iproc-ahci", "generic-ahci";
> -			reg = <0x00110000 0x1000>;
> -			reg-names = "ahci";
> -			interrupts = <GIC_SPI 331 IRQ_TYPE_LEVEL_HIGH>;
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			status = "disabled";
> -
> -			sata5_port0: sata-port@0 {
> -				reg = <0>;
> -				phys = <&sata5_phy0>;
> -				phy-names = "sata-phy";
> -			};
> -		};
> -
> -		sata_phy5: sata_phy@...100 {
> -			compatible = "brcm,iproc-sr-sata-phy";
> -			reg = <0x00112100 0x1000>;
> -			reg-names = "phy";
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			status = "disabled";
> -
> -			sata5_phy0: sata-phy@0 {
> -				reg = <0>;
> -				#phy-cells = <0>;
> -			};
> -		};
> -
> -		sata6: ahci@...000 {
> -			compatible = "brcm,iproc-ahci", "generic-ahci";
> -			reg = <0x00120000 0x1000>;
> -			reg-names = "ahci";
> -			interrupts = <GIC_SPI 333 IRQ_TYPE_LEVEL_HIGH>;
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			status = "disabled";
> -
> -			sata6_port0: sata-port@0 {
> -				reg = <0>;
> -				phys = <&sata6_phy0>;
> -				phy-names = "sata-phy";
> -			};
> -		};
> -
> -		sata_phy6: sata_phy@...100 {
> -			compatible = "brcm,iproc-sr-sata-phy";
> -			reg = <0x00122100 0x1000>;
> -			reg-names = "phy";
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			status = "disabled";
> -
> -			sata6_phy0: sata-phy@0 {
> -				reg = <0>;
> -				#phy-cells = <0>;
> -			};
> -		};
> -
> -		sata7: ahci@...000 {
> -			compatible = "brcm,iproc-ahci", "generic-ahci";
> -			reg = <0x00130000 0x1000>;
> -			reg-names = "ahci";
> -			interrupts = <GIC_SPI 335 IRQ_TYPE_LEVEL_HIGH>;
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			status = "disabled";
> -
> -			sata7_port0: sata-port@0 {
> -				reg = <0>;
> -				phys = <&sata7_phy0>;
> -				phy-names = "sata-phy";
> -			};
> -		};
> -
> -		sata_phy7: sata_phy@...100 {
> -			compatible = "brcm,iproc-sr-sata-phy";
> -			reg = <0x00132100 0x1000>;
> -			reg-names = "phy";
> -			#address-cells = <1>;
> -			#size-cells = <0>;
> -			status = "disabled";
> -
> -			sata7_phy0: sata-phy@0 {
> -				reg = <0>;
> -				#phy-cells = <0>;
> -			};
> -		};
> -	};
> diff --git a/arch/arm64/boot/dts/broadcom/stingray/stingray.dtsi b/arch/arm64/boot/dts/broadcom/stingray/stingray.dtsi
> index b425b12c3ed2..2ffb2c92182a 100644
> --- a/arch/arm64/boot/dts/broadcom/stingray/stingray.dtsi
> +++ b/arch/arm64/boot/dts/broadcom/stingray/stingray.dtsi
> @@ -285,7 +285,6 @@
>  	};
>  
>  	#include "stingray-fs4.dtsi"
> -	#include "stingray-sata.dtsi"
>  	#include "stingray-pcie.dtsi"
>  	#include "stingray-usb.dtsi"
>  
> @@ -309,12 +308,6 @@
>  				#size-cells = <0>;
>  			};
>  
> -			mdio@2 { /* SATA */
> -				reg = <0x2>;
> -				#address-cells = <1>;
> -				#size-cells = <0>;
> -			};
> -
>  			mdio@3 { /* USB */
>  				reg = <0x3>;
>  				#address-cells = <1>;
> 

The entire change looks good to me!

Acked-by: Ray Jui <ray.jui@...adcom.com>

Download attachment "smime.p7s" of type "application/pkcs7-signature" (4151 bytes)

Powered by blists - more mailing lists