* [v2, 3/3] ARM: dts: AM33XX: Add NAND flash device tree data to am335x-evm
@ 2013-04-29 7:55 Gupta, Pekon
2013-04-30 5:05 ` Sekhar Nori
0 siblings, 1 reply; 9+ messages in thread
From: Gupta, Pekon @ 2013-04-29 7:55 UTC (permalink / raw)
To: linux-arm-kernel
From: avinash philip <avinashphilip@ti.com>
NAND flash connected in am335x-evm on GPMC controller. This patch adds
device tree node in am3355-evm with GPMC contoller timing for NAND flash
interface, NAND partition table, ECC scheme, elm handle id.
Signed-off-by: Philip Avinash <avinashphilip@ti.com>
Tested-by: Pekon Gupta <pekon@ti.com>
---
arch/arm/boot/dts/am335x-evm.dts | 98 +++++++++++++++++++++++++++++++++++++-
1 file changed, 97 insertions(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/am335x-evm.dts b/arch/arm/boot/dts/am335x-evm.dts
index 0423298..1c2fd89 100644
--- a/arch/arm/boot/dts/am335x-evm.dts
+++ b/arch/arm/boot/dts/am335x-evm.dts
@@ -26,7 +26,8 @@
am33xx_pinmux: pinmux at 44e10800 {
pinctrl-names = "default";
- pinctrl-0 = <&matrix_keypad_s0 &volume_keys_s0>;
+ pinctrl-0 = <&matrix_keypad_s0 &volume_keys_s0
+ &nandflash_pins_s0>;
matrix_keypad_s0: matrix_keypad_s0 {
pinctrl-single,pins = <
@@ -44,6 +45,26 @@
0x154 0x27 /* spi0_d0.gpio0_3, INPUT | MODE7 */
>;
};
+
+ nandflash_pins_s0: nandflash_pins_s0 {
+ pinctrl-single,pins = <
+ 0x0 0x30 /* gpmc_ad0.gpmc_ad0, INPUT | PULLUP | MODE0 */
+ 0x4 0x30 /* gpmc_ad1.gpmc_ad1, INPUT | PULLUP | MODE0 */
+ 0x8 0x30 /* gpmc_ad2.gpmc_ad2, INPUT | PULLUP | MODE0 */
+ 0xc 0x30 /* gpmc_ad3.gpmc_ad3, INPUT | PULLUP | MODE0 */
+ 0x10 0x30 /* gpmc_ad4.gpmc_ad4, INPUT | PULLUP | MODE0 */
+ 0x14 0x30 /* gpmc_ad5.gpmc_ad5, INPUT | PULLUP | MODE0 */
+ 0x18 0x30 /* gpmc_ad6.gpmc_ad6, INPUT | PULLUP | MODE0 */
+ 0x1c 0x30 /* gpmc_ad7.gpmc_ad7, INPUT | PULLUP | MODE0 */
+ 0x70 0x30 /* gpmc_wait0.gpmc_wait0, INPUT | PULLUP | MODE0 */
+ 0x74 0x37 /* gpmc_wpn.gpio0_30, INPUT | PULLUP | MODE7 */
+ 0x7c 0x8 /* gpmc_csn0.gpmc_csn0, PULL DISA */
+ 0x90 0x8 /* gpmc_advn_ale.gpmc_advn_ale, PULL DISA */
+ 0x94 0x8 /* gpmc_oen_ren.gpmc_oen_ren, PULL DISA */
+ 0x98 0x8 /* gpmc_wen.gpmc_wen, PULL DISA */
+ 0x9c 0x8 /* gpmc_be0n_cle.gpmc_be0n_cle, PULL DISA */
+ >;
+ };
};
ocp {
@@ -102,6 +123,81 @@
reg = <0x48>;
};
};
+
+ elm: elm at 48080000 {
+ status = "okay";
+ };
+
+ gpmc: gpmc at 50000000 {
+ status = "okay";
+ ranges = <0 0 0x08000000 0x10000000>; /* CS0: NAND */
+
+ nand at 0,0 {
+ reg = <0 0 0>; /* CS0, offset 0 */
+ nand-bus-width = <8>;
+ ti,nand-ecc-opt = "bch8";
+
+ gpmc,sync-clk = <0>;
+ gpmc,cs-on = <0>;
+ gpmc,cs-rd-off = <44>;
+ gpmc,cs-wr-off = <44>;
+ gpmc,adv-on = <6>;
+ gpmc,adv-rd-off = <34>;
+ gpmc,adv-wr-off = <44>;
+ gpmc,we-off = <40>;
+ gpmc,oe-off = <54>;
+ gpmc,access = <64>;
+ gpmc,rd-cycle = <82>;
+ gpmc,wr-cycle = <82>;
+ gpmc,wr-access = <40>;
+ gpmc,wr-data-mux-bus = <0>;
+
+ #address-cells = <1>;
+ #size-cells = <1>;
+ elm_id = <&elm>;
+
+ /* MTD partition table */
+ partition at 0 {
+ label = "SPL1";
+ reg = <0x00000000 0x000020000>;
+ };
+
+ partition at 1 {
+ label = "SPL2";
+ reg = <0x00020000 0x00020000>;
+ };
+
+ partition at 2 {
+ label = "SPL3";
+ reg = <0x00040000 0x00020000>;
+ };
+
+ partition at 3 {
+ label = "SPL4";
+ reg = <0x00060000 0x00020000>;
+ };
+
+ partition at 4 {
+ label = "U-boot";
+ reg = <0x00080000 0x001e0000>;
+ };
+
+ partition at 5 {
+ label = "environment";
+ reg = <0x00260000 0x00020000>;
+ };
+
+ partition at 6 {
+ label = "Kernel";
+ reg = <0x00280000 0x00500000>;
+ };
+
+ partition at 7 {
+ label = "File-System";
+ reg = <0x00780000 0x0F880000>;
+ };
+ };
+ };
};
vbat: fixedregulator at 0 {
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [v2, 3/3] ARM: dts: AM33XX: Add NAND flash device tree data to am335x-evm
2013-04-29 7:55 [v2, 3/3] ARM: dts: AM33XX: Add NAND flash device tree data to am335x-evm Gupta, Pekon
@ 2013-04-30 5:05 ` Sekhar Nori
2013-05-08 23:54 ` Tony Lindgren
0 siblings, 1 reply; 9+ messages in thread
From: Sekhar Nori @ 2013-04-30 5:05 UTC (permalink / raw)
To: linux-arm-kernel
On 4/29/2013 1:25 PM, Gupta, Pekon wrote:
> From: avinash philip <avinashphilip@ti.com>
>
> NAND flash connected in am335x-evm on GPMC controller. This patch adds
> device tree node in am3355-evm with GPMC contoller timing for NAND flash
> interface, NAND partition table, ECC scheme, elm handle id.
>
> Signed-off-by: Philip Avinash <avinashphilip@ti.com>
> Tested-by: Pekon Gupta <pekon@ti.com>
>
> ---
> arch/arm/boot/dts/am335x-evm.dts | 98 +++++++++++++++++++++++++++++++++++++-
> 1 file changed, 97 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/am335x-evm.dts b/arch/arm/boot/dts/am335x-evm.dts
> index 0423298..1c2fd89 100644
> --- a/arch/arm/boot/dts/am335x-evm.dts
> +++ b/arch/arm/boot/dts/am335x-evm.dts
> @@ -26,7 +26,8 @@
>
> am33xx_pinmux: pinmux at 44e10800 {
> pinctrl-names = "default";
> - pinctrl-0 = <&matrix_keypad_s0 &volume_keys_s0>;
> + pinctrl-0 = <&matrix_keypad_s0 &volume_keys_s0
> + &nandflash_pins_s0>;
Why add this to the board level fallback (called pinctrl hogs, I think)?
This can be part of nand node you added below so that the pinctrl will
take effect when nand gets probed instead of all the time.
Thanks,
Sekhar
^ permalink raw reply [flat|nested] 9+ messages in thread
* [v2, 3/3] ARM: dts: AM33XX: Add NAND flash device tree data to am335x-evm
2013-04-30 5:05 ` Sekhar Nori
@ 2013-05-08 23:54 ` Tony Lindgren
2013-05-09 4:12 ` Gupta, Pekon
0 siblings, 1 reply; 9+ messages in thread
From: Tony Lindgren @ 2013-05-08 23:54 UTC (permalink / raw)
To: linux-arm-kernel
* Sekhar Nori <nsekhar@ti.com> [130429 22:11]:
>
> On 4/29/2013 1:25 PM, Gupta, Pekon wrote:
> > From: avinash philip <avinashphilip@ti.com>
> >
> > NAND flash connected in am335x-evm on GPMC controller. This patch adds
> > device tree node in am3355-evm with GPMC contoller timing for NAND flash
> > interface, NAND partition table, ECC scheme, elm handle id.
> >
> > Signed-off-by: Philip Avinash <avinashphilip@ti.com>
> > Tested-by: Pekon Gupta <pekon@ti.com>
> >
> > ---
> > arch/arm/boot/dts/am335x-evm.dts | 98 +++++++++++++++++++++++++++++++++++++-
> > 1 file changed, 97 insertions(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/boot/dts/am335x-evm.dts b/arch/arm/boot/dts/am335x-evm.dts
> > index 0423298..1c2fd89 100644
> > --- a/arch/arm/boot/dts/am335x-evm.dts
> > +++ b/arch/arm/boot/dts/am335x-evm.dts
> > @@ -26,7 +26,8 @@
> >
> > am33xx_pinmux: pinmux at 44e10800 {
> > pinctrl-names = "default";
> > - pinctrl-0 = <&matrix_keypad_s0 &volume_keys_s0>;
> > + pinctrl-0 = <&matrix_keypad_s0 &volume_keys_s0
> > + &nandflash_pins_s0>;
>
> Why add this to the board level fallback (called pinctrl hogs, I think)?
> This can be part of nand node you added below so that the pinctrl will
> take effect when nand gets probed instead of all the time.
Yes we should have all the pinctrl entries under the related drivers.
This makes it easy remux pins during runtime if needed, and also
allows unloading pinctrl-single.ko for development.
Regards,
Tony
^ permalink raw reply [flat|nested] 9+ messages in thread
* [v2, 3/3] ARM: dts: AM33XX: Add NAND flash device tree data to am335x-evm
2013-05-08 23:54 ` Tony Lindgren
@ 2013-05-09 4:12 ` Gupta, Pekon
2013-05-20 4:44 ` Gupta, Pekon
0 siblings, 1 reply; 9+ messages in thread
From: Gupta, Pekon @ 2013-05-09 4:12 UTC (permalink / raw)
To: linux-arm-kernel
> > >
> > > am33xx_pinmux: pinmux at 44e10800 {
> > > pinctrl-names = "default";
> > > - pinctrl-0 = <&matrix_keypad_s0 &volume_keys_s0>;
> > > + pinctrl-0 = <&matrix_keypad_s0 &volume_keys_s0
> > > + &nandflash_pins_s0>;
> >
> > Why add this to the board level fallback (called pinctrl hogs, I think)?
> > This can be part of nand node you added below so that the pinctrl will
> > take effect when nand gets probed instead of all the time.
>
> Yes we should have all the pinctrl entries under the related drivers.
> This makes it easy remux pins during runtime if needed, and also
> allows unloading pinctrl-single.ko for development.
>
Yes, accepted. This has been already fixed in v3 of this patch set.
If all fine, then please pull this for next merge..
http://lists.infradead.org/pipermail/linux-mtd/2013-May/046712.html
http://lists.infradead.org/pipermail/linux-mtd/2013-May/046814.html (already pulled)
http://lists.infradead.org/pipermail/linux-mtd/2013-May/046710.html
with regards, pekon
^ permalink raw reply [flat|nested] 9+ messages in thread
* [v2, 3/3] ARM: dts: AM33XX: Add NAND flash device tree data to am335x-evm
2013-05-09 4:12 ` Gupta, Pekon
@ 2013-05-20 4:44 ` Gupta, Pekon
2013-05-30 7:24 ` Benoit Cousson
0 siblings, 1 reply; 9+ messages in thread
From: Gupta, Pekon @ 2013-05-20 4:44 UTC (permalink / raw)
To: linux-arm-kernel
> > > >
> > > > am33xx_pinmux: pinmux at 44e10800 {
> > > > pinctrl-names = "default";
> > > > - pinctrl-0 = <&matrix_keypad_s0 &volume_keys_s0>;
> > > > + pinctrl-0 = <&matrix_keypad_s0 &volume_keys_s0
> > > > + &nandflash_pins_s0>;
> > >
> > > Why add this to the board level fallback (called pinctrl hogs, I think)?
> > > This can be part of nand node you added below so that the pinctrl will
> > > take effect when nand gets probed instead of all the time.
> >
> > Yes we should have all the pinctrl entries under the related drivers.
> > This makes it easy remux pins during runtime if needed, and also
> > allows unloading pinctrl-single.ko for development.
> >
> Yes, accepted. This has been already fixed in v3 of this patch set.
> If all fine, then please pull this for next merge..
>
> http://lists.infradead.org/pipermail/linux-mtd/2013-May/046712.html
>
> http://lists.infradead.org/pipermail/linux-mtd/2013-May/046814.html
>
> http://lists.infradead.org/pipermail/linux-mtd/2013-May/046710.html
>
>
> with regards, pekon
Request you to please accept | provide feedbacks on this patch series.
These are waiting acceptance since Jan-2013, and are necessary for
DT based configs for board having NAND support.
with regards, pekon
^ permalink raw reply [flat|nested] 9+ messages in thread
* [v2, 3/3] ARM: dts: AM33XX: Add NAND flash device tree data to am335x-evm
2013-05-20 4:44 ` Gupta, Pekon
@ 2013-05-30 7:24 ` Benoit Cousson
2013-05-30 7:31 ` Gupta, Pekon
0 siblings, 1 reply; 9+ messages in thread
From: Benoit Cousson @ 2013-05-30 7:24 UTC (permalink / raw)
To: linux-arm-kernel
Hi Pekon,
On 05/20/2013 06:44 AM, Gupta, Pekon wrote:
>
>>>>>
>>>>> am33xx_pinmux: pinmux at 44e10800 {
>>>>> pinctrl-names = "default";
>>>>> - pinctrl-0 = <&matrix_keypad_s0 &volume_keys_s0>;
>>>>> + pinctrl-0 = <&matrix_keypad_s0 &volume_keys_s0
>>>>> + &nandflash_pins_s0>;
>>>>
>>>> Why add this to the board level fallback (called pinctrl hogs, I think)?
>>>> This can be part of nand node you added below so that the pinctrl will
>>>> take effect when nand gets probed instead of all the time.
>>>
>>> Yes we should have all the pinctrl entries under the related drivers.
>>> This makes it easy remux pins during runtime if needed, and also
>>> allows unloading pinctrl-single.ko for development.
>>>
>> Yes, accepted. This has been already fixed in v3 of this patch set.
>> If all fine, then please pull this for next merge..
>>
>> http://lists.infradead.org/pipermail/linux-mtd/2013-May/046712.html
>>
>> http://lists.infradead.org/pipermail/linux-mtd/2013-May/046814.html
>>
>> http://lists.infradead.org/pipermail/linux-mtd/2013-May/046710.html
>>
>>
>> with regards, pekon
>
> Request you to please accept | provide feedbacks on this patch series.
> These are waiting acceptance since Jan-2013, and are necessary for
> DT based configs for board having NAND support.
>
> with regards, pekon
Sorry, I missed that series.
I'm applying it right now.
Thanks,
Benoit
^ permalink raw reply [flat|nested] 9+ messages in thread
* [v2, 3/3] ARM: dts: AM33XX: Add NAND flash device tree data to am335x-evm
2013-05-30 7:24 ` Benoit Cousson
@ 2013-05-30 7:31 ` Gupta, Pekon
2013-05-30 7:36 ` Gupta, Pekon
2013-05-30 7:39 ` Benoit Cousson
0 siblings, 2 replies; 9+ messages in thread
From: Gupta, Pekon @ 2013-05-30 7:31 UTC (permalink / raw)
To: linux-arm-kernel
> Sorry, I missed that series.
>
> I'm applying it right now.
>
No issues.. Please pick newer v4 versions of this series.
Following are rebased, updated and tested on linux-3.10-rc3
[PATCH v4,0/3] http://www.spinics.net/lists/linux-omap/msg91165.html
[PATCH v4,1/3] http://www.spinics.net/lists/linux-omap/msg91166.html
[PATCH v4,2/3] (please skip this one)
instead pick http://www.spinics.net/lists/linux-omap/msg91161.html
[PATCH v4,3/3] http://www.spinics.net/lists/linux-omap/msg91167.html
Thanks..
with regards, pekon
^ permalink raw reply [flat|nested] 9+ messages in thread
* [v2, 3/3] ARM: dts: AM33XX: Add NAND flash device tree data to am335x-evm
2013-05-30 7:31 ` Gupta, Pekon
@ 2013-05-30 7:36 ` Gupta, Pekon
2013-05-30 7:39 ` Benoit Cousson
1 sibling, 0 replies; 9+ messages in thread
From: Gupta, Pekon @ 2013-05-30 7:36 UTC (permalink / raw)
To: linux-arm-kernel
> > Sorry, I missed that series.
> >
> > I'm applying it right now.
> >
> No issues.. Please pick newer v4 versions of this series.
> Following are rebased, updated and tested on linux-3.10-rc3
>
> [PATCH v4,0/3] http://www.spinics.net/lists/linux-omap/msg91165.html
>
> [PATCH v4,1/3] http://www.spinics.net/lists/linux-omap/msg91166.html
>
> [PATCH v4,2/3] (please skip this one)
> instead pick http://www.spinics.net/lists/linux-omap/msg91161.html
>
> [PATCH v4,3/3] (typo)
http://www.spinics.net/lists/linux-omap/msg91164.html
>
>
> Thanks..
> with regards, pekon
>
> ______________________________________________________
> Linux MTD discussion mailing list
> http://lists.infradead.org/mailman/listinfo/linux-mtd/
^ permalink raw reply [flat|nested] 9+ messages in thread
* [v2, 3/3] ARM: dts: AM33XX: Add NAND flash device tree data to am335x-evm
2013-05-30 7:31 ` Gupta, Pekon
2013-05-30 7:36 ` Gupta, Pekon
@ 2013-05-30 7:39 ` Benoit Cousson
1 sibling, 0 replies; 9+ messages in thread
From: Benoit Cousson @ 2013-05-30 7:39 UTC (permalink / raw)
To: linux-arm-kernel
On 05/30/2013 09:31 AM, Gupta, Pekon wrote:
>> Sorry, I missed that series.
>>
>> I'm applying it right now.
>>
> No issues.. Please pick newer v4 versions of this series.
> Following are rebased, updated and tested on linux-3.10-rc3
>
> [PATCH v4,0/3] http://www.spinics.net/lists/linux-omap/msg91165.html
>
> [PATCH v4,1/3] http://www.spinics.net/lists/linux-omap/msg91166.html
>
> [PATCH v4,2/3] (please skip this one)
> instead pick http://www.spinics.net/lists/linux-omap/msg91161.html
>
> [PATCH v4,3/3] http://www.spinics.net/lists/linux-omap/msg91167.html
Could you please rebase on for_3.11/dts and repost the whole stuff. It
looks like Tony already pulled the GPMC node, and the two other ones
cannot apply anymore.
Thanks,
Benoit
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2013-05-30 7:39 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-29 7:55 [v2, 3/3] ARM: dts: AM33XX: Add NAND flash device tree data to am335x-evm Gupta, Pekon
2013-04-30 5:05 ` Sekhar Nori
2013-05-08 23:54 ` Tony Lindgren
2013-05-09 4:12 ` Gupta, Pekon
2013-05-20 4:44 ` Gupta, Pekon
2013-05-30 7:24 ` Benoit Cousson
2013-05-30 7:31 ` Gupta, Pekon
2013-05-30 7:36 ` Gupta, Pekon
2013-05-30 7:39 ` Benoit Cousson
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).