From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Ripard Date: Mon, 12 Dec 2016 13:33:07 +0100 Subject: [U-Boot] [PATCH v2 12/23] sunxi: provide default DRAM config for sun50i in Kconfig In-Reply-To: <66393a39-9e44-e676-98f8-81ca6cf0c235@arm.com> References: <1480902750-839-1-git-send-email-andre.przywara@arm.com> <1480902750-839-13-git-send-email-andre.przywara@arm.com> <20161206105657.z4pe7jpjgjv7qh6e@lukather> <66393a39-9e44-e676-98f8-81ca6cf0c235@arm.com> Message-ID: <20161212123307.3ltdpxrfoxgvao3s@lukather> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Tue, Dec 06, 2016 at 11:21:26AM +0000, Andre Przywara wrote: > Hi, > > On 06/12/16 10:56, Maxime Ripard wrote: > > On Mon, Dec 05, 2016 at 01:52:19AM +0000, Andre Przywara wrote: > >> To avoid enumerating the very same DRAM values in defconfig files > >> for each and every Allwinner A64 board out there, let's put some sane > >> default values in the Kconfig file. > >> Boards with different needs can override them at any time. > > > > Did you check other boards to see what their values were before > > calling it a default? > > I sampled all the boards (two ;-) I have access to and got a 100% > coverage ;-) > As far as I know, this 672 MHz seems to be an Allwinner recommendation, > though it needs to be worked out how stable this is under load. > I have no idea if the ZQ value is similarly common. > > If you have any other data, I am all ears and happy to use a different > safe default if this is the intention. > But we could use a "most common value" approach here to avoid pointless > defconfig entries with the same value for lots of boards, so even if > there are board with can't do 672 MHz, for instance, we still put it in > Kconfig and let that one board override it. Fair enough. Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 801 bytes Desc: not available URL: