diff mbox

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

Message ID 1421220998-3491-2-git-send-email-peter.chen@freescale.com (mailing list archive)
State New, archived
Headers show

Commit Message

Peter Chen Jan. 14, 2015, 7:36 a.m. UTC
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@freescale.com>
---
 arch/arm/boot/dts/imx6sx-sdb-reva.dts              | 158 +++++++++++++++++++++
 .../boot/dts/{imx6sx-sdb.dts => imx6sx-sdb.dtsi}   | 131 -----------------
 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%)

Comments

Peter Chen Jan. 20, 2015, 6:30 a.m. UTC | #1
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@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?

Peter
Shawn Guo Jan. 20, 2015, 7:35 a.m. UTC | #2
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@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.

>  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.

Shawn

> +};
> +
> +&i2c1 {
> +	clock-frequency = <100000>;
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&pinctrl_i2c1>;
> +	status = "okay";
> +
> +	pmic: pfuze100@08 {
> +		compatible = "fsl,pfuze100";
> +		reg = <0x08>;
> +
> +		regulators {
> +			sw1a_reg: sw1ab {
> +				regulator-min-microvolt = <300000>;
> +				regulator-max-microvolt = <1875000>;
> +				regulator-boot-on;
> +				regulator-always-on;
> +				regulator-ramp-delay = <6250>;
> +			};
> +
> +			sw1c_reg: sw1c {
> +				regulator-min-microvolt = <300000>;
> +				regulator-max-microvolt = <1875000>;
> +				regulator-boot-on;
> +				regulator-always-on;
> +				regulator-ramp-delay = <6250>;
> +			};
> +
> +			sw2_reg: sw2 {
> +				regulator-min-microvolt = <800000>;
> +				regulator-max-microvolt = <3300000>;
> +				regulator-boot-on;
> +				regulator-always-on;
> +			};
> +
> +			sw3a_reg: sw3a {
> +				regulator-min-microvolt = <400000>;
> +				regulator-max-microvolt = <1975000>;
> +				regulator-boot-on;
> +				regulator-always-on;
> +			};
> +
> +			sw3b_reg: sw3b {
> +				regulator-min-microvolt = <400000>;
> +				regulator-max-microvolt = <1975000>;
> +				regulator-boot-on;
> +				regulator-always-on;
> +			};
> +
> +			sw4_reg: sw4 {
> +				regulator-min-microvolt = <800000>;
> +				regulator-max-microvolt = <3300000>;
> +			};
> +
> +			swbst_reg: swbst {
> +				regulator-min-microvolt = <5000000>;
> +				regulator-max-microvolt = <5150000>;
> +			};
> +
> +			snvs_reg: vsnvs {
> +				regulator-min-microvolt = <1000000>;
> +				regulator-max-microvolt = <3000000>;
> +				regulator-boot-on;
> +				regulator-always-on;
> +			};
> +
> +			vref_reg: vrefddr {
> +				regulator-boot-on;
> +				regulator-always-on;
> +			};
> +
> +			vgen1_reg: vgen1 {
> +				regulator-min-microvolt = <800000>;
> +				regulator-max-microvolt = <1550000>;
> +				regulator-always-on;
> +			};
> +
> +			vgen2_reg: vgen2 {
> +				regulator-min-microvolt = <800000>;
> +				regulator-max-microvolt = <1550000>;
> +			};
> +
> +			vgen3_reg: vgen3 {
> +				regulator-min-microvolt = <1800000>;
> +				regulator-max-microvolt = <3300000>;
> +				regulator-always-on;
> +			};
> +
> +			vgen4_reg: vgen4 {
> +				regulator-min-microvolt = <1800000>;
> +				regulator-max-microvolt = <3300000>;
> +				regulator-always-on;
> +			};
> +
> +			vgen5_reg: vgen5 {
> +				regulator-min-microvolt = <1800000>;
> +				regulator-max-microvolt = <3300000>;
> +				regulator-always-on;
> +			};
> +
> +			vgen6_reg: vgen6 {
> +				regulator-min-microvolt = <1800000>;
> +				regulator-max-microvolt = <3300000>;
> +				regulator-always-on;
> +			};
> +		};
> +	};
> +};
> +
> +&qspi2 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&pinctrl_qspi2>;
> +	status = "okay";
> +
> +	flash0: s25fl128s@0 {
> +		reg = <0>;
> +		#address-cells = <1>;
> +		#size-cells = <1>;
> +		compatible = "spansion,s25fl128s";
> +		spi-max-frequency = <66000000>;
> +	};
> +
> +	flash1: s25fl128s@1 {
> +		reg = <1>;
> +		#address-cells = <1>;
> +		#size-cells = <1>;
> +		compatible = "spansion,s25fl128s";
> +		spi-max-frequency = <66000000>;
> +	};
> +};
> diff --git a/arch/arm/boot/dts/imx6sx-sdb.dts b/arch/arm/boot/dts/imx6sx-sdb.dtsi
> similarity index 83%
> rename from arch/arm/boot/dts/imx6sx-sdb.dts
> rename to arch/arm/boot/dts/imx6sx-sdb.dtsi
> index cdffe84..7b02aa7 100644
> --- a/arch/arm/boot/dts/imx6sx-sdb.dts
> +++ b/arch/arm/boot/dts/imx6sx-sdb.dtsi
> @@ -13,7 +13,6 @@
>  #include "imx6sx.dtsi"
>  
>  / {
> -	model = "Freescale i.MX6 SoloX SDB Board";
>  	compatible = "fsl,imx6sx-sdb", "fsl,imx6sx";
>  
>  	chosen {
> @@ -169,114 +168,6 @@
>  	status = "okay";
>  };
>  
> -&i2c1 {
> -        clock-frequency = <100000>;
> -        pinctrl-names = "default";
> -        pinctrl-0 = <&pinctrl_i2c1>;
> -        status = "okay";
> -
> -	pmic: pfuze100@08 {
> -		compatible = "fsl,pfuze100";
> -		reg = <0x08>;
> -
> -		regulators {
> -			sw1a_reg: sw1ab {
> -				regulator-min-microvolt = <300000>;
> -				regulator-max-microvolt = <1875000>;
> -				regulator-boot-on;
> -				regulator-always-on;
> -				regulator-ramp-delay = <6250>;
> -			};
> -
> -			sw1c_reg: sw1c {
> -				regulator-min-microvolt = <300000>;
> -				regulator-max-microvolt = <1875000>;
> -				regulator-boot-on;
> -				regulator-always-on;
> -				regulator-ramp-delay = <6250>;
> -			};
> -
> -			sw2_reg: sw2 {
> -				regulator-min-microvolt = <800000>;
> -				regulator-max-microvolt = <3300000>;
> -				regulator-boot-on;
> -				regulator-always-on;
> -			};
> -
> -			sw3a_reg: sw3a {
> -				regulator-min-microvolt = <400000>;
> -				regulator-max-microvolt = <1975000>;
> -				regulator-boot-on;
> -				regulator-always-on;
> -			};
> -
> -			sw3b_reg: sw3b {
> -				regulator-min-microvolt = <400000>;
> -				regulator-max-microvolt = <1975000>;
> -				regulator-boot-on;
> -				regulator-always-on;
> -			};
> -
> -			sw4_reg: sw4 {
> -				regulator-min-microvolt = <800000>;
> -				regulator-max-microvolt = <3300000>;
> -			};
> -
> -			swbst_reg: swbst {
> -				regulator-min-microvolt = <5000000>;
> -				regulator-max-microvolt = <5150000>;
> -			};
> -
> -			snvs_reg: vsnvs {
> -				regulator-min-microvolt = <1000000>;
> -				regulator-max-microvolt = <3000000>;
> -				regulator-boot-on;
> -				regulator-always-on;
> -			};
> -
> -			vref_reg: vrefddr {
> -				regulator-boot-on;
> -				regulator-always-on;
> -			};
> -
> -			vgen1_reg: vgen1 {
> -				regulator-min-microvolt = <800000>;
> -				regulator-max-microvolt = <1550000>;
> -				regulator-always-on;
> -			};
> -
> -			vgen2_reg: vgen2 {
> -				regulator-min-microvolt = <800000>;
> -				regulator-max-microvolt = <1550000>;
> -			};
> -
> -			vgen3_reg: vgen3 {
> -				regulator-min-microvolt = <1800000>;
> -				regulator-max-microvolt = <3300000>;
> -				regulator-always-on;
> -			};
> -
> -			vgen4_reg: vgen4 {
> -				regulator-min-microvolt = <1800000>;
> -				regulator-max-microvolt = <3300000>;
> -				regulator-always-on;
> -			};
> -
> -			vgen5_reg: vgen5 {
> -				regulator-min-microvolt = <1800000>;
> -				regulator-max-microvolt = <3300000>;
> -				regulator-always-on;
> -			};
> -
> -			vgen6_reg: vgen6 {
> -				regulator-min-microvolt = <1800000>;
> -				regulator-max-microvolt = <3300000>;
> -				regulator-always-on;
> -			};
> -		};
> -	};
> -};
> -
>  &i2c4 {
>          clock-frequency = <100000>;
>          pinctrl-names = "default";
> @@ -340,28 +231,6 @@
>  	status = "okay";
>  };
>  
> -&qspi2 {
> -	pinctrl-names = "default";
> -	pinctrl-0 = <&pinctrl_qspi2>;
> -	status = "okay";
> -
> -	flash0: s25fl128s@0 {
> -		reg = <0>;
> -		#address-cells = <1>;
> -		#size-cells = <1>;
> -		compatible = "spansion,s25fl128s";
> -		spi-max-frequency = <66000000>;
> -	};
> -
> -	flash1: s25fl128s@1 {
> -		reg = <1>;
> -		#address-cells = <1>;
> -		#size-cells = <1>;
> -		compatible = "spansion,s25fl128s";
> -		spi-max-frequency = <66000000>;
> -	};
> -};
> -
>  &ssi2 {
>  	status = "okay";
>  };
> -- 
> 1.9.1
>
Anson.Huang@freescale.com Jan. 20, 2015, 7:51 a.m. UTC | #3
Best regards!
Anson Huang


> -----Original Message-----
> From: Peter Chen [mailto:peter.chen@freescale.com]
> Sent: 2015-01-20 2:30 PM
> To: Shawn Guo; Huang Yongcai-B20788
> Cc: kernel@pengutronix.de; robh+dt@kernel.org; pawel.moll@arm.com;
> mark.rutland@arm.com; linux-arm-kernel@lists.infradead.org;
> devicetree@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@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
Shawn Guo Jan. 20, 2015, 8:03 a.m. UTC | #4
On Tue, Jan 20, 2015 at 07:51:31AM +0000, Anson.Huang@freescale.com wrote:
> > > > +&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.

Thanks for the clarification, Anson.

Peter,

OPP is a CPU characteristic defined by data sheet, so we should define
OPP according to data sheet in imx6sx.dtsi, and only overwrite the
property in Rev-B dts per board/PMIC quirk.

Also, please put the rationale why we need OPP quick on Rev-B into
commit log.

Shawn
diff mbox

Patch

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
+	>;
+};
+
+&i2c1 {
+	clock-frequency = <100000>;
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_i2c1>;
+	status = "okay";
+
+	pmic: pfuze100@08 {
+		compatible = "fsl,pfuze100";
+		reg = <0x08>;
+
+		regulators {
+			sw1a_reg: sw1ab {
+				regulator-min-microvolt = <300000>;
+				regulator-max-microvolt = <1875000>;
+				regulator-boot-on;
+				regulator-always-on;
+				regulator-ramp-delay = <6250>;
+			};
+
+			sw1c_reg: sw1c {
+				regulator-min-microvolt = <300000>;
+				regulator-max-microvolt = <1875000>;
+				regulator-boot-on;
+				regulator-always-on;
+				regulator-ramp-delay = <6250>;
+			};
+
+			sw2_reg: sw2 {
+				regulator-min-microvolt = <800000>;
+				regulator-max-microvolt = <3300000>;
+				regulator-boot-on;
+				regulator-always-on;
+			};
+
+			sw3a_reg: sw3a {
+				regulator-min-microvolt = <400000>;
+				regulator-max-microvolt = <1975000>;
+				regulator-boot-on;
+				regulator-always-on;
+			};
+
+			sw3b_reg: sw3b {
+				regulator-min-microvolt = <400000>;
+				regulator-max-microvolt = <1975000>;
+				regulator-boot-on;
+				regulator-always-on;
+			};
+
+			sw4_reg: sw4 {
+				regulator-min-microvolt = <800000>;
+				regulator-max-microvolt = <3300000>;
+			};
+
+			swbst_reg: swbst {
+				regulator-min-microvolt = <5000000>;
+				regulator-max-microvolt = <5150000>;
+			};
+
+			snvs_reg: vsnvs {
+				regulator-min-microvolt = <1000000>;
+				regulator-max-microvolt = <3000000>;
+				regulator-boot-on;
+				regulator-always-on;
+			};
+
+			vref_reg: vrefddr {
+				regulator-boot-on;
+				regulator-always-on;
+			};
+
+			vgen1_reg: vgen1 {
+				regulator-min-microvolt = <800000>;
+				regulator-max-microvolt = <1550000>;
+				regulator-always-on;
+			};
+
+			vgen2_reg: vgen2 {
+				regulator-min-microvolt = <800000>;
+				regulator-max-microvolt = <1550000>;
+			};
+
+			vgen3_reg: vgen3 {
+				regulator-min-microvolt = <1800000>;
+				regulator-max-microvolt = <3300000>;
+				regulator-always-on;
+			};
+
+			vgen4_reg: vgen4 {
+				regulator-min-microvolt = <1800000>;
+				regulator-max-microvolt = <3300000>;
+				regulator-always-on;
+			};
+
+			vgen5_reg: vgen5 {
+				regulator-min-microvolt = <1800000>;
+				regulator-max-microvolt = <3300000>;
+				regulator-always-on;
+			};
+
+			vgen6_reg: vgen6 {
+				regulator-min-microvolt = <1800000>;
+				regulator-max-microvolt = <3300000>;
+				regulator-always-on;
+			};
+		};
+	};
+};
+
+&qspi2 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_qspi2>;
+	status = "okay";
+
+	flash0: s25fl128s@0 {
+		reg = <0>;
+		#address-cells = <1>;
+		#size-cells = <1>;
+		compatible = "spansion,s25fl128s";
+		spi-max-frequency = <66000000>;
+	};
+
+	flash1: s25fl128s@1 {
+		reg = <1>;
+		#address-cells = <1>;
+		#size-cells = <1>;
+		compatible = "spansion,s25fl128s";
+		spi-max-frequency = <66000000>;
+	};
+};
diff --git a/arch/arm/boot/dts/imx6sx-sdb.dts b/arch/arm/boot/dts/imx6sx-sdb.dtsi
similarity index 83%
rename from arch/arm/boot/dts/imx6sx-sdb.dts
rename to arch/arm/boot/dts/imx6sx-sdb.dtsi
index cdffe84..7b02aa7 100644
--- a/arch/arm/boot/dts/imx6sx-sdb.dts
+++ b/arch/arm/boot/dts/imx6sx-sdb.dtsi
@@ -13,7 +13,6 @@ 
 #include "imx6sx.dtsi"
 
 / {
-	model = "Freescale i.MX6 SoloX SDB Board";
 	compatible = "fsl,imx6sx-sdb", "fsl,imx6sx";
 
 	chosen {
@@ -169,114 +168,6 @@ 
 	status = "okay";
 };
 
-&i2c1 {
-        clock-frequency = <100000>;
-        pinctrl-names = "default";
-        pinctrl-0 = <&pinctrl_i2c1>;
-        status = "okay";
-
-	pmic: pfuze100@08 {
-		compatible = "fsl,pfuze100";
-		reg = <0x08>;
-
-		regulators {
-			sw1a_reg: sw1ab {
-				regulator-min-microvolt = <300000>;
-				regulator-max-microvolt = <1875000>;
-				regulator-boot-on;
-				regulator-always-on;
-				regulator-ramp-delay = <6250>;
-			};
-
-			sw1c_reg: sw1c {
-				regulator-min-microvolt = <300000>;
-				regulator-max-microvolt = <1875000>;
-				regulator-boot-on;
-				regulator-always-on;
-				regulator-ramp-delay = <6250>;
-			};
-
-			sw2_reg: sw2 {
-				regulator-min-microvolt = <800000>;
-				regulator-max-microvolt = <3300000>;
-				regulator-boot-on;
-				regulator-always-on;
-			};
-
-			sw3a_reg: sw3a {
-				regulator-min-microvolt = <400000>;
-				regulator-max-microvolt = <1975000>;
-				regulator-boot-on;
-				regulator-always-on;
-			};
-
-			sw3b_reg: sw3b {
-				regulator-min-microvolt = <400000>;
-				regulator-max-microvolt = <1975000>;
-				regulator-boot-on;
-				regulator-always-on;
-			};
-
-			sw4_reg: sw4 {
-				regulator-min-microvolt = <800000>;
-				regulator-max-microvolt = <3300000>;
-			};
-
-			swbst_reg: swbst {
-				regulator-min-microvolt = <5000000>;
-				regulator-max-microvolt = <5150000>;
-			};
-
-			snvs_reg: vsnvs {
-				regulator-min-microvolt = <1000000>;
-				regulator-max-microvolt = <3000000>;
-				regulator-boot-on;
-				regulator-always-on;
-			};
-
-			vref_reg: vrefddr {
-				regulator-boot-on;
-				regulator-always-on;
-			};
-
-			vgen1_reg: vgen1 {
-				regulator-min-microvolt = <800000>;
-				regulator-max-microvolt = <1550000>;
-				regulator-always-on;
-			};
-
-			vgen2_reg: vgen2 {
-				regulator-min-microvolt = <800000>;
-				regulator-max-microvolt = <1550000>;
-			};
-
-			vgen3_reg: vgen3 {
-				regulator-min-microvolt = <1800000>;
-				regulator-max-microvolt = <3300000>;
-				regulator-always-on;
-			};
-
-			vgen4_reg: vgen4 {
-				regulator-min-microvolt = <1800000>;
-				regulator-max-microvolt = <3300000>;
-				regulator-always-on;
-			};
-
-			vgen5_reg: vgen5 {
-				regulator-min-microvolt = <1800000>;
-				regulator-max-microvolt = <3300000>;
-				regulator-always-on;
-			};
-
-			vgen6_reg: vgen6 {
-				regulator-min-microvolt = <1800000>;
-				regulator-max-microvolt = <3300000>;
-				regulator-always-on;
-			};
-		};
-	};
-};
-
 &i2c4 {
         clock-frequency = <100000>;
         pinctrl-names = "default";
@@ -340,28 +231,6 @@ 
 	status = "okay";
 };
 
-&qspi2 {
-	pinctrl-names = "default";
-	pinctrl-0 = <&pinctrl_qspi2>;
-	status = "okay";
-
-	flash0: s25fl128s@0 {
-		reg = <0>;
-		#address-cells = <1>;
-		#size-cells = <1>;
-		compatible = "spansion,s25fl128s";
-		spi-max-frequency = <66000000>;
-	};
-
-	flash1: s25fl128s@1 {
-		reg = <1>;
-		#address-cells = <1>;
-		#size-cells = <1>;
-		compatible = "spansion,s25fl128s";
-		spi-max-frequency = <66000000>;
-	};
-};
-
 &ssi2 {
 	status = "okay";
 };