Best regards! Anson Huang > -----Original Message----- > From: Peter Chen [mailto:peter.chen@xxxxxxxxxxxxx] > Sent: 2015-01-20 2:30 PM > To: Shawn Guo; Huang Yongcai-B20788 > Cc: kernel@xxxxxxxxxxxxxx; robh+dt@xxxxxxxxxx; pawel.moll@xxxxxxx; > mark.rutland@xxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; > devicetree@xxxxxxxxxxxxxxx > Subject: Re: [PATCH 1/2] ARM: imx6sx-sdb: change default board as reva > board > > 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 <peter.chen@xxxxxxxxxxxxx> > > > --- > > > 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? As i.MX6SX SDB Rev-B board uses PFuse 200 which has only one power supply for both VDDARM_IN and VDDSOC_IN, so it needs VDDARM_IN equal VDDSOC_IN, currently we set Rev-B board as default in imx6sx.dtsi, which uses same voltage for both VDDARM_IN and VDDSOC_IN.The Rev-A board uses PFuze 100 which has separate power supply for VDDARM_IN and VDDSOC_IN, so it can use different voltage on VDDARM_IN and VDDSOC_IN according to datasheet. Anson > > Peter > > > -- > > Best Regards, > Peter Chen -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
