| 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
| ||
|
Message-ID: <640bd260-3253-d36d-57c1-ab1ca2e08ddc@broadcom.com>
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