From patchwork Fri Jan 13 09:12:15 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chris Packham X-Patchwork-Id: 714892 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3v0H2v3yL3z9t0H for ; Fri, 13 Jan 2017 20:12:39 +1100 (AEDT) Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=alliedtelesis.co.nz header.i=@alliedtelesis.co.nz header.b="msstZbpC"; dkim-atps=neutral Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751355AbdAMJMh (ORCPT ); Fri, 13 Jan 2017 04:12:37 -0500 Received: from gate2.alliedtelesis.co.nz ([202.36.163.20]:55942 "EHLO gate2.alliedtelesis.co.nz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751146AbdAMJMe (ORCPT ); Fri, 13 Jan 2017 04:12:34 -0500 Received: from mmarshal3.atlnz.lc (mmarshal3.atlnz.lc [10.32.18.43]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by gate2.alliedtelesis.co.nz (Postfix) with ESMTPS id 282438011F; Fri, 13 Jan 2017 22:12:31 +1300 (NZDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=alliedtelesis.co.nz; s=mail; t=1484298751; bh=1jU55132ifmP983lmQkEzbZNpnPF7F7xLwWlHqV7zao=; h=From:To:Cc:Subject:Date; b=msstZbpCkLkqFY2XWkC0/sycOodAFTQXtZRA8fqds3Q0cih+2LamLIqmqvx5ytnDW J11Fh/z0ZHkNHvCoyOoEFIBSHZ1a74aZahsCHicRZGt1mbs8cgIlBBvrrjBWzgMzH8 5KsR73Dm5xLji+R7yWLSUgwukgQAEvBN1BAGFWPk= Received: from smtp (Not Verified[10.32.16.33]) by mmarshal3.atlnz.lc with Trustwave SEG (v7, 5, 6, 8438) id ; Fri, 13 Jan 2017 22:12:30 +1300 Received: from chrisp-dl.atlnz.lc (chrisp-dl.ws.atlnz.lc [10.33.22.30]) by smtp (Postfix) with ESMTP id B97E013EF27; Fri, 13 Jan 2017 22:12:29 +1300 (NZDT) Received: by chrisp-dl.atlnz.lc (Postfix, from userid 1030) id 512421E0BBD; Fri, 13 Jan 2017 22:12:30 +1300 (NZDT) From: Chris Packham To: linux-arm-kernel@lists.infradead.org Cc: Chris Packham , Rob Herring , Mark Rutland , Michael Turquette , Stephen Boyd , Linus Walleij , Jason Cooper , Andrew Lunn , Gregory Clement , Sebastian Hesselbarth , Russell King , Geert Uytterhoeven , Chris Brand , Florian Fainelli , Arnd Bergmann , Thierry Reding , Sudeep Holla , Juri Lelli , Thomas Petazzoni , Laxman Dewangan , Kalyan Kinthada , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, linux-gpio@vger.kernel.org Subject: [PATCHv4 0/5] Support for Marvell switches with integrated CPUs Date: Fri, 13 Jan 2017 22:12:15 +1300 Message-Id: <20170113091222.7132-1-chris.packham@alliedtelesis.co.nz> X-Mailer: git-send-email 2.11.0.24.ge6920cf Sender: linux-gpio-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-gpio@vger.kernel.org The 98DX3236, 98DX3336 and 98DX4251 are a set of switch ASICs with integrated CPUs. They CPU block is common within these product lines and (as far as I can tell/have been told) is based on the Armada XP. There are a few differences due to the fact they have to squeeze the CPU into the same package as the switch. This series is starting to settle down now. The only major change is in "arm: mvebu: support for SMP on 98DX3336 SoC" the other changes are generally cosmetic or collecting acks. Chris Packham (4): clk: mvebu: support for 98DX3236 SoC Changes in v2: - Update devicetree binding documentation for new compatible string Changes in v3: - Add 98dx3236 support to mvebu/clk-corediv.c rather than creating a new driver. - Document mv98dx3236-corediv-clock binding Changes in v4: - None arm: mvebu: support for SMP on 98DX3336 SoC Changes in v2: - Document new enable-method value - Correct some references from 98DX4521 to 98DX3236 Changes in v3: - Simplify mv98dx3236_resume_init by using of_io_request_and_map() Changes in v4: - integrate changes into platsmp.c instead of new init call - avoid duplicated code. - fix error return - Collect ack from Rob arm: mvebu: Add device tree for 98DX3236 SoCs Changes in v2: - Update devicetree binding documentation to reflect that 98DX3336 and 984251 are supersets of 98DX3236. - disable crypto block - disable sdio for 98DX3236, enable for 98DX4251 Changes in v3: - fix typo 4521 -> 4251 - document prestera bindings - rework corediv-clock binding - add label to packet processor node - add new compatible string for DFX server Changes in v4: - Collect ack from Rob arm: mvebu: Add device tree for db-dxbc2 and db-xc3-24g4xg boards Kalyan Kinthada (1): pinctrl: mvebu: pinctrl driver for 98DX3236 SoC Changes in v2: - include sdio support for the 98DX4251 Changes in v3: - None Changes in v4: - Correct some discrepencies between binding and driver. - Collect acks from Rob and Sebastian Documentation/devicetree/bindings/arm/cpus.txt | 1 + .../bindings/arm/marvell/98dx3236-resume-ctrl.txt | 18 ++ .../devicetree/bindings/arm/marvell/98dx3236.txt | 23 ++ .../bindings/clock/mvebu-corediv-clock.txt | 1 + .../devicetree/bindings/clock/mvebu-cpu-clock.txt | 1 + .../devicetree/bindings/net/marvell,prestera.txt | 50 ++++ .../pinctrl/marvell,armada-98dx3236-pinctrl.txt | 46 ++++ arch/arm/boot/dts/armada-xp-98dx3236.dtsi | 254 +++++++++++++++++++++ arch/arm/boot/dts/armada-xp-98dx3336.dtsi | 76 ++++++ arch/arm/boot/dts/armada-xp-98dx4251.dtsi | 90 ++++++++ arch/arm/boot/dts/db-dxbc2.dts | 159 +++++++++++++ arch/arm/boot/dts/db-xc3-24g4xg.dts | 155 +++++++++++++ arch/arm/mach-mvebu/platsmp.c | 86 +++++++ drivers/clk/mvebu/armada-xp.c | 42 ++++ drivers/clk/mvebu/clk-corediv.c | 23 ++ drivers/clk/mvebu/clk-cpu.c | 31 ++- drivers/pinctrl/mvebu/pinctrl-armada-xp.c | 156 +++++++++++++ 17 files changed, 1210 insertions(+), 2 deletions(-) create mode 100644 Documentation/devicetree/bindings/arm/marvell/98dx3236-resume-ctrl.txt create mode 100644 Documentation/devicetree/bindings/arm/marvell/98dx3236.txt create mode 100644 Documentation/devicetree/bindings/net/marvell,prestera.txt create mode 100644 Documentation/devicetree/bindings/pinctrl/marvell,armada-98dx3236-pinctrl.txt create mode 100644 arch/arm/boot/dts/armada-xp-98dx3236.dtsi create mode 100644 arch/arm/boot/dts/armada-xp-98dx3336.dtsi create mode 100644 arch/arm/boot/dts/armada-xp-98dx4251.dtsi create mode 100644 arch/arm/boot/dts/db-dxbc2.dts create mode 100644 arch/arm/boot/dts/db-xc3-24g4xg.dts inter-diff to v3: diff --git a/Documentation/devicetree/bindings/pinctrl/marvell,armada-98dx3236-pinctrl.txt b/Documentation/devicetree/bindings/pinctrl/marvell,armada-98dx3236-pinctrl.txt index d4e6ecdfc853..b5bd23992fdf 100644 --- a/Documentation/devicetree/bindings/pinctrl/marvell,armada-98dx3236-pinctrl.txt +++ b/Documentation/devicetree/bindings/pinctrl/marvell,armada-98dx3236-pinctrl.txt @@ -28,10 +28,10 @@ mpp13 13 gpio, intr(out), dev(ad15) mpp14 14 gpio, i2c0(sck) mpp15 15 gpio, i2c0(sda) mpp16 16 gpio, dev(oe) -mpp17 17 gpio, dev(clk) +mpp17 17 gpio, dev(clkout) mpp18 18 gpio, uart1(txd) mpp19 19 gpio, uart1(rxd), dev(rb) -mpp20 20 gpio, dev(we) +mpp20 20 gpio, dev(we0) mpp21 21 gpio, dev(ad0) mpp22 22 gpio, dev(ad1) mpp23 23 gpio, dev(ad2) diff --git a/arch/arm/mach-mvebu/Makefile b/arch/arm/mach-mvebu/Makefile index 2a2dd8324fb8..6c6497e80a7b 100644 --- a/arch/arm/mach-mvebu/Makefile +++ b/arch/arm/mach-mvebu/Makefile @@ -7,7 +7,6 @@ obj-$(CONFIG_MACH_MVEBU_ANY) += system-controller.o mvebu-soc-id.o ifeq ($(CONFIG_MACH_MVEBU_V7),y) obj-y += cpu-reset.o board-v7.o coherency.o coherency_ll.o pmsu.o pmsu_ll.o -obj-y += pmsu-98dx3236.o obj-$(CONFIG_PM) += pm.o pm-board.o obj-$(CONFIG_SMP) += platsmp.o headsmp.o platsmp-a9.o headsmp-a9.o diff --git a/arch/arm/mach-mvebu/common.h b/arch/arm/mach-mvebu/common.h index 099dabf23461..6b775492cfad 100644 --- a/arch/arm/mach-mvebu/common.h +++ b/arch/arm/mach-mvebu/common.h @@ -27,5 +27,4 @@ void __iomem *mvebu_get_scu_base(void); int mvebu_pm_suspend_init(void (*board_pm_enter)(void __iomem *sdram_reg, u32 srcmd)); -void mv98dx3236_resume_set_cpu_boot_addr(int hw_cpu, void *boot_addr); #endif diff --git a/arch/arm/mach-mvebu/platsmp.c b/arch/arm/mach-mvebu/platsmp.c index 3c9ab9a008ad..59be3ca0464f 100644 --- a/arch/arm/mach-mvebu/platsmp.c +++ b/arch/arm/mach-mvebu/platsmp.c @@ -182,12 +182,57 @@ const struct smp_operations armada_xp_smp_ops __initconst = { #endif }; +CPU_METHOD_OF_DECLARE(armada_xp_smp, "marvell,armada-xp-smp", + &armada_xp_smp_ops); + +struct resume_controller { + u32 resume_control; + u32 resume_boot_addr; +}; + +static const struct resume_controller mv98dx3336_resume_controller = { + .resume_control = 0x08, + .resume_boot_addr = 0x04, +}; + +static const struct of_device_id of_mv98dx3236_resume_table[] = { + { + .compatible = "marvell,98dx3336-resume-ctrl", + .data = (void *)&mv98dx3336_resume_controller, + }, + { /* end of list */ }, +}; + +static int mv98dx3236_resume_set_cpu_boot_addr(int hw_cpu, void *boot_addr) +{ + const struct of_device_id *match; + struct device_node *np; + void __iomem *base; + struct resume_controller *rc; + + WARN_ON(hw_cpu != 1); + + np = of_find_matching_node_and_match(NULL, of_mv98dx3236_resume_table, + &match); + if (!np) + return -ENODEV; + + base = of_io_request_and_map(np, 0, of_node_full_name(np)); + rc = (struct resume_controller *)match->data; + of_node_put(np); + if (IS_ERR(base)) + return PTR_ERR(base); + + writel(0, base + rc->resume_control); + writel(virt_to_phys(boot_addr), base + rc->resume_boot_addr); + + return 0; +} + static int mv98dx3236_boot_secondary(unsigned int cpu, struct task_struct *idle) { int ret, hw_cpu; - pr_info("Booting CPU %d\n", cpu); - hw_cpu = cpu_logical_map(cpu); set_secondary_cpu_clock(hw_cpu); mv98dx3236_resume_set_cpu_boot_addr(hw_cpu, @@ -212,7 +257,7 @@ static int mv98dx3236_boot_secondary(unsigned int cpu, struct task_struct *idle) return 0; } -struct smp_operations mv98dx3236_smp_ops __initdata = { +static const struct smp_operations mv98dx3236_smp_ops __initconst = { .smp_init_cpus = armada_xp_smp_init_cpus, .smp_prepare_cpus = armada_xp_smp_prepare_cpus, .smp_boot_secondary = mv98dx3236_boot_secondary, @@ -223,7 +268,5 @@ struct smp_operations mv98dx3236_smp_ops __initdata = { #endif }; -CPU_METHOD_OF_DECLARE(armada_xp_smp, "marvell,armada-xp-smp", - &armada_xp_smp_ops); CPU_METHOD_OF_DECLARE(mv98dx3236_smp, "marvell,98dx3236-smp", &mv98dx3236_smp_ops); diff --git a/arch/arm/mach-mvebu/pmsu-98dx3236.c b/arch/arm/mach-mvebu/pmsu-98dx3236.c deleted file mode 100644 index 1052674dd439..000000000000 --- a/arch/arm/mach-mvebu/pmsu-98dx3236.c +++ /dev/null @@ -1,52 +0,0 @@ -/** - * CPU resume support for 98DX3236 internal CPU (a.k.a. MSYS). - */ - -#define pr_fmt(fmt) "mv98dx3236-resume: " fmt - -#include -#include -#include -#include -#include "common.h" - -static void __iomem *mv98dx3236_resume_base; -#define MV98DX3236_CPU_RESUME_CTRL_OFFSET 0x08 -#define MV98DX3236_CPU_RESUME_ADDR_OFFSET 0x04 - -static const struct of_device_id of_mv98dx3236_resume_table[] = { - {.compatible = "marvell,98dx3336-resume-ctrl",}, - { /* end of list */ }, -}; - -void mv98dx3236_resume_set_cpu_boot_addr(int hw_cpu, void *boot_addr) -{ - WARN_ON(hw_cpu != 1); - - writel(0, mv98dx3236_resume_base + MV98DX3236_CPU_RESUME_CTRL_OFFSET); - writel(virt_to_phys(boot_addr), mv98dx3236_resume_base + - MV98DX3236_CPU_RESUME_ADDR_OFFSET); -} - -static int __init mv98dx3236_resume_init(void) -{ - struct device_node *np; - void __iomem *base; - - np = of_find_matching_node(NULL, of_mv98dx3236_resume_table); - if (!np) - return 0; - - base = of_io_request_and_map(np, 0, of_node_full_name(np)); - if (IS_ERR(base)) { - pr_err("unable to map registers\n"); - of_node_put(np); - return PTR_ERR(mv98dx3236_resume_base); - } - - mv98dx3236_resume_base = base; - of_node_put(np); - return 0; -} - -early_initcall(mv98dx3236_resume_init); diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-xp.c b/drivers/pinctrl/mvebu/pinctrl-armada-xp.c index 554eeae8cd21..9601d662c7f5 100644 --- a/drivers/pinctrl/mvebu/pinctrl-armada-xp.c +++ b/drivers/pinctrl/mvebu/pinctrl-armada-xp.c @@ -374,8 +374,8 @@ static struct mvebu_mpp_mode mv98dx3236_mpp_modes[] = { MPP_VAR_FUNCTION(0x2, "spi0", "miso", V_98DX3236_PLUS), MPP_VAR_FUNCTION(0x4, "dev", "ad9", V_98DX3236_PLUS)), MPP_MODE(2, - MPP_VAR_FUNCTION(0x0, "gpio", NULL, V_98DX3236_PLUS), - MPP_VAR_FUNCTION(0x2, "spi0", "csk", V_98DX3236_PLUS), + MPP_VAR_FUNCTION(0x0, "gpo", NULL, V_98DX3236_PLUS), + MPP_VAR_FUNCTION(0x2, "spi0", "sck", V_98DX3236_PLUS), MPP_VAR_FUNCTION(0x4, "dev", "ad10", V_98DX3236_PLUS)), MPP_MODE(3, MPP_VAR_FUNCTION(0x0, "gpio", NULL, V_98DX3236_PLUS), @@ -390,7 +390,7 @@ static struct mvebu_mpp_mode mv98dx3236_mpp_modes[] = { MPP_VAR_FUNCTION(0x0, "gpio", NULL, V_98DX3236_PLUS), MPP_VAR_FUNCTION(0x1, "pex", "rsto", V_98DX3236_PLUS), MPP_VAR_FUNCTION(0x2, "sd0", "cmd", V_98DX4251), - MPP_VAR_FUNCTION(0x4, "dev", "bootcs0", V_98DX3236_PLUS)), + MPP_VAR_FUNCTION(0x4, "dev", "bootcs", V_98DX3236_PLUS)), MPP_MODE(6, MPP_VAR_FUNCTION(0x0, "gpo", NULL, V_98DX3236_PLUS), MPP_VAR_FUNCTION(0x2, "sd0", "clk", V_98DX4251), @@ -442,7 +442,8 @@ static struct mvebu_mpp_mode mv98dx3236_mpp_modes[] = { MPP_VAR_FUNCTION(0x3, "uart1", "txd", V_98DX3236_PLUS)), MPP_MODE(19, MPP_VAR_FUNCTION(0x0, "gpio", NULL, V_98DX3236_PLUS), - MPP_VAR_FUNCTION(0x3, "uart1", "rxd", V_98DX3236_PLUS)), + MPP_VAR_FUNCTION(0x3, "uart1", "rxd", V_98DX3236_PLUS), + MPP_VAR_FUNCTION(0x4, "dev", "rb", V_98DX3236_PLUS)), MPP_MODE(20, MPP_VAR_FUNCTION(0x0, "gpo", NULL, V_98DX3236_PLUS), MPP_VAR_FUNCTION(0x4, "dev", "we0", V_98DX3236_PLUS)), @@ -548,7 +549,7 @@ static struct mvebu_mpp_ctrl mv98dx3236_mpp_controls[] = { }; static struct pinctrl_gpio_range mv98dx3236_mpp_gpio_ranges[] = { - MPP_GPIO_RANGE(0, 0, 0, 32), + MPP_GPIO_RANGE(0, 0, 0, 32), }; static int armada_xp_pinctrl_suspend(struct platform_device *pdev,