* [PATCH v2] mtd: atmel_nand: fix the compile error which miss label "err_ecc_ioremap".
@ 2012-09-11 7:10 Josh Wu
2012-09-20 2:03 ` Josh Wu
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Josh Wu @ 2012-09-11 7:10 UTC (permalink / raw)
To: linux-arm-kernel
Add err_ecc_ioremap label to fix following errors:
---8<---
CC drivers/mtd/nand/atmel_nand.o
drivers/mtd/nand/atmel_nand.c: In function 'atmel_nand_probe':
drivers/mtd/nand/atmel_nand.c:1423: error: label 'err_ecc_ioremap' used but not defined
make[3]: *** [drivers/mtd/nand/atmel_nand.o] Error 1
make[2]: *** [drivers/mtd/nand] Error 2
make[1]: *** [drivers/mtd] Error 2
make: *** [drivers] Error 2
--->8---
This error was introduced in b654a9a46fc2100b318050483f8c5b5d0f187303 (mtd: atmel nand: fix gpio missing request)
Signed-off-by: Josh Wu <josh.wu@atmel.com>
---
change logs:
Add detail information for the commit message.
drivers/mtd/nand/atmel_nand.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/mtd/nand/atmel_nand.c b/drivers/mtd/nand/atmel_nand.c
index 6a5a35b..9144557 100644
--- a/drivers/mtd/nand/atmel_nand.c
+++ b/drivers/mtd/nand/atmel_nand.c
@@ -1399,7 +1399,7 @@ static int __init atmel_nand_probe(struct platform_device *pdev)
if (pdev->dev.of_node) {
res = atmel_of_init_port(host, pdev->dev.of_node);
if (res)
- goto err_nand_ioremap;
+ goto err_ecc_ioremap;
} else {
memcpy(&host->board, pdev->dev.platform_data,
sizeof(struct atmel_nand_data));
@@ -1560,6 +1560,7 @@ err_no_card:
platform_set_drvdata(pdev, NULL);
if (host->dma_chan)
dma_release_channel(host->dma_chan);
+err_ecc_ioremap:
iounmap(host->io_base);
err_nand_ioremap:
kfree(host);
--
1.7.9.5
^ permalink raw reply related [flat|nested] 7+ messages in thread* [PATCH v2] mtd: atmel_nand: fix the compile error which miss label "err_ecc_ioremap".
2012-09-11 7:10 [PATCH v2] mtd: atmel_nand: fix the compile error which miss label "err_ecc_ioremap" Josh Wu
@ 2012-09-20 2:03 ` Josh Wu
2012-09-20 5:23 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-23 12:41 ` Artem Bityutskiy
2 siblings, 0 replies; 7+ messages in thread
From: Josh Wu @ 2012-09-20 2:03 UTC (permalink / raw)
To: linux-arm-kernel
Hi, Artem
Do you have any feedback since there is a long time about this patch?
Best Regards,
Josh Wu
On 9/11/2012 3:10 PM, Josh Wu wrote:
> Add err_ecc_ioremap label to fix following errors:
> ---8<---
> CC drivers/mtd/nand/atmel_nand.o
> drivers/mtd/nand/atmel_nand.c: In function 'atmel_nand_probe':
> drivers/mtd/nand/atmel_nand.c:1423: error: label 'err_ecc_ioremap' used but not defined
> make[3]: *** [drivers/mtd/nand/atmel_nand.o] Error 1
> make[2]: *** [drivers/mtd/nand] Error 2
> make[1]: *** [drivers/mtd] Error 2
> make: *** [drivers] Error 2
> --->8---
>
> This error was introduced in b654a9a46fc2100b318050483f8c5b5d0f187303 (mtd: atmel nand: fix gpio missing request)
>
> Signed-off-by: Josh Wu <josh.wu@atmel.com>
> ---
> change logs:
> Add detail information for the commit message.
>
> drivers/mtd/nand/atmel_nand.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/mtd/nand/atmel_nand.c b/drivers/mtd/nand/atmel_nand.c
> index 6a5a35b..9144557 100644
> --- a/drivers/mtd/nand/atmel_nand.c
> +++ b/drivers/mtd/nand/atmel_nand.c
> @@ -1399,7 +1399,7 @@ static int __init atmel_nand_probe(struct platform_device *pdev)
> if (pdev->dev.of_node) {
> res = atmel_of_init_port(host, pdev->dev.of_node);
> if (res)
> - goto err_nand_ioremap;
> + goto err_ecc_ioremap;
> } else {
> memcpy(&host->board, pdev->dev.platform_data,
> sizeof(struct atmel_nand_data));
> @@ -1560,6 +1560,7 @@ err_no_card:
> platform_set_drvdata(pdev, NULL);
> if (host->dma_chan)
> dma_release_channel(host->dma_chan);
> +err_ecc_ioremap:
> iounmap(host->io_base);
> err_nand_ioremap:
> kfree(host);
^ permalink raw reply [flat|nested] 7+ messages in thread* [PATCH v2] mtd: atmel_nand: fix the compile error which miss label "err_ecc_ioremap".
2012-09-11 7:10 [PATCH v2] mtd: atmel_nand: fix the compile error which miss label "err_ecc_ioremap" Josh Wu
2012-09-20 2:03 ` Josh Wu
@ 2012-09-20 5:23 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-20 8:20 ` Nicolas Ferre
2012-09-21 10:16 ` Josh Wu
2012-09-23 12:41 ` Artem Bityutskiy
2 siblings, 2 replies; 7+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2012-09-20 5:23 UTC (permalink / raw)
To: linux-arm-kernel
On 15:10 Tue 11 Sep , Josh Wu wrote:
> Add err_ecc_ioremap label to fix following errors:
> ---8<---
> CC drivers/mtd/nand/atmel_nand.o
> drivers/mtd/nand/atmel_nand.c: In function 'atmel_nand_probe':
> drivers/mtd/nand/atmel_nand.c:1423: error: label 'err_ecc_ioremap' used but not defined
> make[3]: *** [drivers/mtd/nand/atmel_nand.o] Error 1
> make[2]: *** [drivers/mtd/nand] Error 2
> make[1]: *** [drivers/mtd] Error 2
> make: *** [drivers] Error 2
> --->8---
switch to devm_xxx instead this will simplify the error patch and remove patch
Best Regards,
J.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2] mtd: atmel_nand: fix the compile error which miss label "err_ecc_ioremap".
2012-09-20 5:23 ` Jean-Christophe PLAGNIOL-VILLARD
@ 2012-09-20 8:20 ` Nicolas Ferre
2012-09-21 10:16 ` Josh Wu
1 sibling, 0 replies; 7+ messages in thread
From: Nicolas Ferre @ 2012-09-20 8:20 UTC (permalink / raw)
To: linux-arm-kernel
On 09/20/2012 07:23 AM, Jean-Christophe PLAGNIOL-VILLARD :
> On 15:10 Tue 11 Sep , Josh Wu wrote:
>> Add err_ecc_ioremap label to fix following errors:
>> ---8<---
>> CC drivers/mtd/nand/atmel_nand.o
>> drivers/mtd/nand/atmel_nand.c: In function 'atmel_nand_probe':
>> drivers/mtd/nand/atmel_nand.c:1423: error: label 'err_ecc_ioremap' used but not defined
>> make[3]: *** [drivers/mtd/nand/atmel_nand.o] Error 1
>> make[2]: *** [drivers/mtd/nand] Error 2
>> make[1]: *** [drivers/mtd] Error 2
>> make: *** [drivers] Error 2
>> --->8---
> switch to devm_xxx instead this will simplify the error patch and remove patch
As far as I understand, it is a correction patch: so my advice would be
to take this patch for fixing the current code (for 3.7 if I recall well).
If we want to move to the new devm_xxx API, we can do it once the
current code is fixed.
Bye,
--
Nicolas Ferre
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2] mtd: atmel_nand: fix the compile error which miss label "err_ecc_ioremap".
2012-09-20 5:23 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-20 8:20 ` Nicolas Ferre
@ 2012-09-21 10:16 ` Josh Wu
1 sibling, 0 replies; 7+ messages in thread
From: Josh Wu @ 2012-09-21 10:16 UTC (permalink / raw)
To: linux-arm-kernel
Hi, J.C
On 9/20/2012 1:23 PM, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 15:10 Tue 11 Sep , Josh Wu wrote:
>> Add err_ecc_ioremap label to fix following errors:
>> ---8<---
>> CC drivers/mtd/nand/atmel_nand.o
>> drivers/mtd/nand/atmel_nand.c: In function 'atmel_nand_probe':
>> drivers/mtd/nand/atmel_nand.c:1423: error: label 'err_ecc_ioremap' used but not defined
>> make[3]: *** [drivers/mtd/nand/atmel_nand.o] Error 1
>> make[2]: *** [drivers/mtd/nand] Error 2
>> make[1]: *** [drivers/mtd] Error 2
>> make: *** [drivers] Error 2
>> --->8---
> switch to devm_xxx instead this will simplify the error patch and remove patch
understood. I will push a patch which use devm_xxx. That should remove
many code of the error handling. Thanks
Best Regards,
Josh Wu
>
> Best Regards,
> J.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2] mtd: atmel_nand: fix the compile error which miss label "err_ecc_ioremap".
2012-09-11 7:10 [PATCH v2] mtd: atmel_nand: fix the compile error which miss label "err_ecc_ioremap" Josh Wu
2012-09-20 2:03 ` Josh Wu
2012-09-20 5:23 ` Jean-Christophe PLAGNIOL-VILLARD
@ 2012-09-23 12:41 ` Artem Bityutskiy
2012-09-23 13:01 ` Artem Bityutskiy
2 siblings, 1 reply; 7+ messages in thread
From: Artem Bityutskiy @ 2012-09-23 12:41 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, 2012-09-11 at 15:10 +0800, Josh Wu wrote:
> Add err_ecc_ioremap label to fix following errors:
> ---8<---
> CC drivers/mtd/nand/atmel_nand.o
> drivers/mtd/nand/atmel_nand.c: In function 'atmel_nand_probe':
> drivers/mtd/nand/atmel_nand.c:1423: error: label 'err_ecc_ioremap' used but not defined
> make[3]: *** [drivers/mtd/nand/atmel_nand.o] Error 1
> make[2]: *** [drivers/mtd/nand] Error 2
> make[1]: *** [drivers/mtd] Error 2
> make: *** [drivers] Error 2
> --->8---
Pushed to l2-mtd.git, thanks!
--
Best Regards,
Artem Bityutskiy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120923/26001d36/attachment.sig>
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2] mtd: atmel_nand: fix the compile error which miss label "err_ecc_ioremap".
2012-09-23 12:41 ` Artem Bityutskiy
@ 2012-09-23 13:01 ` Artem Bityutskiy
0 siblings, 0 replies; 7+ messages in thread
From: Artem Bityutskiy @ 2012-09-23 13:01 UTC (permalink / raw)
To: linux-arm-kernel
On Sun, 2012-09-23 at 15:41 +0300, Artem Bityutskiy wrote:
> On Tue, 2012-09-11 at 15:10 +0800, Josh Wu wrote:
> > Add err_ecc_ioremap label to fix following errors:
> > ---8<---
> > CC drivers/mtd/nand/atmel_nand.o
> > drivers/mtd/nand/atmel_nand.c: In function 'atmel_nand_probe':
> > drivers/mtd/nand/atmel_nand.c:1423: error: label 'err_ecc_ioremap' used but not defined
> > make[3]: *** [drivers/mtd/nand/atmel_nand.o] Error 1
> > make[2]: *** [drivers/mtd/nand] Error 2
> > make[1]: *** [drivers/mtd] Error 2
> > make: *** [drivers] Error 2
> > --->8---
>
> Pushed to l2-mtd.git, thanks!
Actually squashed it into the bogus patch.
--
Best Regards,
Artem Bityutskiy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120923/d7f838a4/attachment.sig>
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-09-23 13:01 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-11 7:10 [PATCH v2] mtd: atmel_nand: fix the compile error which miss label "err_ecc_ioremap" Josh Wu
2012-09-20 2:03 ` Josh Wu
2012-09-20 5:23 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-20 8:20 ` Nicolas Ferre
2012-09-21 10:16 ` Josh Wu
2012-09-23 12:41 ` Artem Bityutskiy
2012-09-23 13:01 ` Artem Bityutskiy
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).