Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1461616
| 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 | 2016-08-13 13:30 +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> |
| Organization | linux.* mail to news gateway |
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
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