From: s.hauer@pengutronix.de (Sascha Hauer)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] Add device tree file for the armadeus apf27
Date: Mon, 29 Oct 2012 22:10:15 +0100 [thread overview]
Message-ID: <20121029211015.GX1641@pengutronix.de> (raw)
In-Reply-To: <1351268980-25347-1-git-send-email-tremyfr@yahoo.fr>
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 at 10000000 {
> + wdog at 10002000 {
> + status = "okay";
> + };
This node is not necessary. The watchdog is enabled already in the dtsi
file.
Sascha
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
WARNING: multiple messages have this Message-ID (diff)
From: Sascha Hauer <s.hauer@pengutronix.de>
To: Philippe Reynes <tremyfr@yahoo.fr>
Cc: linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
julien.boibessot@armadeus.com, eric.jarrige@armadeus.org
Subject: Re: [PATCH] Add device tree file for the armadeus apf27
Date: Mon, 29 Oct 2012 22:10:15 +0100 [thread overview]
Message-ID: <20121029211015.GX1641@pengutronix.de> (raw)
In-Reply-To: <1351268980-25347-1-git-send-email-tremyfr@yahoo.fr>
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
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
next prev parent reply other threads:[~2012-10-29 21:10 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-26 16:29 [PATCH] Add device tree file for the armadeus apf27 Philippe Reynes
2012-10-26 16:29 ` Philippe Reynes
2012-10-29 21:10 ` Sascha Hauer [this message]
2012-10-29 21:10 ` Sascha Hauer
2012-10-30 19:55 ` Philippe Reynes
2012-10-31 7:32 ` Sascha Hauer
2012-10-31 7:32 ` Sascha Hauer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20121029211015.GX1641@pengutronix.de \
--to=s.hauer@pengutronix.de \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.