From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kumar Gala Subject: Re: [PATCH v3 2/4] arm64: dts: Add Qualcomm MSM8916 SoC and evaluation board dts Date: Thu, 12 Mar 2015 12:33:22 -0500 Message-ID: References: <1426107080-29079-1-git-send-email-galak@codeaurora.org> <1426107080-29079-2-git-send-email-galak@codeaurora.org> <20150312170541.GE30145@leverpostej> Mime-Version: 1.0 (Mac OS X Mail 7.3 \(1878.6\)) Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from smtp.codeaurora.org ([198.145.29.96]:46975 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030264AbbCLRd0 convert rfc822-to-8bit (ORCPT ); Thu, 12 Mar 2015 13:33:26 -0400 In-Reply-To: <20150312170541.GE30145@leverpostej> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Mark Rutland Cc: "linux-arm-msm@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "arm@kernel.org" , "devicetree@vger.kernel.org" , "heiko@sntech.de" On Mar 12, 2015, at 12:05 PM, Mark Rutland wrote= : > Hi Kumar, >=20 >> +/ { >> + model =3D "Qualcomm Technologies, Inc. MSM 8916 MTP"; >> + compatible =3D "qcom,msm8916-mtp", "qcom,msm8916-mtp-smb1360", >> + "qcom,msm8916", "qcom,mtp"; >> +}; >=20 > No /chosen/stdout-path? Nope ;). >=20 > Does your UART driver support earlycon? It does. >=20 > [...] >=20 >> + cpus { >> + #address-cells =3D <1>; >> + #size-cells =3D <0>; >> + >> + CPU0: cpu@0 { >> + device_type =3D "cpu"; >> + compatible =3D "arm,cortex-a53", "arm,armv8"; >> + reg =3D <0x0>; >> + }; >> + >> + CPU1: cpu@1 { >> + device_type =3D "cpu"; >> + compatible =3D "arm,cortex-a53", "arm,armv8"; >> + reg =3D <0x1>; >> + }; >> + >> + CPU2: cpu@2 { >> + device_type =3D "cpu"; >> + compatible =3D "arm,cortex-a53", "arm,armv8"; >> + reg =3D <0x2>; >> + }; >> + >> + CPU3: cpu@3 { >> + device_type =3D "cpu"; >> + compatible =3D "arm,cortex-a53", "arm,armv8"; >> + reg =3D <0x3>; >> + }; >> + }; >=20 > The secondary CPUs need an enable-method. Are you using PSCI or > spin-table? This is on purpose. We aren=92t using either PSCI or spin-table. Righ= t now the dts is for booting on a single core. I can drop CPU1..CPU3 i= f that helps. > Which exception level do the CPUs enter the kernel? >=20 >> + timer { >> + compatible =3D "arm,armv7-timer"; >=20 > This should be "arm,armv8-timer=94. will change >=20 >> + interrupts =3D , >> + , >> + , >> + ; >> + clock-frequency =3D <19200000>; >> + }; >=20 > NAK. CNTFRQ should be programmed on all CPUs prior to entering the > kernel, per the boot protocol. You should not need clock-frequency he= re. Will drop clock-frequency. > [...] >=20 >> + intc: interrupt-controller@b000000 { >> + compatible =3D "qcom,msm-qgic2"; >=20 > This string isn't documented (but seems to be supported by the GIC > driver). There=92s a patch posted to add =91qcom,msm-qgic2=92 to the binding doc= =2E > How does this differ from other GIC implementations? Not sure the exact details, just that its qcom=92s on implementation of= the GIC spec. >=20 >> + interrupt-controller; >> + #interrupt-cells =3D <3>; >> + reg =3D <0x0b000000 0x1000>, <0x0b002000 0x1000>; >> + }; >=20 > No GICH, GICV, maintenance interrupt? Nope. >=20 > Minor nit, but I'd prefer if the reg entries were on individual lines= as > happens in other dts. >=20 > Thanks, > Mark. --=20 Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora For= um, a Linux Foundation Collaborative Project From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030309AbbCLRd3 (ORCPT ); Thu, 12 Mar 2015 13:33:29 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:46975 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030264AbbCLRd0 convert rfc822-to-8bit (ORCPT ); Thu, 12 Mar 2015 13:33:26 -0400 Content-Type: text/plain; charset=windows-1252 Mime-Version: 1.0 (Mac OS X Mail 7.3 \(1878.6\)) Subject: Re: [PATCH v3 2/4] arm64: dts: Add Qualcomm MSM8916 SoC and evaluation board dts From: Kumar Gala In-Reply-To: <20150312170541.GE30145@leverpostej> Date: Thu, 12 Mar 2015 12:33:22 -0500 Cc: "linux-arm-msm@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "arm@kernel.org" , "devicetree@vger.kernel.org" , "heiko@sntech.de" Content-Transfer-Encoding: 8BIT Message-Id: References: <1426107080-29079-1-git-send-email-galak@codeaurora.org> <1426107080-29079-2-git-send-email-galak@codeaurora.org> <20150312170541.GE30145@leverpostej> To: Mark Rutland X-Mailer: Apple Mail (2.1878.6) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mar 12, 2015, at 12:05 PM, Mark Rutland wrote: > Hi Kumar, > >> +/ { >> + model = "Qualcomm Technologies, Inc. MSM 8916 MTP"; >> + compatible = "qcom,msm8916-mtp", "qcom,msm8916-mtp-smb1360", >> + "qcom,msm8916", "qcom,mtp"; >> +}; > > No /chosen/stdout-path? Nope ;). > > Does your UART driver support earlycon? It does. > > [...] > >> + cpus { >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + CPU0: cpu@0 { >> + device_type = "cpu"; >> + compatible = "arm,cortex-a53", "arm,armv8"; >> + reg = <0x0>; >> + }; >> + >> + CPU1: cpu@1 { >> + device_type = "cpu"; >> + compatible = "arm,cortex-a53", "arm,armv8"; >> + reg = <0x1>; >> + }; >> + >> + CPU2: cpu@2 { >> + device_type = "cpu"; >> + compatible = "arm,cortex-a53", "arm,armv8"; >> + reg = <0x2>; >> + }; >> + >> + CPU3: cpu@3 { >> + device_type = "cpu"; >> + compatible = "arm,cortex-a53", "arm,armv8"; >> + reg = <0x3>; >> + }; >> + }; > > The secondary CPUs need an enable-method. Are you using PSCI or > spin-table? This is on purpose. We aren’t using either PSCI or spin-table. Right now the dts is for booting on a single core. I can drop CPU1..CPU3 if that helps. > Which exception level do the CPUs enter the kernel? > >> + timer { >> + compatible = "arm,armv7-timer"; > > This should be "arm,armv8-timer”. will change > >> + interrupts = , >> + , >> + , >> + ; >> + clock-frequency = <19200000>; >> + }; > > NAK. CNTFRQ should be programmed on all CPUs prior to entering the > kernel, per the boot protocol. You should not need clock-frequency here. Will drop clock-frequency. > [...] > >> + intc: interrupt-controller@b000000 { >> + compatible = "qcom,msm-qgic2"; > > This string isn't documented (but seems to be supported by the GIC > driver). There’s a patch posted to add ‘qcom,msm-qgic2’ to the binding doc. > How does this differ from other GIC implementations? Not sure the exact details, just that its qcom’s on implementation of the GIC spec. > >> + interrupt-controller; >> + #interrupt-cells = <3>; >> + reg = <0x0b000000 0x1000>, <0x0b002000 0x1000>; >> + }; > > No GICH, GICV, maintenance interrupt? Nope. > > Minor nit, but I'd prefer if the reg entries were on individual lines as > happens in other dts. > > Thanks, > Mark. -- Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project From mboxrd@z Thu Jan 1 00:00:00 1970 From: galak@codeaurora.org (Kumar Gala) Date: Thu, 12 Mar 2015 12:33:22 -0500 Subject: [PATCH v3 2/4] arm64: dts: Add Qualcomm MSM8916 SoC and evaluation board dts In-Reply-To: <20150312170541.GE30145@leverpostej> References: <1426107080-29079-1-git-send-email-galak@codeaurora.org> <1426107080-29079-2-git-send-email-galak@codeaurora.org> <20150312170541.GE30145@leverpostej> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Mar 12, 2015, at 12:05 PM, Mark Rutland wrote: > Hi Kumar, > >> +/ { >> + model = "Qualcomm Technologies, Inc. MSM 8916 MTP"; >> + compatible = "qcom,msm8916-mtp", "qcom,msm8916-mtp-smb1360", >> + "qcom,msm8916", "qcom,mtp"; >> +}; > > No /chosen/stdout-path? Nope ;). > > Does your UART driver support earlycon? It does. > > [...] > >> + cpus { >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + CPU0: cpu at 0 { >> + device_type = "cpu"; >> + compatible = "arm,cortex-a53", "arm,armv8"; >> + reg = <0x0>; >> + }; >> + >> + CPU1: cpu at 1 { >> + device_type = "cpu"; >> + compatible = "arm,cortex-a53", "arm,armv8"; >> + reg = <0x1>; >> + }; >> + >> + CPU2: cpu at 2 { >> + device_type = "cpu"; >> + compatible = "arm,cortex-a53", "arm,armv8"; >> + reg = <0x2>; >> + }; >> + >> + CPU3: cpu at 3 { >> + device_type = "cpu"; >> + compatible = "arm,cortex-a53", "arm,armv8"; >> + reg = <0x3>; >> + }; >> + }; > > The secondary CPUs need an enable-method. Are you using PSCI or > spin-table? This is on purpose. We aren?t using either PSCI or spin-table. Right now the dts is for booting on a single core. I can drop CPU1..CPU3 if that helps. > Which exception level do the CPUs enter the kernel? > >> + timer { >> + compatible = "arm,armv7-timer"; > > This should be "arm,armv8-timer?. will change > >> + interrupts = , >> + , >> + , >> + ; >> + clock-frequency = <19200000>; >> + }; > > NAK. CNTFRQ should be programmed on all CPUs prior to entering the > kernel, per the boot protocol. You should not need clock-frequency here. Will drop clock-frequency. > [...] > >> + intc: interrupt-controller at b000000 { >> + compatible = "qcom,msm-qgic2"; > > This string isn't documented (but seems to be supported by the GIC > driver). There?s a patch posted to add ?qcom,msm-qgic2? to the binding doc. > How does this differ from other GIC implementations? Not sure the exact details, just that its qcom?s on implementation of the GIC spec. > >> + interrupt-controller; >> + #interrupt-cells = <3>; >> + reg = <0x0b000000 0x1000>, <0x0b002000 0x1000>; >> + }; > > No GICH, GICV, maintenance interrupt? Nope. > > Minor nit, but I'd prefer if the reg entries were on individual lines as > happens in other dts. > > Thanks, > Mark. -- Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project