From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-03.1984.is (mail-03.1984.is [93.95.224.70]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 5393621E70D31 for ; Tue, 29 Aug 2017 12:23:50 -0700 (PDT) Received: from localhost by mail-03.1984.is with esmtpsa (TLSv1.2:DHE-RSA-AES128-SHA:128) (Exim 4.84_2) (envelope-from ) id 1dmm9Z-00012K-3b; Tue, 29 Aug 2017 19:26:25 +0000 To: Ard Biesheuvel , edk2-devel@lists.01.org References: <20170829162751.25226-1-ard.biesheuvel@linaro.org> Cc: leif.lindholm@linaro.org, liming.gao@intel.com, yonghong.zhu@intel.com From: Alan Ott Message-ID: Date: Tue, 29 Aug 2017 15:26:21 -0400 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <20170829162751.25226-1-ard.biesheuvel@linaro.org> Subject: Re: [PATCH edk2-platforms] Platform/OverdriveBoard: move device tree compilation into build X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 29 Aug 2017 19:23:51 -0000 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit On 08/29/2017 12:27 PM, Ard Biesheuvel wrote: > Use the proposed BaseTools support for device tree compilation to > build the device tree binary from source at build time. Give it its > own .inf and a build rule so the tools take care of everything. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Ard Biesheuvel > --- > Platform/AMD/OverdriveBoard/FdtBlob/styx-overdrive.dtb | Bin 9357 -> 0 bytes > Platform/AMD/OverdriveBoard/FdtBlob/styx-overdrive.dts | 510 -------------------- > Platform/AMD/OverdriveBoard/OverdriveBoard.dsc | 1 + > Platform/AMD/OverdriveBoard/OverdriveBoard.fdf | 9 +- > Platform/AMD/OverdriveBoard/OverdriveBoardDeviceTree/OverdriveBoardDeviceTree.dts | 487 +++++++++++++++++++ > Platform/AMD/OverdriveBoard/OverdriveBoardDeviceTree/OverdriveBoardDeviceTree.inf | 28 ++ > 6 files changed, 521 insertions(+), 514 deletions(-) I like this in principle, but what's the 23 line difference in the DTS and/or should it be in a separate patch from the patch that moves the file so the difference is clear? > > diff --git a/Platform/AMD/OverdriveBoard/FdtBlob/styx-overdrive.dtb b/Platform/AMD/OverdriveBoard/FdtBlob/styx-overdrive.dtb > deleted file mode 100644 > index c8e5fd980bce..000000000000 > Binary files a/Platform/AMD/OverdriveBoard/FdtBlob/styx-overdrive.dtb and /dev/null differ > diff --git a/Platform/AMD/OverdriveBoard/FdtBlob/styx-overdrive.dts b/Platform/AMD/OverdriveBoard/FdtBlob/styx-overdrive.dts > deleted file mode 100644 > index 4039f666004a..000000000000 > --- a/Platform/AMD/OverdriveBoard/FdtBlob/styx-overdrive.dts > +++ /dev/null > @@ -1,510 +0,0 @@ > -/* > - * DTS file for AMD Seattle (Rev.B) Overdrive Development Board > - * > - * Copyright 2015 - 2016 ADVANCED MICRO DEVICES, INC. All Rights Reserved. > - * > - * This program and the accompanying materials are licensed and made available > - * under the terms and conditions of the BSD License which accompanies this > - * distribution. The full text of the license may be found at > - * http://opensource.org/licenses/bsd-license.php > - * > - * THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, > - * WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR > - * IMPLIED. > - * > - */ > - > -/dts-v1/; > - > -/ { > - model = "AMD Seattle (Rev.B) Development Board (Overdrive)"; > - compatible = "amd,seattle-overdrive", "amd,seattle"; > - interrupt-parent = <0x1>; > - #address-cells = <0x2>; > - #size-cells = <0x2>; > - > - interrupt-controller@e1101000 { > - compatible = "arm,gic-400", "arm,cortex-a15-gic"; > - interrupt-controller; > - #interrupt-cells = <0x3>; > - #address-cells = <0x2>; > - #size-cells = <0x2>; > - reg = <0x0 0xe1110000 0x0 0x1000>, > - <0x0 0xe112f000 0x0 0x2000>, > - <0x0 0xe1140000 0x0 0x2000>, > - <0x0 0xe1160000 0x0 0x2000>; > - interrupts = <0x1 0x9 0xf04>; > - ranges = <0x0 0x0 0x0 0xe1100000 0x0 0x100000>; > - linux,phandle = <0x1>; > - phandle = <0x1>; > - > - v2m@e0080000 { > - compatible = "arm,gic-v2m-frame"; > - msi-controller; > - reg = <0x0 0x80000 0x0 0x1000>; > - linux,phandle = <0x4>; > - phandle = <0x4>; > - }; > - }; > - > - timer { > - compatible = "arm,armv8-timer"; > - interrupts = <0x1 0xd 0xff04>, > - <0x1 0xe 0xff04>, > - <0x1 0xb 0xff04>, > - <0x1 0xa 0xff04>; > - }; > - > - smb { > - compatible = "simple-bus"; > - #address-cells = <0x2>; > - #size-cells = <0x2>; > - ranges; > - /* > - * dma-ranges is 40-bit address space containing: > - * - GICv2m MSI register is at 0xe0080000 > - * - DRAM range [0x8000000000 to 0xffffffffff] > - */ > - dma-ranges = <0x0 0x0 0x0 0x0 0x100 0x0>; > - > - clk100mhz_0 { > - compatible = "fixed-clock"; > - #clock-cells = <0x0>; > - clock-frequency = <100000000>; > - clock-output-names = "adl3clk_100mhz"; > - }; > - > - clk375mhz { > - compatible = "fixed-clock"; > - #clock-cells = <0x0>; > - clock-frequency = <375000000>; > - clock-output-names = "ccpclk_375mhz"; > - }; > - > - clk333mhz { > - compatible = "fixed-clock"; > - #clock-cells = <0x0>; > - clock-frequency = <333000000>; > - clock-output-names = "sataclk_333mhz"; > - linux,phandle = <0x2>; > - phandle = <0x2>; > - }; > - > - clk500mhz_0 { > - compatible = "fixed-clock"; > - #clock-cells = <0x0>; > - clock-frequency = <500000000>; > - clock-output-names = "pcieclk_500mhz"; > - }; > - > - clk500mhz_1 { > - compatible = "fixed-clock"; > - #clock-cells = <0x0>; > - clock-frequency = <500000000>; > - clock-output-names = "dmaclk_500mhz"; > - }; > - > - clk250mhz_4 { > - compatible = "fixed-clock"; > - #clock-cells = <0x0>; > - clock-frequency = <250000000>; > - clock-output-names = "miscclk_250mhz"; > - linux,phandle = <0xd>; > - phandle = <0xd>; > - }; > - > - clk100mhz_1 { > - compatible = "fixed-clock"; > - #clock-cells = <0x0>; > - clock-frequency = <100000000>; > - clock-output-names = "uartspiclk_100mhz"; > - linux,phandle = <0x3>; > - phandle = <0x3>; > - }; > - > - sata0_smmu: smmu@e0200000 { > - compatible = "arm,mmu-401"; > - reg = <0 0xe0200000 0 0x10000>; > - #global-interrupts = <1>; > - interrupts = /* Uses combined intr for both > - * global and context > - */ > - <0 332 4>, > - <0 332 4>; > - #iommu-cells = <2>; > - dma-coherent; > - }; > - > - sata1_smmu: smmu@e0c00000 { > - compatible = "arm,mmu-401"; > - reg = <0 0xe0c00000 0 0x10000>; > - #global-interrupts = <1>; > - interrupts = /* Uses combined intr for both > - * global and context > - */ > - <0 331 4>, > - <0 331 4>; > - #iommu-cells = <2>; > - dma-coherent; > - }; > - > - sata@e0300000 { > - compatible = "snps,dwc-ahci"; > - reg = <0x0 0xe0300000 0x0 0xf0000>; > - interrupts = <0x0 0x163 0x4>; > - clocks = <0x2>; > - dma-coherent; > - iommus = <&sata0_smmu 0x00 0x1f>; /* 0-31 */ > - }; > - > - sata@e0d00000 { > - status = "disabled"; > - compatible = "snps,dwc-ahci"; > - reg = <0x0 0xe0d00000 0x0 0xf0000>; > - interrupts = <0x0 0x162 0x4>; > - clocks = <0x2>; > - dma-coherent; > - iommus = <&sata1_smmu 0x00 0x1f>; /* 0-31 */ > - }; > - > - i2c@e1000000 { > - compatible = "snps,designware-i2c"; > - reg = <0x0 0xe1000000 0x0 0x1000>; > - interrupts = <0x0 0x165 0x4>; > - clocks = <0xd>; > - }; > - > - i2c@e0050000 { > - compatible = "snps,designware-i2c"; > - reg = <0x0 0xe0050000 0x0 0x1000>; > - interrupts = <0x0 0x154 0x4>; > - clocks = <0xd>; > - }; > - > - serial@e1010000 { > - compatible = "arm,pl011", "arm,primecell"; > - reg = <0x0 0xe1010000 0x0 0x1000>; > - interrupts = <0x0 0x148 0x4>; > - clocks = <0x3 0x3>; > - clock-names = "uartclk", "apb_pclk"; > - }; > - > - ssp@e1020000 { > - compatible = "arm,pl022", "arm,primecell"; > - reg = <0x0 0xe1020000 0x0 0x1000>; > - spi-controller; > - interrupts = <0x0 0x14a 0x4>; > - clocks = <0x3>; > - clock-names = "apb_pclk"; > - }; > - > - ssp@e1030000 { > - compatible = "arm,pl022", "arm,primecell"; > - reg = <0x0 0xe1030000 0x0 0x1000>; > - spi-controller; > - interrupts = <0x0 0x149 0x4>; > - clocks = <0x3>; > - clock-names = "apb_pclk"; > - num-cs = <0x1>; > - #address-cells = <0x1>; > - #size-cells = <0x0>; > - > - sdcard@0 { > - compatible = "mmc-spi-slot"; > - reg = <0x0>; > - spi-max-frequency = <20000000>; > - voltage-ranges = <3200 3400>; > - pl022,hierarchy = <0x0>; > - pl022,interface = <0x0>; > - pl022,com-mode = <0x0>; > - pl022,rx-level-trig = <0x0>; > - pl022,tx-level-trig = <0x0>; > - }; > - }; > - > - gpio@e1050000 { /* [0 : 7] */ > - compatible = "arm,pl061", "arm,primecell"; > - #gpio-cells = <0x2>; > - reg = <0x0 0xe1050000 0x0 0x1000>; > - gpio-controller; > - interrupt-controller; > - #interrupt-cells = <0x2>; > - interrupts = <0x0 0x166 0x4>; > - clocks = <0x3>; > - clock-names = "apb_pclk"; > - }; > - > - gpio@e0020000 { /* [8 : 15] */ > - status = "disabled"; > - compatible = "arm,pl061", "arm,primecell"; > - #gpio-cells = <0x2>; > - reg = <0x0 0xe0020000 0x0 0x1000>; > - gpio-controller; > - interrupt-controller; > - #interrupt-cells = <0x2>; > - interrupts = <0x0 0x16e 0x4>; > - clocks = <0x3>; > - clock-names = "apb_pclk"; > - }; > - > - gpio@e0030000 { /* [16 : 23] */ > - status = "disabled"; > - compatible = "arm,pl061", "arm,primecell"; > - #gpio-cells = <0x2>; > - reg = <0x0 0xe0030000 0x0 0x1000>; > - gpio-controller; > - interrupt-controller; > - #interrupt-cells = <0x2>; > - interrupts = <0x0 0x16d 0x4>; > - clocks = <0x3>; > - clock-names = "apb_pclk"; > - }; > - > - gpio@e0080000 { /* [24] */ > - compatible = "arm,pl061", "arm,primecell"; > - #gpio-cells = <0x2>; > - reg = <0x0 0xe0080000 0x0 0x1000>; > - gpio-controller; > - interrupt-controller; > - #interrupt-cells = <0x2>; > - interrupts = <0x0 0x169 0x4>; > - clocks = <0x3>; > - clock-names = "apb_pclk"; > - }; > - > - ccp: ccp@e0100000 { > - compatible = "amd,ccp-seattle-v1a"; > - reg = <0x0 0xe0100000 0x0 0x10000>; > - interrupts = <0x0 0x3 0x4>; > - dma-coherent; > - amd,zlib-support = <0x1>; > - }; > - > - pcie: pcie@f0000000 { > - compatible = "pci-host-ecam-generic"; > - #address-cells = <0x3>; > - #size-cells = <0x2>; > - #interrupt-cells = <0x1>; > - iommu-map = <0x0 &pcie_smmu 0x0 0x10000>; > - device_type = "pci"; > - bus-range = <0x0 0x7f>; > - msi-parent = <0x4>; > - reg = <0x0 0xf0000000 0x0 0x10000000>; > - interrupt-map-mask = <0xff00 0x0 0x0 0x7>; > - interrupt-map = <0x1100 0x0 0x0 0x1 0x1 0x0 0x0 0x0 0x120 0x1>, > - <0x1100 0x0 0x0 0x2 0x1 0x0 0x0 0x0 0x121 0x1>, > - <0x1100 0x0 0x0 0x3 0x1 0x0 0x0 0x0 0x122 0x1>, > - <0x1100 0x0 0x0 0x4 0x1 0x0 0x0 0x0 0x123 0x1>, > - > - <0x1200 0x0 0x0 0x1 0x1 0x0 0x0 0x0 0x124 0x1>, > - <0x1200 0x0 0x0 0x2 0x1 0x0 0x0 0x0 0x125 0x1>, > - <0x1200 0x0 0x0 0x3 0x1 0x0 0x0 0x0 0x126 0x1>, > - <0x1200 0x0 0x0 0x4 0x1 0x0 0x0 0x0 0x127 0x1>, > - > - <0x1300 0x0 0x0 0x1 0x1 0x0 0x0 0x0 0x128 0x1>, > - <0x1300 0x0 0x0 0x2 0x1 0x0 0x0 0x0 0x129 0x1>, > - <0x1300 0x0 0x0 0x3 0x1 0x0 0x0 0x0 0x12a 0x1>, > - <0x1300 0x0 0x0 0x4 0x1 0x0 0x0 0x0 0x12b 0x1>; > - dma-coherent; > - dma-ranges = <0x43000000 0x0 0x0 0x0 0x0 0x100 0x0>; > - ranges = <0x1000000 0x0 0x00000000 0x0 0xefff0000 0x00 0x00010000>, /* I/O Memory (size=64K) */ > - <0x2000000 0x0 0x40000000 0x0 0x40000000 0x00 0x80000000>, /* 32-bit MMIO (size=2G) */ > - <0x3000000 0x1 0x00000000 0x1 0x00000000 0x7f 0x00000000>; /* 64-bit MMIO (size= 124G) */ > - }; > - > - pcie_smmu: smmu@e0a00000 { > - compatible = "arm,mmu-401"; > - reg = <0 0xe0a00000 0 0x10000>; > - #global-interrupts = <1>; > - interrupts = /* Uses combined intr for both > - * global and context > - */ > - <0 333 4>, > - <0 333 4>; > - #iommu-cells = <1>; > - dma-coherent; > - }; > - > - ccn@0xe8000000 { > - compatible = "arm,ccn-504"; > - reg = <0x0 0xe8000000 0x0 0x1000000>; > - interrupts = <0x0 0x17c 0x4>; > - }; > - > - gwdt@e0bb0000 { > - status = "disabled"; > - compatible = "arm,sbsa-gwdt"; > - reg = <0x0 0xe0bb0000 0x0 0x10000 > - 0x0 0xe0bc0000 0x0 0x10000>; > - reg-names = "refresh", "control"; > - interrupts = <0x0 0x151 0x4>; > - interrupt-names = "ws0"; > - }; > - > - kcs@e0010000 { > - status = "disabled"; > - compatible = "ipmi-kcs"; > - device_type = "ipmi"; > - reg = <0x0 0xe0010000 0 0x8>; > - interrupts = <0 389 4>; > - interrupt-names = "ipmi_kcs"; > - reg-size = <1>; > - reg-spacing = <4>; > - }; > - > - clk250mhz_0 { > - compatible = "fixed-clock"; > - #clock-cells = <0x0>; > - clock-frequency = <250000000>; > - clock-output-names = "xgmacclk0_dma_250mhz"; > - linux,phandle = <0x5>; > - phandle = <0x5>; > - }; > - > - clk250mhz_1 { > - compatible = "fixed-clock"; > - #clock-cells = <0x0>; > - clock-frequency = <250000000>; > - clock-output-names = "xgmacclk0_ptp_250mhz"; > - linux,phandle = <0x6>; > - phandle = <0x6>; > - }; > - > - clk250mhz_2 { > - compatible = "fixed-clock"; > - #clock-cells = <0x0>; > - clock-frequency = <250000000>; > - clock-output-names = "xgmacclk1_dma_250mhz"; > - linux,phandle = <0x7>; > - phandle = <0x7>; > - }; > - > - clk250mhz_3 { > - compatible = "fixed-clock"; > - #clock-cells = <0x0>; > - clock-frequency = <250000000>; > - clock-output-names = "xgmacclk1_ptp_250mhz"; > - linux,phandle = <0x8>; > - phandle = <0x8>; > - }; > - > - phy@e1240800 { > - status = "disabled"; > - compatible = "amd,xgbe-phy-seattle-v1a"; > - reg = <0x0 0xe1240800 0x0 0x0400>, /* SERDES RX/TX0 */ > - <0x0 0xe1250000 0x0 0x0060>, /* SERDES IR 1/2 */ > - <0x0 0xe12500f8 0x0 0x0004>; /* SERDES IR 2/2 */ > - interrupts = <0x0 0x143 0x4>; > - amd,speed-set = <0x0>; > - amd,serdes-blwc = <0x1 0x1 0x0>; > - amd,serdes-cdr-rate = <0x2 0x2 0x7>; > - amd,serdes-pq-skew = <0xa 0xa 0x12>; > - amd,serdes-tx-amp = <0xf 0xf 0xa>; > - amd,serdes-dfe-tap-config = <0x3 0x3 0x1>; > - amd,serdes-dfe-tap-enable = <0x0 0x0 0x7f>; > - linux,phandle = <0x9>; > - phandle = <0x9>; > - }; > - > - phy@e1240c00 { > - status = "disabled"; > - compatible = "amd,xgbe-phy-seattle-v1a"; > - reg = <0x0 0xe1240c00 0x0 0x0400>, /* SERDES RX/TX0 */ > - <0x0 0xe1250080 0x0 0x0060>, /* SERDES IR 1/2 */ > - <0x0 0xe12500fc 0x0 0x0004>; /* SERDES IR 2/2 */ > - interrupts = <0x0 0x142 0x4>; > - amd,speed-set = <0x0>; > - amd,serdes-blwc = <0x1 0x1 0x0>; > - amd,serdes-cdr-rate = <0x2 0x2 0x7>; > - amd,serdes-pq-skew = <0xa 0xa 0x12>; > - amd,serdes-tx-amp = <0xf 0xf 0xa>; > - amd,serdes-dfe-tap-config = <0x3 0x3 0x1>; > - amd,serdes-dfe-tap-enable = <0x0 0x0 0x7f>; > - linux,phandle = <0xa>; > - phandle = <0xa>; > - }; > - > - xgmac0_smmu: smmu@e0600000 { > - compatible = "arm,mmu-401"; > - reg = <0 0xe0600000 0 0x10000>; > - #global-interrupts = <1>; > - interrupts = /* Uses combined intr for both > - * global and context > - */ > - <0 336 4>, > - <0 336 4>; > - #iommu-cells = <2>; > - dma-coherent; > - }; > - > - xgmac1_smmu: smmu@e0800000 { > - compatible = "arm,mmu-401"; > - reg = <0 0xe0800000 0 0x10000>; > - #global-interrupts = <1>; > - interrupts = /* Uses combined intr for both > - * global and context > - */ > - <0 335 4>, > - <0 335 4>; > - #iommu-cells = <2>; > - dma-coherent; > - }; > - > - xgmac@e0700000 { > - status = "disabled"; > - compatible = "amd,xgbe-seattle-v1a"; > - reg = <0x0 0xe0700000 0x0 0x80000 0x0 0xe0780000 0x0 0x80000>; > - interrupts = <0x0 0x145 0x4>, > - <0x0 0x15a 0x1>, > - <0x0 0x15b 0x1>, > - <0x0 0x15c 0x1>, > - <0x0 0x15d 0x1>; > - amd,per-channel-interrupt; > - mac-address = [02 a1 a2 a3 a4 a5]; > - clocks = <0x5 0x6>; > - clock-names = "dma_clk", "ptp_clk"; > - phy-handle = <0x9>; > - phy-mode = "xgmii"; > - dma-coherent; > - iommus = <&xgmac0_smmu 0x00 0x1f>; /* 0-31 */ > - linux,phandle = <0xb>; > - phandle = <0xb>; > - }; > - > - xgmac@e0900000 { > - status = "disabled"; > - compatible = "amd,xgbe-seattle-v1a"; > - reg = <0x0 0xe0900000 0x0 0x80000 0x0 0xe0980000 0x0 0x80000>; > - interrupts = <0x0 0x144 0x4>, > - <0x0 0x155 0x1>, > - <0x0 0x156 0x1>, > - <0x0 0x157 0x1>, > - <0x0 0x158 0x1>; > - amd,per-channel-interrupt; > - mac-address = [02 b1 b2 b3 b4 b5]; > - clocks = <0x7 0x8>; > - clock-names = "dma_clk", "ptp_clk"; > - phy-handle = <0xa>; > - phy-mode = "xgmii"; > - dma-coherent; > - iommus = <&xgmac1_smmu 0x00 0x1f>; /* 0-31 */ > - linux,phandle = <0xc>; > - phandle = <0xc>; > - }; > - }; > - > - chosen { > - stdout-path = "/smb/serial@e1010000"; > - /* Note: > - * Linux support for pci-probe-only DT is not > - * stable. Disable this for now and let Linux > - * take care of the resource assignment. > - */ > - // linux,pci-probe-only; > - }; > - > - psci { > - compatible = "arm,psci-0.2", "arm,psci"; > - method = "smc"; > - }; > -}; > diff --git a/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc b/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc > index bc69c586e929..081a4f3cd002 100644 > --- a/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc > +++ b/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc > @@ -600,6 +600,7 @@ DEFINE DO_FLASHER = FALSE > # > # FDT support > # > + Platform/AMD/OverdriveBoard/OverdriveBoardDeviceTree/OverdriveBoardDeviceTree.inf > EmbeddedPkg/Drivers/DtPlatformDxe/DtPlatformDxe.inf { > > FdtLib|EmbeddedPkg/Library/FdtLib/FdtLib.inf > diff --git a/Platform/AMD/OverdriveBoard/OverdriveBoard.fdf b/Platform/AMD/OverdriveBoard/OverdriveBoard.fdf > index 23e57befcdd9..aa2a1a6fa053 100644 > --- a/Platform/AMD/OverdriveBoard/OverdriveBoard.fdf > +++ b/Platform/AMD/OverdriveBoard/OverdriveBoard.fdf > @@ -145,10 +145,7 @@ READ_LOCK_STATUS = TRUE > # FDT support > # > INF EmbeddedPkg/Drivers/DtPlatformDxe/DtPlatformDxe.inf > - > - FILE FREEFORM = 25462CDA-221F-47DF-AC1D-259CFAA4E326 { > - SECTION RAW = Platform/AMD/OverdriveBoard/FdtBlob/styx-overdrive.dtb > - } > + INF RuleOverride = DTB Platform/AMD/OverdriveBoard/OverdriveBoardDeviceTree/OverdriveBoardDeviceTree.inf > > # > # PCI support > @@ -413,3 +410,7 @@ READ_LOCK_STATUS = TRUE > RAW ASL |.aml > } > > +[Rule.Common.USER_DEFINED.DTB] > + FILE FREEFORM = $(NAMED_GUID) { > + RAW BIN |.dtb > + } > diff --git a/Platform/AMD/OverdriveBoard/OverdriveBoardDeviceTree/OverdriveBoardDeviceTree.dts b/Platform/AMD/OverdriveBoard/OverdriveBoardDeviceTree/OverdriveBoardDeviceTree.dts > new file mode 100644 > index 000000000000..2c05fdbb8b71 > --- /dev/null > +++ b/Platform/AMD/OverdriveBoard/OverdriveBoardDeviceTree/OverdriveBoardDeviceTree.dts > @@ -0,0 +1,487 @@ > +/* > + * DTS file for AMD Seattle (Rev.B) Overdrive Development Board > + * > + * Copyright 2015 - 2016 ADVANCED MICRO DEVICES, INC. All Rights Reserved. > + * Copyright 2015 - 2017 Linaro, Ltd. All Rights Reserved. > + * > + * This program and the accompanying materials are licensed and made available > + * under the terms and conditions of the BSD License which accompanies this > + * distribution. The full text of the license may be found at > + * http://opensource.org/licenses/bsd-license.php > + * > + * THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, > + * WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR > + * IMPLIED. > + * > + */ > + > +#define GIC_SPI 0 > +#define GIC_PPI 1 > + > +#define IRQ_TYPE_NONE 0 > +#define IRQ_TYPE_EDGE_RISING 1 > +#define IRQ_TYPE_EDGE_FALLING 2 > +#define IRQ_TYPE_LEVEL_HIGH 4 > +#define IRQ_TYPE_LEVEL_LOW 8 > + > +#define GIC_CPU_MASK(num) (((1 << (num)) - 1) << 8) > + > +/dts-v1/; > + > +/ { > + model = "AMD Seattle (Rev.B) Development Board (Overdrive)"; > + compatible = "amd,seattle-overdrive", "amd,seattle"; > + interrupt-parent = <&gic>; > + #address-cells = <0x2>; > + #size-cells = <0x2>; > + > + gic: interrupt-controller@e1101000 { > + compatible = "arm,gic-400", "arm,cortex-a15-gic"; > + interrupt-controller; > + #interrupt-cells = <0x3>; > + #address-cells = <0x2>; > + #size-cells = <0x2>; > + reg = <0x0 0xe1110000 0x0 0x1000>, > + <0x0 0xe112f000 0x0 0x2000>, > + <0x0 0xe1140000 0x0 0x2000>, > + <0x0 0xe1160000 0x0 0x2000>; > + interrupts = ; > + ranges = <0x0 0x0 0x0 0xe1100000 0x0 0x100000>; > + > + msi: v2m@e0080000 { > + compatible = "arm,gic-v2m-frame"; > + reg = <0x0 0x80000 0x0 0x1000>; > + msi-controller; > + }; > + }; > + > + timer { > + compatible = "arm,armv8-timer"; > + interrupts = , > + , > + , > + ; > + }; > + > + smb { > + compatible = "simple-bus"; > + #address-cells = <0x2>; > + #size-cells = <0x2>; > + ranges; > + /* > + * dma-ranges is 40-bit address space containing: > + * - GICv2m MSI register is at 0xe0080000 > + * - DRAM range [0x8000000000 to 0xffffffffff] > + */ > + dma-ranges = <0x0 0x0 0x0 0x0 0x100 0x0>; > + > + adl3clk: clk100mhz_0 { > + compatible = "fixed-clock"; > + #clock-cells = <0x0>; > + clock-frequency = <100000000>; > + clock-output-names = "adl3clk_100mhz"; > + }; > + > + ccpclk: clk375mhz { > + compatible = "fixed-clock"; > + #clock-cells = <0x0>; > + clock-frequency = <375000000>; > + clock-output-names = "ccpclk_375mhz"; > + }; > + > + sataclk: clk333mhz { > + compatible = "fixed-clock"; > + #clock-cells = <0x0>; > + clock-frequency = <333000000>; > + clock-output-names = "sataclk_333mhz"; > + }; > + > + pcieclk: clk500mhz_0 { > + compatible = "fixed-clock"; > + #clock-cells = <0x0>; > + clock-frequency = <500000000>; > + clock-output-names = "pcieclk_500mhz"; > + }; > + > + dmaclk: clk500mhz_1 { > + compatible = "fixed-clock"; > + #clock-cells = <0x0>; > + clock-frequency = <500000000>; > + clock-output-names = "dmaclk_500mhz"; > + }; > + > + miscclk: clk250mhz_4 { > + compatible = "fixed-clock"; > + #clock-cells = <0x0>; > + clock-frequency = <250000000>; > + clock-output-names = "miscclk_250mhz"; > + }; > + > + uartspiclk: clk100mhz_1 { > + compatible = "fixed-clock"; > + #clock-cells = <0x0>; > + clock-frequency = <100000000>; > + clock-output-names = "uartspiclk_100mhz"; > + }; > + > + sata0_smmu: smmu@e0200000 { > + compatible = "arm,mmu-401"; > + reg = <0 0xe0200000 0 0x10000>; > + > + /* Uses combined intr for both global and context */ > + #global-interrupts = <1>; > + interrupts = , > + ; > + #iommu-cells = <2>; > + dma-coherent; > + }; > + > + sata1_smmu: smmu@e0c00000 { > + compatible = "arm,mmu-401"; > + reg = <0 0xe0c00000 0 0x10000>; > + > + /* Uses combined intr for both global and context */ > + #global-interrupts = <1>; > + interrupts = , > + ; > + #iommu-cells = <2>; > + dma-coherent; > + }; > + > + sata@e0300000 { > + compatible = "snps,dwc-ahci"; > + reg = <0x0 0xe0300000 0x0 0xf0000>; > + interrupts = ; > + clocks = <&sataclk>; > + dma-coherent; > + iommus = <&sata0_smmu 0x00 0x1f>; /* 0-31 */ > + }; > + > + sata@e0d00000 { > + status = "disabled"; > + compatible = "snps,dwc-ahci"; > + reg = <0x0 0xe0d00000 0x0 0xf0000>; > + interrupts = ; > + clocks = <&sataclk>; > + dma-coherent; > + iommus = <&sata1_smmu 0x00 0x1f>; /* 0-31 */ > + }; > + > + i2c@e1000000 { > + compatible = "snps,designware-i2c"; > + reg = <0x0 0xe1000000 0x0 0x1000>; > + interrupts = ; > + clocks = <&miscclk>; > + }; > + > + i2c@e0050000 { > + compatible = "snps,designware-i2c"; > + reg = <0x0 0xe0050000 0x0 0x1000>; > + interrupts = ; > + clocks = <&miscclk>; > + }; > + > + serial@e1010000 { > + compatible = "arm,pl011", "arm,primecell"; > + reg = <0x0 0xe1010000 0x0 0x1000>; > + interrupts = ; > + clocks = <&uartspiclk &uartspiclk>; > + clock-names = "uartclk", "apb_pclk"; > + }; > + > + ssp@e1020000 { > + compatible = "arm,pl022", "arm,primecell"; > + reg = <0x0 0xe1020000 0x0 0x1000>; > + spi-controller; > + interrupts = ; > + clocks = <&uartspiclk>; > + clock-names = "apb_pclk"; > + }; > + > + ssp@e1030000 { > + compatible = "arm,pl022", "arm,primecell"; > + reg = <0x0 0xe1030000 0x0 0x1000>; > + spi-controller; > + interrupts = ; > + clocks = <&uartspiclk>; > + clock-names = "apb_pclk"; > + num-cs = <0x1>; > + #address-cells = <0x1>; > + #size-cells = <0x0>; > + > + sdcard@0 { > + compatible = "mmc-spi-slot"; > + reg = <0x0>; > + spi-max-frequency = <20000000>; > + voltage-ranges = <3200 3400>; > + pl022,hierarchy = <0x0>; > + pl022,interface = <0x0>; > + pl022,com-mode = <0x0>; > + pl022,rx-level-trig = <0x0>; > + pl022,tx-level-trig = <0x0>; > + }; > + }; > + > + gpio@e1050000 { /* [0 : 7] */ > + compatible = "arm,pl061", "arm,primecell"; > + #gpio-cells = <0x2>; > + reg = <0x0 0xe1050000 0x0 0x1000>; > + gpio-controller; > + interrupt-controller; > + #interrupt-cells = <0x2>; > + interrupts = ; > + clocks = <&uartspiclk>; > + clock-names = "apb_pclk"; > + }; > + > + gpio@e0020000 { /* [8 : 15] */ > + status = "disabled"; > + compatible = "arm,pl061", "arm,primecell"; > + #gpio-cells = <0x2>; > + reg = <0x0 0xe0020000 0x0 0x1000>; > + gpio-controller; > + interrupt-controller; > + #interrupt-cells = <0x2>; > + interrupts = ; > + clocks = <&uartspiclk>; > + clock-names = "apb_pclk"; > + }; > + > + gpio@e0030000 { /* [16 : 23] */ > + status = "disabled"; > + compatible = "arm,pl061", "arm,primecell"; > + #gpio-cells = <0x2>; > + reg = <0x0 0xe0030000 0x0 0x1000>; > + gpio-controller; > + interrupt-controller; > + #interrupt-cells = <0x2>; > + interrupts = ; > + clocks = <&uartspiclk>; > + clock-names = "apb_pclk"; > + }; > + > + gpio@e0080000 { /* [24] */ > + compatible = "arm,pl061", "arm,primecell"; > + #gpio-cells = <0x2>; > + reg = <0x0 0xe0080000 0x0 0x1000>; > + gpio-controller; > + interrupt-controller; > + #interrupt-cells = <0x2>; > + interrupts = ; > + clocks = <&uartspiclk>; > + clock-names = "apb_pclk"; > + }; > + > + ccp: ccp@e0100000 { > + compatible = "amd,ccp-seattle-v1a"; > + reg = <0x0 0xe0100000 0x0 0x10000>; > + interrupts = ; > + dma-coherent; > + amd,zlib-support = <0x1>; > + }; > + > + pcie: pcie@f0000000 { > + compatible = "pci-host-ecam-generic"; > + #address-cells = <0x3>; > + #size-cells = <0x2>; > + #interrupt-cells = <0x1>; > + iommu-map = <0x0 &pcie_smmu 0x0 0x10000>; > + device_type = "pci"; > + bus-range = <0x0 0x7f>; > + msi-parent = <&msi>; > + reg = <0x0 0xf0000000 0x0 0x10000000>; > + interrupt-map-mask = <0xff00 0x0 0x0 0x7>; > + interrupt-map = <0x1100 0x0 0x0 0x1 &gic 0x0 0x0 GIC_SPI 0x120 IRQ_TYPE_LEVEL_HIGH>, > + <0x1100 0x0 0x0 0x2 &gic 0x0 0x0 GIC_SPI 0x121 IRQ_TYPE_LEVEL_HIGH>, > + <0x1100 0x0 0x0 0x3 &gic 0x0 0x0 GIC_SPI 0x122 IRQ_TYPE_LEVEL_HIGH>, > + <0x1100 0x0 0x0 0x4 &gic 0x0 0x0 GIC_SPI 0x123 IRQ_TYPE_LEVEL_HIGH>, > + > + <0x1200 0x0 0x0 0x1 &gic 0x0 0x0 GIC_SPI 0x124 IRQ_TYPE_LEVEL_HIGH>, > + <0x1200 0x0 0x0 0x2 &gic 0x0 0x0 GIC_SPI 0x125 IRQ_TYPE_LEVEL_HIGH>, > + <0x1200 0x0 0x0 0x3 &gic 0x0 0x0 GIC_SPI 0x126 IRQ_TYPE_LEVEL_HIGH>, > + <0x1200 0x0 0x0 0x4 &gic 0x0 0x0 GIC_SPI 0x127 IRQ_TYPE_LEVEL_HIGH>, > + > + <0x1300 0x0 0x0 0x1 &gic 0x0 0x0 GIC_SPI 0x128 IRQ_TYPE_LEVEL_HIGH>, > + <0x1300 0x0 0x0 0x2 &gic 0x0 0x0 GIC_SPI 0x129 IRQ_TYPE_LEVEL_HIGH>, > + <0x1300 0x0 0x0 0x3 &gic 0x0 0x0 GIC_SPI 0x12a IRQ_TYPE_LEVEL_HIGH>, > + <0x1300 0x0 0x0 0x4 &gic 0x0 0x0 GIC_SPI 0x12b IRQ_TYPE_LEVEL_HIGH>; > + dma-coherent; > + dma-ranges = <0x43000000 0x0 0x0 0x0 0x0 0x100 0x0>; > + ranges = <0x1000000 0x0 0x00000000 0x0 0xefff0000 0x00 0x00010000>, /* I/O Memory (size=64K) */ > + <0x2000000 0x0 0x40000000 0x0 0x40000000 0x00 0x80000000>, /* 32-bit MMIO (size=2G) */ > + <0x3000000 0x1 0x00000000 0x1 0x00000000 0x7f 0x00000000>; /* 64-bit MMIO (size= 124G) */ > + }; > + > + pcie_smmu: smmu@e0a00000 { > + compatible = "arm,mmu-401"; > + reg = <0 0xe0a00000 0 0x10000>; > + > + /* Uses combined intr for both global and context */ > + #global-interrupts = <1>; > + interrupts = , > + ; > + #iommu-cells = <1>; > + dma-coherent; > + }; > + > + ccn@0xe8000000 { > + compatible = "arm,ccn-504"; > + reg = <0x0 0xe8000000 0x0 0x1000000>; > + interrupts = ; > + }; > + > + gwdt@e0bb0000 { > + status = "disabled"; > + compatible = "arm,sbsa-gwdt"; > + reg = <0x0 0xe0bb0000 0x0 0x10000 > + 0x0 0xe0bc0000 0x0 0x10000>; > + reg-names = "refresh", "control"; > + interrupts = ; > + interrupt-names = "ws0"; > + }; > + > + kcs@e0010000 { > + status = "disabled"; > + compatible = "ipmi-kcs"; > + device_type = "ipmi"; > + reg = <0x0 0xe0010000 0 0x8>; > + interrupts = ; > + interrupt-names = "ipmi_kcs"; > + reg-size = <1>; > + reg-spacing = <4>; > + }; > + > + xgmacclk0_dma: clk250mhz_0 { > + compatible = "fixed-clock"; > + #clock-cells = <0x0>; > + clock-frequency = <250000000>; > + clock-output-names = "xgmacclk0_dma_250mhz"; > + }; > + > + xgmacclk0_ptp: clk250mhz_1 { > + compatible = "fixed-clock"; > + #clock-cells = <0x0>; > + clock-frequency = <250000000>; > + clock-output-names = "xgmacclk0_ptp_250mhz"; > + }; > + > + xgmacclk1_dma: clk250mhz_2 { > + compatible = "fixed-clock"; > + #clock-cells = <0x0>; > + clock-frequency = <250000000>; > + clock-output-names = "xgmacclk1_dma_250mhz"; > + }; > + > + xgmacclk1_ptp: clk250mhz_3 { > + compatible = "fixed-clock"; > + #clock-cells = <0x0>; > + clock-frequency = <250000000>; > + clock-output-names = "xgmacclk1_ptp_250mhz"; > + }; > + > + xgmac0_phy: phy@e1240800 { > + compatible = "amd,xgbe-phy-seattle-v1a"; > + reg = <0x0 0xe1240800 0x0 0x0400>, /* SERDES RX/TX0 */ > + <0x0 0xe1250000 0x0 0x0060>, /* SERDES IR 1/2 */ > + <0x0 0xe12500f8 0x0 0x0004>; /* SERDES IR 2/2 */ > + interrupts = ; > + amd,speed-set = <0x0>; > + amd,serdes-blwc = <0x1 0x1 0x0>; > + amd,serdes-cdr-rate = <0x2 0x2 0x7>; > + amd,serdes-pq-skew = <0xa 0xa 0x12>; > + amd,serdes-tx-amp = <0xf 0xf 0xa>; > + amd,serdes-dfe-tap-config = <0x3 0x3 0x1>; > + amd,serdes-dfe-tap-enable = <0x0 0x0 0x7f>; > + status = "disabled"; > + }; > + > + xgmac1_phy: phy@e1240c00 { > + compatible = "amd,xgbe-phy-seattle-v1a"; > + reg = <0x0 0xe1240c00 0x0 0x0400>, /* SERDES RX/TX0 */ > + <0x0 0xe1250080 0x0 0x0060>, /* SERDES IR 1/2 */ > + <0x0 0xe12500fc 0x0 0x0004>; /* SERDES IR 2/2 */ > + interrupts = ; > + amd,speed-set = <0x0>; > + amd,serdes-blwc = <0x1 0x1 0x0>; > + amd,serdes-cdr-rate = <0x2 0x2 0x7>; > + amd,serdes-pq-skew = <0xa 0xa 0x12>; > + amd,serdes-tx-amp = <0xf 0xf 0xa>; > + amd,serdes-dfe-tap-config = <0x3 0x3 0x1>; > + amd,serdes-dfe-tap-enable = <0x0 0x0 0x7f>; > + status = "disabled"; > + }; > + > + xgmac0_smmu: smmu@e0600000 { > + compatible = "arm,mmu-401"; > + reg = <0 0xe0600000 0 0x10000>; > + > + /* Uses combined intr for both global and context */ > + #global-interrupts = <1>; > + interrupts = , > + ; > + #iommu-cells = <2>; > + dma-coherent; > + }; > + > + xgmac1_smmu: smmu@e0800000 { > + compatible = "arm,mmu-401"; > + reg = <0 0xe0800000 0 0x10000>; > + > + /* Uses combined intr for both global and context */ > + #global-interrupts = <1>; > + interrupts = , > + ; > + #iommu-cells = <2>; > + dma-coherent; > + }; > + > + xgmac@e0700000 { > + compatible = "amd,xgbe-seattle-v1a"; > + reg = <0x0 0xe0700000 0x0 0x80000 0x0 0xe0780000 0x0 0x80000>; > + interrupts = , > + , > + , > + , > + ; > + amd,per-channel-interrupt; > + mac-address = [02 a1 a2 a3 a4 a5]; > + clocks = <&xgmacclk0_dma &xgmacclk0_ptp>; > + clock-names = "dma_clk", "ptp_clk"; > + phy-handle = <&xgmac0_phy>; > + phy-mode = "xgmii"; > + > + iommus = <&xgmac0_smmu 0x00 0x1f>; /* 0-31 */ > + dma-coherent; > + status = "disabled"; > + }; > + > + xgmac@e0900000 { > + compatible = "amd,xgbe-seattle-v1a"; > + reg = <0x0 0xe0900000 0x0 0x80000 0x0 0xe0980000 0x0 0x80000>; > + interrupts = , > + , > + , > + , > + ; > + amd,per-channel-interrupt; > + mac-address = [02 b1 b2 b3 b4 b5]; > + clocks = <&xgmacclk1_dma &xgmacclk1_ptp>; > + clock-names = "dma_clk", "ptp_clk"; > + phy-handle = <&xgmac1_phy>; > + phy-mode = "xgmii"; > + > + iommus = <&xgmac1_smmu 0x00 0x1f>; /* 0-31 */ > + dma-coherent; > + status = "disabled"; > + }; > + }; > + > + chosen { > + stdout-path = "/smb/serial@e1010000"; > + }; > + > + psci { > + compatible = "arm,psci-0.2", "arm,psci"; > + method = "smc"; > + }; > +}; > diff --git a/Platform/AMD/OverdriveBoard/OverdriveBoardDeviceTree/OverdriveBoardDeviceTree.inf b/Platform/AMD/OverdriveBoard/OverdriveBoardDeviceTree/OverdriveBoardDeviceTree.inf > new file mode 100644 > index 000000000000..9ecf993cafac > --- /dev/null > +++ b/Platform/AMD/OverdriveBoard/OverdriveBoardDeviceTree/OverdriveBoardDeviceTree.inf > @@ -0,0 +1,28 @@ > +## @file > +# > +# Device tree description of the AMD Seattle Overdrive platform > +# > +# Copyright (c) 2017, Linaro Ltd. All rights reserved. > +# > +# This program and the accompanying materials > +# are licensed and made available under the terms and conditions of the BSD License > +# which accompanies this distribution. The full text of the license may be found at > +# http://opensource.org/licenses/bsd-license.php > +# > +# THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, > +# WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. > +# > +## > + > +[Defines] > + INF_VERSION = 0x00010019 > + BASE_NAME = OverdriveBoardDeviceTree > + FILE_GUID = 25462CDA-221F-47DF-AC1D-259CFAA4E326 # gDtPlatformDefaultDtbFileGuid > + MODULE_TYPE = USER_DEFINED > + VERSION_STRING = 1.0 > + > +[Sources] > + OverdriveBoardDeviceTree.dts > + > +[Packages] > + MdePkg/MdePkg.dec