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 <kbel...@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

Reply via email to