Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932887AbbHYVOK (ORCPT ); Tue, 25 Aug 2015 17:14:10 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:55616 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751965AbbHYVOH (ORCPT ); Tue, 25 Aug 2015 17:14:07 -0400 Date: Tue, 25 Aug 2015 14:14:05 -0700 From: Stephen Boyd To: Varadarajan Narayanan Cc: Mark Rutland , devicetree@vger.kernel.org, Russell King , Pawel Moll , Ian Campbell , linux-arm-msm@vger.kernel.org, Andy Gross , linux-kernel@vger.kernel.org, Rob Herring , Georgi Djakov , Kumar Gala , David Brown , linux-soc@vger.kernel.org, Lina Iyer , linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v3] qcom: ipq40xx: Add basic board/dts support for IPQ40XX SoC Message-ID: <20150825211405.GJ19120@codeaurora.org> References: <20150824092813.GA15581@codeaurora.org> <20150824224928.GE14330@codeaurora.org> <20150825081042.GA14028@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150825081042.GA14028@codeaurora.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2233 Lines: 61 On 08/25, Varadarajan Narayanan wrote: > On Mon, Aug 24, 2015 at 03:49:28PM -0700, Stephen Boyd wrote: > > On 08/24, Varadarajan Narayanan wrote: > > > > + compatible = "qcom,ipq40xx-r3pc", "qcom,ipq40xx"; > > > + > > > + memory { > > > + device_type = "memory"; > > > + reg = <0x80000000 0x20000000>; /* 512MB */ > > > + }; > > > > Doesn't the bootloader fill the memory node for us? Why do we > > need this? > > > > > + > > > + chosen { > > > + bootargs = "root=/dev/ram rw init=/init console=ttyMSM0,115200n8 initrd=0x82000000,0x000E2246"; > > > + }; > > > > Please don't add bootargs. Use stdout-path for the console part > > and everything else should be done by the bootloader or is the > > defaults. > > Since this is for the emulation platform, we don't have the bootloader. Why are we upstreaming an emulation platform? I assume you have some sort of scripts or something to load the kernel image into the emulator, so those scripts could go and modify the dtb in place to populate the initrd and commandline. If you need pointers on that feel free to contact me directly. > > > > + compatible = "arm,armv7-timer"; > > > + interrupts = <1 2 0xf08>, > > > + <1 3 0xf08>, > > > + <1 4 0xf08>, > > > + <1 1 0xf08>; > > > + clock-frequency = <20833333>; > > > > Drop this clock-frequency part if you can. The hardware should > > properly report the frequency. > > Cannot drop this. This is the ref-clock frequency. For the > chip it is 48MHz, for emulation is ~20MHz. Similar to > https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/arch/arm/boot/dts/qcom-msm8974.dtsi#n89 And the hardware is not emulated properly to report the actual frequency that the timer is running at? That's sad. At the least, this should be in the board specific dtsi file instead of the SoC specific file, because it's a board quirk. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/