[{"id":1776633,"web_url":"http://patchwork.ozlabs.org/comment/1776633/","msgid":"<20170927214213.qjz7qalvvmhpxxmy@rob-hp-laptop>","list_archive_url":null,"date":"2017-09-27T21:42:13","subject":"Re: [PATCH v6 2/3] arm: dts: add Nuvoton NPCM750 device tree","submitter":{"id":62529,"url":"http://patchwork.ozlabs.org/api/people/62529/","name":"Rob Herring (Arm)","email":"robh@kernel.org"},"content":"On Tue, Sep 19, 2017 at 03:40:00PM -0700, Brendan Higgins wrote:\n> Add a common device tree for all Nuvoton NPCM750 BMCs and a board\n> specific device tree for the NPCM750 (Poleg) evaluation board.\n> \n> Signed-off-by: Brendan Higgins \n> Reviewed-by: Tomer Maimon \n> Reviewed-by: Avi Fishman \n> Reviewed-by: Joel Stanley \n> Tested-by: Tomer Maimon \n> Tested-by: Avi Fishman \n> ---\n> .../arm/cpu-enable-method/nuvoton,npcm7xx-smp | 42 ++++\n> .../devicetree/bindings/arm/npcm/npcm.txt | 6 +\n> arch/arm/boot/dts/nuvoton-npcm750-evb.dts | 48 +++++\n> arch/arm/boot/dts/nuvoton-npcm750.dtsi | 211 +++++++++++++++++++++\n> include/dt-bindings/clock/nuvoton,npcm7xx-clks.h | 39 ++++\n> 5 files changed, 346 insertions(+)\n> create mode 100644 Documentation/devicetree/bindings/arm/cpu-enable-method/nuvoton,npcm7xx-smp\n> create mode 100644 Documentation/devicetree/bindings/arm/npcm/npcm.txt\n> create mode 100644 arch/arm/boot/dts/nuvoton-npcm750-evb.dts\n> create mode 100644 arch/arm/boot/dts/nuvoton-npcm750.dtsi\n> create mode 100644 include/dt-bindings/clock/nuvoton,npcm7xx-clks.h\n> \n> diff --git a/Documentation/devicetree/bindings/arm/cpu-enable-method/nuvoton,npcm7xx-smp b/Documentation/devicetree/bindings/arm/cpu-enable-method/nuvoton,npcm7xx-smp\n> new file mode 100644\n> index 000000000000..e81f85b400cf\n> --- /dev/null\n> +++ b/Documentation/devicetree/bindings/arm/cpu-enable-method/nuvoton,npcm7xx-smp\n> @@ -0,0 +1,42 @@\n> +=========================================================\n> +Secondary CPU enable-method \"nuvoton,npcm7xx-smp\" binding\n> +=========================================================\n> +\n> +To apply to all CPUs, a single \"nuvoton,npcm7xx-smp\" enable method should be\n> +defined in the \"cpus\" node.\n> +\n> +Enable method name:\t\"nuvoton,npcm7xx-smp\"\n> +Compatible machines:\t\"nuvoton,npcm750\"\n> +Compatible CPUs:\t\"arm,cortex-a9\"\n> +Related properties:\t(none)\n> +\n> +Note:\n> +This enable method needs valid nodes compatible with \"arm,cortex-a9-scu\" and\n> +\"nuvoton,npcm750-gcr\".\n> +\n> +Example:\n> +\n> +\tcpus {\n> +\t\t#address-cells = <1>;\n> +\t\t#size-cells = <0>;\n> +\t\tenable-method = \"nuvoton,npcm7xx-smp\";\n> +\n> +\t\tcpu@0 {\n> +\t\t\tdevice_type = \"cpu\";\n> +\t\t\tcompatible = \"arm,cortex-a9\";\n> +\t\t\tclocks = <&clk NPCM7XX_CLK_CPU>;\n> +\t\t\tclock-names = \"clk_cpu\";\n> +\t\t\treg = <0>;\n> +\t\t\tnext-level-cache = <&L2>;\n> +\t\t};\n> +\n> +\t\tcpu@1 {\n> +\t\t\tdevice_type = \"cpu\";\n> +\t\t\tcompatible = \"arm,cortex-a9\";\n> +\t\t\tclocks = <&clk NPCM7XX_CLK_CPU>;\n> +\t\t\tclock-names = \"clk_cpu\";\n> +\t\t\treg = <1>;\n> +\t\t\tnext-level-cache = <&L2>;\n> +\t\t};\n> +\t};\n> +\n> diff --git a/Documentation/devicetree/bindings/arm/npcm/npcm.txt b/Documentation/devicetree/bindings/arm/npcm/npcm.txt\n> new file mode 100644\n> index 000000000000..2d87d9ecea85\n> --- /dev/null\n> +++ b/Documentation/devicetree/bindings/arm/npcm/npcm.txt\n> @@ -0,0 +1,6 @@\n> +NPCM Platforms Device Tree Bindings\n> +-----------------------------------\n> +NPCM750 SoC\n> +Required root node properties:\n> +\t- compatible = \"nuvoton,npcm750\";\n> +\n> diff --git a/arch/arm/boot/dts/nuvoton-npcm750-evb.dts b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts\n> new file mode 100644\n> index 000000000000..a0675e584125\n> --- /dev/null\n> +++ b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts\n> @@ -0,0 +1,48 @@\n> +/*\n> + * DTS file for all NPCM750 SoCs\n> + *\n> + * Copyright 2012 Tomer Maimon \n> + *\n> + * The code contained herein is licensed under the GNU General Public\n> + * License. You may obtain a copy of the GNU General Public License\n> + * Version 2 or later at the following locations:\n> + *\n> + * http://www.opensource.org/licenses/gpl-license.html\n> + * http://www.gnu.org/copyleft/gpl.html\n> + */\n> +\n> +/dts-v1/;\n> +#include \"nuvoton-npcm750.dtsi\"\n> +\n> +/ {\n> +\tmodel = \"Nuvoton npcm750 Development Board (Device Tree)\";\n> +\tcompatible = \"nuvoton,npcm750\";\n> +\n> +\tchosen {\n> +\t\tstdout-path = &serial3;\n> +\t};\n> +\n> +\tmemory {\n> +\t\treg = <0 0x40000000>;\n> +\t};\n> +};\n> +\n> +&watchdog1 {\n> +\tstatus = \"okay\";\n> +};\n> +\n> +&serial0 {\n> +\tstatus = \"okay\";\n> +};\n> +\n> +&serial1 {\n> +\tstatus = \"okay\";\n> +};\n> +\n> +&serial2 {\n> +\tstatus = \"okay\";\n> +};\n> +\n> +&serial3 {\n> +\tstatus = \"okay\";\n> +};\n> diff --git a/arch/arm/boot/dts/nuvoton-npcm750.dtsi b/arch/arm/boot/dts/nuvoton-npcm750.dtsi\n> new file mode 100644\n> index 000000000000..5d8a48e44274\n> --- /dev/null\n> +++ b/arch/arm/boot/dts/nuvoton-npcm750.dtsi\n> @@ -0,0 +1,211 @@\n> +/*\n> + * DTSi file for the NPCM750 SoC\n> + *\n> + * Copyright 2012 Tomer Maimon \n> + *\n> + * The code contained herein is licensed under the GNU General Public\n> + * License. You may obtain a copy of the GNU General Public License\n> + * Version 2 or later at the following locations:\n> + *\n> + * http://www.opensource.org/licenses/gpl-license.html\n> + * http://www.gnu.org/copyleft/gpl.html\n> + */\n> +\n> +#include \n> +#include \n> +\n> +/ {\n> +\t#address-cells = <1>;\n> +\t#size-cells = <1>;\n> +\tinterrupt-parent = <&gic>;\n> +\n> +\tcpus {\n> +\t\t#address-cells = <1>;\n> +\t\t#size-cells = <0>;\n> +\t\tenable-method = \"nuvoton,npcm7xx-smp\";\n> +\n> +\t\tcpu@0 {\n> +\t\t\tdevice_type = \"cpu\";\n> +\t\t\tcompatible = \"arm,cortex-a9\";\n> +\t\t\tclocks = <&clk NPCM7XX_CLK_CPU>;\n> +\t\t\tclock-names = \"clk_cpu\";\n> +\t\t\treg = <0>;\n> +\t\t\tnext-level-cache = <&l2>;\n> +\t\t};\n> +\n> +\t\tcpu@1 {\n> +\t\t\tdevice_type = \"cpu\";\n> +\t\t\tcompatible = \"arm,cortex-a9\";\n> +\t\t\tclocks = <&clk NPCM7XX_CLK_CPU>;\n> +\t\t\tclock-names = \"clk_cpu\";\n> +\t\t\treg = <1>;\n> +\t\t\tnext-level-cache = <&l2>;\n> +\t\t};\n> +\t};\n> +\n\n> +\tgcr: gcr@f0800000 {\n> +\t\tcompatible = \"nuvoton,npcm750-gcr\", \"syscon\",\n> +\t\t\t\"simple-mfd\";\n> +\t\treg = <0xf0800000 0x1000>;\n> +\t};\n> +\n> +\tscu: scu@f03fe000 {\n> +\t\tcompatible = \"arm,cortex-a9-scu\";\n> +\t\treg = <0xf03fe000 0x1000>;\n> +\t};\n> +\n> +\tl2: cache-controller@f03fc000 {\n> +\t\tcompatible = \"arm,pl310-cache\";\n> +\t\treg = <0xf03fc000 0x1000>;\n> +\t\tinterrupts = <0 21 4>;\n> +\t\tcache-unified;\n> +\t\tcache-level = <2>;\n> +\t\tclocks = <&clk NPCM7XX_CLK_AXI>;\n> +\t};\n> +\n> +\tgic: interrupt-controller@f03ff000 {\n> +\t\tcompatible = \"arm,cortex-a9-gic\";\n> +\t\tinterrupt-controller;\n> +\t\t#interrupt-cells = <3>;\n> +\t\treg = <0xf03ff000 0x1000>,\n> +\t\t <0xf03fe100 0x100>;\n> +\t};\n> +\n> +\ttimer@f03fe600 {\n> +\t\tcompatible = \"arm,cortex-a9-twd-timer\";\n> +\t\treg = <0xf03fe600 0x20>;\n> +\t\tinterrupts = <1 13 0x304>;\n> +\t\tclocks = <&clk NPCM7XX_CLK_TIMER>;\n> +\t};\n\nAll these nodes with a memory mapped address should go under a bus node.\n\n> +\n> +\tahb {\n> +\t\t#address-cells = <1>;\n> +\t\t#size-cells = <1>;\n> +\t\tcompatible = \"simple-bus\";\n> +\t\tinterrupt-parent = <&gic>;\n> +\t\tranges = <0x80000000 0x80000000 0x40000000\n> +\t\t\t 0xc0000000 0xc0000000 0x00002000\n> +\t\t\t 0xc0008000 0xc0008000 0x00001000\n> +\t\t\t 0xe0800000 0xe0800000 0x00001000\n> +\t\t\t 0xe1000000 0xe1000000 0x00001000\n> +\t\t\t 0xe8000000 0xe8000000 0x08000000\n\nThese addresses don't appear to be used. These are coming later? They \ncould be collapsed down into 2 entries.\n\n<0x80000000 0x80000000 0x40010000>\n<0xe0800000 0xe0800000 0x0f800000>\n\n> +\t\t\t /* APB start */\n> +\t\t\t 0xf0000000 0xf0000000 0x00005000\n> +\t\t\t 0xf0007000 0xf0007000 0x00005000\n> +\t\t\t 0xf0010000 0xf0010000 0x00008000\n> +\t\t\t 0xf0080000 0xf0080000 0x00010000\n> +\t\t\t 0xf009f000 0xf009f000 0x00001000\n> +\t\t\t 0xf0100000 0xf0100000 0x00005000\n> +\t\t\t 0xf0180000 0xf0180000 0x0000b000\n> +\t\t\t 0xf0200000 0xf0200000 0x00002000\n\nNot necessary to be so fine grained and shouldn't just be 1:1. So \nfor these just:\n\n<0 0xf0000000 0x00900000>\n\n\n> +\t\t\t /* APB end */\n> +\t\t\t 0xf0800000 0xf0800000 0x000fc000\n> +\t\t\t 0xf8000000 0xf8000000 0x02000000\n> +\t\t\t 0xfb000000 0xfb000000 0x00002000>;\n\n> +\n> +\t\tclk: clock-controller@f0801000 {\n> +\t\t\tcompatible = \"nuvoton,npcm750-clk\";\n> +\t\t\t#clock-cells = <1>;\n> +\t\t\treg = <0xf0801000 0x1000>;\n\nThen this becomes: 0x801000 0x1000\n\n> +\t\t\tstatus = \"okay\";\n> +\t\t};\n> +\n> +\t\t/* external clock signal rg1refck, supplied by the phy */\n> +\t\tclk-rg1refck {\n> +\t\t\tcompatible = \"fixed-clock\";\n> +\t\t\t#clock-cells = <0>;\n> +\t\t\tclock-frequency = <125000000>;\n> +\t\t};\n> +\n> +\t\t/* external clock signal rg2refck, supplied by the phy */\n> +\t\tclk-rg2refck {\n> +\t\t\tcompatible = \"fixed-clock\";\n> +\t\t\t#clock-cells = <0>;\n> +\t\t\tclock-frequency = <125000000>;\n> +\t\t};\n> +\n> +\t\tclk-xin {\n> +\t\t\tcompatible = \"fixed-clock\";\n> +\t\t\t#clock-cells = <0>;\n> +\t\t\tclock-frequency = <50000000>;\n> +\t\t};\n\nThese clocks are not on the bus, so move them out of the bus node to the \ntop level.\n\n> +\n> +\t\tapb {\n> +\t\t\t#address-cells = <1>;\n> +\t\t\t#size-cells = <1>;\n> +\t\t\tcompatible = \"simple-bus\";\n> +\t\t\tinterrupt-parent = <&gic>;\n> +\t\t\tranges = <0xf0000000 0xf0000000 0x00005000\n> +\t\t\t\t 0xf0007000 0xf0007000 0x00005000\n> +\t\t\t\t 0xf0010000 0xf0010000 0x00008000\n> +\t\t\t\t 0xf0080000 0xf0080000 0x00010000\n> +\t\t\t\t 0xf009f000 0xf009f000 0x00001000\n> +\t\t\t\t 0xf0100000 0xf0100000 0x00005000\n> +\t\t\t\t 0xf0180000 0xf0180000 0x0000b000\n> +\t\t\t\t 0xf0200000 0xf0200000 0x00002000>;\n\nWith above changes this can be just: <0 0 0x300000>\n\n> +\n> +\t\t\ttimer0: timer@f0000000 {\n> +\t\t\t\tcompatible = \"nuvoton,npcm750-timer\";\n> +\t\t\t\tinterrupts = <0 32 4>;\n> +\t\t\t\treg = <0xf0000000 0x1000>;\n> +\t\t\t\tclocks = <&clk NPCM7XX_CLK_TIMER>;\n> +\t\t\t};\n> +\n> +\t\t\twatchdog0: watchdog@f0008000 {\n> +\t\t\t\tcompatible = \"nuvoton,npcm750-wdt\";\n> +\t\t\t\tinterrupts = <0 47 4>;\n> +\t\t\t\treg = <0xf0008000 0x1000>;\n> +\t\t\t\tstatus = \"disabled\";\n> +\t\t\t\tclocks = <&clk NPCM7XX_CLK_TIMER>;\n> +\t\t\t};\n> +\n> +\t\t\twatchdog1: watchdog@f0009000 {\n> +\t\t\t\tcompatible = \"nuvoton,npcm750-wdt\";\n> +\t\t\t\tinterrupts = <0 48 4>;\n> +\t\t\t\treg = <0xf0009000 0x1000>;\n> +\t\t\t\tstatus = \"disabled\";\n> +\t\t\t\tclocks = <&clk NPCM7XX_CLK_TIMER>;\n> +\t\t\t};\n> +\n> +\t\t\twatchdog2: watchdog@f000a000 {\n> +\t\t\t\tcompatible = \"nuvoton,npcm750-wdt\";\n> +\t\t\t\tinterrupts = <0 49 4>;\n> +\t\t\t\treg = <0xf000a000 0x1000>;\n> +\t\t\t\tstatus = \"disabled\";\n> +\t\t\t\tclocks = <&clk NPCM7XX_CLK_TIMER>;\n> +\t\t\t};\n> +\n> +\t\t\tserial0: serial0@f0001000 {\n\nSorry I miss this earlier, but need to drop the 0 in the node names. \nIOW, should be \"serial@f0001000\".\n\n> +\t\t\t\tcompatible = \"nuvoton,npcm750-uart\";\n> +\t\t\t\treg = <0xf0001000 0x1000>;\n> +\t\t\t\tclocks = <&clk NPCM7XX_CLK_UART_CORE>;\n> +\t\t\t\tinterrupts = <0 2 4>;\n> +\t\t\t\tstatus = \"disabled\";\n> +\t\t\t};\n> +\n> +\t\t\tserial1: serial1@f0002000 {\n> +\t\t\t\tcompatible = \"nuvoton,npcm750-uart\";\n> +\t\t\t\treg = <0xf0002000 0x1000>;\n> +\t\t\t\tclocks = <&clk NPCM7XX_CLK_UART_CORE>;\n> +\t\t\t\tinterrupts = <0 3 4>;\n> +\t\t\t\tstatus = \"disabled\";\n> +\t\t\t};\n> +\n> +\t\t\tserial2: serial2@f0003000 {\n> +\t\t\t\tcompatible = \"nuvoton,npcm750-uart\";\n> +\t\t\t\treg = <0xf0003000 0x1000>;\n> +\t\t\t\tclocks = <&clk NPCM7XX_CLK_UART_CORE>;\n> +\t\t\t\tinterrupts = <0 4 4>;\n> +\t\t\t\tstatus = \"disabled\";\n> +\t\t\t};\n> +\n> +\t\t\tserial3: serial3@f0004000 {\n> +\t\t\t\tcompatible = \"nuvoton,npcm750-uart\";\n> +\t\t\t\treg = <0xf0004000 0x1000>;\n> +\t\t\t\tclocks = <&clk NPCM7XX_CLK_UART_CORE>;\n> +\t\t\t\tinterrupts = <0 5 4>;\n> +\t\t\t\tstatus = \"disabled\";\n> +\t\t\t};\n> +\t\t};\n> +\t};\n> +};","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"hCmNgYK4\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y2WWq0YyRz9t6B\n\tfor ;\n\tThu, 28 Sep 2017 07:42:47 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxK6L-00019Q-DB; Wed, 27 Sep 2017 21:42:41 +0000","from mail-pf0-f195.google.com ([209.85.192.195])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dxK6G-0000k2-Ex for linux-arm-kernel@lists.infradead.org;\n\tWed, 27 Sep 2017 21:42:38 +0000","by mail-pf0-f195.google.com with SMTP id f84so7783991pfj.3\n\tfor ;\n\tWed, 27 Sep 2017 14:42:16 -0700 (PDT)","from localhost ([70.35.39.2]) by smtp.gmail.com with ESMTPSA id\n\tq73sm24161438pfl.147.2017.09.27.14.42.14\n\t(version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256);\n\tWed, 27 Sep 2017 14:42:14 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=wSBiqLR3Sen13JlS/UsoWwNL2rEXIi/ZoVlhyx3dfKU=;\n\tb=hCmNgYK4WfF6WH\n\tROi1+tihiSWoQG5s+EvvwVpBSyMWr+VMeQV7NusOGgHdFJtadwJX2tIJOWCTuQk+aFBKFpYiGtpNf\n\tnFtWj9TSu57Uyb5/U5sm+Ig8Rj2x0OfNDqjtoJon/SB0XjR9yXikWJWsE/GUe4mMDX161H1CQuxDx\n\tmJMILYOexlWhw1f71WE/VmeHEefDAUkRkRBBcEsoht9Y7nF7wPfCJ1/mZtTCCspdYgLTOvKpqvqpW\n\tXEd/HVLX1pxrgxkQCXnJ005h3tdsg1FM8AmT+XwtDkz/oZt/FLlrA6Yr9pewWX8cFzXQfHANYHHbO\n\toRkS/61Vq0rt/HsYEdYA==;","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:in-reply-to:user-agent;\n\tbh=zv/ywiAqDAAoyrw07Lu+jz+zrNCGG4qzsJIqvLTc69Y=;\n\tb=jYQK3nIRewsInNdqjPSUp5OgdVZGDTDvpQTdPMGwB9GULdB9ksZlOqH4Xf9o0kGXTA\n\tWCmB4zXeLgWJPTdb3LXdOvyJFxLoPxc7ngCHY/Y59w5o6tzoxsoSCfY23eL878CfuHPY\n\tUs3adk68waywwYWMeZpsERVj8B6z6VgY+21cTEQzP8O0Ovo6VJp7xyCF2OvfYtMkhqxi\n\t1k4TW5PZod6sazmHaYq3ftg9Fwtkvyo00AdGP9F/LD8NOpiImm0jBxQqp1wReK2GjIxO\n\tQRQUjD1L8h5WzkZS9teF9xa6hbkB3j0aHBSYLPjbYwOeGJsUseWxhaZ5p7LnkRkAhXrW\n\tMUuA==","X-Gm-Message-State":"AHPjjUhCTy2A4ch8EmTldcUQvM9mMaDuKre2arFtL5vniEvC/QjGXHbb\n\tWZqnkPaDR+dqZ9kAGcYBGA==","X-Google-Smtp-Source":"AOwi7QB5l7hwJHay88HX7T4yLAei2LeFobrznswYfecJoj9hyFjM1ax8o8U8+dFFkgS5Wv2BCWoqCA==","X-Received":"by 10.84.232.134 with SMTP id i6mr2294995plk.190.1506548535419; \n\tWed, 27 Sep 2017 14:42:15 -0700 (PDT)","Date":"Wed, 27 Sep 2017 16:42:13 -0500","From":"Rob Herring ","To":"Brendan Higgins ","Subject":"Re: [PATCH v6 2/3] arm: dts: add Nuvoton NPCM750 device tree","Message-ID":"<20170927214213.qjz7qalvvmhpxxmy@rob-hp-laptop>","References":"<20170919224001.22284-1-brendanhiggins@google.com>\n\t<20170919224001.22284-3-brendanhiggins@google.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170919224001.22284-3-brendanhiggins@google.com>","User-Agent":"NeoMutt/20170113 (1.7.2)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170927_144236_615805_17333C6C ","X-CRM114-Status":"GOOD ( 19.92 )","X-Spam-Score":"-4.2 (----)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-4.2 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [209.85.192.195 listed in list.dnswl.org]\n\t-2.8 RCVD_IN_MSPIKE_H2 RBL: Average reputation (+2)\n\t[209.85.192.195 listed in wl.mailspike.net]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t0.2 FREEMAIL_ENVFROM_END_DIGIT Envelope-from freemail username ends\n\tin digit (robherring2[at]gmail.com)\n\t0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\n\tprovider (robherring2[at]gmail.com)\n\t0.0 HEADER_FROM_DIFFERENT_DOMAINS From and EnvelopeFrom 2nd level\n\tmail domains are different\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t0.2 FREEMAIL_FORGED_FROMDOMAIN 2nd level domains in From and\n\tEnvelopeFrom freemail headers are different","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"mark.rutland@arm.com, devicetree@vger.kernel.org, f.fainelli@gmail.com, \n\ttmaimon77@gmail.com, avifishman70@gmail.com, openbmc@lists.ozlabs.org,\n\tlinux@armlinux.org.uk, linux-kernel@vger.kernel.org,\n\traltherr@google.com, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1790852,"web_url":"http://patchwork.ozlabs.org/comment/1790852/","msgid":"","list_archive_url":null,"date":"2017-10-19T19:04:36","subject":"Re: [PATCH v6 2/3] arm: dts: add Nuvoton NPCM750 device tree","submitter":{"id":69647,"url":"http://patchwork.ozlabs.org/api/people/69647/","name":"Brendan Higgins","email":"brendanhiggins@google.com"},"content":"Sorry for the delay. A couple questions:\n\nOn Wed, Sep 27, 2017 at 2:42 PM, Rob Herring wrote:\n> On Tue, Sep 19, 2017 at 03:40:00PM -0700, Brendan Higgins wrote:\n>> Add a common device tree for all Nuvoton NPCM750 BMCs and a board\n>> specific device tree for the NPCM750 (Poleg) evaluation board.\n>>\n>> Signed-off-by: Brendan Higgins \n>> Reviewed-by: Tomer Maimon \n>> Reviewed-by: Avi Fishman \n>> Reviewed-by: Joel Stanley \n>> Tested-by: Tomer Maimon \n>> Tested-by: Avi Fishman \n>> ---\n>> .../arm/cpu-enable-method/nuvoton,npcm7xx-smp | 42 ++++\n>> .../devicetree/bindings/arm/npcm/npcm.txt | 6 +\n>> arch/arm/boot/dts/nuvoton-npcm750-evb.dts | 48 +++++\n>> arch/arm/boot/dts/nuvoton-npcm750.dtsi | 211 +++++++++++++++++++++\n>> include/dt-bindings/clock/nuvoton,npcm7xx-clks.h | 39 ++++\n>> 5 files changed, 346 insertions(+)\n>> create mode 100644 Documentation/devicetree/bindings/arm/cpu-enable-method/nuvoton,npcm7xx-smp\n>> create mode 100644 Documentation/devicetree/bindings/arm/npcm/npcm.txt\n>> create mode 100644 arch/arm/boot/dts/nuvoton-npcm750-evb.dts\n>> create mode 100644 arch/arm/boot/dts/nuvoton-npcm750.dtsi\n>> create mode 100644 include/dt-bindings/clock/nuvoton,npcm7xx-clks.h\n>>\n>> diff --git a/Documentation/devicetree/bindings/arm/cpu-enable-method/nuvoton,npcm7xx-smp b/Documentation/devicetree/bindings/arm/cpu-enable-method/nuvoton,npcm7xx-smp\n>> new file mode 100644\n>> index 000000000000..e81f85b400cf\n>> --- /dev/null\n>> +++ b/Documentation/devicetree/bindings/arm/cpu-enable-method/nuvoton,npcm7xx-smp\n>> @@ -0,0 +1,42 @@\n>> +=========================================================\n>> +Secondary CPU enable-method \"nuvoton,npcm7xx-smp\" binding\n>> +=========================================================\n>> +\n>> +To apply to all CPUs, a single \"nuvoton,npcm7xx-smp\" enable method should be\n>> +defined in the \"cpus\" node.\n>> +\n>> +Enable method name: \"nuvoton,npcm7xx-smp\"\n>> +Compatible machines: \"nuvoton,npcm750\"\n>> +Compatible CPUs: \"arm,cortex-a9\"\n>> +Related properties: (none)\n>> +\n>> +Note:\n>> +This enable method needs valid nodes compatible with \"arm,cortex-a9-scu\" and\n>> +\"nuvoton,npcm750-gcr\".\n>> +\n>> +Example:\n>> +\n>> + cpus {\n>> + #address-cells = <1>;\n>> + #size-cells = <0>;\n>> + enable-method = \"nuvoton,npcm7xx-smp\";\n>> +\n>> + cpu@0 {\n>> + device_type = \"cpu\";\n>> + compatible = \"arm,cortex-a9\";\n>> + clocks = <&clk NPCM7XX_CLK_CPU>;\n>> + clock-names = \"clk_cpu\";\n>> + reg = <0>;\n>> + next-level-cache = <&L2>;\n>> + };\n>> +\n>> + cpu@1 {\n>> + device_type = \"cpu\";\n>> + compatible = \"arm,cortex-a9\";\n>> + clocks = <&clk NPCM7XX_CLK_CPU>;\n>> + clock-names = \"clk_cpu\";\n>> + reg = <1>;\n>> + next-level-cache = <&L2>;\n>> + };\n>> + };\n>> +\n>> diff --git a/Documentation/devicetree/bindings/arm/npcm/npcm.txt b/Documentation/devicetree/bindings/arm/npcm/npcm.txt\n>> new file mode 100644\n>> index 000000000000..2d87d9ecea85\n>> --- /dev/null\n>> +++ b/Documentation/devicetree/bindings/arm/npcm/npcm.txt\n>> @@ -0,0 +1,6 @@\n>> +NPCM Platforms Device Tree Bindings\n>> +-----------------------------------\n>> +NPCM750 SoC\n>> +Required root node properties:\n>> + - compatible = \"nuvoton,npcm750\";\n>> +\n>> diff --git a/arch/arm/boot/dts/nuvoton-npcm750-evb.dts b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts\n>> new file mode 100644\n>> index 000000000000..a0675e584125\n>> --- /dev/null\n>> +++ b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts\n>> @@ -0,0 +1,48 @@\n>> +/*\n>> + * DTS file for all NPCM750 SoCs\n>> + *\n>> + * Copyright 2012 Tomer Maimon \n>> + *\n>> + * The code contained herein is licensed under the GNU General Public\n>> + * License. You may obtain a copy of the GNU General Public License\n>> + * Version 2 or later at the following locations:\n>> + *\n>> + * http://www.opensource.org/licenses/gpl-license.html\n>> + * http://www.gnu.org/copyleft/gpl.html\n>> + */\n>> +\n>> +/dts-v1/;\n>> +#include \"nuvoton-npcm750.dtsi\"\n>> +\n>> +/ {\n>> + model = \"Nuvoton npcm750 Development Board (Device Tree)\";\n>> + compatible = \"nuvoton,npcm750\";\n>> +\n>> + chosen {\n>> + stdout-path = &serial3;\n>> + };\n>> +\n>> + memory {\n>> + reg = <0 0x40000000>;\n>> + };\n>> +};\n>> +\n>> +&watchdog1 {\n>> + status = \"okay\";\n>> +};\n>> +\n>> +&serial0 {\n>> + status = \"okay\";\n>> +};\n>> +\n>> +&serial1 {\n>> + status = \"okay\";\n>> +};\n>> +\n>> +&serial2 {\n>> + status = \"okay\";\n>> +};\n>> +\n>> +&serial3 {\n>> + status = \"okay\";\n>> +};\n>> diff --git a/arch/arm/boot/dts/nuvoton-npcm750.dtsi b/arch/arm/boot/dts/nuvoton-npcm750.dtsi\n>> new file mode 100644\n>> index 000000000000..5d8a48e44274\n>> --- /dev/null\n>> +++ b/arch/arm/boot/dts/nuvoton-npcm750.dtsi\n>> @@ -0,0 +1,211 @@\n>> +/*\n>> + * DTSi file for the NPCM750 SoC\n>> + *\n>> + * Copyright 2012 Tomer Maimon \n>> + *\n>> + * The code contained herein is licensed under the GNU General Public\n>> + * License. You may obtain a copy of the GNU General Public License\n>> + * Version 2 or later at the following locations:\n>> + *\n>> + * http://www.opensource.org/licenses/gpl-license.html\n>> + * http://www.gnu.org/copyleft/gpl.html\n>> + */\n>> +\n>> +#include \n>> +#include \n>> +\n>> +/ {\n>> + #address-cells = <1>;\n>> + #size-cells = <1>;\n>> + interrupt-parent = <&gic>;\n>> +\n>> + cpus {\n>> + #address-cells = <1>;\n>> + #size-cells = <0>;\n>> + enable-method = \"nuvoton,npcm7xx-smp\";\n>> +\n>> + cpu@0 {\n>> + device_type = \"cpu\";\n>> + compatible = \"arm,cortex-a9\";\n>> + clocks = <&clk NPCM7XX_CLK_CPU>;\n>> + clock-names = \"clk_cpu\";\n>> + reg = <0>;\n>> + next-level-cache = <&l2>;\n>> + };\n>> +\n>> + cpu@1 {\n>> + device_type = \"cpu\";\n>> + compatible = \"arm,cortex-a9\";\n>> + clocks = <&clk NPCM7XX_CLK_CPU>;\n>> + clock-names = \"clk_cpu\";\n>> + reg = <1>;\n>> + next-level-cache = <&l2>;\n>> + };\n>> + };\n>> +\n>\n>> + gcr: gcr@f0800000 {\n>> + compatible = \"nuvoton,npcm750-gcr\", \"syscon\",\n>> + \"simple-mfd\";\n>> + reg = <0xf0800000 0x1000>;\n>> + };\n>> +\n>> + scu: scu@f03fe000 {\n>> + compatible = \"arm,cortex-a9-scu\";\n>> + reg = <0xf03fe000 0x1000>;\n>> + };\n>> +\n>> + l2: cache-controller@f03fc000 {\n>> + compatible = \"arm,pl310-cache\";\n>> + reg = <0xf03fc000 0x1000>;\n>> + interrupts = <0 21 4>;\n>> + cache-unified;\n>> + cache-level = <2>;\n>> + clocks = <&clk NPCM7XX_CLK_AXI>;\n>> + };\n>> +\n>> + gic: interrupt-controller@f03ff000 {\n>> + compatible = \"arm,cortex-a9-gic\";\n>> + interrupt-controller;\n>> + #interrupt-cells = <3>;\n>> + reg = <0xf03ff000 0x1000>,\n>> + <0xf03fe100 0x100>;\n>> + };\n>> +\n>> + timer@f03fe600 {\n>> + compatible = \"arm,cortex-a9-twd-timer\";\n>> + reg = <0xf03fe600 0x20>;\n>> + interrupts = <1 13 0x304>;\n>> + clocks = <&clk NPCM7XX_CLK_TIMER>;\n>> + };\n>\n> All these nodes with a memory mapped address should go under a bus node.\n>\n>> +\n>> + ahb {\n>> + #address-cells = <1>;\n>> + #size-cells = <1>;\n>> + compatible = \"simple-bus\";\n>> + interrupt-parent = <&gic>;\n>> + ranges = <0x80000000 0x80000000 0x40000000\n>> + 0xc0000000 0xc0000000 0x00002000\n>> + 0xc0008000 0xc0008000 0x00001000\n>> + 0xe0800000 0xe0800000 0x00001000\n>> + 0xe1000000 0xe1000000 0x00001000\n>> + 0xe8000000 0xe8000000 0x08000000\n>\n> These addresses don't appear to be used. These are coming later? They\n> could be collapsed down into 2 entries.\n\nYep, the other addresses will be used in later patch sets. This is part of why I\ndid not think that it made sense to do address translation here. I thought that\nmapping such a large range of addresses would not make it any easier to read,\nquite the opposite. At least addresses 'mapped' to the original address\ncorrespond to the datasheet. So I am guessing that is not what you are asking\nme to do.\n\nMost of the large mappings (on the orders of MB) correspond to memory\nlocations mapped on either one of the SPI busses or PCIe busses, which when\nimplemented would likely get their own busses under ahb.\n\nAre you asking that I remap just the addresses for controlling SoC devices\n(0xf0xxxxxx), leave the external busses as is, and then remap those within their\nbusses when we get to that point?\n\nI will try to implement what I think you are asking me to do and get a patch out\nlater today. Hopefully that will make this discussion easier.\n\n>\n> <0x80000000 0x80000000 0x40010000>\n> <0xe0800000 0xe0800000 0x0f800000>\n>\n>> + /* APB start */\n>> + 0xf0000000 0xf0000000 0x00005000\n>> + 0xf0007000 0xf0007000 0x00005000\n>> + 0xf0010000 0xf0010000 0x00008000\n>> + 0xf0080000 0xf0080000 0x00010000\n>> + 0xf009f000 0xf009f000 0x00001000\n>> + 0xf0100000 0xf0100000 0x00005000\n>> + 0xf0180000 0xf0180000 0x0000b000\n>> + 0xf0200000 0xf0200000 0x00002000\n>\n> Not necessary to be so fine grained and shouldn't just be 1:1. So\n> for these just:\n>\n> <0 0xf0000000 0x00900000>\n>\n>\n>> + /* APB end */\n>> + 0xf0800000 0xf0800000 0x000fc000\n>> + 0xf8000000 0xf8000000 0x02000000\n>> + 0xfb000000 0xfb000000 0x00002000>;\n>\n>> +\n>> + clk: clock-controller@f0801000 {\n>> + compatible = \"nuvoton,npcm750-clk\";\n>> + #clock-cells = <1>;\n>> + reg = <0xf0801000 0x1000>;\n>\n> Then this becomes: 0x801000 0x1000\n>\n>> + status = \"okay\";\n>> + };\n>> +\n>> + /* external clock signal rg1refck, supplied by the phy */\n>> + clk-rg1refck {\n>> + compatible = \"fixed-clock\";\n>> + #clock-cells = <0>;\n>> + clock-frequency = <125000000>;\n>> + };\n>> +\n>> + /* external clock signal rg2refck, supplied by the phy */\n>> + clk-rg2refck {\n>> + compatible = \"fixed-clock\";\n>> + #clock-cells = <0>;\n>> + clock-frequency = <125000000>;\n>> + };\n>> +\n>> + clk-xin {\n>> + compatible = \"fixed-clock\";\n>> + #clock-cells = <0>;\n>> + clock-frequency = <50000000>;\n>> + };\n>\n> These clocks are not on the bus, so move them out of the bus node to the\n> top level.\n>\n>> +\n>> + apb {\n>> + #address-cells = <1>;\n>> + #size-cells = <1>;\n>> + compatible = \"simple-bus\";\n>> + interrupt-parent = <&gic>;\n>> + ranges = <0xf0000000 0xf0000000 0x00005000\n>> + 0xf0007000 0xf0007000 0x00005000\n>> + 0xf0010000 0xf0010000 0x00008000\n>> + 0xf0080000 0xf0080000 0x00010000\n>> + 0xf009f000 0xf009f000 0x00001000\n>> + 0xf0100000 0xf0100000 0x00005000\n>> + 0xf0180000 0xf0180000 0x0000b000\n>> + 0xf0200000 0xf0200000 0x00002000>;\n>\n> With above changes this can be just: <0 0 0x300000>\n>\n>> +\n>> + timer0: timer@f0000000 {\n>> + compatible = \"nuvoton,npcm750-timer\";\n>> + interrupts = <0 32 4>;\n>> + reg = <0xf0000000 0x1000>;\n>> + clocks = <&clk NPCM7XX_CLK_TIMER>;\n>> + };\n>> +\n>> + watchdog0: watchdog@f0008000 {\n>> + compatible = \"nuvoton,npcm750-wdt\";\n>> + interrupts = <0 47 4>;\n>> + reg = <0xf0008000 0x1000>;\n>> + status = \"disabled\";\n>> + clocks = <&clk NPCM7XX_CLK_TIMER>;\n>> + };\n>> +\n>> + watchdog1: watchdog@f0009000 {\n>> + compatible = \"nuvoton,npcm750-wdt\";\n>> + interrupts = <0 48 4>;\n>> + reg = <0xf0009000 0x1000>;\n>> + status = \"disabled\";\n>> + clocks = <&clk NPCM7XX_CLK_TIMER>;\n>> + };\n>> +\n>> + watchdog2: watchdog@f000a000 {\n>> + compatible = \"nuvoton,npcm750-wdt\";\n>> + interrupts = <0 49 4>;\n>> + reg = <0xf000a000 0x1000>;\n>> + status = \"disabled\";\n>> + clocks = <&clk NPCM7XX_CLK_TIMER>;\n>> + };\n>> +\n>> + serial0: serial0@f0001000 {\n>\n> Sorry I miss this earlier, but need to drop the 0 in the node names.\n> IOW, should be \"serial@f0001000\".\n>\n>> + compatible = \"nuvoton,npcm750-uart\";\n>> + reg = <0xf0001000 0x1000>;\n>> + clocks = <&clk NPCM7XX_CLK_UART_CORE>;\n>> + interrupts = <0 2 4>;\n>> + status = \"disabled\";\n>> + };\n>> +\n>> + serial1: serial1@f0002000 {\n>> + compatible = \"nuvoton,npcm750-uart\";\n>> + reg = <0xf0002000 0x1000>;\n>> + clocks = <&clk NPCM7XX_CLK_UART_CORE>;\n>> + interrupts = <0 3 4>;\n>> + status = \"disabled\";\n>> + };\n>> +\n>> + serial2: serial2@f0003000 {\n>> + compatible = \"nuvoton,npcm750-uart\";\n>> + reg = <0xf0003000 0x1000>;\n>> + clocks = <&clk NPCM7XX_CLK_UART_CORE>;\n>> + interrupts = <0 4 4>;\n>> + status = \"disabled\";\n>> + };\n>> +\n>> + serial3: serial3@f0004000 {\n>> + compatible = \"nuvoton,npcm750-uart\";\n>> + reg = <0xf0004000 0x1000>;\n>> + clocks = <&clk NPCM7XX_CLK_UART_CORE>;\n>> + interrupts = <0 5 4>;\n>> + status = \"disabled\";\n>> + };\n>> + };\n>> + };\n>> +};\n\nThanks!","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"AqmqwnZE\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=google.com header.i=@google.com\n\theader.b=\"csbZoaxz\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3yHyzm063qz9t5R\n\tfor ;\n\tFri, 20 Oct 2017 06:05:08 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e5G7t-0001YE-97; Thu, 19 Oct 2017 19:05:05 +0000","from mail-qt0-x242.google.com ([2607:f8b0:400d:c0d::242])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e5G7o-0001EJ-63 for linux-arm-kernel@lists.infradead.org;\n\tThu, 19 Oct 2017 19:05:03 +0000","by mail-qt0-x242.google.com with SMTP id 31so15740953qtz.9\n\tfor ;\n\tThu, 19 Oct 2017 12:04:38 -0700 (PDT)","by 10.140.81.200 with HTTP; Thu, 19 Oct 2017 12:04:36 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:To:Subject:Message-ID:Date:From:\n\tReferences:In-Reply-To:MIME-Version:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=xm2Xp4tGXtnEGqFcGx4XuG6eY6b4mSooAWN7t8gNvBI=;\n\tb=AqmqwnZEwTOVwP\n\t0Jsgdhdpca8YQ4b5mzVkFbfKKHiosza64C2tuZm4P6PBE4LaBHQS/1//kVVHnMTDGGRUolDjAz12y\n\tsqhHB7WV9TX3N6G4I29zl+w3sFHrlJeeUTi3AosT3Jv/OJs51dn/y60vVOZCqI0/VfhL9ivlPxA/A\n\tu3pwbWypjYQuJG6wv1nc6636rLfUfPyWoogXVrBQV9TQr047WlQTwKu6aW/16vexgbm3NY2TfjP6A\n\tayjG2poAn00uxZcL9YBxwYPoV3S5+ktnVJyS2ffgKVlax6hdqjBAtKttiZmlLAepz6Rlqzgl8m/lp\n\tCpTwTbiY3QBpByWr1v5g==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com;\n\ts=20161025; \n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:cc; bh=9AoI0/pUzVfxoP2shWnSGarsBZowbnLa6qVnKYP2HOw=;\n\tb=csbZoaxz9ZQG5R8FLYRWEWQ6gLnjc+0i4F9SFOLVDtQ2nZk6q47M/syu1QuWTtQT6Z\n\tJ547yC0EtnRmxCNiY2Z1VbSFyW9DS/glR7xSM6E3yxXxTOp+7d2W+NMlnSlnQ0ZsO6U0\n\taxxYloYWXsF7J8ANA32NMsJ6CZq20Lz2Aw5Z7AP2aQakZjqZjPyYTf40wPfI0XMQ9ip3\n\t98rQqITxKF0fa2ByT8AULAXHOZlu15qHMxKv8JGex8xrYTuMkniQ+UZo7+j8dFcDPKhQ\n\tIGmp8MIW2MugbzDzwppuqATcMybXIwZwjJK9YFdcDSVs6pZ3zJX9MKD55ZM50Xl/RbHz\n\tezEg=="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:in-reply-to:references:from:date\n\t:message-id:subject:to:cc;\n\tbh=9AoI0/pUzVfxoP2shWnSGarsBZowbnLa6qVnKYP2HOw=;\n\tb=e26gT4svp5JQmAbDeN6O5LDi4h99IJuBv5TxrvMQH94QbC6W9sWOMZWPekry0hZLBS\n\toxuWao5YvBYpUki9UbypqokzD0/k7AcbaWXYiBD8yh9ifNoKMek1w7WxkniouMuOTV1Q\n\tEZvtUqTOYo41js0Dxs58rJ4CHvA2vjzYohsxTh5Gl+W5h3An8IVrx+97SFNUmPOHrg+T\n\tbH4ZHR/nbqIBSVcWT7A4r8owwR84ut/SN/XL23PK80mAX41TzKd7rCf/wezzN2RFOPM4\n\t7e26iZfkP8rhPlsMrLRaKEh9rb5H+izO8jLKEYk1oJTaE6PTutv++14/4/xGkinDkQu4\n\t/vng==","X-Gm-Message-State":"AMCzsaX/j8Ym7rGAJ8CC506T94H2gQdVfu3qSWYYkj6SVNW91qKrJf0R\n\tRoLLwxooSjjRTZLdh7/nRtYstDomeDvNu2pK5wjaxw==","X-Google-Smtp-Source":"ABhQp+QtQLZwD5K6pIHjFBfbZI6tT3pbOBcNEwETRXb+tmzFnqwgeT8hB/rF0x1j3SvT3rJ11kyY1/RuuuRM3QJN3GY=","X-Received":"by 10.237.57.135 with SMTP id m7mr3475038qte.4.1508439877185;\n\tThu, 19 Oct 2017 12:04:37 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<20170927214213.qjz7qalvvmhpxxmy@rob-hp-laptop>","References":"<20170919224001.22284-1-brendanhiggins@google.com>\n\t<20170919224001.22284-3-brendanhiggins@google.com>\n\t<20170927214213.qjz7qalvvmhpxxmy@rob-hp-laptop>","From":"Brendan Higgins ","Date":"Thu, 19 Oct 2017 12:04:36 -0700","Message-ID":"","Subject":"Re: [PATCH v6 2/3] arm: dts: add Nuvoton NPCM750 device tree","To":"Rob Herring ","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171019_120500_400934_C275C680 ","X-CRM114-Status":"GOOD ( 26.29 )","X-Spam-Score":"-2.0 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-2.0 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno\n\ttrust [2607:f8b0:400d:c0d:0:0:0:242 listed in] [list.dnswl.org]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"Mark Rutland ,\n\tdevicetree ,\n\tFlorian Fainelli , tmaimon77@gmail.com,\n\tAvi Fishman ,\n\tOpenBMC Maillist ,\n\tRussell King ,\n\tLinux Kernel Mailing List ,\n\tRick Altherr , linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1795533,"web_url":"http://patchwork.ozlabs.org/comment/1795533/","msgid":"","list_archive_url":null,"date":"2017-10-29T20:33:40","subject":"Re: [PATCH v6 2/3] arm: dts: add Nuvoton NPCM750 device tree","submitter":{"id":72291,"url":"http://patchwork.ozlabs.org/api/people/72291/","name":"Tomer Maimon","email":"tmaimon77@gmail.com"},"content":"On 20 September 2017 at 00:40, Brendan Higgins\n wrote:\n> Add a common device tree for all Nuvoton NPCM750 BMCs and a board\n> specific device tree for the NPCM750 (Poleg) evaluation board.\n>\n> Signed-off-by: Brendan Higgins \n> Reviewed-by: Tomer Maimon \n> Reviewed-by: Avi Fishman \n> Reviewed-by: Joel Stanley \n> Tested-by: Tomer Maimon \n> Tested-by: Avi Fishman \n> ---\n> .../arm/cpu-enable-method/nuvoton,npcm7xx-smp | 42 ++++\n> .../devicetree/bindings/arm/npcm/npcm.txt | 6 +\n> arch/arm/boot/dts/nuvoton-npcm750-evb.dts | 48 +++++\n> arch/arm/boot/dts/nuvoton-npcm750.dtsi | 211 +++++++++++++++++++++\n> include/dt-bindings/clock/nuvoton,npcm7xx-clks.h | 39 ++++\n> 5 files changed, 346 insertions(+)\n> create mode 100644 Documentation/devicetree/bindings/arm/cpu-enable-method/nuvoton,npcm7xx-smp\n> create mode 100644 Documentation/devicetree/bindings/arm/npcm/npcm.txt\n> create mode 100644 arch/arm/boot/dts/nuvoton-npcm750-evb.dts\n> create mode 100644 arch/arm/boot/dts/nuvoton-npcm750.dtsi\n> create mode 100644 include/dt-bindings/clock/nuvoton,npcm7xx-clks.h\n>\n> diff --git a/Documentation/devicetree/bindings/arm/cpu-enable-method/nuvoton,npcm7xx-smp b/Documentation/devicetree/bindings/arm/cpu-enable-method/nuvoton,npcm7xx-smp\n> new file mode 100644\n> index 000000000000..e81f85b400cf\n> --- /dev/null\n> +++ b/Documentation/devicetree/bindings/arm/cpu-enable-method/nuvoton,npcm7xx-smp\n> @@ -0,0 +1,42 @@\n> +=========================================================\n> +Secondary CPU enable-method \"nuvoton,npcm7xx-smp\" binding\n> +=========================================================\n> +\n> +To apply to all CPUs, a single \"nuvoton,npcm7xx-smp\" enable method should be\n> +defined in the \"cpus\" node.\n> +\n> +Enable method name: \"nuvoton,npcm7xx-smp\"\n> +Compatible machines: \"nuvoton,npcm750\"\n> +Compatible CPUs: \"arm,cortex-a9\"\n> +Related properties: (none)\n> +\n> +Note:\n> +This enable method needs valid nodes compatible with \"arm,cortex-a9-scu\" and\n> +\"nuvoton,npcm750-gcr\".\n> +\n> +Example:\n> +\n> + cpus {\n> + #address-cells = <1>;\n> + #size-cells = <0>;\n> + enable-method = \"nuvoton,npcm7xx-smp\";\n> +\n> + cpu@0 {\n> + device_type = \"cpu\";\n> + compatible = \"arm,cortex-a9\";\n> + clocks = <&clk NPCM7XX_CLK_CPU>;\n> + clock-names = \"clk_cpu\";\n> + reg = <0>;\n> + next-level-cache = <&L2>;\n> + };\n> +\n> + cpu@1 {\n> + device_type = \"cpu\";\n> + compatible = \"arm,cortex-a9\";\n> + clocks = <&clk NPCM7XX_CLK_CPU>;\n> + clock-names = \"clk_cpu\";\n> + reg = <1>;\n> + next-level-cache = <&L2>;\n> + };\n> + };\n> +\n> diff --git a/Documentation/devicetree/bindings/arm/npcm/npcm.txt b/Documentation/devicetree/bindings/arm/npcm/npcm.txt\n> new file mode 100644\n> index 000000000000..2d87d9ecea85\n> --- /dev/null\n> +++ b/Documentation/devicetree/bindings/arm/npcm/npcm.txt\n> @@ -0,0 +1,6 @@\n> +NPCM Platforms Device Tree Bindings\n> +-----------------------------------\n> +NPCM750 SoC\n> +Required root node properties:\n> + - compatible = \"nuvoton,npcm750\";\n> +\n> diff --git a/arch/arm/boot/dts/nuvoton-npcm750-evb.dts b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts\n> new file mode 100644\n> index 000000000000..a0675e584125\n> --- /dev/null\n> +++ b/arch/arm/boot/dts/nuvoton-npcm750-evb.dts\n> @@ -0,0 +1,48 @@\n> +/*\n> + * DTS file for all NPCM750 SoCs\n> + *\n> + * Copyright 2012 Tomer Maimon \n> + *\n> + * The code contained herein is licensed under the GNU General Public\n> + * License. You may obtain a copy of the GNU General Public License\n> + * Version 2 or later at the following locations:\n> + *\n> + * http://www.opensource.org/licenses/gpl-license.html\n> + * http://www.gnu.org/copyleft/gpl.html\n> + */\n> +\n> +/dts-v1/;\n> +#include \"nuvoton-npcm750.dtsi\"\n> +\n> +/ {\n> + model = \"Nuvoton npcm750 Development Board (Device Tree)\";\n> + compatible = \"nuvoton,npcm750\";\n> +\n> + chosen {\n> + stdout-path = &serial3;\n> + };\n> +\n> + memory {\n> + reg = <0 0x40000000>;\n> + };\n> +};\n> +\n> +&watchdog1 {\n> + status = \"okay\";\n> +};\n> +\n> +&serial0 {\n> + status = \"okay\";\n> +};\n> +\n> +&serial1 {\n> + status = \"okay\";\n> +};\n> +\n> +&serial2 {\n> + status = \"okay\";\n> +};\n> +\n> +&serial3 {\n> + status = \"okay\";\n> +};\n> diff --git a/arch/arm/boot/dts/nuvoton-npcm750.dtsi b/arch/arm/boot/dts/nuvoton-npcm750.dtsi\n> new file mode 100644\n> index 000000000000..5d8a48e44274\n> --- /dev/null\n> +++ b/arch/arm/boot/dts/nuvoton-npcm750.dtsi\n> @@ -0,0 +1,211 @@\n> +/*\n> + * DTSi file for the NPCM750 SoC\n> + *\n> + * Copyright 2012 Tomer Maimon \n> + *\n> + * The code contained herein is licensed under the GNU General Public\n> + * License. You may obtain a copy of the GNU General Public License\n> + * Version 2 or later at the following locations:\n> + *\n> + * http://www.opensource.org/licenses/gpl-license.html\n> + * http://www.gnu.org/copyleft/gpl.html\n> + */\n> +\n> +#include \n> +#include \n> +\n> +/ {\n> + #address-cells = <1>;\n> + #size-cells = <1>;\n> + interrupt-parent = <&gic>;\n> +\n> + cpus {\n> + #address-cells = <1>;\n> + #size-cells = <0>;\n> + enable-method = \"nuvoton,npcm7xx-smp\";\n> +\n> + cpu@0 {\n> + device_type = \"cpu\";\n> + compatible = \"arm,cortex-a9\";\n> + clocks = <&clk NPCM7XX_CLK_CPU>;\n> + clock-names = \"clk_cpu\";\n> + reg = <0>;\n> + next-level-cache = <&l2>;\n> + };\n> +\n> + cpu@1 {\n> + device_type = \"cpu\";\n> + compatible = \"arm,cortex-a9\";\n> + clocks = <&clk NPCM7XX_CLK_CPU>;\n> + clock-names = \"clk_cpu\";\n> + reg = <1>;\n> + next-level-cache = <&l2>;\n> + };\n> + };\n> +\n> + gcr: gcr@f0800000 {\n> + compatible = \"nuvoton,npcm750-gcr\", \"syscon\",\n> + \"simple-mfd\";\n> + reg = <0xf0800000 0x1000>;\n> + };\n> +\n> + scu: scu@f03fe000 {\n> + compatible = \"arm,cortex-a9-scu\";\n> + reg = <0xf03fe000 0x1000>;\n> + };\n> +\n> + l2: cache-controller@f03fc000 {\n> + compatible = \"arm,pl310-cache\";\n> + reg = <0xf03fc000 0x1000>;\n> + interrupts = <0 21 4>;\n> + cache-unified;\n> + cache-level = <2>;\n> + clocks = <&clk NPCM7XX_CLK_AXI>;\n> + };\n> +\n> + gic: interrupt-controller@f03ff000 {\n> + compatible = \"arm,cortex-a9-gic\";\n> + interrupt-controller;\n> + #interrupt-cells = <3>;\n> + reg = <0xf03ff000 0x1000>,\n> + <0xf03fe100 0x100>;\n> + };\n> +\n> + timer@f03fe600 {\n> + compatible = \"arm,cortex-a9-twd-timer\";\n> + reg = <0xf03fe600 0x20>;\n> + interrupts = <1 13 0x304>;\n> + clocks = <&clk NPCM7XX_CLK_TIMER>;\n> + };\n> +\n> + ahb {\n> + #address-cells = <1>;\n> + #size-cells = <1>;\n> + compatible = \"simple-bus\";\n> + interrupt-parent = <&gic>;\n> + ranges = <0x80000000 0x80000000 0x40000000\n> + 0xc0000000 0xc0000000 0x00002000\n> + 0xc0008000 0xc0008000 0x00001000\n> + 0xe0800000 0xe0800000 0x00001000\n> + 0xe1000000 0xe1000000 0x00001000\n> + 0xe8000000 0xe8000000 0x08000000\n> + /* APB start */\n> + 0xf0000000 0xf0000000 0x00005000\n> + 0xf0007000 0xf0007000 0x00005000\n> + 0xf0010000 0xf0010000 0x00008000\n> + 0xf0080000 0xf0080000 0x00010000\n> + 0xf009f000 0xf009f000 0x00001000\n> + 0xf0100000 0xf0100000 0x00005000\n> + 0xf0180000 0xf0180000 0x0000b000\n> + 0xf0200000 0xf0200000 0x00002000\n> + /* APB end */\n> + 0xf0800000 0xf0800000 0x000fc000\n> + 0xf8000000 0xf8000000 0x02000000\n> + 0xfb000000 0xfb000000 0x00002000>;\n> +\n> + clk: clock-controller@f0801000 {\n> + compatible = \"nuvoton,npcm750-clk\";\n> + #clock-cells = <1>;\n> + reg = <0xf0801000 0x1000>;\n> + status = \"okay\";\n> + };\n> +\n> + /* external clock signal rg1refck, supplied by the phy */\n> + clk-rg1refck {\n> + compatible = \"fixed-clock\";\n> + #clock-cells = <0>;\n> + clock-frequency = <125000000>;\n> + };\n> +\n> + /* external clock signal rg2refck, supplied by the phy */\n> + clk-rg2refck {\n> + compatible = \"fixed-clock\";\n> + #clock-cells = <0>;\n> + clock-frequency = <125000000>;\n> + };\n> +\n> + clk-xin {\n> + compatible = \"fixed-clock\";\n> + #clock-cells = <0>;\n> + clock-frequency = <50000000>;\n> + };\n> +\n> + apb {\n> + #address-cells = <1>;\n> + #size-cells = <1>;\n> + compatible = \"simple-bus\";\n> + interrupt-parent = <&gic>;\n> + ranges = <0xf0000000 0xf0000000 0x00005000\n> + 0xf0007000 0xf0007000 0x00005000\n> + 0xf0010000 0xf0010000 0x00008000\n> + 0xf0080000 0xf0080000 0x00010000\n> + 0xf009f000 0xf009f000 0x00001000\n> + 0xf0100000 0xf0100000 0x00005000\n> + 0xf0180000 0xf0180000 0x0000b000\n> + 0xf0200000 0xf0200000 0x00002000>;\n> +\n> + timer0: timer@f0000000 {\n> + compatible = \"nuvoton,npcm750-timer\";\n> + interrupts = <0 32 4>;\n> + reg = <0xf0000000 0x1000>;\n> + clocks = <&clk NPCM7XX_CLK_TIMER>;\n> + };\n\nI started the upstream process of NPCM7xx clocksource driver, I got\nthe following comment:\nhttps://www.spinics.net/lists/devicetree/msg197916.html\n\nIs it possible to modify the the timer DT entry to:\n\ntimer0: timer@f0008000 {\n compatible = \"nuvoton,npcm7xx-timer\";\n interrupts = <0 32 4>;\n reg = <0xf0008000 0x1000>;\n clocks = <&clk NPCM7XX_CLK_TIMER>;\n};\n\n> +\n> + watchdog0: watchdog@f0008000 {\n> + compatible = \"nuvoton,npcm750-wdt\";\n> + interrupts = <0 47 4>;\n> + reg = <0xf0008000 0x1000>;\n> + status = \"disabled\";\n> + clocks = <&clk NPCM7XX_CLK_TIMER>;\n> + };\n> +\n> + watchdog1: watchdog@f0009000 {\n> + compatible = \"nuvoton,npcm750-wdt\";\n> + interrupts = <0 48 4>;\n> + reg = <0xf0009000 0x1000>;\n> + status = \"disabled\";\n> + clocks = <&clk NPCM7XX_CLK_TIMER>;\n> + };\n> +\n> + watchdog2: watchdog@f000a000 {\n> + compatible = \"nuvoton,npcm750-wdt\";\n> + interrupts = <0 49 4>;\n> + reg = <0xf000a000 0x1000>;\n> + status = \"disabled\";\n> + clocks = <&clk NPCM7XX_CLK_TIMER>;\n> + };\n> +\n> + serial0: serial0@f0001000 {\n> + compatible = \"nuvoton,npcm750-uart\";\n> + reg = <0xf0001000 0x1000>;\n> + clocks = <&clk NPCM7XX_CLK_UART_CORE>;\n> + interrupts = <0 2 4>;\n> + status = \"disabled\";\n> + };\n> +\n> + serial1: serial1@f0002000 {\n> + compatible = \"nuvoton,npcm750-uart\";\n> + reg = <0xf0002000 0x1000>;\n> + clocks = <&clk NPCM7XX_CLK_UART_CORE>;\n> + interrupts = <0 3 4>;\n> + status = \"disabled\";\n> + };\n> +\n> + serial2: serial2@f0003000 {\n> + compatible = \"nuvoton,npcm750-uart\";\n> + reg = <0xf0003000 0x1000>;\n> + clocks = <&clk NPCM7XX_CLK_UART_CORE>;\n> + interrupts = <0 4 4>;\n> + status = \"disabled\";\n> + };\n> +\n> + serial3: serial3@f0004000 {\n> + compatible = \"nuvoton,npcm750-uart\";\n> + reg = <0xf0004000 0x1000>;\n> + clocks = <&clk NPCM7XX_CLK_UART_CORE>;\n> + interrupts = <0 5 4>;\n> + status = \"disabled\";\n> + };\n> + };\n> + };\n> +};\n> diff --git a/include/dt-bindings/clock/nuvoton,npcm7xx-clks.h b/include/dt-bindings/clock/nuvoton,npcm7xx-clks.h\n> new file mode 100644\n> index 000000000000..c69d3bbf7e42\n> --- /dev/null\n> +++ b/include/dt-bindings/clock/nuvoton,npcm7xx-clks.h\n> @@ -0,0 +1,39 @@\n> +/*\n> + * Copyright (C) 2016 Nuvoton Technologies, tali.perry@nuvoton.com\n> + *\n> + * This software is licensed under the terms of the GNU General Public\n> + * License version 2, as published by the Free Software Foundation, and\n> + * may be copied, distributed, and modified under those terms.\n> + */\n> +\n> +#ifndef _DT_BINDINGS_CLK_NPCM7XX_H\n> +#define _DT_BINDINGS_CLK_NPCM7XX_H\n> +\n> +#define NPCM7XX_CLK_PLL0 0\n> +#define NPCM7XX_CLK_PLL1 1\n> +#define NPCM7XX_CLK_PLL2 2\n> +#define NPCM7XX_CLK_GFX 3\n> +#define NPCM7XX_CLK_APB1 4\n> +#define NPCM7XX_CLK_APB2 5\n> +#define NPCM7XX_CLK_APB3 6\n> +#define NPCM7XX_CLK_APB4 7\n> +#define NPCM7XX_CLK_APB5 8\n> +#define NPCM7XX_CLK_MC 9\n> +#define NPCM7XX_CLK_CPU 10\n> +#define NPCM7XX_CLK_SPI0 11\n> +#define NPCM7XX_CLK_SPI3 12\n> +#define NPCM7XX_CLK_SPIX 13\n> +#define NPCM7XX_CLK_UART_CORE 14\n> +#define NPCM7XX_CLK_TIMER 15\n> +#define NPCM7XX_CLK_HOST_UART 16\n> +#define NPCM7XX_CLK_MMC 17\n> +#define NPCM7XX_CLK_SDHC 18\n> +#define NPCM7XX_CLK_ADC 19\n> +#define NPCM7XX_CLK_GFX_MEM 20\n> +#define NPCM7XX_CLK_USB_BRIDGE 21\n> +#define NPCM7XX_CLK_AXI 22\n> +#define NPCM7XX_CLK_AHB 23\n> +#define NPCM7XX_CLK_EMC 24\n> +#define NPCM7XX_CLK_GMAC 25\n> +\n> +#endif\n> --\n> 2.14.1.690.gbb1197296e-goog\n>\n\nThanks!","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"Obx+lc2t\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"AnVdiFZe\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3yQ8Ty1vh9z9t4c\n\tfor ;\n\tMon, 30 Oct 2017 07:34:14 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e8uHW-0006kX-TL; Sun, 29 Oct 2017 20:34:06 +0000","from mail-io0-x243.google.com ([2607:f8b0:4001:c06::243])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1e8uHS-0006j2-I2 for linux-arm-kernel@lists.infradead.org;\n\tSun, 29 Oct 2017 20:34:05 +0000","by mail-io0-x243.google.com with SMTP id b186so22865580iof.8\n\tfor ;\n\tSun, 29 Oct 2017 13:33:42 -0700 (PDT)","by 10.2.28.66 with HTTP; Sun, 29 Oct 2017 13:33:40 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:To:Subject:Message-ID:Date:From:\n\tReferences:In-Reply-To:MIME-Version:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=5AYAw2RMsvDXxME8vYs+tnNQyocu6tOCgDkP+DFj9bA=;\n\tb=Obx+lc2tZCvkqa\n\tyyLaIAnigT0D6X0FjLAFcR8wsFFc2mDLGLSLGxEXsD4NglOp+6rdC1mQYUmV+8fFWh94BZo0Inpx/\n\t+vcE7Csc0V+Xkozc0NmJvsJZ0aIJ/8Cdn2NkqztOMNsLxd8aiAtKJgCuHV1T0RHgnbsQWDErqdpY5\n\t9yhbPUrCC7hUs4VoSpcaI5NmioR3UTLHC8+iHqqW36nXsE4a+HcnfpDFRQqsjlG46tl8lgNBF4YNL\n\tqtCd+nmH3Wu0VnggaECNyZ3E098/EOFjzT2tFACPyF2cuz8/goUSzSYheaJmTEVzNltRlku0FFrU0\n\tlR1EfYxqCuB9C+8YKbNA==;","v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:cc; bh=9+VLfBuxumNYjB5XCWgd5t3EjNbnkdzHkrwLE5SBd0E=;\n\tb=AnVdiFZeW+X26jh3zEuVaAut/CUT9f1+kZKMbJkGJ7ghfCD9/+oTkv6jfjro2eSuTS\n\tt3LS0UgGUJ8MmVn2RuTLPgsgOpFdVfUnd6oCEWHPBhBq4rWckfNn+2aWZbWwd1faPt0P\n\tyoaSVk14oFDS/wnSW2mvg+KhRifN4USrTYuvbMgYnVM+4v+6kk5UBHXxu5XMq37f2JB9\n\t5BbxFkJZw1q16Lki1Yf+XY5j6O0hQTQ6wlpfhiWlrwFTmURgkXKfG5m8ixfYGCGzIhKd\n\tZfUO7ww182ydNt+nr5RAyb8P5+HRkfU1enSyCsYvpc96ZJ8F2Qdgfgc3kfJ9QalFs+sn\n\tuR2A=="],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:in-reply-to:references:from:date\n\t:message-id:subject:to:cc;\n\tbh=9+VLfBuxumNYjB5XCWgd5t3EjNbnkdzHkrwLE5SBd0E=;\n\tb=pJ6hL231r9bupUmj9dEBcWdFigbtVCpnzdTufaUeALb+B7OUqT+raHgB0tmqYJ6X8o\n\thfOjWoaVv7XOw+3FFlTRJnuiVx0BT+oEv7X7qOfO1YgJ7gKw9SBM2wsiiOqcP5gkSKFI\n\tujXivEwiYJSkSicQNB5yNJ++/vtjKJrb+zyvuyXCVOiVxZVhQ+Vxw2wLLGuQScTuvF1e\n\teVkI96dhiGHMr3GNOkpAoFJRGQe5B/Ra3UkjJ7X9q5T2dvCAVFAX6Q9wvmgMgS+XMQZA\n\tXFi5R+Um5vUXcdCXl6sucLyvuDDTDhfCSjHxGA7h0Aeq6lck78BE/w3OYoSq/psj74+6\n\ts+MQ==","X-Gm-Message-State":"AMCzsaX3rhV/RBgFJG/PvDxgd5TYFH078uIy/abOAGriB8s8+LsCrdHa\n\tT42hFvdHW8LXn6KR6rHEY1uU9poANWEJbJJw2Kg=","X-Google-Smtp-Source":"ABhQp+TF+DWoSJwGmtl6hgk68GJ3bw+aeUlrZoVqc1vdS5eh4nOuC+ziD8ZJpjXeyOwDQiMvgsHrU9gnSliNKwhfFUk=","X-Received":"by 10.107.20.137 with SMTP id 131mr8205454iou.137.1509309221192; \n\tSun, 29 Oct 2017 13:33:41 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<20170919224001.22284-3-brendanhiggins@google.com>","References":"<20170919224001.22284-1-brendanhiggins@google.com>\n\t<20170919224001.22284-3-brendanhiggins@google.com>","From":"Tomer Maimon ","Date":"Sun, 29 Oct 2017 22:33:40 +0200","Message-ID":"","Subject":"Re: [PATCH v6 2/3] arm: dts: add Nuvoton NPCM750 device tree","To":"Brendan Higgins ","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171029_133402_687683_652400C1 ","X-CRM114-Status":"GOOD ( 19.99 )","X-Spam-Score":"-1.8 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.8 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno\n\ttrust [2607:f8b0:4001:c06:0:0:0:243 listed in] [list.dnswl.org]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t0.2 FREEMAIL_ENVFROM_END_DIGIT Envelope-from freemail username ends\n\tin digit (tmaimon77[at]gmail.com)\n\t0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\n\tprovider (tmaimon77[at]gmail.com)\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED Message has a DKIM or DK signature,\n\tnot necessarily valid\n\t-0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from\n\tauthor's domain","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"Mark Rutland ,\n\tdevicetree ,\n\tFlorian Fainelli ,\n\tAvi Fishman , \n\tOpenBMC Maillist ,\n\tRussell King - ARM Linux ,\n\tLinux Kernel Mailing List ,\n\tRob Herring , Rick Altherr , \n\tlinux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1797581,"web_url":"http://patchwork.ozlabs.org/comment/1797581/","msgid":"","list_archive_url":null,"date":"2017-11-02T00:41:15","subject":"Re: [PATCH v6 2/3] arm: dts: add Nuvoton NPCM750 device tree","submitter":{"id":67416,"url":"http://patchwork.ozlabs.org/api/people/67416/","name":"Rob Herring","email":"robh+dt@kernel.org"},"content":"On Sun, Oct 29, 2017 at 3:33 PM, Tomer Maimon wrote:\n> On 20 September 2017 at 00:40, Brendan Higgins\n> wrote:\n>> Add a common device tree for all Nuvoton NPCM750 BMCs and a board\n>> specific device tree for the NPCM750 (Poleg) evaluation board.\n>>\n\n>> + timer0: timer@f0000000 {\n>> + compatible = \"nuvoton,npcm750-timer\";\n>> + interrupts = <0 32 4>;\n>> + reg = <0xf0000000 0x1000>;\n>> + clocks = <&clk NPCM7XX_CLK_TIMER>;\n>> + };\n>\n> I started the upstream process of NPCM7xx clocksource driver, I got\n> the following comment:\n> https://www.spinics.net/lists/devicetree/msg197916.html\n>\n> Is it possible to modify the the timer DT entry to:\n>\n> timer0: timer@f0008000 {\n> compatible = \"nuvoton,npcm7xx-timer\";\n\nPlease don't. Using 750 is correct. Using 7xx (i.e. wildcards) is not.\n\nIf there's subsequent SoCs with the same block, then you can do:\n'\"nuvoton,npcm760-timer\", \"nuvoton,npcm750-timer\"'.\n\nRob","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"uB60BeR6\"; dkim-atps=neutral","mail.kernel.org;\n\tdmarc=none (p=none dis=none) header.from=kernel.org","mail.kernel.org;\n\tspf=none smtp.mailfrom=robh+dt@kernel.org"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3yS5rg2FF4z9t34\n\tfor ;\n\tThu, 2 Nov 2017 11:42:10 +1100 (AEDT)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1eA3a6-0005Jp-LN; Thu, 02 Nov 2017 00:42:02 +0000","from mail.kernel.org ([198.145.29.99])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1eA3a2-0005IK-Us for linux-arm-kernel@lists.infradead.org;\n\tThu, 02 Nov 2017 00:42:00 +0000","from mail-qt0-f177.google.com (mail-qt0-f177.google.com\n\t[209.85.216.177])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128\n\tbits)) (No client certificate requested)\n\tby mail.kernel.org (Postfix) with ESMTPSA id 431642192D\n\tfor ;\n\tThu, 2 Nov 2017 00:41:37 +0000 (UTC)","by mail-qt0-f177.google.com with SMTP id v41so4855434qtv.12\n\tfor ;\n\tWed, 01 Nov 2017 17:41:37 -0700 (PDT)","by 10.12.130.134 with HTTP; Wed, 1 Nov 2017 17:41:15 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:To:Subject:Message-ID:Date:From:\n\tReferences:In-Reply-To:MIME-Version:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=x+q1GX6hIjoh9IyuEjJGKJdYwcqkm4Bz+L8s3baezm4=;\n\tb=uB60BeR616kx5N\n\tYLLVwUh86zjglcIRFbowewvCd4Qw3oNrhmGmG/HJCtAYRPNy1EBfbHHt4EY64f+nI7tvLz3EwO6OR\n\tEK+2+7yKO4o7Yid3dbMLNq1tiBcE//elDBQDp3vk5UwZUv1lyaBVO4EQ8H2RRyoc49SSGkxOUasHD\n\tXRc47ehp6ea6QE6dnFjBXrgCXJm0EkUqnipVCGsqbMPSpk3ZwB/g1fArSedIv2a03l1/C4dnEKJZI\n\tflK0OU3KIE1E+9zHX6vPVIu4XNW3WHLWMqDbxouIvAeGy+AyLjThEh03m+PY0AjRfpnc6WBmvk2Yt\n\t8LGpeH6dlmIInxl6Y48A==;","DMARC-Filter":"OpenDMARC Filter v1.3.2 mail.kernel.org 431642192D","X-Gm-Message-State":"AMCzsaVMe1R2N2Xqs7eD7BanuA2xJWRuRrvULqmSr0zPqyrykqnc0DRq\n\tueXLGn6NHVDw8FYgwl+3Ge5e9mWtYQOndEE7FQ==","X-Google-Smtp-Source":"ABhQp+SsNjv1ypLO+Rv9+1A9Pnt/MyopcfVXit0J2Cw7EtwVO7SRkal/DlH0u0IReVSROTUB7tIZeHvN1xXhUvCKmc0=","X-Received":"by 10.200.8.239 with SMTP id y44mr2715082qth.341.1509583296458; \n\tWed, 01 Nov 2017 17:41:36 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"","References":"<20170919224001.22284-1-brendanhiggins@google.com>\n\t<20170919224001.22284-3-brendanhiggins@google.com>\n\t","From":"Rob Herring ","Date":"Wed, 1 Nov 2017 19:41:15 -0500","X-Gmail-Original-Message-ID":"","Message-ID":"","Subject":"Re: [PATCH v6 2/3] arm: dts: add Nuvoton NPCM750 device tree","To":"Tomer Maimon ","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20171101_174159_022199_13E9C035 ","X-CRM114-Status":"GOOD ( 15.09 )","X-Spam-Score":"-6.4 (------)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-6.4 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t0.5 RCVD_IN_SORBS_SPAM RBL: SORBS: sender is a spam source\n\t[209.85.216.177 listed in dnsbl.sorbs.net]\n\t-5.0 RCVD_IN_DNSWL_HI RBL: Sender listed at http://www.dnswl.org/,\n\thigh trust [198.145.29.99 listed in list.dnswl.org]\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"Mark Rutland ,\n\tdevicetree ,\n\tFlorian Fainelli ,\n\tAvi Fishman , \n\tOpenBMC Maillist ,\n\tBrendan Higgins ,\n\tRussell King - ARM Linux ,\n\tLinux Kernel Mailing List ,\n\tRick Altherr ,\n\t\"linux-arm-kernel@lists.infradead.org\"\n\t","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}}]