[PATCH 1/2] ARM: imx6sx-sdb: change default board as reva board

[Anson.Huang at freescale.com]

Best regards!
Anson Huang


> -----Original Message-----
> From: Peter Chen [mailto:peter.chen at freescale.com]
> Sent: 2015-01-20 2:30 PM
> To: Shawn Guo; Huang Yongcai-B20788
> Cc: kernel at pengutronix.de; robh+dt at kernel.org; pawel.moll at arm.com;
> mark.rutland at arm.com; linux-arm-kernel at lists.infradead.org;
> devicetree at vger.kernel.org
> 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 at freescale.com>
> > > ---
> > >  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