* [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx
@ 2013-05-14 11:15 Ezequiel Garcia
2013-05-14 11:15 ` [PATCH 1/5] mtd: nand: pxa3xx: Set info->use_dma properly Ezequiel Garcia
` (6 more replies)
0 siblings, 7 replies; 19+ messages in thread
From: Ezequiel Garcia @ 2013-05-14 11:15 UTC (permalink / raw)
To: linux-mtd
Cc: Thomas Petazzoni, Lior Amsalem, Ezequiel Garcia, Artem Bityutskiy,
Lei Wen, Haojian Zhuang, Maen Suleiman, Gregory Clement,
Neil Zhang, Chao Xie
The Armada 370 and Armada XP SoC include a NAND controller (NFCv2) that
is similar to the one in PXA SoC. Therefore, it should be possible to support
Armada NAND using pxa3xx-nand driver.
While this effort is not ready yet, here is a set of patches that are needed
for such work.
A small summary of the patches:
* 1/5 is a fix for a 2-year regression that removed DMA support.
* 2/5 and 3/5 are needed to build the driver on platforms other PXA.
* 4/5 and 5/5 add ONFI-compliant device detection and are needed
to successfully probe newer flash devices.
Applies cleanly on top of l2-mtd.git.
Thanks!
Ezequiel Garcia (5):
mtd: nand: pxa3xx: Set info->use_dma properly
mtd: nand: pxa3xx: Use of_machine_is_compatible()
mtd: nand: pxa3xx: Fix MODULE_DEVICE_TABLE declaration
mtd: nand: pxa3xx: Add address support for READID command
mtd: nand: pxa3xx: Add support for Read parameter page command
drivers/mtd/nand/pxa3xx_nand.c | 16 ++++++++++++++--
1 file changed, 14 insertions(+), 2 deletions(-)
--
1.8.1.5
^ permalink raw reply [flat|nested] 19+ messages in thread
* [PATCH 1/5] mtd: nand: pxa3xx: Set info->use_dma properly
2013-05-14 11:15 [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx Ezequiel Garcia
@ 2013-05-14 11:15 ` Ezequiel Garcia
2013-05-14 11:15 ` [PATCH 2/5] mtd: nand: pxa3xx: Use of_machine_is_compatible() Ezequiel Garcia
` (5 subsequent siblings)
6 siblings, 0 replies; 19+ messages in thread
From: Ezequiel Garcia @ 2013-05-14 11:15 UTC (permalink / raw)
To: linux-mtd
Cc: Thomas Petazzoni, Lior Amsalem, Artem Bityutskiy, Lei Wen,
Haojian Zhuang, Maen Suleiman, Ezequiel Garcia, Gregory Clement,
Neil Zhang, Chao Xie
From: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
Currently, the variable info->use_dma is never set and always
zero-valued which means the driver never does DMA transfers.
We fix this by simply setting info->use_dma to the module parameter,
also named 'use_dma'. Note that the module parameter has the same name,
but different semantics.
This fixes a regression introduced by the below commit
which removed the info->use_dma variable set.
commit 4eb2da8994042d68e84e31138788429a102da2ea
Author: Lei Wen <leiwen@marvell.com>
Date: Mon Feb 28 10:32:13 2011 +0800
mtd: pxa3xx_nand: unify prepare command
Before the above commit, the driver had use_dma=1 on all NAND commands
except on CMD_STATUS. This behavior is long lost and we are not
recovering in this patch, either.
This was spotted and verified only by human inspection. No testing of
any kind has been performed for lack of hardware.
Cc: Lei Wen <leiwen@marvell.com>
Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
---
drivers/mtd/nand/pxa3xx_nand.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
index 14d3db5..cef1eeb 100644
--- a/drivers/mtd/nand/pxa3xx_nand.c
+++ b/drivers/mtd/nand/pxa3xx_nand.c
@@ -506,6 +506,7 @@ static int prepare_command_pool(struct pxa3xx_nand_info *info, int command,
info->buf_count = 0;
info->oob_size = 0;
info->use_ecc = 0;
+ info->use_dma = (use_dma) ? 1 : 0;
info->is_ready = 0;
info->retcode = ERR_NONE;
if (info->cs != 0)
--
1.8.1.5
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH 2/5] mtd: nand: pxa3xx: Use of_machine_is_compatible()
2013-05-14 11:15 [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx Ezequiel Garcia
2013-05-14 11:15 ` [PATCH 1/5] mtd: nand: pxa3xx: Set info->use_dma properly Ezequiel Garcia
@ 2013-05-14 11:15 ` Ezequiel Garcia
2013-05-14 11:15 ` [PATCH 3/5] mtd: nand: pxa3xx: Fix MODULE_DEVICE_TABLE declaration Ezequiel Garcia
` (4 subsequent siblings)
6 siblings, 0 replies; 19+ messages in thread
From: Ezequiel Garcia @ 2013-05-14 11:15 UTC (permalink / raw)
To: linux-mtd
Cc: Thomas Petazzoni, Lior Amsalem, Artem Bityutskiy, Lei Wen,
Haojian Zhuang, Maen Suleiman, Ezequiel Garcia, Gregory Clement,
Neil Zhang, Chao Xie
From: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
This patch replaces cpu_is_pxa3xx() with of_machine_is_compatible()
which allows to build this driver for other platforms than ARCH_PXA.
Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
---
drivers/mtd/nand/pxa3xx_nand.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
index cef1eeb..8386e9a 100644
--- a/drivers/mtd/nand/pxa3xx_nand.c
+++ b/drivers/mtd/nand/pxa3xx_nand.c
@@ -1094,7 +1094,7 @@ static int alloc_nand_resource(struct platform_device *pdev)
* bindings. It can be removed once we have a prober DMA controller
* framework for DT.
*/
- if (pdev->dev.of_node && cpu_is_pxa3xx()) {
+ if (pdev->dev.of_node && of_machine_is_compatible("marvell,pxa3xx")) {
info->drcmr_dat = 97;
info->drcmr_cmd = 99;
} else {
--
1.8.1.5
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH 3/5] mtd: nand: pxa3xx: Fix MODULE_DEVICE_TABLE declaration
2013-05-14 11:15 [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx Ezequiel Garcia
2013-05-14 11:15 ` [PATCH 1/5] mtd: nand: pxa3xx: Set info->use_dma properly Ezequiel Garcia
2013-05-14 11:15 ` [PATCH 2/5] mtd: nand: pxa3xx: Use of_machine_is_compatible() Ezequiel Garcia
@ 2013-05-14 11:15 ` Ezequiel Garcia
2013-05-14 11:15 ` [PATCH 4/5] mtd: nand: pxa3xx: Add address support for READID command Ezequiel Garcia
` (3 subsequent siblings)
6 siblings, 0 replies; 19+ messages in thread
From: Ezequiel Garcia @ 2013-05-14 11:15 UTC (permalink / raw)
To: linux-mtd
Cc: Thomas Petazzoni, Lior Amsalem, Artem Bityutskiy, Lei Wen,
Haojian Zhuang, Maen Suleiman, Ezequiel Garcia, Gregory Clement,
Neil Zhang, Chao Xie
From: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
This module's device table is incorrectly declared using
i2c_pxa_dt_ids, instead of pxa3xx_nand_dt_ids.
Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
---
drivers/mtd/nand/pxa3xx_nand.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
index 8386e9a..e4e697f 100644
--- a/drivers/mtd/nand/pxa3xx_nand.c
+++ b/drivers/mtd/nand/pxa3xx_nand.c
@@ -1185,7 +1185,7 @@ static struct of_device_id pxa3xx_nand_dt_ids[] = {
{ .compatible = "marvell,pxa3xx-nand" },
{}
};
-MODULE_DEVICE_TABLE(of, i2c_pxa_dt_ids);
+MODULE_DEVICE_TABLE(of, pxa3xx_nand_dt_ids);
static int pxa3xx_nand_probe_dt(struct platform_device *pdev)
{
--
1.8.1.5
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH 4/5] mtd: nand: pxa3xx: Add address support for READID command
2013-05-14 11:15 [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx Ezequiel Garcia
` (2 preceding siblings ...)
2013-05-14 11:15 ` [PATCH 3/5] mtd: nand: pxa3xx: Fix MODULE_DEVICE_TABLE declaration Ezequiel Garcia
@ 2013-05-14 11:15 ` Ezequiel Garcia
2013-05-14 11:15 ` [PATCH 5/5] mtd: nand: pxa3xx: Add support for Read parameter page command Ezequiel Garcia
` (2 subsequent siblings)
6 siblings, 0 replies; 19+ messages in thread
From: Ezequiel Garcia @ 2013-05-14 11:15 UTC (permalink / raw)
To: linux-mtd
Cc: Thomas Petazzoni, Lior Amsalem, Artem Bityutskiy, Lei Wen,
Haojian Zhuang, Maen Suleiman, Ezequiel Garcia, Gregory Clement,
Neil Zhang, Chao Xie
From: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
This allows to support READID ONFI command which sends 0x20
as address together with the 0x90 READID command.
This is required to detect ONFI compliant devices.
Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
---
drivers/mtd/nand/pxa3xx_nand.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
index e4e697f..36780e2 100644
--- a/drivers/mtd/nand/pxa3xx_nand.c
+++ b/drivers/mtd/nand/pxa3xx_nand.c
@@ -596,6 +596,7 @@ static int prepare_command_pool(struct pxa3xx_nand_info *info, int command,
info->ndcb0 |= NDCB0_CMD_TYPE(3)
| NDCB0_ADDR_CYC(1)
| cmd;
+ info->ndcb1 = (column & 0xFF);
info->data_size = 8;
break;
--
1.8.1.5
^ permalink raw reply related [flat|nested] 19+ messages in thread
* [PATCH 5/5] mtd: nand: pxa3xx: Add support for Read parameter page command
2013-05-14 11:15 [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx Ezequiel Garcia
` (3 preceding siblings ...)
2013-05-14 11:15 ` [PATCH 4/5] mtd: nand: pxa3xx: Add address support for READID command Ezequiel Garcia
@ 2013-05-14 11:15 ` Ezequiel Garcia
2013-05-20 12:37 ` [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx Ezequiel Garcia
2013-06-17 5:53 ` Igor Grinberg
6 siblings, 0 replies; 19+ messages in thread
From: Ezequiel Garcia @ 2013-05-14 11:15 UTC (permalink / raw)
To: linux-mtd
Cc: Thomas Petazzoni, Lior Amsalem, Artem Bityutskiy, Lei Wen,
Haojian Zhuang, Maen Suleiman, Ezequiel Garcia, Gregory Clement,
Neil Zhang, Chao Xie
From: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
This command is required to identify ONFI-compliant devices.
Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
---
drivers/mtd/nand/pxa3xx_nand.c | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
index 36780e2..d63d378 100644
--- a/drivers/mtd/nand/pxa3xx_nand.c
+++ b/drivers/mtd/nand/pxa3xx_nand.c
@@ -590,6 +590,16 @@ static int prepare_command_pool(struct pxa3xx_nand_info *info, int command,
| addr_cycle;
break;
+ case NAND_CMD_PARAM:
+ cmd = NAND_CMD_PARAM;
+ info->buf_count = 256;
+ info->ndcb0 |= NDCB0_CMD_TYPE(0)
+ | NDCB0_ADDR_CYC(1)
+ | cmd;
+ info->ndcb1 = (column & 0xFF);
+ info->data_size = 256;
+ break;
+
case NAND_CMD_READID:
cmd = host->cmdset->read_id;
info->buf_count = host->read_id_bytes;
--
1.8.1.5
^ permalink raw reply related [flat|nested] 19+ messages in thread
* Re: [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx
2013-05-14 11:15 [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx Ezequiel Garcia
` (4 preceding siblings ...)
2013-05-14 11:15 ` [PATCH 5/5] mtd: nand: pxa3xx: Add support for Read parameter page command Ezequiel Garcia
@ 2013-05-20 12:37 ` Ezequiel Garcia
2013-06-07 14:22 ` Ezequiel Garcia
2013-06-17 5:53 ` Igor Grinberg
6 siblings, 1 reply; 19+ messages in thread
From: Ezequiel Garcia @ 2013-05-20 12:37 UTC (permalink / raw)
To: Ezequiel Garcia
Cc: Thomas Petazzoni, Lior Amsalem, Eric Miao, Artem Bityutskiy,
Lei Wen, Haojian Zhuang, Maen Suleiman, linux-mtd,
Gregory Clement, Neil Zhang, Chao Xie
Hi,
(adding Eric Miao in Cc)
On Tue, May 14, 2013 at 08:15:20AM -0300, Ezequiel Garcia wrote:
> The Armada 370 and Armada XP SoC include a NAND controller (NFCv2) that
> is similar to the one in PXA SoC. Therefore, it should be possible to support
> Armada NAND using pxa3xx-nand driver.
>
> While this effort is not ready yet, here is a set of patches that are needed
> for such work.
>
> A small summary of the patches:
>
> * 1/5 is a fix for a 2-year regression that removed DMA support.
>
> * 2/5 and 3/5 are needed to build the driver on platforms other PXA.
>
> * 4/5 and 5/5 add ONFI-compliant device detection and are needed
> to successfully probe newer flash devices.
>
Since this patch affects directly the PXA hardware, I'd like to have some
response from PXA maintainers.
Guys, could you take a look at this?
Thanks a lot,
--
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx
2013-05-20 12:37 ` [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx Ezequiel Garcia
@ 2013-06-07 14:22 ` Ezequiel Garcia
2013-06-08 7:22 ` Haojian Zhuang
2013-07-01 7:21 ` Artem Bityutskiy
0 siblings, 2 replies; 19+ messages in thread
From: Ezequiel Garcia @ 2013-06-07 14:22 UTC (permalink / raw)
To: Ezequiel Garcia
Cc: Thomas Petazzoni, Lior Amsalem, Eric Miao, Artem Bityutskiy,
Lei Wen, Haojian Zhuang, Maen Suleiman, linux-mtd,
Gregory Clement, Neil Zhang, Chao Xie
On Mon, May 20, 2013 at 9:37 AM, Ezequiel Garcia
<ezequiel.garcia@free-electrons.com> wrote:
> Hi,
>
> (adding Eric Miao in Cc)
>
> On Tue, May 14, 2013 at 08:15:20AM -0300, Ezequiel Garcia wrote:
>> The Armada 370 and Armada XP SoC include a NAND controller (NFCv2) that
>> is similar to the one in PXA SoC. Therefore, it should be possible to support
>> Armada NAND using pxa3xx-nand driver.
>>
>> While this effort is not ready yet, here is a set of patches that are needed
>> for such work.
>>
>> A small summary of the patches:
>>
>> * 1/5 is a fix for a 2-year regression that removed DMA support.
>>
>> * 2/5 and 3/5 are needed to build the driver on platforms other PXA.
>>
>> * 4/5 and 5/5 add ONFI-compliant device detection and are needed
>> to successfully probe newer flash devices.
>>
>
> Since this patch affects directly the PXA hardware, I'd like to have some
> response from PXA maintainers.
>
> Guys, could you take a look at this?
>
Ping?
--
Ezequiel
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx
2013-06-07 14:22 ` Ezequiel Garcia
@ 2013-06-08 7:22 ` Haojian Zhuang
2013-06-08 9:00 ` Haojian Zhuang
2013-07-01 7:21 ` Artem Bityutskiy
1 sibling, 1 reply; 19+ messages in thread
From: Haojian Zhuang @ 2013-06-08 7:22 UTC (permalink / raw)
To: Ezequiel Garcia, Daniel Mack, Igor Grinberg, Vasut Marek
Cc: Thomas Petazzoni, Lior Amsalem, Eric Miao, Artem Bityutskiy,
Lei Wen, Maen Suleiman, linux-mtd, Ezequiel Garcia,
Gregory Clement, Neil Zhang, Chao Xie
On Fri, Jun 7, 2013 at 10:22 PM, Ezequiel Garcia <elezegarcia@gmail.com> wrote:
> On Mon, May 20, 2013 at 9:37 AM, Ezequiel Garcia
> <ezequiel.garcia@free-electrons.com> wrote:
>> Hi,
>>
>> (adding Eric Miao in Cc)
>>
>> On Tue, May 14, 2013 at 08:15:20AM -0300, Ezequiel Garcia wrote:
>>> The Armada 370 and Armada XP SoC include a NAND controller (NFCv2) that
>>> is similar to the one in PXA SoC. Therefore, it should be possible to support
>>> Armada NAND using pxa3xx-nand driver.
>>>
>>> While this effort is not ready yet, here is a set of patches that are needed
>>> for such work.
>>>
>>> A small summary of the patches:
>>>
>>> * 1/5 is a fix for a 2-year regression that removed DMA support.
>>>
>>> * 2/5 and 3/5 are needed to build the driver on platforms other PXA.
>>>
>>> * 4/5 and 5/5 add ONFI-compliant device detection and are needed
>>> to successfully probe newer flash devices.
>>>
>>
>> Since this patch affects directly the PXA hardware, I'd like to have some
>> response from PXA maintainers.
>>
>> Guys, could you take a look at this?
>>
>
> Ping?
>
> --
> Ezequiel
Hi Ezequiel,
As I didn't have the test environment, I looped some Marvell guys. But
they didn't
have any time on test it.
So let's loop Marek, Daniel & Igor. I hope that we could help us.
By the way, my zylonite pxa310 fails on running uboot. I can't fix it right now.
Regards
Haojian
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx
2013-06-08 7:22 ` Haojian Zhuang
@ 2013-06-08 9:00 ` Haojian Zhuang
2013-06-08 18:40 ` Ezequiel Garcia
0 siblings, 1 reply; 19+ messages in thread
From: Haojian Zhuang @ 2013-06-08 9:00 UTC (permalink / raw)
To: Ezequiel Garcia, Daniel Mack, Igor Grinberg, Vasut Marek
Cc: Thomas Petazzoni, Lior Amsalem, Eric Miao, Artem Bityutskiy,
Lei Wen, Maen Suleiman, linux-mtd, Ezequiel Garcia,
Gregory Clement, Neil Zhang, Chao Xie
On Sat, Jun 8, 2013 at 3:22 PM, Haojian Zhuang <haojian.zhuang@gmail.com> wrote:
> On Fri, Jun 7, 2013 at 10:22 PM, Ezequiel Garcia <elezegarcia@gmail.com> wrote:
>> On Mon, May 20, 2013 at 9:37 AM, Ezequiel Garcia
>> <ezequiel.garcia@free-electrons.com> wrote:
>>> Hi,
>>>
>>> (adding Eric Miao in Cc)
>>>
>>> On Tue, May 14, 2013 at 08:15:20AM -0300, Ezequiel Garcia wrote:
>>>> The Armada 370 and Armada XP SoC include a NAND controller (NFCv2) that
>>>> is similar to the one in PXA SoC. Therefore, it should be possible to support
>>>> Armada NAND using pxa3xx-nand driver.
>>>>
>>>> While this effort is not ready yet, here is a set of patches that are needed
>>>> for such work.
>>>>
>>>> A small summary of the patches:
>>>>
>>>> * 1/5 is a fix for a 2-year regression that removed DMA support.
>>>>
>>>> * 2/5 and 3/5 are needed to build the driver on platforms other PXA.
>>>>
>>>> * 4/5 and 5/5 add ONFI-compliant device detection and are needed
>>>> to successfully probe newer flash devices.
>>>>
>>>
>>> Since this patch affects directly the PXA hardware, I'd like to have some
>>> response from PXA maintainers.
>>>
>>> Guys, could you take a look at this?
>>>
>>
>> Ping?
>>
>> --
>> Ezequiel
>
> Hi Ezequiel,
>
> As I didn't have the test environment, I looped some Marvell guys. But
> they didn't
> have any time on test it.
>
> So let's loop Marek, Daniel & Igor. I hope that we could help us.
>
> By the way, my zylonite pxa310 fails on running uboot. I can't fix it right now.
>
> Regards
> Haojian
I'm OK on these patches.
Reviewed-by: Haojian Zhuang <haojian.zhuang@gmail.com>
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx
2013-06-08 9:00 ` Haojian Zhuang
@ 2013-06-08 18:40 ` Ezequiel Garcia
0 siblings, 0 replies; 19+ messages in thread
From: Ezequiel Garcia @ 2013-06-08 18:40 UTC (permalink / raw)
To: Haojian Zhuang
Cc: Thomas Petazzoni, Lior Amsalem, Ezequiel Garcia, Maen Suleiman,
Eric Miao, Artem Bityutskiy, Lei Wen, Daniel Mack, Vasut Marek,
linux-mtd, Igor Grinberg, Gregory Clement, Neil Zhang, Chao Xie
On Sat, Jun 08, 2013 at 05:00:41PM +0800, Haojian Zhuang wrote:
> On Sat, Jun 8, 2013 at 3:22 PM, Haojian Zhuang <haojian.zhuang@gmail.com> wrote:
> > On Fri, Jun 7, 2013 at 10:22 PM, Ezequiel Garcia <elezegarcia@gmail.com> wrote:
> >> On Mon, May 20, 2013 at 9:37 AM, Ezequiel Garcia
> >> <ezequiel.garcia@free-electrons.com> wrote:
> >>> Hi,
> >>>
> >>> (adding Eric Miao in Cc)
> >>>
> >>> On Tue, May 14, 2013 at 08:15:20AM -0300, Ezequiel Garcia wrote:
> >>>> The Armada 370 and Armada XP SoC include a NAND controller (NFCv2) that
> >>>> is similar to the one in PXA SoC. Therefore, it should be possible to support
> >>>> Armada NAND using pxa3xx-nand driver.
> >>>>
> >>>> While this effort is not ready yet, here is a set of patches that are needed
> >>>> for such work.
> >>>>
> >>>> A small summary of the patches:
> >>>>
> >>>> * 1/5 is a fix for a 2-year regression that removed DMA support.
> >>>>
> >>>> * 2/5 and 3/5 are needed to build the driver on platforms other PXA.
> >>>>
> >>>> * 4/5 and 5/5 add ONFI-compliant device detection and are needed
> >>>> to successfully probe newer flash devices.
> >>>>
> >>>
> >>> Since this patch affects directly the PXA hardware, I'd like to have some
> >>> response from PXA maintainers.
> >>>
> >>> Guys, could you take a look at this?
> >>>
> >>
> >> Ping?
> >>
> >> --
> >> Ezequiel
> >
> > Hi Ezequiel,
> >
> > As I didn't have the test environment, I looped some Marvell guys. But
> > they didn't
> > have any time on test it.
> >
> > So let's loop Marek, Daniel & Igor. I hope that we could help us.
> >
Great, thanks a lot! Let's hope they can give it a test...
>
> I'm OK on these patches.
>
> Reviewed-by: Haojian Zhuang <haojian.zhuang@gmail.com>
Great!
Thanks a lot,
--
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx
2013-05-14 11:15 [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx Ezequiel Garcia
` (5 preceding siblings ...)
2013-05-20 12:37 ` [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx Ezequiel Garcia
@ 2013-06-17 5:53 ` Igor Grinberg
2013-06-17 11:03 ` Ezequiel Garcia
6 siblings, 1 reply; 19+ messages in thread
From: Igor Grinberg @ 2013-06-17 5:53 UTC (permalink / raw)
To: Ezequiel Garcia
Cc: Thomas Petazzoni, Lior Amsalem, Artem Bityutskiy, Lei Wen,
Haojian Zhuang, Maen Suleiman, linux-mtd, Gregory Clement,
Neil Zhang, Chao Xie
Hi Ezequiel,
On 05/14/13 14:15, Ezequiel Garcia wrote:
> The Armada 370 and Armada XP SoC include a NAND controller (NFCv2) that
> is similar to the one in PXA SoC. Therefore, it should be possible to support
> Armada NAND using pxa3xx-nand driver.
>
> While this effort is not ready yet, here is a set of patches that are needed
> for such work.
>
> A small summary of the patches:
>
> * 1/5 is a fix for a 2-year regression that removed DMA support.
>
> * 2/5 and 3/5 are needed to build the driver on platforms other PXA.
>
> * 4/5 and 5/5 add ONFI-compliant device detection and are needed
> to successfully probe newer flash devices.
>
> Applies cleanly on top of l2-mtd.git.
>
> Thanks!
>
> Ezequiel Garcia (5):
> mtd: nand: pxa3xx: Set info->use_dma properly
> mtd: nand: pxa3xx: Use of_machine_is_compatible()
> mtd: nand: pxa3xx: Fix MODULE_DEVICE_TABLE declaration
> mtd: nand: pxa3xx: Add address support for READID command
> mtd: nand: pxa3xx: Add support for Read parameter page command
tested on cm-x300, upstream 3.10-rc5 and no regressions seen.
Tested-by: Nikita Kiryanov <nikita@compulab.co.il>
Acked-by: Igor Grinberg <grinberg@compulab.co.il>
>
> drivers/mtd/nand/pxa3xx_nand.c | 16 ++++++++++++++--
> 1 file changed, 14 insertions(+), 2 deletions(-)
>
--
Regards,
Igor.
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx
2013-06-17 5:53 ` Igor Grinberg
@ 2013-06-17 11:03 ` Ezequiel Garcia
2013-06-29 14:17 ` Ezequiel Garcia
0 siblings, 1 reply; 19+ messages in thread
From: Ezequiel Garcia @ 2013-06-17 11:03 UTC (permalink / raw)
To: Igor Grinberg
Cc: Thomas Petazzoni, Lior Amsalem, Ezequiel Garcia, Artem Bityutskiy,
Lei Wen, Haojian Zhuang, Maen Suleiman, linux-mtd,
Gregory Clement, Neil Zhang, Chao Xie
Nikita, Igor:
On Mon, Jun 17, 2013 at 08:53:25AM +0300, Igor Grinberg wrote:
> On 05/14/13 14:15, Ezequiel Garcia wrote:
> > The Armada 370 and Armada XP SoC include a NAND controller (NFCv2) that
> > is similar to the one in PXA SoC. Therefore, it should be possible to support
> > Armada NAND using pxa3xx-nand driver.
> >
> > While this effort is not ready yet, here is a set of patches that are needed
> > for such work.
> >
> > A small summary of the patches:
> >
> > * 1/5 is a fix for a 2-year regression that removed DMA support.
> >
> > * 2/5 and 3/5 are needed to build the driver on platforms other PXA.
> >
> > * 4/5 and 5/5 add ONFI-compliant device detection and are needed
> > to successfully probe newer flash devices.
> >
> > Applies cleanly on top of l2-mtd.git.
> >
> > Thanks!
> >
> > Ezequiel Garcia (5):
> > mtd: nand: pxa3xx: Set info->use_dma properly
> > mtd: nand: pxa3xx: Use of_machine_is_compatible()
> > mtd: nand: pxa3xx: Fix MODULE_DEVICE_TABLE declaration
> > mtd: nand: pxa3xx: Add address support for READID command
> > mtd: nand: pxa3xx: Add support for Read parameter page command
>
> tested on cm-x300, upstream 3.10-rc5 and no regressions seen.
>
> Tested-by: Nikita Kiryanov <nikita@compulab.co.il>
> Acked-by: Igor Grinberg <grinberg@compulab.co.il>
>
Thanks a lot for the review and the test!
Artem, do you think this could be queued for v3.11?
--
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx
2013-06-17 11:03 ` Ezequiel Garcia
@ 2013-06-29 14:17 ` Ezequiel Garcia
0 siblings, 0 replies; 19+ messages in thread
From: Ezequiel Garcia @ 2013-06-29 14:17 UTC (permalink / raw)
To: Artem Bityutskiy
Cc: Thomas Petazzoni, Lior Amsalem, Ezequiel Garcia, Artem Bityutskiy,
Lei Wen, Haojian Zhuang, Maen Suleiman, linux-mtd, Igor Grinberg,
Gregory Clement, Neil Zhang, Chao Xie
Hi Artem,
On Mon, Jun 17, 2013 at 08:03:05AM -0300, Ezequiel Garcia wrote:
> Nikita, Igor:
>
> On Mon, Jun 17, 2013 at 08:53:25AM +0300, Igor Grinberg wrote:
> > On 05/14/13 14:15, Ezequiel Garcia wrote:
> > > The Armada 370 and Armada XP SoC include a NAND controller (NFCv2) that
> > > is similar to the one in PXA SoC. Therefore, it should be possible to support
> > > Armada NAND using pxa3xx-nand driver.
> > >
> > > While this effort is not ready yet, here is a set of patches that are needed
> > > for such work.
> > >
> > > A small summary of the patches:
> > >
> > > * 1/5 is a fix for a 2-year regression that removed DMA support.
> > >
> > > * 2/5 and 3/5 are needed to build the driver on platforms other PXA.
> > >
> > > * 4/5 and 5/5 add ONFI-compliant device detection and are needed
> > > to successfully probe newer flash devices.
> > >
> > > Applies cleanly on top of l2-mtd.git.
> > >
> > > Thanks!
> > >
> > > Ezequiel Garcia (5):
> > > mtd: nand: pxa3xx: Set info->use_dma properly
> > > mtd: nand: pxa3xx: Use of_machine_is_compatible()
> > > mtd: nand: pxa3xx: Fix MODULE_DEVICE_TABLE declaration
> > > mtd: nand: pxa3xx: Add address support for READID command
> > > mtd: nand: pxa3xx: Add support for Read parameter page command
> >
> > tested on cm-x300, upstream 3.10-rc5 and no regressions seen.
> >
> > Tested-by: Nikita Kiryanov <nikita@compulab.co.il>
> > Acked-by: Igor Grinberg <grinberg@compulab.co.il>
> >
>
> Thanks a lot for the review and the test!
>
> Artem, do you think this could be queued for v3.11?
>
I guess it's already too late to have this merged for v3.11, right?
Given this has Tested-by, Acked-by and Reviewed-by, I think it should
be ready to apply. If there's anything you want me to review,
just let me know.
Thanks!
--
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx
2013-06-07 14:22 ` Ezequiel Garcia
2013-06-08 7:22 ` Haojian Zhuang
@ 2013-07-01 7:21 ` Artem Bityutskiy
2013-07-01 13:41 ` Ezequiel Garcia
2013-07-20 14:36 ` Ezequiel Garcia
1 sibling, 2 replies; 19+ messages in thread
From: Artem Bityutskiy @ 2013-07-01 7:21 UTC (permalink / raw)
To: Ezequiel Garcia
Cc: Thomas Petazzoni, Lior Amsalem, Eric Miao, Lei Wen,
Haojian Zhuang, Maen Suleiman, linux-mtd, Ezequiel Garcia,
Gregory Clement, Neil Zhang, Chao Xie
On Fri, 2013-06-07 at 11:22 -0300, Ezequiel Garcia wrote:
> > Since this patch affects directly the PXA hardware, I'd like to have some
> > response from PXA maintainers.
> >
> > Guys, could you take a look at this?
> >
>
> Ping?
Hi Ezequiel,
sorry for long delay. I simply had no time to look an any of my
open-source things, too busy.
I've pushed this series to l2-mtd.git, thanks!
--
Best Regards,
Artem Bityutskiy
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx
2013-07-01 7:21 ` Artem Bityutskiy
@ 2013-07-01 13:41 ` Ezequiel Garcia
2013-07-20 14:36 ` Ezequiel Garcia
1 sibling, 0 replies; 19+ messages in thread
From: Ezequiel Garcia @ 2013-07-01 13:41 UTC (permalink / raw)
To: Artem Bityutskiy
Cc: Thomas Petazzoni, Lior Amsalem, Ezequiel Garcia, Eric Miao,
Lei Wen, Haojian Zhuang, Maen Suleiman, linux-mtd,
Gregory Clement, Neil Zhang, Chao Xie
On Mon, Jul 01, 2013 at 10:21:16AM +0300, Artem Bityutskiy wrote:
> On Fri, 2013-06-07 at 11:22 -0300, Ezequiel Garcia wrote:
> > > Since this patch affects directly the PXA hardware, I'd like to have some
> > > response from PXA maintainers.
> > >
> > > Guys, could you take a look at this?
> > >
> >
> > Ping?
>
>
> sorry for long delay. I simply had no time to look an any of my
> open-source things, too busy.
>
No worries...
> I've pushed this series to l2-mtd.git, thanks!
>
Great, thanks!
--
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx
2013-07-01 7:21 ` Artem Bityutskiy
2013-07-01 13:41 ` Ezequiel Garcia
@ 2013-07-20 14:36 ` Ezequiel Garcia
2013-07-22 6:39 ` Brian Norris
1 sibling, 1 reply; 19+ messages in thread
From: Ezequiel Garcia @ 2013-07-20 14:36 UTC (permalink / raw)
To: Artem Bityutskiy
Cc: Thomas Petazzoni, Lior Amsalem, Ezequiel Garcia, Eric Miao,
Lei Wen, Haojian Zhuang, Maen Suleiman, linux-mtd,
Gregory Clement, Neil Zhang, Chao Xie
Hi Artem,
On Mon, Jul 01, 2013 at 10:21:16AM +0300, Artem Bityutskiy wrote:
> On Fri, 2013-06-07 at 11:22 -0300, Ezequiel Garcia wrote:
> > > Since this patch affects directly the PXA hardware, I'd like to have some
> > > response from PXA maintainers.
> > >
> > > Guys, could you take a look at this?
> > >
> >
> > Ping?
>
>
> sorry for long delay. I simply had no time to look an any of my
> open-source things, too busy.
>
> I've pushed this series to l2-mtd.git, thanks!
>
Thanks!
Any chance you can push this to some branch that's included in linux-next?
(when you get back to work, of course)
--
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx
2013-07-20 14:36 ` Ezequiel Garcia
@ 2013-07-22 6:39 ` Brian Norris
2013-07-22 14:10 ` Ezequiel Garcia
0 siblings, 1 reply; 19+ messages in thread
From: Brian Norris @ 2013-07-22 6:39 UTC (permalink / raw)
To: Ezequiel Garcia
Cc: Thomas Petazzoni, Lior Amsalem, Ezequiel Garcia, Eric Miao,
Artem Bityutskiy, Lei Wen, Haojian Zhuang, Maen Suleiman,
linux-mtd, Gregory Clement, Neil Zhang, Chao Xie
Hi Ezequiel,
On Sat, Jul 20, 2013 at 7:36 AM, Ezequiel Garcia
<ezequiel.garcia@free-electrons.com> wrote:
> Hi Artem,
>
> On Mon, Jul 01, 2013 at 10:21:16AM +0300, Artem Bityutskiy wrote:
>> On Fri, 2013-06-07 at 11:22 -0300, Ezequiel Garcia wrote:
>> > > Since this patch affects directly the PXA hardware, I'd like to have some
>> > > response from PXA maintainers.
>> > >
>> > > Guys, could you take a look at this?
>> > >
>> >
>> > Ping?
>>
>>
>> sorry for long delay. I simply had no time to look an any of my
>> open-source things, too busy.
>>
>> I've pushed this series to l2-mtd.git, thanks!
>>
>
> Thanks!
>
> Any chance you can push this to some branch that's included in linux-next?
> (when you get back to work, of course)
Artem's l2-mtd.git 'master' branch is already included in linux-next.
Your patch series appears to be included as the following commit (and
its parents):
commit 3f81a731d931cc66abec2d0c205c4a6ecda7dac9
Author: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
Date: Tue May 14 08:15:25 2013 -0300
mtd: nand: pxa3xx: Add support for Read parameter page command
Regards,
Brian
^ permalink raw reply [flat|nested] 19+ messages in thread
* Re: [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx
2013-07-22 6:39 ` Brian Norris
@ 2013-07-22 14:10 ` Ezequiel Garcia
0 siblings, 0 replies; 19+ messages in thread
From: Ezequiel Garcia @ 2013-07-22 14:10 UTC (permalink / raw)
To: Brian Norris
Cc: Thomas Petazzoni, Lior Amsalem, Ezequiel Garcia, Eric Miao,
Artem Bityutskiy, Lei Wen, Haojian Zhuang, Maen Suleiman,
linux-mtd, Gregory Clement, Neil Zhang, Chao Xie
On Sun, Jul 21, 2013 at 11:39:49PM -0700, Brian Norris wrote:
> Hi Ezequiel,
>
> On Sat, Jul 20, 2013 at 7:36 AM, Ezequiel Garcia
> <ezequiel.garcia@free-electrons.com> wrote:
> > Hi Artem,
> >
> > On Mon, Jul 01, 2013 at 10:21:16AM +0300, Artem Bityutskiy wrote:
> >> On Fri, 2013-06-07 at 11:22 -0300, Ezequiel Garcia wrote:
> >> > > Since this patch affects directly the PXA hardware, I'd like to have some
> >> > > response from PXA maintainers.
> >> > >
> >> > > Guys, could you take a look at this?
> >> > >
> >> >
> >> > Ping?
> >>
> >>
> >> sorry for long delay. I simply had no time to look an any of my
> >> open-source things, too busy.
> >>
> >> I've pushed this series to l2-mtd.git, thanks!
> >>
> >
> > Thanks!
> >
> > Any chance you can push this to some branch that's included in linux-next?
> > (when you get back to work, of course)
>
> Artem's l2-mtd.git 'master' branch is already included in linux-next.
Indeed.
I checked before asking but somehow thought it wasn't...
Sorry for the noise and thanks!
--
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 19+ messages in thread
end of thread, other threads:[~2013-07-22 14:10 UTC | newest]
Thread overview: 19+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-05-14 11:15 [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx Ezequiel Garcia
2013-05-14 11:15 ` [PATCH 1/5] mtd: nand: pxa3xx: Set info->use_dma properly Ezequiel Garcia
2013-05-14 11:15 ` [PATCH 2/5] mtd: nand: pxa3xx: Use of_machine_is_compatible() Ezequiel Garcia
2013-05-14 11:15 ` [PATCH 3/5] mtd: nand: pxa3xx: Fix MODULE_DEVICE_TABLE declaration Ezequiel Garcia
2013-05-14 11:15 ` [PATCH 4/5] mtd: nand: pxa3xx: Add address support for READID command Ezequiel Garcia
2013-05-14 11:15 ` [PATCH 5/5] mtd: nand: pxa3xx: Add support for Read parameter page command Ezequiel Garcia
2013-05-20 12:37 ` [PATCH 0/5] mtd: nand: Fixes and improvements on pxa3xx Ezequiel Garcia
2013-06-07 14:22 ` Ezequiel Garcia
2013-06-08 7:22 ` Haojian Zhuang
2013-06-08 9:00 ` Haojian Zhuang
2013-06-08 18:40 ` Ezequiel Garcia
2013-07-01 7:21 ` Artem Bityutskiy
2013-07-01 13:41 ` Ezequiel Garcia
2013-07-20 14:36 ` Ezequiel Garcia
2013-07-22 6:39 ` Brian Norris
2013-07-22 14:10 ` Ezequiel Garcia
2013-06-17 5:53 ` Igor Grinberg
2013-06-17 11:03 ` Ezequiel Garcia
2013-06-29 14:17 ` Ezequiel Garcia
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).