devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Benoit Cousson <bcousson@baylibre.com>
To: Nishanth Menon <nm@ti.com>
Cc: Tony Lindgren <tony@atomide.com>,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-omap@vger.kernel.org
Subject: Re: [PATCH] ARM: dts: Add basic devices for AM3517-craneboard
Date: Wed, 18 Dec 2013 17:57:00 +0100	[thread overview]
Message-ID: <52B1D3DC.1010304@baylibre.com> (raw)
In-Reply-To: <52B0A9C8.7070201@ti.com>

Hi Nishanth,

On 17/12/2013 20:45, Nishanth Menon wrote:
> On 12/09/2013 03:55 PM, Nishanth Menon wrote:
>> Craneboard is a hardware development platform based on the Sitara
>> AM3517 ARM Cortex - A8 microprocessor device - see [1] for more
>> details. Add basic devices for craneboard as replacement for the board
>> file scheduled for removal as part of device tree conversion
>>
>> [1] http://craneboard.org
>>
>> Signed-off-by: Nishanth Menon <nm@ti.com>
>> ---
>
> gentle ping.. had'nt seen a response on this patch. Could we queue
> this up for v3.14-rc1?

Yep, it looks good to me.
But if you don't mind I'll start pushing my branch after Xmas :-).

Thanks,
Benoit


>> Based on:
>> git://git.kernel.org/pub/scm/linux/kernel/git/tmlind/linux-omap.git
>> branch: omap-for-v3.14/omap3-board-removal 736e812 ARM: OMAP2+: Remove unused platform init code and headers
>>
>> Bootlog: http://pastebin.mozilla.org/3744412
>>
>>   arch/arm/boot/dts/Makefile              |    1 +
>>   arch/arm/boot/dts/am3517-craneboard.dts |  174 +++++++++++++++++++++++++++++++
>>   2 files changed, 175 insertions(+)
>>   create mode 100644 arch/arm/boot/dts/am3517-craneboard.dts
>>
>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
>> index fc37bca..ad155fc 100644
>> --- a/arch/arm/boot/dts/Makefile
>> +++ b/arch/arm/boot/dts/Makefile
>> @@ -205,6 +205,7 @@ dtb-$(CONFIG_ARCH_OMAP2PLUS) += omap2420-h4.dtb \
>>   	am335x-boneblack.dtb \
>>   	am335x-nano.dtb \
>>   	am335x-base0033.dtb \
>> +	am3517-craneboard.dtb \
>>   	am3517-evm.dtb \
>>   	am3517_mt_ventoux.dtb \
>>   	am43x-epos-evm.dtb \
>> diff --git a/arch/arm/boot/dts/am3517-craneboard.dts b/arch/arm/boot/dts/am3517-craneboard.dts
>> new file mode 100644
>> index 0000000..2d40b3f
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/am3517-craneboard.dts
>> @@ -0,0 +1,174 @@
>> +/*
>> + * See craneboard.org for more details
>> + *
>> + * Copyright (C) 2013 Texas Instruments Incorporated - http://www.ti.com/
>> + *
>> + * 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.
>> + */
>> +/dts-v1/;
>> +
>> +#include "am3517.dtsi"
>> +
>> +/ {
>> +	model = "TI AM3517 CraneBoard (TMDSEVM3517)";
>> +	compatible = "ti,am3517-craneboard", "ti,am3517", "ti,omap3";
>> +
>> +	memory {
>> +		device_type = "memory";
>> +		reg = <0x80000000 0x10000000>;	/* 256 MB */
>> +	};
>> +
>> +	vbat: fixedregulator@0 {
>> +		compatible = "regulator-fixed";
>> +		regulator-name = "vbat";
>> +		regulator-min-microvolt = <5000000>;
>> +		regulator-max-microvolt = <5000000>;
>> +		regulator-boot-on;
>> +	};
>> +};
>> +
>> +&davinci_emac {
>> +	status = "okay";
>> +};
>> +
>> +&davinci_mdio {
>> +	status = "okay";
>> +};
>> +
>> +&i2c1 {
>> +	clock-frequency = <2600000>;
>> +
>> +	tps: tps@2d {
>> +		reg = <0x2d>;
>> +	};
>> +};
>> +
>> +&i2c2 {
>> +	clock-frequency = <400000>;
>> +	/* goes to expansion connector */
>> +	status = "disabled";
>> +};
>> +
>> +&i2c3 {
>> +	clock-frequency = <400000>;
>> +	/* goes to expansion connector */
>> +	status = "disabled";
>> +};
>> +
>> +&mmc1 {
>> +	vmmc-supply = <&vdd2_reg>;
>> +	bus-width = <8>;
>> +};
>> +
>> +&mmc2 {
>> +	/* goes to expansion connector */
>> +	status = "disabled";
>> +};
>> +
>> +&mmc3 {
>> +	/* goes to expansion connector */
>> +	status = "disabled";
>> +};
>> +
>> +#include "tps65910.dtsi"
>> +
>> +&omap3_pmx_core {
>> +	tps_pins: pinmux_tps_pins {
>> +		pinctrl-single,pins = <
>> +			0x1b0 (PIN_INPUT_PULLUP | MUX_MODE0) /* sys_nirq.sys_nirq */
>> +		>;
>> +	};
>> +};
>> +
>> +&tps {
>> +	pinctrl-names = "default";
>> +	pinctrl-0 = <&tps_pins>;
>> +
>> +	interrupts = <7>; /* SYS_NIRQ cascaded to intc */
>> +	interrupt-parent = <&intc>;
>> +
>> +	ti,en-ck32k-xtal;
>> +
>> +	vcc1-supply = <&vbat>;
>> +	vcc2-supply = <&vbat>;
>> +	vcc3-supply = <&vbat>;
>> +	vcc4-supply = <&vbat>;
>> +	vcc5-supply = <&vbat>;
>> +	vcc6-supply = <&vbat>;
>> +	vcc7-supply = <&vbat>;
>> +	vccio-supply = <&vbat>;
>> +
>> +	regulators {
>> +		vrtc_reg: regulator@0 {
>> +			regulator-always-on;
>> +		};
>> +
>> +		vio_reg: regulator@1 {
>> +			regulator-always-on;
>> +		};
>> +
>> +		/*
>> +		 * Unused:
>> +		 * VDIG1=2.7V,300mA max
>> +		 * VDIG2=1.8V,300mA max
>> +		 */
>> +
>> +		vpll_reg: regulator@7 {
>> +			/* VDDS_DPLL_1V8 */
>> +			regulator-min-microvolt = <1800000>;
>> +			regulator-max-microvolt = <1800000>;
>> +			regulator-always-on;
>> +		};
>> +
>> +		vaux1_reg: regulator@9 {
>> +			/* VDDS_SRAM_1V8 */
>> +			regulator-min-microvolt = <1800000>;
>> +			regulator-max-microvolt = <1800000>;
>> +			regulator-always-on;
>> +		};
>> +
>> +		vaux2_reg: regulator@10 {
>> +			/* VDDA1P8V_USBPHY */
>> +			regulator-min-microvolt = <1800000>;
>> +			regulator-max-microvolt = <1800000>;
>> +			regulator-always-on;
>> +		};
>> +
>> +		/* VAUX33 unused */
>> +
>> +		vdac_reg: regulator@8 {
>> +			/* VDDA_DAC_1V8 */
>> +			regulator-min-microvolt = <1800000>;
>> +			regulator-max-microvolt = <1800000>;
>> +			regulator-always-on;
>> +		};
>> +
>> +		vmmc_reg: regulator@12 {
>> +			/* VDDA3P3V_USBPHY */
>> +			regulator-min-microvolt = <3300000>;
>> +			regulator-max-microvolt = <3300000>;
>> +			regulator-always-on;
>> +		};
>> +
>> +		vdd1_reg: regulator@2 {
>> +			/* VDD_CORE */
>> +			regulator-name = "vdd_core";
>> +			regulator-min-microvolt = <1200000>;
>> +			regulator-max-microvolt = <1200000>;
>> +			regulator-boot-on;
>> +			regulator-always-on;
>> +		};
>> +
>> +		vdd2_reg: regulator@3 {
>> +			/* VDDSHV_3V3 */
>> +			regulator-name = "vdd_shv";
>> +			regulator-min-microvolt = <3300000>;
>> +			regulator-max-microvolt = <3300000>;
>> +			regulator-always-on;
>> +		};
>> +
>> +		/* VDD3 unused */
>> +	};
>> +};
>>
>
>


-- 
Benoît Cousson
BayLibre
Embedded Linux Technology Lab
www.baylibre.com
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2013-12-18 16:57 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-09 21:55 [PATCH] ARM: dts: Add basic devices for AM3517-craneboard Nishanth Menon
2013-12-17 19:45 ` Nishanth Menon
2013-12-18 16:57   ` Benoit Cousson [this message]
2013-12-18 17:02     ` Nishanth Menon
2013-12-18 17:16       ` Benoit Cousson
2014-01-20 16:43         ` Nishanth Menon
2014-01-29 13:30           ` Nishanth Menon

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=52B1D3DC.1010304@baylibre.com \
    --to=bcousson@baylibre.com \
    --cc=devicetree@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=nm@ti.com \
    --cc=tony@atomide.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).