From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Chen Subject: Re: [PATCH 1/2] ARM: imx6sx-sdb: change default board as reva board Date: Tue, 20 Jan 2015 14:30:18 +0800 Message-ID: <20150120063016.GD13645@shlinux2> References: <1421220998-3491-1-git-send-email-peter.chen@freescale.com> <1421220998-3491-2-git-send-email-peter.chen@freescale.com> <20150120073527.GT2766@dragon> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20150120073527.GT2766@dragon> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Shawn Guo , b20788@freescale.com Cc: mark.rutland@arm.com, devicetree@vger.kernel.org, pawel.moll@arm.com, robh+dt@kernel.org, kernel@pengutronix.de, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org On Tue, Jan 20, 2015 at 03:35:29PM +0800, Shawn Guo wrote: > On Wed, Jan 14, 2015 at 03:36:37PM +0800, Peter Chen wrote: > > The imx6sx sdb board has two revisions, the current mainline one > > is reva which is mainly for internal use. In this commit, we > > rename imx6sx-sdb.dts to imx6sx-sdb.dtsi, and move the reva dedicated > > contents to imx6sx-sdb-reva.dts. > > > > Signed-off-by: Peter Chen > > --- > > arch/arm/boot/dts/imx6sx-sdb-reva.dts | 158 +++++++++++++++++++++ > > .../boot/dts/{imx6sx-sdb.dts => imx6sx-sdb.dtsi} | 131 ----------------- > > arch/arm/boot/dts/Makefile needs an update. > Thanks, I will. > > 2 files changed, 158 insertions(+), 131 deletions(-) > > create mode 100644 arch/arm/boot/dts/imx6sx-sdb-reva.dts > > rename arch/arm/boot/dts/{imx6sx-sdb.dts => imx6sx-sdb.dtsi} (83%) > > > > diff --git a/arch/arm/boot/dts/imx6sx-sdb-reva.dts b/arch/arm/boot/dts/imx6sx-sdb-reva.dts > > new file mode 100644 > > index 0000000..e7a8c34 > > --- /dev/null > > +++ b/arch/arm/boot/dts/imx6sx-sdb-reva.dts > > @@ -0,0 +1,158 @@ > > +/* > > + * Copyright (C) 2014-2015 Freescale Semiconductor, Inc. > > + * > > + * This program is free software; you can redistribute it and/or modify > > + * it under the terms of the GNU General Public License version 2 as > > + * published by the Free Software Foundation. > > + */ > > + > > +#include "imx6sx-sdb.dtsi" > > + > > +/ { > > + model = "Freescale i.MX6 SoloX SDB RevA Board"; > > +}; > > + > > +&cpu0 { > > + operating-points = < > > + /* kHz uV */ > > + 996000 1250000 > > + 792000 1175000 > > + 396000 1075000 > > + >; > > + fsl,soc-operating-points = < > > + /* ARM kHz SOC uV */ > > + 996000 1175000 > > + 792000 1175000 > > + 396000 1175000 > > + >; > > I'm wondering why different revision of the board require a different > OPP settings. > Anson, do you know the reason? Peter -- Best Regards, Peter Chen