From mboxrd@z Thu Jan  1 00:00:00 1970
From: Nikolai Zhubr <n-a-zhubr@yandex.ru>
Date: Wed, 26 Dec 2018 17:37:01 +0300
Subject: [U-Boot] [linux-sunxi] [PATCH] Add NFC register definitions for
 Allwinner A20 SoC
In-Reply-To: <20181226112810.ieywd5zvuhtyd3cm@plaes.org>
References: <5C2367F6.4060704@yandex.ru>
 <20181226112810.ieywd5zvuhtyd3cm@plaes.org>
Message-ID: <5C23920D.1000501@yandex.ru>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi,

26.12.2018 14:28, Priit Laes:
>> And while at it, why not finally add the DT register definitions for
>> Allwinner A20 NFC. I'm certainly not the author of these, and they have been
[...]
>> +			nand_pins_a: nand_base0 at 0 {
>> +				allwinner,pins = "PC0", "PC1", "PC2",
>> +						"PC5", "PC8", "PC9", "PC10",
>> +						"PC11", "PC12", "PC13", "PC14",
>> +						"PC15", "PC16";
>> +				allwinner,function = "nand0";
>> +				allwinner,drive =<0>;
>> +				allwinner,pull =<0>;
>
> allwinner,{pins,function,drive,pull} syntax has been deprecated in
> favor or general pinctrl syntax.
>
> You can check how sun5i.dtsi defines these blocks.

Well, I could certainly do such editing, but I'm not sure if something 
related in the tree will also need to be updated accordingly, therefore 
I'd prefer to refrain.
I'm rather a u-boot user than a u-boot developer, I can do some simple 
searching and testing, but probably not refactoring.
Thing is, however, the original Allwinner's implementation of nand boot 
procedure for A20 appeared to be so messy and fragile and inconvenient 
that using u-boot's mainline approach is basically the only reasonable 
choice for this device. (And btw great thanks to all developers who 
implemented it!)


Regards,
Nikolai

>
>
>> +			};
>> +			nand_cs0_pins_a: nand_cs at 0 {
>> +				allwinner,pins = "PC4";
>> +				allwinner,function = "nand0";
>> +				allwinner,drive =<0>;
>> +				allwinner,pull =<0>;
>> +			};
>> +			nand_cs1_pins_a: nand_cs at 1 {
>> +				allwinner,pins = "PC3";
>> +				allwinner,function = "nand0";
>> +				allwinner,drive =<0>;
>> +				allwinner,pull =<0>;
>> +			};
>> +			nand_cs2_pins_a: nand_cs at 2 {
>> +				allwinner,pins = "PC17";
>> +				allwinner,function = "nand0";
>> +				allwinner,drive =<0>;
>> +				allwinner,pull =<0>;
>> +			};
>> +			nand_cs3_pins_a: nand_cs at 3 {
>> +				allwinner,pins = "PC18";
>> +				allwinner,function = "nand0";
>> +				allwinner,drive =<0>;
>> +				allwinner,pull =<0>;
>> +			};
>> +			nand_cs4_pins_a: nand_cs at 4 {
>> +				allwinner,pins = "PC19";
>> +				allwinner,function = "nand0";
>> +				allwinner,drive =<0>;
>> +				allwinner,pull =<0>;
>> +			};
>> +			nand_cs5_pins_a: nand_cs at 5 {
>> +				allwinner,pins = "PC20";
>> +				allwinner,function = "nand0";
>> +				allwinner,drive =<0>;
>> +				allwinner,pull =<0>;
>> +			};
>> +			nand_cs6_pins_a: nand_cs at 6 {
>> +				allwinner,pins = "PC21";
>> +				allwinner,function = "nand0";
>> +				allwinner,drive =<0>;
>> +				allwinner,pull =<0>;
>> +			};
>> +			nand_cs7_pins_a: nand_cs at 7 {
>> +				allwinner,pins = "PC22";
>> +				allwinner,function = "nand0";
>> +				allwinner,drive =<0>;
>> +				allwinner,pull =<0>;
>> +			};
>> +			nand_rb0_pins_a: nand_rb at 0 {
>> +				allwinner,pins = "PC6";
>> +				allwinner,function = "nand0";
>> +				allwinner,drive =<0>;
>> +				allwinner,pull =<0>;
>> +			};
>> +			nand_rb1_pins_a: nand_rb at 1 {
>> +				allwinner,pins = "PC7";
>> +				allwinner,function = "nand0";
>> +				allwinner,drive =<0>;
>> +				allwinner,pull =<0>;
>> +			};
>> 		};
>>
>> 		timer at 1c20c00 {
>> --
>> 2.1.2
>>
>> --
>> You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
>> To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe at googlegroups.com.
>> For more options, visit https://groups.google.com/d/optout.
>