Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751965AbdHAMqg (ORCPT ); Tue, 1 Aug 2017 08:46:36 -0400 Received: from mail-wr0-f196.google.com ([209.85.128.196]:33151 "EHLO mail-wr0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751857AbdHAMqd (ORCPT ); Tue, 1 Aug 2017 08:46:33 -0400 Subject: Re: [PATCH v5 2/2] arm64: dts: Add Mediatek SoC MT2712 and evaluation board dts and Makefile To: Yingjoe Chen , YT Shen Cc: Rob Herring , Mark Rutland , devicetree@vger.kernel.org, Jason Cooper , srv_heupstream@mediatek.com, Marc Zyngier , Catalin Marinas , Will Deacon , linux-kernel@vger.kernel.org, Mars Cheng , linux-serial@vger.kernel.org, Greg Kroah-Hartman , linux-mediatek@lists.infradead.org, Thomas Gleixner , linux-arm-kernel@lists.infradead.org References: <1501241835-47849-1-git-send-email-yt.shen@mediatek.com> <1501241835-47849-3-git-send-email-yt.shen@mediatek.com> <1501577481.32089.3.camel@mtksdaap41> From: Matthias Brugger Message-ID: <85f5d713-a599-e4cc-f1b4-2e0dedf2797f@gmail.com> Date: Tue, 1 Aug 2017 14:46:28 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <1501577481.32089.3.camel@mtksdaap41> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1114 Lines: 42 On 08/01/2017 10:51 AM, Yingjoe Chen wrote: > On Fri, 2017-07-28 at 19:37 +0800, YT Shen wrote: >> diff --git a/arch/arm64/boot/dts/mediatek/mt2712e.dtsi b/arch/arm64/boot/dts/mediatek/mt2712e.dtsi >> new file mode 100644 >> index 0000000..1e135af >> --- /dev/null >> +++ b/arch/arm64/boot/dts/mediatek/mt2712e.dtsi > > <...> > >> + timer { >> + compatible = "arm,armv8-timer"; >> + interrupt-parent = <&gic>; >> + interrupts = > + (GIC_CPU_MASK_RAW(0x13) | IRQ_TYPE_LEVEL_LOW)>, >> + > + (GIC_CPU_MASK_RAW(0x13) | IRQ_TYPE_LEVEL_LOW)>, >> + > + (GIC_CPU_MASK_RAW(0x13) | IRQ_TYPE_LEVEL_LOW)>, >> + > + (GIC_CPU_MASK_RAW(0x13) | IRQ_TYPE_LEVEL_LOW)>; >> + }; >> + >> + soc { >> + #address-cells = <2>; >> + #size-cells = <2>; >> + compatible = "simple-bus"; >> + ranges; > > Matthias, > > I notice this have soc node. > Do we need to get rid of it? > Good catch. Yes please get rid of it. We should avoid artificial bus nodes. As soc is no real bus, we should get rid of it. Thanks, Matthias