* [PATCH] ARM: dts: cm-t3x: add NAND support @ 2014-12-28 13:15 Dmitry Lifshitz 2014-12-28 13:55 ` Igor Grinberg 0 siblings, 1 reply; 9+ messages in thread From: Dmitry Lifshitz @ 2014-12-28 13:15 UTC (permalink / raw) To: Benoît Cousson, Tony Lindgren, linux-omap, devicetree, linux-arm-kernel Cc: Dmitry Lifshitz Add NAND support Signed-off-by: Dmitry Lifshitz <lifshitz@compulab.co.il> --- arch/arm/boot/dts/omap3-cm-t3x.dtsi | 58 +++++++++++++++++++++++++++++++++ arch/arm/boot/dts/omap3-sbc-t3517.dts | 4 ++ arch/arm/boot/dts/omap3-sbc-t3530.dts | 10 ++---- arch/arm/boot/dts/omap3-sbc-t3730.dts | 5 ++- 4 files changed, 68 insertions(+), 9 deletions(-) diff --git a/arch/arm/boot/dts/omap3-cm-t3x.dtsi b/arch/arm/boot/dts/omap3-cm-t3x.dtsi index 6ea6d46..4d091ca 100644 --- a/arch/arm/boot/dts/omap3-cm-t3x.dtsi +++ b/arch/arm/boot/dts/omap3-cm-t3x.dtsi @@ -259,3 +259,61 @@ pinctrl-names = "default"; pinctrl-0 = <&mcbsp2_pins>; }; + +&gpmc { + ranges = <0 0 0x00000000 0x01000000>; + + nand@0,0 { + reg = <0 0 4>; /* CS0, offset 0, IO size 4 */ + nand-bus-width = <8>; + gpmc,device-width = <1>; + ti,nand-ecc-opt = "sw"; + + gpmc,cs-on-ns = <0>; + gpmc,cs-rd-off-ns = <120>; + gpmc,cs-wr-off-ns = <120>; + + gpmc,adv-on-ns = <0>; + gpmc,adv-rd-off-ns = <120>; + gpmc,adv-wr-off-ns = <120>; + + gpmc,we-on-ns = <6>; + gpmc,we-off-ns = <90>; + + gpmc,oe-on-ns = <6>; + gpmc,oe-off-ns = <90>; + + gpmc,page-burst-access-ns = <6>; + gpmc,access-ns = <72>; + gpmc,cycle2cycle-delay-ns = <60>; + + gpmc,rd-cycle-ns = <120>; + gpmc,wr-cycle-ns = <120>; + gpmc,wr-access-ns = <186>; + gpmc,wr-data-mux-bus-ns = <90>; + + #address-cells = <1>; + #size-cells = <1>; + + partition@0 { + label = "xloader"; + reg = <0 0x80000>; + }; + partition@0x80000 { + label = "uboot"; + reg = <0x80000 0x1e0000>; + }; + partition@0x260000 { + label = "uboot environment"; + reg = <0x260000 0x40000>; + }; + partition@0x2a0000 { + label = "linux"; + reg = <0x2a0000 0x400000>; + }; + partition@0x6a0000 { + label = "rootfs"; + reg = <0x6a0000 0x1f880000>; + }; + }; +}; diff --git a/arch/arm/boot/dts/omap3-sbc-t3517.dts b/arch/arm/boot/dts/omap3-sbc-t3517.dts index 1798653..c2d5c28 100644 --- a/arch/arm/boot/dts/omap3-sbc-t3517.dts +++ b/arch/arm/boot/dts/omap3-sbc-t3517.dts @@ -69,3 +69,7 @@ }; }; +&gpmc { + ranges = <4 0 0x2d000000 0x01000000>, /* SB-T35 SMSC9x Eth */ + <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ +}; diff --git a/arch/arm/boot/dts/omap3-sbc-t3530.dts b/arch/arm/boot/dts/omap3-sbc-t3530.dts index c994f0f..834bc78 100644 --- a/arch/arm/boot/dts/omap3-sbc-t3530.dts +++ b/arch/arm/boot/dts/omap3-sbc-t3530.dts @@ -26,14 +26,10 @@ }; }; -/* - * The following ranges correspond to SMSC9x eth chips on CM-T3530 CoM and - * SB-T35 baseboard respectively. - * This setting includes both chips in SBC-T3530 board device tree. - */ &gpmc { - ranges = <5 0 0x2c000000 0x01000000>, - <4 0 0x2d000000 0x01000000>; + ranges = <5 0 0x2c000000 0x01000000>, /* CM-T3x30 SMSC9x Eth */ + <4 0 0x2d000000 0x01000000>, /* SB-T35 SMSC9x Eth */ + <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ }; &mmc1 { diff --git a/arch/arm/boot/dts/omap3-sbc-t3730.dts b/arch/arm/boot/dts/omap3-sbc-t3730.dts index 5bdddf2..73c7bf4 100644 --- a/arch/arm/boot/dts/omap3-sbc-t3730.dts +++ b/arch/arm/boot/dts/omap3-sbc-t3730.dts @@ -27,8 +27,9 @@ }; &gpmc { - ranges = <5 0 0x2c000000 0x01000000>, - <4 0 0x2d000000 0x01000000>; + ranges = <5 0 0x2c000000 0x01000000>, /* CM-T3x30 SMSC9x Eth */ + <4 0 0x2d000000 0x01000000>, /* SB-T35 SMSC9x Eth */ + <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ }; &dss { -- 1.7.5.4 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] ARM: dts: cm-t3x: add NAND support 2014-12-28 13:15 [PATCH] ARM: dts: cm-t3x: add NAND support Dmitry Lifshitz @ 2014-12-28 13:55 ` Igor Grinberg 2014-12-28 14:30 ` [PATCH v2] " Dmitry Lifshitz 0 siblings, 1 reply; 9+ messages in thread From: Igor Grinberg @ 2014-12-28 13:55 UTC (permalink / raw) To: Dmitry Lifshitz, Benoît Cousson, Tony Lindgren, linux-omap, devicetree, linux-arm-kernel Hi Dima, On 12/28/14 15:15, Dmitry Lifshitz wrote: > Add NAND support > > Signed-off-by: Dmitry Lifshitz <lifshitz@compulab.co.il> > --- > arch/arm/boot/dts/omap3-cm-t3x.dtsi | 58 +++++++++++++++++++++++++++++++++ > arch/arm/boot/dts/omap3-sbc-t3517.dts | 4 ++ > arch/arm/boot/dts/omap3-sbc-t3530.dts | 10 ++---- > arch/arm/boot/dts/omap3-sbc-t3730.dts | 5 ++- > 4 files changed, 68 insertions(+), 9 deletions(-) It seems that you have missed the omap3-cm-t3x30.dtsi. If after this patch you build omap3-cm-t3530.dts or omap3-cm-t3730.dts, it will not have NAND, while they should get it as well. -- Regards, Igor. ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH v2] ARM: dts: cm-t3x: add NAND support 2014-12-28 13:55 ` Igor Grinberg @ 2014-12-28 14:30 ` Dmitry Lifshitz 2014-12-28 14:54 ` Igor Grinberg 2014-12-29 13:06 ` Roger Quadros 0 siblings, 2 replies; 9+ messages in thread From: Dmitry Lifshitz @ 2014-12-28 14:30 UTC (permalink / raw) To: Benoît Cousson, Tony Lindgren, linux-omap, devicetree, linux-arm-kernel Cc: Dmitry Lifshitz CM-T3517, CM-T3530 and CM-T3730 features NAND storage chip connected to GPMC bus. Add GPMC DT entry into the root DT file omap3-cm-t3x.dtsi, common for all three modules. NAND timings are calculated to be safe for CM-T3x devices as it works now in non DT boot (in this case the timings are updated by U-Boot). Update GPMC ranges in boards DT files to include all connected devices. Signed-off-by: Dmitry Lifshitz <lifshitz@compulab.co.il> --- v2 * Update GPMC ranges with NAND values in omap3-cm-t3x30.dtsi. * Add commit message details arch/arm/boot/dts/omap3-cm-t3x.dtsi | 58 +++++++++++++++++++++++++++++++++ arch/arm/boot/dts/omap3-cm-t3x30.dtsi | 3 +- arch/arm/boot/dts/omap3-sbc-t3517.dts | 4 ++ arch/arm/boot/dts/omap3-sbc-t3530.dts | 10 ++---- arch/arm/boot/dts/omap3-sbc-t3730.dts | 5 ++- 5 files changed, 70 insertions(+), 10 deletions(-) diff --git a/arch/arm/boot/dts/omap3-cm-t3x.dtsi b/arch/arm/boot/dts/omap3-cm-t3x.dtsi index 6ea6d46..4d091ca 100644 --- a/arch/arm/boot/dts/omap3-cm-t3x.dtsi +++ b/arch/arm/boot/dts/omap3-cm-t3x.dtsi @@ -259,3 +259,61 @@ pinctrl-names = "default"; pinctrl-0 = <&mcbsp2_pins>; }; + +&gpmc { + ranges = <0 0 0x00000000 0x01000000>; + + nand@0,0 { + reg = <0 0 4>; /* CS0, offset 0, IO size 4 */ + nand-bus-width = <8>; + gpmc,device-width = <1>; + ti,nand-ecc-opt = "sw"; + + gpmc,cs-on-ns = <0>; + gpmc,cs-rd-off-ns = <120>; + gpmc,cs-wr-off-ns = <120>; + + gpmc,adv-on-ns = <0>; + gpmc,adv-rd-off-ns = <120>; + gpmc,adv-wr-off-ns = <120>; + + gpmc,we-on-ns = <6>; + gpmc,we-off-ns = <90>; + + gpmc,oe-on-ns = <6>; + gpmc,oe-off-ns = <90>; + + gpmc,page-burst-access-ns = <6>; + gpmc,access-ns = <72>; + gpmc,cycle2cycle-delay-ns = <60>; + + gpmc,rd-cycle-ns = <120>; + gpmc,wr-cycle-ns = <120>; + gpmc,wr-access-ns = <186>; + gpmc,wr-data-mux-bus-ns = <90>; + + #address-cells = <1>; + #size-cells = <1>; + + partition@0 { + label = "xloader"; + reg = <0 0x80000>; + }; + partition@0x80000 { + label = "uboot"; + reg = <0x80000 0x1e0000>; + }; + partition@0x260000 { + label = "uboot environment"; + reg = <0x260000 0x40000>; + }; + partition@0x2a0000 { + label = "linux"; + reg = <0x2a0000 0x400000>; + }; + partition@0x6a0000 { + label = "rootfs"; + reg = <0x6a0000 0x1f880000>; + }; + }; +}; diff --git a/arch/arm/boot/dts/omap3-cm-t3x30.dtsi b/arch/arm/boot/dts/omap3-cm-t3x30.dtsi index 9a4a3ab..d9e92b6 100644 --- a/arch/arm/boot/dts/omap3-cm-t3x30.dtsi +++ b/arch/arm/boot/dts/omap3-cm-t3x30.dtsi @@ -50,7 +50,8 @@ #include "omap-gpmc-smsc911x.dtsi" &gpmc { - ranges = <5 0 0x2c000000 0x01000000>; + ranges = <5 0 0x2c000000 0x01000000>, /* CM-T3x30 SMSC9x Eth */ + <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ smsc1: ethernet@gpmc { compatible = "smsc,lan9221", "smsc,lan9115"; diff --git a/arch/arm/boot/dts/omap3-sbc-t3517.dts b/arch/arm/boot/dts/omap3-sbc-t3517.dts index 1798653..c2d5c28 100644 --- a/arch/arm/boot/dts/omap3-sbc-t3517.dts +++ b/arch/arm/boot/dts/omap3-sbc-t3517.dts @@ -69,3 +69,7 @@ }; }; +&gpmc { + ranges = <4 0 0x2d000000 0x01000000>, /* SB-T35 SMSC9x Eth */ + <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ +}; diff --git a/arch/arm/boot/dts/omap3-sbc-t3530.dts b/arch/arm/boot/dts/omap3-sbc-t3530.dts index c994f0f..834bc78 100644 --- a/arch/arm/boot/dts/omap3-sbc-t3530.dts +++ b/arch/arm/boot/dts/omap3-sbc-t3530.dts @@ -26,14 +26,10 @@ }; }; -/* - * The following ranges correspond to SMSC9x eth chips on CM-T3530 CoM and - * SB-T35 baseboard respectively. - * This setting includes both chips in SBC-T3530 board device tree. - */ &gpmc { - ranges = <5 0 0x2c000000 0x01000000>, - <4 0 0x2d000000 0x01000000>; + ranges = <5 0 0x2c000000 0x01000000>, /* CM-T3x30 SMSC9x Eth */ + <4 0 0x2d000000 0x01000000>, /* SB-T35 SMSC9x Eth */ + <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ }; &mmc1 { diff --git a/arch/arm/boot/dts/omap3-sbc-t3730.dts b/arch/arm/boot/dts/omap3-sbc-t3730.dts index 5bdddf2..73c7bf4 100644 --- a/arch/arm/boot/dts/omap3-sbc-t3730.dts +++ b/arch/arm/boot/dts/omap3-sbc-t3730.dts @@ -27,8 +27,9 @@ }; &gpmc { - ranges = <5 0 0x2c000000 0x01000000>, - <4 0 0x2d000000 0x01000000>; + ranges = <5 0 0x2c000000 0x01000000>, /* CM-T3x30 SMSC9x Eth */ + <4 0 0x2d000000 0x01000000>, /* SB-T35 SMSC9x Eth */ + <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ }; &dss { -- 1.7.5.4 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH v2] ARM: dts: cm-t3x: add NAND support 2014-12-28 14:30 ` [PATCH v2] " Dmitry Lifshitz @ 2014-12-28 14:54 ` Igor Grinberg 2014-12-29 13:06 ` Roger Quadros 1 sibling, 0 replies; 9+ messages in thread From: Igor Grinberg @ 2014-12-28 14:54 UTC (permalink / raw) To: Dmitry Lifshitz, Benoît Cousson, Tony Lindgren, linux-omap, devicetree, linux-arm-kernel On 12/28/14 16:30, Dmitry Lifshitz wrote: > CM-T3517, CM-T3530 and CM-T3730 features NAND storage chip connected to > GPMC bus. > > Add GPMC DT entry into the root DT file omap3-cm-t3x.dtsi, common for > all three modules. > > NAND timings are calculated to be safe for CM-T3x devices as it works > now in non DT boot (in this case the timings are updated by U-Boot). > > Update GPMC ranges in boards DT files to include all connected devices. > > Signed-off-by: Dmitry Lifshitz <lifshitz@compulab.co.il> Acked-by: Igor Grinberg <grinberg@compulab.co.il> -- Regards, Igor. ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] ARM: dts: cm-t3x: add NAND support 2014-12-28 14:30 ` [PATCH v2] " Dmitry Lifshitz 2014-12-28 14:54 ` Igor Grinberg @ 2014-12-29 13:06 ` Roger Quadros 2014-12-30 7:39 ` Dmitry Lifshitz 1 sibling, 1 reply; 9+ messages in thread From: Roger Quadros @ 2014-12-29 13:06 UTC (permalink / raw) To: Dmitry Lifshitz, Benoît Cousson, Tony Lindgren, linux-omap, devicetree, linux-arm-kernel, grinberg Hi Dmitry, On 28/12/14 16:30, Dmitry Lifshitz wrote: > CM-T3517, CM-T3530 and CM-T3730 features NAND storage chip connected to > GPMC bus. > > Add GPMC DT entry into the root DT file omap3-cm-t3x.dtsi, common for > all three modules. > > NAND timings are calculated to be safe for CM-T3x devices as it works > now in non DT boot (in this case the timings are updated by U-Boot). I didn't get this part. You provide the GPMC timings in the DT now so they will override u-boot configured timings. > > Update GPMC ranges in boards DT files to include all connected devices. > > Signed-off-by: Dmitry Lifshitz <lifshitz@compulab.co.il> > --- > > v2 * Update GPMC ranges with NAND values in omap3-cm-t3x30.dtsi. > * Add commit message details > > arch/arm/boot/dts/omap3-cm-t3x.dtsi | 58 +++++++++++++++++++++++++++++++++ > arch/arm/boot/dts/omap3-cm-t3x30.dtsi | 3 +- > arch/arm/boot/dts/omap3-sbc-t3517.dts | 4 ++ > arch/arm/boot/dts/omap3-sbc-t3530.dts | 10 ++---- > arch/arm/boot/dts/omap3-sbc-t3730.dts | 5 ++- > 5 files changed, 70 insertions(+), 10 deletions(-) > > diff --git a/arch/arm/boot/dts/omap3-cm-t3x.dtsi b/arch/arm/boot/dts/omap3-cm-t3x.dtsi > index 6ea6d46..4d091ca 100644 > --- a/arch/arm/boot/dts/omap3-cm-t3x.dtsi > +++ b/arch/arm/boot/dts/omap3-cm-t3x.dtsi > @@ -259,3 +259,61 @@ > pinctrl-names = "default"; > pinctrl-0 = <&mcbsp2_pins>; > }; > + > +&gpmc { > + ranges = <0 0 0x00000000 0x01000000>; Isn't this ranges property redundant as it will be overridden by the board specific dts? > + > + nand@0,0 { > + reg = <0 0 4>; /* CS0, offset 0, IO size 4 */ > + nand-bus-width = <8>; > + gpmc,device-width = <1>; > + ti,nand-ecc-opt = "sw"; any particular reason for sticking with "sw"? Isn't it better to use at least "ham1" or migrate to stronger schemes like "bch4" or "bch8"? "ham1" scheme is compatible with ROM loader so the boot partition could be flashed from Linux. For bch4/8 you will have to keep boot partition/s locked from Linux access as we don't yet support mixed ECC schemes among partitions. > + > + gpmc,cs-on-ns = <0>; > + gpmc,cs-rd-off-ns = <120>; > + gpmc,cs-wr-off-ns = <120>; > + > + gpmc,adv-on-ns = <0>; > + gpmc,adv-rd-off-ns = <120>; > + gpmc,adv-wr-off-ns = <120>; > + > + gpmc,we-on-ns = <6>; > + gpmc,we-off-ns = <90>; > + > + gpmc,oe-on-ns = <6>; > + gpmc,oe-off-ns = <90>; > + > + gpmc,page-burst-access-ns = <6>; > + gpmc,access-ns = <72>; > + gpmc,cycle2cycle-delay-ns = <60>; > + > + gpmc,rd-cycle-ns = <120>; > + gpmc,wr-cycle-ns = <120>; > + gpmc,wr-access-ns = <186>; > + gpmc,wr-data-mux-bus-ns = <90>; > + > + #address-cells = <1>; > + #size-cells = <1>; > + > + partition@0 { > + label = "xloader"; > + reg = <0 0x80000>; > + }; > + partition@0x80000 { > + label = "uboot"; > + reg = <0x80000 0x1e0000>; > + }; > + partition@0x260000 { > + label = "uboot environment"; > + reg = <0x260000 0x40000>; > + }; > + partition@0x2a0000 { > + label = "linux"; > + reg = <0x2a0000 0x400000>; > + }; > + partition@0x6a0000 { > + label = "rootfs"; > + reg = <0x6a0000 0x1f880000>; > + }; > + }; > +}; > diff --git a/arch/arm/boot/dts/omap3-cm-t3x30.dtsi b/arch/arm/boot/dts/omap3-cm-t3x30.dtsi > index 9a4a3ab..d9e92b6 100644 > --- a/arch/arm/boot/dts/omap3-cm-t3x30.dtsi > +++ b/arch/arm/boot/dts/omap3-cm-t3x30.dtsi > @@ -50,7 +50,8 @@ > #include "omap-gpmc-smsc911x.dtsi" > > &gpmc { > - ranges = <5 0 0x2c000000 0x01000000>; > + ranges = <5 0 0x2c000000 0x01000000>, /* CM-T3x30 SMSC9x Eth */ > + <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ Isn't this ranges property redundant as it will anyways be overridden by the board specific dts? > > smsc1: ethernet@gpmc { > compatible = "smsc,lan9221", "smsc,lan9115"; > diff --git a/arch/arm/boot/dts/omap3-sbc-t3517.dts b/arch/arm/boot/dts/omap3-sbc-t3517.dts > index 1798653..c2d5c28 100644 > --- a/arch/arm/boot/dts/omap3-sbc-t3517.dts > +++ b/arch/arm/boot/dts/omap3-sbc-t3517.dts > @@ -69,3 +69,7 @@ > }; > }; > > +&gpmc { > + ranges = <4 0 0x2d000000 0x01000000>, /* SB-T35 SMSC9x Eth */ > + <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ > +}; > diff --git a/arch/arm/boot/dts/omap3-sbc-t3530.dts b/arch/arm/boot/dts/omap3-sbc-t3530.dts > index c994f0f..834bc78 100644 > --- a/arch/arm/boot/dts/omap3-sbc-t3530.dts > +++ b/arch/arm/boot/dts/omap3-sbc-t3530.dts > @@ -26,14 +26,10 @@ > }; > }; > > -/* > - * The following ranges correspond to SMSC9x eth chips on CM-T3530 CoM and > - * SB-T35 baseboard respectively. > - * This setting includes both chips in SBC-T3530 board device tree. > - */ > &gpmc { > - ranges = <5 0 0x2c000000 0x01000000>, > - <4 0 0x2d000000 0x01000000>; > + ranges = <5 0 0x2c000000 0x01000000>, /* CM-T3x30 SMSC9x Eth */ > + <4 0 0x2d000000 0x01000000>, /* SB-T35 SMSC9x Eth */ > + <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ > }; > > &mmc1 { > diff --git a/arch/arm/boot/dts/omap3-sbc-t3730.dts b/arch/arm/boot/dts/omap3-sbc-t3730.dts > index 5bdddf2..73c7bf4 100644 > --- a/arch/arm/boot/dts/omap3-sbc-t3730.dts > +++ b/arch/arm/boot/dts/omap3-sbc-t3730.dts > @@ -27,8 +27,9 @@ > }; > > &gpmc { > - ranges = <5 0 0x2c000000 0x01000000>, > - <4 0 0x2d000000 0x01000000>; > + ranges = <5 0 0x2c000000 0x01000000>, /* CM-T3x30 SMSC9x Eth */ > + <4 0 0x2d000000 0x01000000>, /* SB-T35 SMSC9x Eth */ > + <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ > }; > > &dss { > cheers, -roger ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] ARM: dts: cm-t3x: add NAND support 2014-12-29 13:06 ` Roger Quadros @ 2014-12-30 7:39 ` Dmitry Lifshitz 2015-01-08 0:16 ` Tony Lindgren 0 siblings, 1 reply; 9+ messages in thread From: Dmitry Lifshitz @ 2014-12-30 7:39 UTC (permalink / raw) To: Roger Quadros Cc: Benoît Cousson, Tony Lindgren, linux-omap, devicetree, linux-arm-kernel, grinberg Hi Roger, On 12/29/2014 03:06 PM, Roger Quadros wrote: > Hi Dmitry, > > On 28/12/14 16:30, Dmitry Lifshitz wrote: >> CM-T3517, CM-T3530 and CM-T3730 features NAND storage chip connected to >> GPMC bus. >> >> Add GPMC DT entry into the root DT file omap3-cm-t3x.dtsi, common for >> all three modules. >> >> NAND timings are calculated to be safe for CM-T3x devices as it works >> now in non DT boot (in this case the timings are updated by U-Boot). > > I didn't get this part. You provide the GPMC timings in the DT now so they will override > u-boot configured timings. > In the board files for CM-T3x modules we did not override NAND timings configured in U-Boot. Now, with DT boot, we do not want to rely on boot loader settings wherever is possible. So, we specify the timings explicitly. In order to avoid regression, the timings setup is the same as we do in U-Boot. >> >> Update GPMC ranges in boards DT files to include all connected devices. >> >> Signed-off-by: Dmitry Lifshitz <lifshitz@compulab.co.il> >> --- >> >> v2 * Update GPMC ranges with NAND values in omap3-cm-t3x30.dtsi. >> * Add commit message details >> >> arch/arm/boot/dts/omap3-cm-t3x.dtsi | 58 +++++++++++++++++++++++++++++++++ >> arch/arm/boot/dts/omap3-cm-t3x30.dtsi | 3 +- >> arch/arm/boot/dts/omap3-sbc-t3517.dts | 4 ++ >> arch/arm/boot/dts/omap3-sbc-t3530.dts | 10 ++---- >> arch/arm/boot/dts/omap3-sbc-t3730.dts | 5 ++- >> 5 files changed, 70 insertions(+), 10 deletions(-) >> >> diff --git a/arch/arm/boot/dts/omap3-cm-t3x.dtsi b/arch/arm/boot/dts/omap3-cm-t3x.dtsi >> index 6ea6d46..4d091ca 100644 >> --- a/arch/arm/boot/dts/omap3-cm-t3x.dtsi >> +++ b/arch/arm/boot/dts/omap3-cm-t3x.dtsi >> @@ -259,3 +259,61 @@ >> pinctrl-names = "default"; >> pinctrl-0 = <&mcbsp2_pins>; >> }; >> + >> +&gpmc { >> + ranges = <0 0 0x00000000 0x01000000>; > > Isn't this ranges property redundant as it will be overridden by the board specific dts? > >> + >> + nand@0,0 { >> + reg = <0 0 4>; /* CS0, offset 0, IO size 4 */ >> + nand-bus-width = <8>; >> + gpmc,device-width = <1>; >> + ti,nand-ecc-opt = "sw"; > > any particular reason for sticking with "sw"? Isn't it better to use at least "ham1" > or migrate to stronger schemes like "bch4" or "bch8"? > > "ham1" scheme is compatible with ROM loader so the boot partition could be flashed from Linux. > For bch4/8 you will have to keep boot partition/s locked from Linux access as we don't yet > support mixed ECC schemes among partitions. > The regression issue should be considered here. Our customers have thousands of boards running rootfs with "sw" ecc. Updating the kernel is not a reason for reformatting the NAND. >> + >> + gpmc,cs-on-ns = <0>; >> + gpmc,cs-rd-off-ns = <120>; >> + gpmc,cs-wr-off-ns = <120>; >> + >> + gpmc,adv-on-ns = <0>; >> + gpmc,adv-rd-off-ns = <120>; >> + gpmc,adv-wr-off-ns = <120>; >> + >> + gpmc,we-on-ns = <6>; >> + gpmc,we-off-ns = <90>; >> + >> + gpmc,oe-on-ns = <6>; >> + gpmc,oe-off-ns = <90>; >> + >> + gpmc,page-burst-access-ns = <6>; >> + gpmc,access-ns = <72>; >> + gpmc,cycle2cycle-delay-ns = <60>; >> + >> + gpmc,rd-cycle-ns = <120>; >> + gpmc,wr-cycle-ns = <120>; >> + gpmc,wr-access-ns = <186>; >> + gpmc,wr-data-mux-bus-ns = <90>; >> + >> + #address-cells = <1>; >> + #size-cells = <1>; >> + >> + partition@0 { >> + label = "xloader"; >> + reg = <0 0x80000>; >> + }; >> + partition@0x80000 { >> + label = "uboot"; >> + reg = <0x80000 0x1e0000>; >> + }; >> + partition@0x260000 { >> + label = "uboot environment"; >> + reg = <0x260000 0x40000>; >> + }; >> + partition@0x2a0000 { >> + label = "linux"; >> + reg = <0x2a0000 0x400000>; >> + }; >> + partition@0x6a0000 { >> + label = "rootfs"; >> + reg = <0x6a0000 0x1f880000>; >> + }; >> + }; >> +}; >> diff --git a/arch/arm/boot/dts/omap3-cm-t3x30.dtsi b/arch/arm/boot/dts/omap3-cm-t3x30.dtsi >> index 9a4a3ab..d9e92b6 100644 >> --- a/arch/arm/boot/dts/omap3-cm-t3x30.dtsi >> +++ b/arch/arm/boot/dts/omap3-cm-t3x30.dtsi >> @@ -50,7 +50,8 @@ >> #include "omap-gpmc-smsc911x.dtsi" >> >> &gpmc { >> - ranges = <5 0 0x2c000000 0x01000000>; >> + ranges = <5 0 0x2c000000 0x01000000>, /* CM-T3x30 SMSC9x Eth */ >> + <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ > > Isn't this ranges property redundant as it will anyways be overridden by the board specific dts? > The ranges are specified here (and other files below) by design. We built a tree like structure to organize DT files - d234e4239 "ARM: dts: sbc-t3x: refactor DT support" It allows to inherit/override properties common for different boards. Our customers do not have to deal with a correct GPMC ranges settings, once they base on omap3-cm-t3x.dts core module DT file (in case they do not connect additional device to the bus). This is the point of Igor's comment to the previous patch version. >> >> smsc1: ethernet@gpmc { >> compatible = "smsc,lan9221", "smsc,lan9115"; >> diff --git a/arch/arm/boot/dts/omap3-sbc-t3517.dts b/arch/arm/boot/dts/omap3-sbc-t3517.dts >> index 1798653..c2d5c28 100644 >> --- a/arch/arm/boot/dts/omap3-sbc-t3517.dts >> +++ b/arch/arm/boot/dts/omap3-sbc-t3517.dts >> @@ -69,3 +69,7 @@ >> }; >> }; >> >> +&gpmc { >> + ranges = <4 0 0x2d000000 0x01000000>, /* SB-T35 SMSC9x Eth */ >> + <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ >> +}; >> diff --git a/arch/arm/boot/dts/omap3-sbc-t3530.dts b/arch/arm/boot/dts/omap3-sbc-t3530.dts >> index c994f0f..834bc78 100644 >> --- a/arch/arm/boot/dts/omap3-sbc-t3530.dts >> +++ b/arch/arm/boot/dts/omap3-sbc-t3530.dts >> @@ -26,14 +26,10 @@ >> }; >> }; >> >> -/* >> - * The following ranges correspond to SMSC9x eth chips on CM-T3530 CoM and >> - * SB-T35 baseboard respectively. >> - * This setting includes both chips in SBC-T3530 board device tree. >> - */ >> &gpmc { >> - ranges = <5 0 0x2c000000 0x01000000>, >> - <4 0 0x2d000000 0x01000000>; >> + ranges = <5 0 0x2c000000 0x01000000>, /* CM-T3x30 SMSC9x Eth */ >> + <4 0 0x2d000000 0x01000000>, /* SB-T35 SMSC9x Eth */ >> + <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ >> }; >> >> &mmc1 { >> diff --git a/arch/arm/boot/dts/omap3-sbc-t3730.dts b/arch/arm/boot/dts/omap3-sbc-t3730.dts >> index 5bdddf2..73c7bf4 100644 >> --- a/arch/arm/boot/dts/omap3-sbc-t3730.dts >> +++ b/arch/arm/boot/dts/omap3-sbc-t3730.dts >> @@ -27,8 +27,9 @@ >> }; >> >> &gpmc { >> - ranges = <5 0 0x2c000000 0x01000000>, >> - <4 0 0x2d000000 0x01000000>; >> + ranges = <5 0 0x2c000000 0x01000000>, /* CM-T3x30 SMSC9x Eth */ >> + <4 0 0x2d000000 0x01000000>, /* SB-T35 SMSC9x Eth */ >> + <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ >> }; >> >> &dss { >> > > cheers, > -roger > > > Thanks, Dmitry ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] ARM: dts: cm-t3x: add NAND support 2014-12-30 7:39 ` Dmitry Lifshitz @ 2015-01-08 0:16 ` Tony Lindgren 2015-01-08 10:30 ` Roger Quadros 0 siblings, 1 reply; 9+ messages in thread From: Tony Lindgren @ 2015-01-08 0:16 UTC (permalink / raw) To: Dmitry Lifshitz Cc: Roger Quadros, Benoît Cousson, linux-omap, devicetree, linux-arm-kernel, grinberg * Dmitry Lifshitz <lifshitz@compulab.co.il> [141229 23:42]: > On 12/29/2014 03:06 PM, Roger Quadros wrote: > >On 28/12/14 16:30, Dmitry Lifshitz wrote: > >>--- a/arch/arm/boot/dts/omap3-cm-t3x30.dtsi > >>+++ b/arch/arm/boot/dts/omap3-cm-t3x30.dtsi > >>@@ -50,7 +50,8 @@ > >> #include "omap-gpmc-smsc911x.dtsi" > >> > >> &gpmc { > >>- ranges = <5 0 0x2c000000 0x01000000>; > >>+ ranges = <5 0 0x2c000000 0x01000000>, /* CM-T3x30 SMSC9x Eth */ > >>+ <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ > > > >Isn't this ranges property redundant as it will anyways be overridden by the board specific dts? > > > > The ranges are specified here (and other files below) by design. > > We built a tree like structure to organize DT files - > d234e4239 "ARM: dts: sbc-t3x: refactor DT support" > > It allows to inherit/override properties common for different boards. > > Our customers do not have to deal with a correct GPMC ranges settings, once > they base on omap3-cm-t3x.dts core module DT file (in case they do not > connect additional device to the bus). > > This is the point of Igor's comment to the previous patch version. Roger, do you still have pending issues with this patch? Regards, Tony ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] ARM: dts: cm-t3x: add NAND support 2015-01-08 0:16 ` Tony Lindgren @ 2015-01-08 10:30 ` Roger Quadros 2015-01-13 16:01 ` Tony Lindgren 0 siblings, 1 reply; 9+ messages in thread From: Roger Quadros @ 2015-01-08 10:30 UTC (permalink / raw) To: Tony Lindgren, Dmitry Lifshitz Cc: Benoît Cousson, linux-omap, devicetree, linux-arm-kernel, grinberg On 08/01/15 02:16, Tony Lindgren wrote: > * Dmitry Lifshitz <lifshitz@compulab.co.il> [141229 23:42]: >> On 12/29/2014 03:06 PM, Roger Quadros wrote: >>> On 28/12/14 16:30, Dmitry Lifshitz wrote: >>>> --- a/arch/arm/boot/dts/omap3-cm-t3x30.dtsi >>>> +++ b/arch/arm/boot/dts/omap3-cm-t3x30.dtsi >>>> @@ -50,7 +50,8 @@ >>>> #include "omap-gpmc-smsc911x.dtsi" >>>> >>>> &gpmc { >>>> - ranges = <5 0 0x2c000000 0x01000000>; >>>> + ranges = <5 0 0x2c000000 0x01000000>, /* CM-T3x30 SMSC9x Eth */ >>>> + <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ >>> >>> Isn't this ranges property redundant as it will anyways be overridden by the board specific dts? >>> >> >> The ranges are specified here (and other files below) by design. >> >> We built a tree like structure to organize DT files - >> d234e4239 "ARM: dts: sbc-t3x: refactor DT support" >> >> It allows to inherit/override properties common for different boards. >> >> Our customers do not have to deal with a correct GPMC ranges settings, once >> they base on omap3-cm-t3x.dts core module DT file (in case they do not >> connect additional device to the bus). >> >> This is the point of Igor's comment to the previous patch version. > > Roger, do you still have pending issues with this patch? Sorry missed this one. Thanks for the explanations Dmitry. Acked-by: Roger Quadros <rogerq@ti.com> cheers, -roger ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2] ARM: dts: cm-t3x: add NAND support 2015-01-08 10:30 ` Roger Quadros @ 2015-01-13 16:01 ` Tony Lindgren 0 siblings, 0 replies; 9+ messages in thread From: Tony Lindgren @ 2015-01-13 16:01 UTC (permalink / raw) To: Roger Quadros Cc: Dmitry Lifshitz, Benoît Cousson, linux-omap, devicetree, linux-arm-kernel, grinberg * Roger Quadros <rogerq@ti.com> [150108 02:33]: > On 08/01/15 02:16, Tony Lindgren wrote: > > * Dmitry Lifshitz <lifshitz@compulab.co.il> [141229 23:42]: > >> On 12/29/2014 03:06 PM, Roger Quadros wrote: > >>> On 28/12/14 16:30, Dmitry Lifshitz wrote: > >>>> --- a/arch/arm/boot/dts/omap3-cm-t3x30.dtsi > >>>> +++ b/arch/arm/boot/dts/omap3-cm-t3x30.dtsi > >>>> @@ -50,7 +50,8 @@ > >>>> #include "omap-gpmc-smsc911x.dtsi" > >>>> > >>>> &gpmc { > >>>> - ranges = <5 0 0x2c000000 0x01000000>; > >>>> + ranges = <5 0 0x2c000000 0x01000000>, /* CM-T3x30 SMSC9x Eth */ > >>>> + <0 0 0x00000000 0x01000000>; /* CM-T3x NAND */ > >>> > >>> Isn't this ranges property redundant as it will anyways be overridden by the board specific dts? > >>> > >> > >> The ranges are specified here (and other files below) by design. > >> > >> We built a tree like structure to organize DT files - > >> d234e4239 "ARM: dts: sbc-t3x: refactor DT support" > >> > >> It allows to inherit/override properties common for different boards. > >> > >> Our customers do not have to deal with a correct GPMC ranges settings, once > >> they base on omap3-cm-t3x.dts core module DT file (in case they do not > >> connect additional device to the bus). > >> > >> This is the point of Igor's comment to the previous patch version. > > > > Roger, do you still have pending issues with this patch? > > Sorry missed this one. Thanks for the explanations Dmitry. > > Acked-by: Roger Quadros <rogerq@ti.com> Applying into omap-for-v3.20/dt thanks. Tony ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2015-01-13 16:01 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-12-28 13:15 [PATCH] ARM: dts: cm-t3x: add NAND support Dmitry Lifshitz 2014-12-28 13:55 ` Igor Grinberg 2014-12-28 14:30 ` [PATCH v2] " Dmitry Lifshitz 2014-12-28 14:54 ` Igor Grinberg 2014-12-29 13:06 ` Roger Quadros 2014-12-30 7:39 ` Dmitry Lifshitz 2015-01-08 0:16 ` Tony Lindgren 2015-01-08 10:30 ` Roger Quadros 2015-01-13 16:01 ` Tony Lindgren
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).