Re: [PATCH v4 3/6] ARM: dts: dra7: add support for parallel NAND flash

2014-05-14 Thread Roger Quadros
Hi Pekon, On 05/12/2014 12:05 PM, Gupta, Pekon wrote: From: Javier Martinez Canillas [mailto:jav...@dowhile0.org] [...] Newer platforms have upgraded version of GPMC engine which supports BCH16 ECC scheme in hardware. Thus the GPMC address space was expanded to include some extra registers

RE: [PATCH v4 3/6] ARM: dts: dra7: add support for parallel NAND flash

2014-05-14 Thread Gupta, Pekon
Hi Roger, For now, I'll use GPMC address-space size = 0x380 as it matches with actual hardware and is working. How did you get 0x380? From DRA7 TRM, GPMC address range is 0x5000 : 0x5000 02D0 So the address-space size should be 0x2D4 (as last register@2D0 is 32-bits wide) I think that

Re: [PATCH v4 3/6] ARM: dts: dra7: add support for parallel NAND flash

2014-05-14 Thread Roger Quadros
On 05/14/2014 11:25 AM, Roger Quadros wrote: Hi Pekon, On 05/12/2014 12:05 PM, Gupta, Pekon wrote: From: Javier Martinez Canillas [mailto:jav...@dowhile0.org] [...] Newer platforms have upgraded version of GPMC engine which supports BCH16 ECC scheme in hardware. Thus the GPMC address

RE: [PATCH v4 3/6] ARM: dts: dra7: add support for parallel NAND flash

2014-05-14 Thread Gupta, Pekon
From: Quadros, Roger [...] For now, I'll use GPMC address-space size = 0x380 as it matches with actual hardware and is working. How did you get 0x380? From DRA7 TRM, GPMC address range is 0x5000 : 0x5000 02D0 So the address-space size should be 0x2D4 (as last register@2D0 is 32-bits

Re: [PATCH v4 3/6] ARM: dts: dra7: add support for parallel NAND flash

2014-05-14 Thread Roger Quadros
On 05/14/2014 12:09 PM, Gupta, Pekon wrote: From: Quadros, Roger [...] For now, I'll use GPMC address-space size = 0x380 as it matches with actual hardware and is working. How did you get 0x380? From DRA7 TRM, GPMC address range is 0x5000 : 0x5000 02D0 So the address-space size

Re: [PATCH v4 3/6] ARM: dts: dra7: add support for parallel NAND flash

2014-05-14 Thread Javier Martinez Canillas
On Wed, May 14, 2014 at 11:17 AM, Roger Quadros rog...@ti.com wrote: On 05/14/2014 12:09 PM, Gupta, Pekon wrote: From: Quadros, Roger [...] For now, I'll use GPMC address-space size = 0x380 as it matches with actual hardware and is working. How did you get 0x380? From DRA7 TRM, GPMC

RE: [PATCH v4 3/6] ARM: dts: dra7: add support for parallel NAND flash

2014-05-12 Thread Gupta, Pekon
Hello, From: Javier Martinez Canillas [mailto:jav...@dowhile0.org] On Fri, May 9, 2014 at 10:46 PM, Pekon Gupta pe...@ti.com wrote: From: Minal Shah minalks...@gmail.com [...] +gpmc { + status = okay; + pinctrl-names = default; + pinctrl-0 = nand_flash_x16; + ranges

Re: [PATCH v4 3/6] ARM: dts: dra7: add support for parallel NAND flash

2014-05-12 Thread Javier Martinez Canillas
Hello Pekon, On Mon, May 12, 2014 at 9:03 AM, Gupta, Pekon pe...@ti.com wrote: Hello, From: Javier Martinez Canillas [mailto:jav...@dowhile0.org] On Fri, May 9, 2014 at 10:46 PM, Pekon Gupta pe...@ti.com wrote: From: Minal Shah minalks...@gmail.com [...] +gpmc { + status = okay; +

RE: [PATCH v4 3/6] ARM: dts: dra7: add support for parallel NAND flash

2014-05-12 Thread Gupta, Pekon
From: Javier Martinez Canillas [mailto:jav...@dowhile0.org] [...] Newer platforms have upgraded version of GPMC engine which supports BCH16 ECC scheme in hardware. Thus the GPMC address space was expanded to include some extra registers required for BCH16 ECC [2]. I see and did the GPMC

Re: [PATCH v4 3/6] ARM: dts: dra7: add support for parallel NAND flash

2014-05-12 Thread Javier Martinez Canillas
On Mon, May 12, 2014 at 11:05 AM, Gupta, Pekon pe...@ti.com wrote: From: Javier Martinez Canillas [mailto:jav...@dowhile0.org] [...] Newer platforms have upgraded version of GPMC engine which supports BCH16 ECC scheme in hardware. Thus the GPMC address space was expanded to include some extra

Re: [PATCH v4 3/6] ARM: dts: dra7: add support for parallel NAND flash

2014-05-10 Thread Javier Martinez Canillas
Hello Pekon, On Fri, May 9, 2014 at 10:46 PM, Pekon Gupta pe...@ti.com wrote: From: Minal Shah minalks...@gmail.com DRA7xx platform has in-build GPMC and ELM h/w engines which can be used for accessing externel NAND flash device. This patch: - adds generic DT binding in dra7.dtsi for

[PATCH v4 3/6] ARM: dts: dra7: add support for parallel NAND flash

2014-05-09 Thread Pekon Gupta
From: Minal Shah minalks...@gmail.com DRA7xx platform has in-build GPMC and ELM h/w engines which can be used for accessing externel NAND flash device. This patch: - adds generic DT binding in dra7.dtsi for enabling GPMC and ELM h/w engines - adds DT binding for Micron NAND Flash (MT29F2G16AADWP)