diff mbox

Add device tree file for the armadeus apf27

Message ID 1351268980-25347-1-git-send-email-tremyfr@yahoo.fr (mailing list archive)
State New, archived
Headers show

Commit Message

Philippe Reynes Oct. 26, 2012, 4:29 p.m. UTC
Signed-off-by: Philippe Reynes <tremyfr@yahoo.fr>
Signed-off-by: Eric Jarrige <eric.jarrige@armadeus.org>
---
 arch/arm/boot/dts/imx27-apf27.dts |   96 +++++++++++++++++++++++++++++++++++++
 1 files changed, 96 insertions(+), 0 deletions(-)
 create mode 100644 arch/arm/boot/dts/imx27-apf27.dts

Comments

Sascha Hauer Oct. 29, 2012, 9:10 p.m. UTC | #1
On Fri, Oct 26, 2012 at 06:29:40PM +0200, Philippe Reynes wrote:
> Signed-off-by: Philippe Reynes <tremyfr@yahoo.fr>
> Signed-off-by: Eric Jarrige <eric.jarrige@armadeus.org>
> ---
>  arch/arm/boot/dts/imx27-apf27.dts |   96 +++++++++++++++++++++++++++++++++++++
>  1 files changed, 96 insertions(+), 0 deletions(-)
>  create mode 100644 arch/arm/boot/dts/imx27-apf27.dts
> 
> diff --git a/arch/arm/boot/dts/imx27-apf27.dts b/arch/arm/boot/dts/imx27-apf27.dts
> new file mode 100644
> index 0000000..b7d11e0
> --- /dev/null
> +++ b/arch/arm/boot/dts/imx27-apf27.dts
> @@ -0,0 +1,96 @@
> +/*
> + * Copyright 2012 Philippe Reynes <tremyfr@yahoo.fr>
> + * Copyright 2012 Armadeus Systems <support@armadeus.com>
> + *
> + * Based on code which is: Copyright 2012 Sascha Hauer, Pengutronix
> + *
> + * The code contained herein is licensed under the GNU General Public
> + * License. You may obtain a copy of the GNU General Public License
> + * Version 2 or later at the following locations:
> + *
> + * http://www.opensource.org/licenses/gpl-license.html
> + * http://www.gnu.org/copyleft/gpl.html
> + */
> +
> +/dts-v1/;
> +/include/ "imx27.dtsi"
> +
> +/ {
> +	model = "Armadeus apf27";
> +	compatible = "armadeus,imx27-apf27", "fsl,imx27";
> +
> +	memory {
> +		reg = <0xa0000000 0x04000000>;
> +	};
> +
> +	clocks {
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +
> +		osc26m {
> +			compatible = "fsl,imx-osc26m", "fixed-clock";
> +			clock-frequency = <33554432>;

Is this really correct? The Datasheet specificies 26MHz, some boards
have 27Mhz, but 33?

> +		};
> +	};
> +
> +	soc {
> +		aipi@10000000 {
> +			wdog@10002000 {
> +				status = "okay";
> +			};

This node is not necessary. The watchdog is enabled already in the dtsi
file.

Sascha
Philippe Reynes Oct. 30, 2012, 7:55 p.m. UTC | #2
Hi Sascha,
>> +        osc26m {

>> +            compatible = "fsl,imx-osc26m", "fixed-clock";
>> +            clock-frequency = <33554432>;

>Is this really correct? The Datasheet specificies 26MHz, some boards
>have 27Mhz, but 33?

The clock value on this board is  32.768kHz, so 32768 * 1024.
I've tried others value, all others produce weird behaviour on the serial.

>> +    soc {
>> +        aipi@10000000 {
>> +            wdog@10002000 {
>> +                status = "okay";
>> +            };

>This node is not necessary. The watchdog is enabled already in the dtsi
>file.

Yes, you're right, I'll send a v2 of this patch without the watchdog.

Regards,
Philippe
Sascha Hauer Oct. 31, 2012, 7:32 a.m. UTC | #3
On Tue, Oct 30, 2012 at 07:55:44PM +0000, Philippe Reynes wrote:
> 
> 
> Hi Sascha,
> >> +        osc26m {
> 
> >> +            compatible = "fsl,imx-osc26m", "fixed-clock";
> >> +            clock-frequency = <33554432>;
> 
> >Is this really correct? The Datasheet specificies 26MHz, some boards
> >have 27Mhz, but 33?
> 
> The clock value on this board is  32.768kHz, so 32768 * 1024.
> I've tried others value, all others produce weird behaviour on the serial.

Ok, so your board runs with the 32.768KHz clock input. I didn't know
there are boards that use this as the reference clock. It seems this
is not handled in the current clk code. Could you test the patch I just
sent to the mailing list (Cced you) and provide a Tested-by tag if it
works? The clock frequency provided in the device tree shouldn't make
a difference then anymore. You should specify it to 0 if you have
nothing connected there.

Sascha
diff mbox

Patch

diff --git a/arch/arm/boot/dts/imx27-apf27.dts b/arch/arm/boot/dts/imx27-apf27.dts
new file mode 100644
index 0000000..b7d11e0
--- /dev/null
+++ b/arch/arm/boot/dts/imx27-apf27.dts
@@ -0,0 +1,96 @@ 
+/*
+ * Copyright 2012 Philippe Reynes <tremyfr@yahoo.fr>
+ * Copyright 2012 Armadeus Systems <support@armadeus.com>
+ *
+ * Based on code which is: Copyright 2012 Sascha Hauer, Pengutronix
+ *
+ * The code contained herein is licensed under the GNU General Public
+ * License. You may obtain a copy of the GNU General Public License
+ * Version 2 or later at the following locations:
+ *
+ * http://www.opensource.org/licenses/gpl-license.html
+ * http://www.gnu.org/copyleft/gpl.html
+ */
+
+/dts-v1/;
+/include/ "imx27.dtsi"
+
+/ {
+	model = "Armadeus apf27";
+	compatible = "armadeus,imx27-apf27", "fsl,imx27";
+
+	memory {
+		reg = <0xa0000000 0x04000000>;
+	};
+
+	clocks {
+		#address-cells = <1>;
+		#size-cells = <0>;
+
+		osc26m {
+			compatible = "fsl,imx-osc26m", "fixed-clock";
+			clock-frequency = <33554432>;
+		};
+	};
+
+	soc {
+		aipi@10000000 {
+			wdog@10002000 {
+				status = "okay";
+			};
+
+			serial@1000a000 {
+				status = "okay";
+			};
+
+			ethernet@1002b000 {
+				status = "okay";
+			};
+
+		};
+
+		nand@d8000000 {
+			status = "okay";
+			nand-bus-width = <16>;
+			nand-ecc-mode = "hw";
+			nand-on-flash-bbt;
+
+			partition@0 {
+				label = "u-boot";
+				reg = <0x0 0x100000>;
+			};
+
+			partition@100000 {
+				label = "env";
+				reg = <0x100000 0x80000>;
+			};
+
+			partition@180000 {
+				label = "env2";
+				reg = <0x180000 0x80000>;
+			};
+
+			partition@200000 {
+				label = "firmware";
+				reg = <0x200000 0x80000>;
+			};
+
+			partition@280000 {
+				label = "dtb";
+				reg = <0x280000 0x80000>;
+			};
+
+			partition@300000 {
+				label = "kernel";
+				reg = <0x300000 0x500000>;
+			};
+
+			partition@800000 {
+				label = "rootfs";
+				reg = <0x800000 0xf800000>;
+			};
+		};
+
+	};
+
+};