* [PATCH 0/2] arm64: Add basic support for Marvell Berlin4CT SoC @ 2015-07-21 14:17 Jisheng Zhang 2015-07-21 14:17 ` [PATCH 1/2] arm64: dts: Add dts files " Jisheng Zhang 2015-07-21 14:17 ` [PATCH 2/2] arm64: Enable Marvell Berlin SoC family in Kconfig and defconfig Jisheng Zhang 0 siblings, 2 replies; 7+ messages in thread From: Jisheng Zhang @ 2015-07-21 14:17 UTC (permalink / raw) To: linux-arm-kernel Marvell Berlin4CT is a SoC based on 64bit ARMv8 architecture. It contains quad CA53 cores. This SoC shares many HW IP with BG2Q and other berlin series. This patchset was tested on Berlin4CT DMP board, and boot to shell ok. Jisheng Zhang (2): arm64: dts: Add dts files for Marvell Berlin4CT SoC arm64: Enable Marvell Berlin SoC family in Kconfig and defconfig arch/arm64/Kconfig.platforms | 6 + arch/arm64/boot/dts/Makefile | 1 + arch/arm64/boot/dts/marvell/Makefile | 5 + arch/arm64/boot/dts/marvell/berlin4ct-dmp.dts | 68 +++++++++++ arch/arm64/boot/dts/marvell/berlin4ct.dtsi | 160 ++++++++++++++++++++++++++ arch/arm64/configs/defconfig | 1 + 6 files changed, 241 insertions(+) create mode 100644 arch/arm64/boot/dts/marvell/Makefile create mode 100644 arch/arm64/boot/dts/marvell/berlin4ct-dmp.dts create mode 100644 arch/arm64/boot/dts/marvell/berlin4ct.dtsi -- 2.1.4 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] arm64: dts: Add dts files for Marvell Berlin4CT SoC 2015-07-21 14:17 [PATCH 0/2] arm64: Add basic support for Marvell Berlin4CT SoC Jisheng Zhang @ 2015-07-21 14:17 ` Jisheng Zhang 2015-07-21 14:34 ` Mark Rutland 2015-07-21 15:03 ` Sudeep Holla 2015-07-21 14:17 ` [PATCH 2/2] arm64: Enable Marvell Berlin SoC family in Kconfig and defconfig Jisheng Zhang 1 sibling, 2 replies; 7+ messages in thread From: Jisheng Zhang @ 2015-07-21 14:17 UTC (permalink / raw) To: linux-arm-kernel Add initial dtsi file to support Marvell Berlin4CT SoC with support of quad Cortex-A53 CPUs. It also adds dts file for Marvell Berlin4CT DMP board which is based on Berlin4CT SoC. Signed-off-by: Jisheng Zhang <jszhang@marvell.com> --- arch/arm64/boot/dts/Makefile | 1 + arch/arm64/boot/dts/marvell/Makefile | 5 + arch/arm64/boot/dts/marvell/berlin4ct-dmp.dts | 68 +++++++++++ arch/arm64/boot/dts/marvell/berlin4ct.dtsi | 160 ++++++++++++++++++++++++++ 4 files changed, 234 insertions(+) create mode 100644 arch/arm64/boot/dts/marvell/Makefile create mode 100644 arch/arm64/boot/dts/marvell/berlin4ct-dmp.dts create mode 100644 arch/arm64/boot/dts/marvell/berlin4ct.dtsi diff --git a/arch/arm64/boot/dts/Makefile b/arch/arm64/boot/dts/Makefile index 0c57290..41e348b 100644 --- a/arch/arm64/boot/dts/Makefile +++ b/arch/arm64/boot/dts/Makefile @@ -1,6 +1,7 @@ dts-dirs += amd dts-dirs += apm dts-dirs += arm +dts-dirs += marvell dts-dirs += cavium dts-dirs += exynos dts-dirs += freescale diff --git a/arch/arm64/boot/dts/marvell/Makefile b/arch/arm64/boot/dts/marvell/Makefile new file mode 100644 index 0000000..e2f6afa --- /dev/null +++ b/arch/arm64/boot/dts/marvell/Makefile @@ -0,0 +1,5 @@ +dtb-$(CONFIG_ARCH_BERLIN) += berlin4ct-dmp.dtb + +always := $(dtb-y) +subdir-y := $(dts-dirs) +clean-files := *.dtb diff --git a/arch/arm64/boot/dts/marvell/berlin4ct-dmp.dts b/arch/arm64/boot/dts/marvell/berlin4ct-dmp.dts new file mode 100644 index 0000000..a990c8d --- /dev/null +++ b/arch/arm64/boot/dts/marvell/berlin4ct-dmp.dts @@ -0,0 +1,68 @@ +/* + * Copyright (C) 2015 Marvell Technology Group Ltd. + * + * Author: Jisheng Zhang <jszhang@marvell.com> + * + * This file is dual-licensed: you can use it either under the terms + * of the GPLv2 or the X11 license, at your option. Note that this dual + * licensing only applies to this file, and not this project as a + * whole. + * + * a) This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License as + * published by the Free Software Foundation; either version 2 of the + * License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * Or, alternatively, + * + * b) Permission is hereby granted, free of charge, to any person + * obtaining a copy of this software and associated documentation + * files (the "Software"), to deal in the Software without + * restriction, including without limitation the rights to use, + * copy, modify, merge, publish, distribute, sublicense, and/or + * sell copies of the Software, and to permit persons to whom the + * Software is furnished to do so, subject to the following + * conditions: + * + * The above copyright notice and this permission notice shall be + * included in all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, + * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES + * OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND + * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT + * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, + * WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR + * OTHER DEALINGS IN THE SOFTWARE. + */ + +/dts-v1/; + +/memreserve/ 0x00000000 0x01000000; + +#include "berlin4ct.dtsi" + +/ { + model = "MARVELL BG4CT DMP BOARD"; + compatible = "marvell,berlin4ct-dmp", "marvell,berlin4ct", "marvell,berlin"; + + chosen { + bootargs = "earlyprintk"; + stdout-path = "serial0:115200n8"; + }; + + memory { + device_type = "memory"; + reg = <0 0x0 0 0x80000000>; + }; +}; + +&uart0 { + status = "okay"; +}; diff --git a/arch/arm64/boot/dts/marvell/berlin4ct.dtsi b/arch/arm64/boot/dts/marvell/berlin4ct.dtsi new file mode 100644 index 0000000..0b667f2 --- /dev/null +++ b/arch/arm64/boot/dts/marvell/berlin4ct.dtsi @@ -0,0 +1,160 @@ +/* + * Copyright (C) 2015 Marvell Technology Group Ltd. + * + * Author: Jisheng Zhang <jszhang@marvell.com> + * + * This file is dual-licensed: you can use it either under the terms + * of the GPLv2 or the X11 license, at your option. Note that this dual + * licensing only applies to this file, and not this project as a + * whole. + * + * a) This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License as + * published by the Free Software Foundation; either version 2 of the + * License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * Or, alternatively, + * + * b) Permission is hereby granted, free of charge, to any person + * obtaining a copy of this software and associated documentation + * files (the "Software"), to deal in the Software without + * restriction, including without limitation the rights to use, + * copy, modify, merge, publish, distribute, sublicense, and/or + * sell copies of the Software, and to permit persons to whom the + * Software is furnished to do so, subject to the following + * conditions: + * + * The above copyright notice and this permission notice shall be + * included in all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, + * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES + * OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND + * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT + * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, + * WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR + * OTHER DEALINGS IN THE SOFTWARE. + */ + +#include <dt-bindings/interrupt-controller/arm-gic.h> + +/ { + compatible = "marvell,berlin"; + interrupt-parent = <&gic>; + #address-cells = <2>; + #size-cells = <2>; + + aliases { + serial0 = &uart0; + }; + + psci { + compatible = "arm,psci-0.2"; + method = "smc"; + }; + + cpus { + #address-cells = <1>; + #size-cells = <0>; + + cpu at 0 { + compatible = "arm,cortex-a53", "arm,armv8"; + device_type = "cpu"; + reg = <0x0>; + enable-method = "psci"; + }; + + cpu at 1 { + compatible = "arm,cortex-a53", "arm,armv8"; + device_type = "cpu"; + reg = <0x1>; + enable-method = "psci"; + }; + + cpu at 2 { + compatible = "arm,cortex-a53", "arm,armv8"; + device_type = "cpu"; + reg = <0x2>; + enable-method = "psci"; + }; + + cpu at 3 { + compatible = "arm,cortex-a53", "arm,armv8"; + device_type = "cpu"; + reg = <0x3>; + enable-method = "psci"; + }; + }; + + pmu { + compatible = "arm,armv8-pmuv3"; + interrupts = <GIC_SPI 23 IRQ_TYPE_LEVEL_HIGH>, + <GIC_SPI 24 IRQ_TYPE_LEVEL_HIGH>, + <GIC_SPI 25 IRQ_TYPE_LEVEL_HIGH>, + <GIC_SPI 26 IRQ_TYPE_LEVEL_HIGH>; + }; + + timer { + compatible = "arm,armv8-timer"; + interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, + <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, + <GIC_PPI 11 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>, + <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>; + }; + + soc { + compatible = "simple-bus"; + #address-cells = <1>; + #size-cells = <1>; + ranges = <0 0 0xf7000000 0x1000000>; + + osc: osc { + compatible = "fixed-clock"; + #clock-cells = <0>; + clock-frequency = <25000000>; + }; + + gic: interrupt-controller at 901000 { + compatible = "arm,gic-400"; + #interrupt-cells = <3>; + interrupt-controller; + reg = <0x901000 0x1000>, + <0x902000 0x1000>, + <0x904000 0x2000>, + <0x906000 0x2000>; + interrupts = <GIC_PPI 9 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>; + }; + + apb at fc0000 { + compatible = "simple-bus"; + #address-cells = <1>; + #size-cells = <1>; + ranges = <0 0xfc0000 0x10000>; + interrupt-parent = <&sic>; + + sic: interrupt-controller at 1000 { + compatible = "snps,dw-apb-ictl"; + reg = <0x1000 0x30>; + interrupt-controller; + #interrupt-cells = <1>; + interrupt-parent = <&gic>; + interrupts = <GIC_SPI 17 IRQ_TYPE_LEVEL_HIGH>; + }; + + uart0: uart at d000 { + compatible = "snps,dw-apb-uart"; + reg = <0xd000 0x100>; + interrupts = <8>; + clocks = <&osc>; + reg-shift = <2>; + status = "disabled"; + }; + }; + }; +}; -- 2.1.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 1/2] arm64: dts: Add dts files for Marvell Berlin4CT SoC 2015-07-21 14:17 ` [PATCH 1/2] arm64: dts: Add dts files " Jisheng Zhang @ 2015-07-21 14:34 ` Mark Rutland 2015-07-21 14:50 ` Jisheng Zhang 2015-07-21 15:03 ` Sudeep Holla 1 sibling, 1 reply; 7+ messages in thread From: Mark Rutland @ 2015-07-21 14:34 UTC (permalink / raw) To: linux-arm-kernel Hi, > +/dts-v1/; > + > +/memreserve/ 0x00000000 0x01000000; What's this reservation for? Given you're using PSCI I can't see why we'd expect the kernel to map but not use some memory. > + > +#include "berlin4ct.dtsi" > + > +/ { > + model = "MARVELL BG4CT DMP BOARD"; > + compatible = "marvell,berlin4ct-dmp", "marvell,berlin4ct", "marvell,berlin"; > + > + chosen { > + bootargs = "earlyprintk"; > + stdout-path = "serial0:115200n8"; > + }; You shouldn't need those bootargs; "earlyprintk" does nothing on arm64. [...] > + gic: interrupt-controller at 901000 { > + compatible = "arm,gic-400"; > + #interrupt-cells = <3>; > + interrupt-controller; > + reg = <0x901000 0x1000>, > + <0x902000 0x1000>, > + <0x904000 0x2000>, > + <0x906000 0x2000>; > + interrupts = <GIC_PPI 9 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>; > + }; > + I believe the size of the cpu interface (the second reg entry) should be 0x2000, as GICC_DIR is at offset 0x1000. Otherwise this looks fine. Thanks, Mark. ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] arm64: dts: Add dts files for Marvell Berlin4CT SoC 2015-07-21 14:34 ` Mark Rutland @ 2015-07-21 14:50 ` Jisheng Zhang 2015-07-21 14:56 ` Mark Rutland 0 siblings, 1 reply; 7+ messages in thread From: Jisheng Zhang @ 2015-07-21 14:50 UTC (permalink / raw) To: linux-arm-kernel Dear Mark, Thanks a lot for so quick review! On Tue, 21 Jul 2015 15:34:26 +0100 Mark Rutland <mark.rutland@arm.com> wrote: > Hi, > > > +/dts-v1/; > > + > > +/memreserve/ 0x00000000 0x01000000; > > What's this reservation for? This is reserved for some firmwares' usage. > > Given you're using PSCI I can't see why we'd expect the kernel to map > but not use some memory. Is it acceptable that we make memory start at 0x01000000 instead of reservation? > > > + > > +#include "berlin4ct.dtsi" > > + > > +/ { > > + model = "MARVELL BG4CT DMP BOARD"; > > + compatible = "marvell,berlin4ct-dmp", "marvell,berlin4ct", "marvell,berlin"; > > + > > + chosen { > > + bootargs = "earlyprintk"; > > + stdout-path = "serial0:115200n8"; > > + }; > > You shouldn't need those bootargs; "earlyprintk" does nothing on arm64. will fix in a newer version > > [...] > > > + gic: interrupt-controller at 901000 { > > + compatible = "arm,gic-400"; > > + #interrupt-cells = <3>; > > + interrupt-controller; > > + reg = <0x901000 0x1000>, > > + <0x902000 0x1000>, > > + <0x904000 0x2000>, > > + <0x906000 0x2000>; > > + interrupts = <GIC_PPI 9 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>; > > + }; > > + > > I believe the size of the cpu interface (the second reg entry) should be > 0x2000, as GICC_DIR is at offset 0x1000. Oh, Yes! Thanks for pointing this out. > > Otherwise this looks fine. > > Thanks, > Mark. ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] arm64: dts: Add dts files for Marvell Berlin4CT SoC 2015-07-21 14:50 ` Jisheng Zhang @ 2015-07-21 14:56 ` Mark Rutland 0 siblings, 0 replies; 7+ messages in thread From: Mark Rutland @ 2015-07-21 14:56 UTC (permalink / raw) To: linux-arm-kernel On Tue, Jul 21, 2015 at 03:50:06PM +0100, Jisheng Zhang wrote: > Dear Mark, > > Thanks a lot for so quick review! > > On Tue, 21 Jul 2015 15:34:26 +0100 > Mark Rutland <mark.rutland@arm.com> wrote: > > > Hi, > > > > > +/dts-v1/; > > > + > > > +/memreserve/ 0x00000000 0x01000000; > > > > What's this reservation for? > > This is reserved for some firmwares' usage. > > > > > Given you're using PSCI I can't see why we'd expect the kernel to map > > but not use some memory. > > Is it acceptable that we make memory start at 0x01000000 instead of reservation? Yes, though it would still be worth a comment as to why that memory can't be used. Thanks, Mark. ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] arm64: dts: Add dts files for Marvell Berlin4CT SoC 2015-07-21 14:17 ` [PATCH 1/2] arm64: dts: Add dts files " Jisheng Zhang 2015-07-21 14:34 ` Mark Rutland @ 2015-07-21 15:03 ` Sudeep Holla 1 sibling, 0 replies; 7+ messages in thread From: Sudeep Holla @ 2015-07-21 15:03 UTC (permalink / raw) To: linux-arm-kernel On 21/07/15 15:17, Jisheng Zhang wrote: > Add initial dtsi file to support Marvell Berlin4CT SoC with > support of quad Cortex-A53 CPUs. > > It also adds dts file for Marvell Berlin4CT DMP board which is > based on Berlin4CT SoC. > > Signed-off-by: Jisheng Zhang <jszhang@marvell.com> > --- > arch/arm64/boot/dts/Makefile | 1 + > arch/arm64/boot/dts/marvell/Makefile | 5 + > arch/arm64/boot/dts/marvell/berlin4ct-dmp.dts | 68 +++++++++++ > arch/arm64/boot/dts/marvell/berlin4ct.dtsi | 160 ++++++++++++++++++++++++++ > 4 files changed, 234 insertions(+) > create mode 100644 arch/arm64/boot/dts/marvell/Makefile > create mode 100644 arch/arm64/boot/dts/marvell/berlin4ct-dmp.dts > create mode 100644 arch/arm64/boot/dts/marvell/berlin4ct.dtsi > [..] > diff --git a/arch/arm64/boot/dts/marvell/berlin4ct-dmp.dts b/arch/arm64/boot/dts/marvell/berlin4ct-dmp.dts > new file mode 100644 > index 0000000..a990c8d [..] > +/dts-v1/; > + > +/memreserve/ 0x00000000 0x01000000; > + > +#include "berlin4ct.dtsi" > + > +/ { > + model = "MARVELL BG4CT DMP BOARD"; > + compatible = "marvell,berlin4ct-dmp", "marvell,berlin4ct", "marvell,berlin"; > + > + chosen { > + bootargs = "earlyprintk"; I think you need to use earlycon instead ? > diff --git a/arch/arm64/boot/dts/marvell/berlin4ct.dtsi b/arch/arm64/boot/dts/marvell/berlin4ct.dtsi > new file mode 100644 > index 0000000..0b667f2 > --- /dev/null > +++ b/arch/arm64/boot/dts/marvell/berlin4ct.dtsi [..] > + > + pmu { > + compatible = "arm,armv8-pmuv3"; > + interrupts = <GIC_SPI 23 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 24 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 25 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 26 IRQ_TYPE_LEVEL_HIGH>; Better to add interrupt-affinity as per pmu bindings[1] Regards, Sudeep [1] Documentation/devicetree/bindings/arm/pmu.txt ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 2/2] arm64: Enable Marvell Berlin SoC family in Kconfig and defconfig 2015-07-21 14:17 [PATCH 0/2] arm64: Add basic support for Marvell Berlin4CT SoC Jisheng Zhang 2015-07-21 14:17 ` [PATCH 1/2] arm64: dts: Add dts files " Jisheng Zhang @ 2015-07-21 14:17 ` Jisheng Zhang 1 sibling, 0 replies; 7+ messages in thread From: Jisheng Zhang @ 2015-07-21 14:17 UTC (permalink / raw) To: linux-arm-kernel This patch introduces ARCH_BERLIN to enable Marvell Berlin SoC family in Kconfig and defconfig. Signed-off-by: Jisheng Zhang <jszhang@marvell.com> --- arch/arm64/Kconfig.platforms | 6 ++++++ arch/arm64/configs/defconfig | 1 + 2 files changed, 7 insertions(+) diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms index f7e4327..37b8fc5 100644 --- a/arch/arm64/Kconfig.platforms +++ b/arch/arm64/Kconfig.platforms @@ -1,5 +1,11 @@ menu "Platform selection" +config ARCH_BERLIN + bool "Marvell Berlin SoC Family" + select DW_APB_ICTL + help + This enables support for Marvell Berlin SoC Family + config ARCH_EXYNOS bool help diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig index 506b466..83843f9 100644 --- a/arch/arm64/configs/defconfig +++ b/arch/arm64/configs/defconfig @@ -31,6 +31,7 @@ CONFIG_MODULES=y CONFIG_MODULE_UNLOAD=y # CONFIG_BLK_DEV_BSG is not set # CONFIG_IOSCHED_DEADLINE is not set +CONFIG_ARCH_BERLIN=y CONFIG_ARCH_EXYNOS7=y CONFIG_ARCH_FSL_LS2085A=y CONFIG_ARCH_HISI=y -- 2.1.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2015-07-21 15:03 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-07-21 14:17 [PATCH 0/2] arm64: Add basic support for Marvell Berlin4CT SoC Jisheng Zhang 2015-07-21 14:17 ` [PATCH 1/2] arm64: dts: Add dts files " Jisheng Zhang 2015-07-21 14:34 ` Mark Rutland 2015-07-21 14:50 ` Jisheng Zhang 2015-07-21 14:56 ` Mark Rutland 2015-07-21 15:03 ` Sudeep Holla 2015-07-21 14:17 ` [PATCH 2/2] arm64: Enable Marvell Berlin SoC family in Kconfig and defconfig Jisheng Zhang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).