[PATCH v5 5/7] riscv: dts: Add device tree for Microchip Icicle Kit
Bin Meng
bmeng.cn at gmail.com
Fri Dec 11 14:24:57 CET 2020
Hi Padmarao,
On Fri, Dec 11, 2020 at 7:37 PM Padmarao Begari <padmarao.b at gmail.com> wrote:
>
> Hi Bin,
>
> On Fri, Dec 11, 2020 at 2:55 PM Bin Meng <bmeng.cn at gmail.com> wrote:
>>
>> Hi Padmarao,
>>
>> On Fri, Dec 11, 2020 at 4:32 PM Padmarao Begari <padmarao.b at gmail.com> wrote:
>> >
>> > Hi Bin,
>> >
>> > On Fri, Dec 11, 2020 at 1:31 PM Bin Meng <bmeng.cn at gmail.com> wrote:
>> >>
>> >> Hi Padmarao,
>> >>
>> >> On Thu, Dec 3, 2020 at 4:44 AM Padmarao Begari
>> >> <padmarao.begari at microchip.com> wrote:
>> >> >
>> >> > Add device tree for Microchip PolarFire SoC Icicle Kit.
>> >> >
>> >> > Signed-off-by: Padmarao Begari <padmarao.begari at microchip.com>
>> >> > Reviewed-by: Anup Patel <anup.patel at wdc.com>
>> >>
>> >> Sorry 2 more warnings, please check below:
>> >>
>> >> > ---
>> >> > arch/riscv/dts/Makefile | 1 +
>> >> > .../dts/microchip-mpfs-icicle-kit-u-boot.dtsi | 14 +
>> >> > arch/riscv/dts/microchip-mpfs-icicle-kit.dts | 421 ++++++++++++++++++
>> >> > 3 files changed, 436 insertions(+)
>> >> > create mode 100644 arch/riscv/dts/microchip-mpfs-icicle-kit-u-boot.dtsi
>> >> > create mode 100644 arch/riscv/dts/microchip-mpfs-icicle-kit.dts
>> >> >
>> >> > diff --git a/arch/riscv/dts/Makefile b/arch/riscv/dts/Makefile
>> >> > index 3a6f96c67d..01331b0aa1 100644
>> >> > --- a/arch/riscv/dts/Makefile
>> >> > +++ b/arch/riscv/dts/Makefile
>> >> > @@ -3,6 +3,7 @@
>> >> > dtb-$(CONFIG_TARGET_AX25_AE350) += ae350_32.dtb ae350_64.dtb
>> >> > dtb-$(CONFIG_TARGET_SIFIVE_FU540) += hifive-unleashed-a00.dtb
>> >> > dtb-$(CONFIG_TARGET_SIPEED_MAIX) += k210-maix-bit.dtb
>> >> > +dtb-$(CONFIG_TARGET_MICROCHIP_ICICLE) += microchip-mpfs-icicle-kit.dtb
>> >> >
>> >> > targets += $(dtb-y)
>> >> >
>> >> > diff --git a/arch/riscv/dts/microchip-mpfs-icicle-kit-u-boot.dtsi b/arch/riscv/dts/microchip-mpfs-icicle-kit-u-boot.dtsi
>> >> > new file mode 100644
>> >> > index 0000000000..f60283fb6b
>> >> > --- /dev/null
>> >> > +++ b/arch/riscv/dts/microchip-mpfs-icicle-kit-u-boot.dtsi
>> >> > @@ -0,0 +1,14 @@
>> >> > +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
>> >> > +/*
>> >> > + * Copyright (C) 2020 Microchip Technology Inc.
>> >> > + * Padmarao Begari <padmarao.begari at microchip.com>
>> >> > + */
>> >> > +
>> >> > +/ {
>> >> > + aliases {
>> >> > + cpu1 = &cpu1;
>> >> > + cpu2 = &cpu2;
>> >> > + cpu3 = &cpu3;
>> >> > + cpu4 = &cpu4;
>> >> > + };
>> >> > +};
>> >> > diff --git a/arch/riscv/dts/microchip-mpfs-icicle-kit.dts b/arch/riscv/dts/microchip-mpfs-icicle-kit.dts
>> >> > new file mode 100644
>> >> > index 0000000000..f5478bf201
>> >> > --- /dev/null
>> >> > +++ b/arch/riscv/dts/microchip-mpfs-icicle-kit.dts
>> >> > @@ -0,0 +1,421 @@
>> >> > +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
>> >> > +/* Copyright (c) 2020 Microchip Technology Inc */
>> >> > +
>> >> > +/dts-v1/;
>> >> > +#include "dt-bindings/clock/microchip,mpfs-clock.h"
>> >> > +
>> >> > +/* Clock frequency (in Hz) of the rtcclk */
>> >> > +#define RTCCLK_FREQ 1000000
>> >> > +
>> >> > +/ {
>> >> > + #address-cells = <2>;
>> >> > + #size-cells = <2>;
>> >> > + model = "Microchip MPFS Icicle Kit";
>> >> > + compatible = "microchip,mpfs-icicle-kit";
>> >> > +
>> >> > + aliases {
>> >> > + serial0 = &uart0;
>> >> > + ethernet0 = &emac1;
>> >> > + };
>> >> > +
>> >> > + chosen {
>> >> > + stdout-path = "serial0";
>> >> > + };
>> >> > +
>> >> > + cpucomplex: cpus {
>> >> > + #address-cells = <1>;
>> >> > + #size-cells = <0>;
>> >> > + timebase-frequency = <RTCCLK_FREQ>;
>> >> > + cpu0: cpu at 0 {
>> >> > + clocks = <&clkcfg CLK_CPU>;
>> >> > + compatible = "sifive,e51", "sifive,rocket0", "riscv";
>> >> > + device_type = "cpu";
>> >> > + i-cache-block-size = <64>;
>> >> > + i-cache-sets = <128>;
>> >> > + i-cache-size = <16384>;
>> >> > + reg = <0>;
>> >> > + riscv,isa = "rv64imac";
>> >> > + status = "disabled";
>> >> > + operating-points = <
>> >> > + /* kHz uV */
>> >> > + 600000 1100000
>> >> > + 300000 950000
>> >> > + 150000 750000
>> >> > + >;
>> >> > + cpu0intc: interrupt-controller {
>> >> > + #interrupt-cells = <1>;
>> >> > + compatible = "riscv,cpu-intc";
>> >> > + interrupt-controller;
>> >> > + };
>> >> > + };
>> >> > + cpu1: cpu at 1 {
>> >> > + clocks = <&clkcfg CLK_CPU>;
>> >> > + compatible = "sifive,u54-mc", "sifive,rocket0", "riscv";
>> >> > + d-cache-block-size = <64>;
>> >> > + d-cache-sets = <64>;
>> >> > + d-cache-size = <32768>;
>> >> > + d-tlb-sets = <1>;
>> >> > + d-tlb-size = <32>;
>> >> > + device_type = "cpu";
>> >> > + i-cache-block-size = <64>;
>> >> > + i-cache-sets = <64>;
>> >> > + i-cache-size = <32768>;
>> >> > + i-tlb-sets = <1>;
>> >> > + i-tlb-size = <32>;
>> >> > + mmu-type = "riscv,sv39";
>> >> > + reg = <1>;
>> >> > + riscv,isa = "rv64imafdc";
>> >> > + tlb-split;
>> >> > + status = "okay";
>> >> > + operating-points = <
>> >> > + /* kHz uV */
>> >> > + 600000 1100000
>> >> > + 300000 950000
>> >> > + 150000 750000
>> >> > + >;
>> >> > + cpu1intc: interrupt-controller {
>> >> > + #interrupt-cells = <1>;
>> >> > + compatible = "riscv,cpu-intc";
>> >> > + interrupt-controller;
>> >> > + };
>> >> > + };
>> >> > + cpu2: cpu at 2 {
>> >> > + clocks = <&clkcfg CLK_CPU>;
>> >> > + compatible = "sifive,u54-mc", "sifive,rocket0", "riscv";
>> >> > + d-cache-block-size = <64>;
>> >> > + d-cache-sets = <64>;
>> >> > + d-cache-size = <32768>;
>> >> > + d-tlb-sets = <1>;
>> >> > + d-tlb-size = <32>;
>> >> > + device_type = "cpu";
>> >> > + i-cache-block-size = <64>;
>> >> > + i-cache-sets = <64>;
>> >> > + i-cache-size = <32768>;
>> >> > + i-tlb-sets = <1>;
>> >> > + i-tlb-size = <32>;
>> >> > + mmu-type = "riscv,sv39";
>> >> > + reg = <2>;
>> >> > + riscv,isa = "rv64imafdc";
>> >> > + tlb-split;
>> >> > + status = "okay";
>> >> > + operating-points = <
>> >> > + /* kHz uV */
>> >> > + 600000 1100000
>> >> > + 300000 950000
>> >> > + 150000 750000
>> >> > + >;
>> >> > + cpu2intc: interrupt-controller {
>> >> > + #interrupt-cells = <1>;
>> >> > + compatible = "riscv,cpu-intc";
>> >> > + interrupt-controller;
>> >> > + };
>> >> > + };
>> >> > + cpu3: cpu at 3 {
>> >> > + clocks = <&clkcfg CLK_CPU>;
>> >> > + compatible = "sifive,u54-mc", "sifive,rocket0", "riscv";
>> >> > + d-cache-block-size = <64>;
>> >> > + d-cache-sets = <64>;
>> >> > + d-cache-size = <32768>;
>> >> > + d-tlb-sets = <1>;
>> >> > + d-tlb-size = <32>;
>> >> > + device_type = "cpu";
>> >> > + i-cache-block-size = <64>;
>> >> > + i-cache-sets = <64>;
>> >> > + i-cache-size = <32768>;
>> >> > + i-tlb-sets = <1>;
>> >> > + i-tlb-size = <32>;
>> >> > + mmu-type = "riscv,sv39";
>> >> > + reg = <3>;
>> >> > + riscv,isa = "rv64imafdc";
>> >> > + tlb-split;
>> >> > + status = "okay";
>> >> > + operating-points = <
>> >> > + /* kHz uV */
>> >> > + 600000 1100000
>> >> > + 300000 950000
>> >> > + 150000 750000
>> >> > + >;
>> >> > + cpu3intc: interrupt-controller {
>> >> > + #interrupt-cells = <1>;
>> >> > + compatible = "riscv,cpu-intc";
>> >> > + interrupt-controller;
>> >> > + };
>> >> > + };
>> >> > + cpu4: cpu at 4 {
>> >> > + clocks = <&clkcfg CLK_CPU>;
>> >> > + compatible = "sifive,u54-mc", "sifive,rocket0", "riscv";
>> >> > + d-cache-block-size = <64>;
>> >> > + d-cache-sets = <64>;
>> >> > + d-cache-size = <32768>;
>> >> > + d-tlb-sets = <1>;
>> >> > + d-tlb-size = <32>;
>> >> > + device_type = "cpu";
>> >> > + i-cache-block-size = <64>;
>> >> > + i-cache-sets = <64>;
>> >> > + i-cache-size = <32768>;
>> >> > + i-tlb-sets = <1>;
>> >> > + i-tlb-size = <32>;
>> >> > + mmu-type = "riscv,sv39";
>> >> > + reg = <4>;
>> >> > + riscv,isa = "rv64imafdc";
>> >> > + tlb-split;
>> >> > + status = "okay";
>> >> > + operating-points = <
>> >> > + /* kHz uV */
>> >> > + 600000 1100000
>> >> > + 300000 950000
>> >> > + 150000 750000
>> >> > + >;
>> >> > + cpu4intc: interrupt-controller {
>> >> > + #interrupt-cells = <1>;
>> >> > + compatible = "riscv,cpu-intc";
>> >> > + interrupt-controller;
>> >> > + };
>> >> > + };
>> >> > + };
>> >> > + ddr: memory at 80000000 {
>> >> > + device_type = "memory";
>> >> > + reg = <0x0 0x80000000 0x0 0x40000000>;
>> >> > + clocks = <&clkcfg CLK_DDRC>;
>> >> > + };
>> >> > + soc: soc {
>> >> > + #address-cells = <2>;
>> >> > + #size-cells = <2>;
>> >> > + compatible = "microchip,mpfs-icicle-kit", "simple-bus";
>> >> > + ranges;
>> >> > + clint0: clint at 2000000 {
>> >> > + compatible = "riscv,clint0";
>> >> > + interrupts-extended = <&cpu0intc 3 &cpu0intc 7
>> >> > + &cpu1intc 3 &cpu1intc 7
>> >> > + &cpu2intc 3 &cpu2intc 7
>> >> > + &cpu3intc 3 &cpu3intc 7
>> >> > + &cpu4intc 3 &cpu4intc 7>;
>> >> > + reg = <0x0 0x2000000 0x0 0x10000>;
>> >> > + reg-names = "control";
>> >> > + clock-frequency = <RTCCLK_FREQ>;
>> >> > + };
>> >> > + cachecontroller: cache-controller at 2010000 {
>> >> > + compatible = "sifive,fu540-c000-ccache", "cache";
>> >> > + cache-block-size = <64>;
>> >> > + cache-level = <2>;
>> >> > + cache-sets = <1024>;
>> >> > + cache-size = <2097152>;
>> >> > + cache-unified;
>> >> > + interrupt-parent = <&plic>;
>> >> > + interrupts = <1 2 3>;
>> >> > + reg = <0x0 0x2010000 0x0 0x1000>;
>> >> > + };
>> >> > + plic: interrupt-controller at c000000 {
>> >> > + #interrupt-cells = <1>;
>> >> > + compatible = "sifive,plic-1.0.0";
>> >> > + reg = <0x0 0xc000000 0x0 0x4000000>;
>> >> > + riscv,max-priority = <7>;
>> >> > + riscv,ndev = <186>;
>> >> > + interrupt-controller;
>> >> > + interrupts-extended = <
>> >> > + &cpu0intc 11
>> >> > + &cpu1intc 11 &cpu1intc 9
>> >> > + &cpu2intc 11 &cpu2intc 9
>> >> > + &cpu3intc 11 &cpu3intc 9
>> >> > + &cpu4intc 11 &cpu4intc 9>;
>> >> > + };
>> >> > + uart0: serial at 20000000 {
>> >> > + compatible = "ns16550a";
>> >> > + reg = <0x0 0x20000000 0x0 0x400>;
>> >> > + reg-io-width = <4>;
>> >> > + reg-shift = <2>;
>> >> > + interrupt-parent = <&plic>;
>> >> > + interrupts = <90>;
>> >> > + clock-frequency = <150000000>;
>> >> > + clocks = <&clkcfg CLK_MMUART0>;
>> >> > + status = "okay";
>> >> > + };
>> >> > + refclk: refclk {
>> >> > + compatible = "fixed-clock";
>> >> > + #clock-cells = <0>;
>> >> > + clock-frequency = <600000000>;
>> >> > + clock-output-names = "msspllclk";
>> >> > + };
>> >>
>> >> Right now this is put under /soc, but with newer dtc, we got warning like:
>> >> Warning (simple_bus_reg): /soc/refclk: missing or empty reg/ranges property
>> >>
>> >> Should we move this node to / instead?
>> >>
>> >
>> > ok
>> >
>> >>
>> >> > + clkcfg: clkcfg at 20002000 {
>> >> > + compatible = "microchip,mpfs-clkcfg";
>> >> > + reg = <0x0 0x20002000 0x0 0x1000>;
>> >> > + reg-names = "mss_sysreg";
>> >> > + clocks = <&refclk>;
>> >> > + #clock-cells = <1>;
>> >> > + clock-output-names = "cpu", "axi", "ahb", "envm",
>> >> > + "mac0", "mac1", "mmc", "timer",
>> >> > + "mmuart0", "mmuart1", "mmuart2",
>> >> > + "mmuart3", "mmuart4", "spi0", "spi1",
>> >> > + "i2c0", "i2c1", "can0", "can1", "usb",
>> >> > + "reserved", "rtc", "qspi", "gpio0",
>> >> > + "gpio1", "gpio2", "ddrc", "fic0",
>> >> > + "fic1", "fic2", "fic3", "athena",
>> >> > + "cfm";
>> >> > + };
>> >> > + emmc: mmc at 20008000 {
>> >> > + compatible = "cdns,sd4hc";
>> >> > + reg = <0x0 0x20008000 0x0 0x1000>;
>> >> > + interrupt-parent = <&plic>;
>> >> > + interrupts = <88 89>;
>> >> > + pinctrl-names = "default";
>> >> > + clocks = <&clkcfg CLK_MMC>;
>> >> > + bus-width = <4>;
>> >> > + cap-mmc-highspeed;
>> >> > + mmc-ddr-3_3v;
>> >> > + max-frequency = <200000000>;
>> >> > + non-removable;
>> >> > + no-sd;
>> >> > + no-sdio;
>> >> > + voltage-ranges = <3300 3300>;
>> >> > + status = "okay";
>> >> > + };
>> >> > + sdcard: sd at 20008000 {
>> >> > + compatible = "cdns,sd4hc";
>> >> > + reg = <0x0 0x20008000 0x0 0x1000>;
>> >> > + interrupt-parent = <&plic>;
>> >> > + interrupts = <88>;
>> >> > + pinctrl-names = "default";
>> >> > + clocks = <&clkcfg CLK_MMC>;
>> >> > + bus-width = <4>;
>> >> > + disable-wp;
>> >> > + cap-sd-highspeed;
>> >> > + card-detect-delay = <200>;
>> >> > + sd-uhs-sdr12;
>> >> > + sd-uhs-sdr25;
>> >> > + sd-uhs-sdr50;
>> >> > + sd-uhs-sdr104;
>> >> > + max-frequency = <200000000>;
>> >> > + status = "disabled";
>> >> > + };
>> >> > + uart1: serial at 20100000 {
>> >> > + compatible = "ns16550a";
>> >> > + reg = <0x0 0x20100000 0x0 0x400>;
>> >> > + reg-io-width = <4>;
>> >> > + reg-shift = <2>;
>> >> > + interrupt-parent = <&plic>;
>> >> > + interrupts = <91>;
>> >> > + clock-frequency = <150000000>;
>> >> > + clocks = <&clkcfg CLK_MMUART1>;
>> >> > + status = "okay";
>> >> > + };
>> >> > + uart2: serial at 20102000 {
>> >> > + compatible = "ns16550a";
>> >> > + reg = <0x0 0x20102000 0x0 0x400>;
>> >> > + reg-io-width = <4>;
>> >> > + reg-shift = <2>;
>> >> > + interrupt-parent = <&plic>;
>> >> > + interrupts = <92>;
>> >> > + clock-frequency = <150000000>;
>> >> > + clocks = <&clkcfg CLK_MMUART2>;
>> >> > + status = "okay";
>> >> > + };
>> >> > + uart3: serial at 20104000 {
>> >> > + compatible = "ns16550a";
>> >> > + reg = <0x0 0x20104000 0x0 0x400>;
>> >> > + reg-io-width = <4>;
>> >> > + reg-shift = <2>;
>> >> > + interrupt-parent = <&plic>;
>> >> > + interrupts = <93>;
>> >> > + clock-frequency = <150000000>;
>> >> > + clocks = <&clkcfg CLK_MMUART3>;
>> >> > + status = "okay";
>> >> > + };
>> >> > + i2c0: i2c at 02010a000 {
>> >> > + #address-cells = <1>;
>> >> > + #size-cells = <0>;
>> >> > + compatible = "microchip,mpfs-mss-i2c";
>> >> > + reg = <0x0 0x2010a000 0x0 0x1000>;
>> >> > + interrupt-parent = <&plic>;
>> >> > + interrupts = <58>;
>> >> > + clocks = <&clkcfg CLK_I2C0>;
>> >> > + status = "disabled";
>> >> > + };
>> >> > + i2c1: i2c at 02010b000 {
>> >> > + #address-cells = <1>;
>> >> > + #size-cells = <0>;
>> >> > + compatible = "microchip,mpfs-mss-i2c";
>> >> > + reg = <0x0 0x2010b000 0x0 0x1000>;
>> >> > + interrupt-parent = <&plic>;
>> >> > + interrupts = <61>;
>> >> > + clocks = <&clkcfg CLK_I2C1>;
>> >> > + status = "disabled";
>> >> > + pac193x at 0x10 {
>> >> > + compatible = "microchip,pac1934";
>> >> > + reg = <0x10>;
>> >> > + samp-rate = <64>;
>> >> > + status = "disabled";
>> >> > + ch1: channel at 0 {
>> >>
>> >> Please remove @0, otherwise with newer dtc, we got:
>> >>
>> >
>> > ok, will remove @0
>> >
>> > Is the channel at 1, channel at 2, channel at 3 ok? or do we need to remove @1, at 2, @3 also?
>> >
>>
>> Same warning. We should remove them too.
>>
>
> Ok, will remove @1, at 2, @3
>
> Can I replace channel at 0 with channel0, channel at 1 with channel1, channel at 2 with channel2, channel at 3 with channel3?
Yes.
>
> The ethernet subnode 'ethernet-phy at 8' is using @8 for the device phy address, is this node also giving warning?
No, because there is a reg = <8>.
Regards,
Bin
More information about the U-Boot
mailing list