Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1461616

Re: [PATCH v2 3/4] ARM: dts: da850,da850-evm: Add an aemif node and use it for the NAND

Path csiph.com!news.freedyn.net!newsfeed.datemas.de!weretis.net!feeder1.news.weretis.net!feeder4.news.weretis.net!news.unit0.net!news.panservice.it!bofh.it!news.nic.it!robomod
From Karl Beldan <kbeldan@baylibre.com>
Newsgroups linux.kernel
Subject Re: [PATCH v2 3/4] ARM: dts: da850,da850-evm: Add an aemif node and use it for the NAND
Date Sat, 13 Aug 2016 13:30:01 +0200
Message-ID <s5Fix-dP-1@gated-at.bofh.it> (permalink)
References <s4G6Z-8uN-11@gated-at.bofh.it> <s4GgG-79-35@gated-at.bofh.it>
X-Original-To devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=8jOJ9EZTm/mEOn6lclivgAAo3YAhooFCWqYUtjFzjoU=; b=yfSJfImjbrYzCN5+arsmhOEvOMYpQ+kU7t6AdTbJnc4JB1LZpBRzL5lvaA/4SQP80i PIhNWE/a4ae6ihcRc6whrjTzpdHlDIk74eHDQ2oR9Gm9E8sAX5TSCNhxMDQ/nIrwaThz WJhlB+bC0aBwsaF/uj0Zx0i5ZKKq6DBhvvvFWZxW62FRT0g+QLdEmJupeHpxmlIARCF4 GzW4lSDFxXSFxgHqw6s47Pkq8vYmvJo6u70OpRCrcZG/A9aY9t/ugtTGST58YtsfgvHP ftKA5AtAoN1YhNqSUN4NP+LQTTa/rMiNB+qNofpBZ3vj64XxKo5w/lVoKNODv8h9ofQa Ttcw==
X-Google-Dkim-Signature v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=8jOJ9EZTm/mEOn6lclivgAAo3YAhooFCWqYUtjFzjoU=; b=YI5SFOaMUc2e7XLYY/rY6NjU9Pe6VrtyeMWXDSAxEVme29APb/ObZoaE6zhS29OvvP 68FsCHFPe4pfSEGGoV0Clo2qUY3DPRyVkwJ+05uYa7fYn4MtbgTJGzSV+3V2QAe2Twmv YYp2tcBvqoZA2q/JhABq2BxomzvoBc5kF6x+p8NYJ66Gehio32nPHZyFQK0btziAH9Pu hiJFnmashkxVv8fvdz6HiTCl/SIiog+vwKemWvyPSx3GeBO/1qd9+D//WT63H03iOS6w Vg/jnIf0A9ngZfQOd1OH+LQz6RzLdM+RigWegWlIfLFvcoT8dvioVU7cDAqIy3co9J1R G5Dg==
X-Gm-Message-State AEkooutZFko49KtFF4Vo33rrmeFl+66kWyrzqhJemEmEK70s7GWUniXNDlvRyeCsPAKIj4Oa
X-Received by 10.194.133.200 with SMTP id pe8mr19499042wjb.14.1471087614810; Sat, 13 Aug 2016 04:26:54 -0700 (PDT)
MIME-Version 1.0
Content-Type text/plain; charset=us-ascii
Content-Disposition inline
User-Agent Mutt/1.6.0 (2016-04-01)
Sender robomod@news.nic.it
List-ID <linux-kernel.vger.kernel.org>
X-Mailing-List linux-kernel@vger.kernel.org
Approved robomod@news.nic.it
Lines 56
Organization linux.* mail to news gateway
X-Original-Cc linux-kernel@vger.kernel.org, Rob Herring <robh+dt@kernel.org>, Mark Rutland <mark.rutland@arm.com>, Russell King <linux@armlinux.org.uk>, Santosh Shilimkar <ssantosh@kernel.org>, Sekhar Nori <nsekhar@ti.com>, Kevin Hilman <khilman@baylibre.com>, Karl Beldan <karl.beldan+oss@gmail.com>
X-Original-Date Sat, 13 Aug 2016 11:27:40 +0000
X-Original-Message-ID <20160813112740.GA27739@gobelin>
X-Original-References <20160810110032.29295-1-kbeldan@baylibre.com> <20160810110032.29295-4-kbeldan@baylibre.com>
X-Original-Sender linux-kernel-owner@vger.kernel.org
Xref csiph.com linux.kernel:1461616

Show key headers only | View raw


On Wed, Aug 10, 2016 at 11:00:31AM +0000, Karl Beldan wrote:
> Currently the davinci da8xx boards use the mach-davinci aemif code.
> Instantiating an aemif node into the DT allows to use the ti-aemif
> memory driver and is another step to better DT support.
> This change adds an aemif node in the dtsi while retiring the nand_cs3
> node. The NAND is now instantiated in the dts as a subnode of the aemif
> one along with its pins.
> 
> Signed-off-by: Karl Beldan <kbeldan@baylibre.com>
> ---
>  arch/arm/boot/dts/da850-evm.dts | 49 ++++++++++++++++++++++++++++++++++++-----
>  arch/arm/boot/dts/da850.dtsi    | 19 +++++++---------
>  2 files changed, 52 insertions(+), 16 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
> index 1a15db8..eedcc59 100644
> --- a/arch/arm/boot/dts/da850-evm.dts
> +++ b/arch/arm/boot/dts/da850-evm.dts
> @@ -29,6 +29,20 @@
>  					0x04 0x00011000 0x000ff000
>  				>;
>  			};
> +			nand_pins: nand_pins {
> +				pinctrl-single,bits = <
> +					/* EMA_WAIT[0], EMA_OE, EMA_WE, EMA_CS[4], EMA_CS[3] */
> +					0x1c 0x10110110  0xf0ff0ff0
> +					/*
> +					 * EMA_D[0], EMA_D[1], EMA_D[2],
> +					 * EMA_D[3], EMA_D[4], EMA_D[5],
> +					 * EMA_D[6], EMA_D[7]
> +					 */
> +					0x24 0x11111111  0xffffffff
> +					/* EMA_A[1], EMA_A[2] */
> +					0x30 0x01100000  0x0ff00000
> +				>;
> +			};
>  		};
>  		serial0: serial@42000 {
>  			status = "okay";
> @@ -131,11 +145,6 @@
>  			status = "okay";
>  		};
>  	};
> -	nand_cs3@62000000 {
> -		status = "okay";
> -		pinctrl-names = "default";
> -		pinctrl-0 = <&nand_cs3_pins>;
> -	};
>  	vbat: fixedregulator@0 {
>  		compatible = "regulator-fixed";
>  		regulator-name = "vbat";

It has no functionnal impact but with this change nand_cs3_pins becomes
useless and I forgot to get rid of it, so it'll make a v3.
 
Karl

Back to linux.kernel | Previous | Next | Find similar | Unroll thread


Thread

Re: [PATCH v2 3/4] ARM: dts: da850,da850-evm: Add an aemif node and  use it for the NAND Karl Beldan <kbeldan@baylibre.com> - 2016-08-13 13:30 +0200

csiph-web