* [PATCH] mtd: pxa3xx_nand: remove unused variable 'mtd'
@ 2011-06-03 7:34 ` Axel Lin
0 siblings, 0 replies; 10+ messages in thread
From: Axel Lin @ 2011-06-03 7:34 UTC (permalink / raw)
To: linux-kernel; +Cc: Haojian Zhuang, Artem Bityutskiy, Eric Miao, linux-mtd
Remove unused variable 'mtd' to eliminate below warning.
CC drivers/mtd/nand/pxa3xx_nand.o
drivers/mtd/nand/pxa3xx_nand.c: In function 'pxa3xx_nand_suspend':
drivers/mtd/nand/pxa3xx_nand.c:1167: warning: unused variable 'mtd'
drivers/mtd/nand/pxa3xx_nand.c: In function 'pxa3xx_nand_resume':
drivers/mtd/nand/pxa3xx_nand.c:1180: warning: unused variable 'mtd'
Signed-off-by: Axel Lin <axel.lin@gmail.com>
---
drivers/mtd/nand/pxa3xx_nand.c | 2 --
1 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
index 1fb3b3a..5cae8ae 100644
--- a/drivers/mtd/nand/pxa3xx_nand.c
+++ b/drivers/mtd/nand/pxa3xx_nand.c
@@ -1164,7 +1164,6 @@ static int pxa3xx_nand_probe(struct platform_device *pdev)
static int pxa3xx_nand_suspend(struct platform_device *pdev, pm_message_t state)
{
struct pxa3xx_nand_info *info = platform_get_drvdata(pdev);
- struct mtd_info *mtd = info->mtd;
if (info->state) {
dev_err(&pdev->dev, "driver busy, state = %d\n", info->state);
@@ -1177,7 +1176,6 @@ static int pxa3xx_nand_suspend(struct platform_device *pdev, pm_message_t state)
static int pxa3xx_nand_resume(struct platform_device *pdev)
{
struct pxa3xx_nand_info *info = platform_get_drvdata(pdev);
- struct mtd_info *mtd = info->mtd;
nand_writel(info, NDTR0CS0, info->ndtr0cs0);
nand_writel(info, NDTR1CS0, info->ndtr1cs0);
--
1.7.4.1
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [PATCH] mtd: pxa3xx_nand: remove unused variable 'mtd'
2011-06-03 7:34 ` Axel Lin
@ 2011-06-03 7:44 ` Daniel Mack
-1 siblings, 0 replies; 10+ messages in thread
From: Daniel Mack @ 2011-06-03 7:44 UTC (permalink / raw)
To: Axel Lin
Cc: Artem Bityutskiy, Eric Miao, linux-kernel, Haojian Zhuang,
linux-mtd, Lei Wen
Hi Alex,
On Fri, Jun 3, 2011 at 9:34 AM, Axel Lin <axel.lin@gmail.com> wrote:
> Remove unused variable 'mtd' to eliminate below warning.
>
> CC drivers/mtd/nand/pxa3xx_nand.o
> drivers/mtd/nand/pxa3xx_nand.c: In function 'pxa3xx_nand_suspend':
> drivers/mtd/nand/pxa3xx_nand.c:1167: warning: unused variable 'mtd'
> drivers/mtd/nand/pxa3xx_nand.c: In function 'pxa3xx_nand_resume':
> drivers/mtd/nand/pxa3xx_nand.c:1180: warning: unused variable 'mtd'
Do you actually use this driver? I'm asking because there's some sort
of nasty regression in the changes between 2.6.38 and 2.6.39 which
makes the driver fail to work on my PXA300 based board. Lei and me
tried to nail these issues down, but we didn't succeed yet.
Daniel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] mtd: pxa3xx_nand: remove unused variable 'mtd'
@ 2011-06-03 7:44 ` Daniel Mack
0 siblings, 0 replies; 10+ messages in thread
From: Daniel Mack @ 2011-06-03 7:44 UTC (permalink / raw)
To: Axel Lin
Cc: linux-kernel, Haojian Zhuang, Artem Bityutskiy, Eric Miao,
linux-mtd, Lei Wen
Hi Alex,
On Fri, Jun 3, 2011 at 9:34 AM, Axel Lin <axel.lin@gmail.com> wrote:
> Remove unused variable 'mtd' to eliminate below warning.
>
> CC drivers/mtd/nand/pxa3xx_nand.o
> drivers/mtd/nand/pxa3xx_nand.c: In function 'pxa3xx_nand_suspend':
> drivers/mtd/nand/pxa3xx_nand.c:1167: warning: unused variable 'mtd'
> drivers/mtd/nand/pxa3xx_nand.c: In function 'pxa3xx_nand_resume':
> drivers/mtd/nand/pxa3xx_nand.c:1180: warning: unused variable 'mtd'
Do you actually use this driver? I'm asking because there's some sort
of nasty regression in the changes between 2.6.38 and 2.6.39 which
makes the driver fail to work on my PXA300 based board. Lei and me
tried to nail these issues down, but we didn't succeed yet.
Daniel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] mtd: pxa3xx_nand: remove unused variable 'mtd'
2011-06-03 7:44 ` Daniel Mack
@ 2011-06-03 7:58 ` Axel Lin
-1 siblings, 0 replies; 10+ messages in thread
From: Axel Lin @ 2011-06-03 7:58 UTC (permalink / raw)
To: Daniel Mack
Cc: Artem Bityutskiy, Eric Miao, linux-kernel, Haojian Zhuang,
linux-mtd, Lei Wen
> Do you actually use this driver? I'm asking because there's some sort
> of nasty regression in the changes between 2.6.38 and 2.6.39 which
> makes the driver fail to work on my PXA300 based board. Lei and me
> tried to nail these issues down, but we didn't succeed yet.
hi Daniel,
I no longer have the device handy.
Last time I use this driver on a pxa310 device is running on 2.6.37 kernel.
Regards,
Axel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] mtd: pxa3xx_nand: remove unused variable 'mtd'
@ 2011-06-03 7:58 ` Axel Lin
0 siblings, 0 replies; 10+ messages in thread
From: Axel Lin @ 2011-06-03 7:58 UTC (permalink / raw)
To: Daniel Mack
Cc: linux-kernel, Haojian Zhuang, Artem Bityutskiy, Eric Miao,
linux-mtd, Lei Wen
> Do you actually use this driver? I'm asking because there's some sort
> of nasty regression in the changes between 2.6.38 and 2.6.39 which
> makes the driver fail to work on my PXA300 based board. Lei and me
> tried to nail these issues down, but we didn't succeed yet.
hi Daniel,
I no longer have the device handy.
Last time I use this driver on a pxa310 device is running on 2.6.37 kernel.
Regards,
Axel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] mtd: pxa3xx_nand: remove unused variable 'mtd'
2011-06-03 7:34 ` Axel Lin
@ 2011-06-03 13:00 ` Daniel Mack
-1 siblings, 0 replies; 10+ messages in thread
From: Daniel Mack @ 2011-06-03 13:00 UTC (permalink / raw)
To: Axel Lin
Cc: Artem Bityutskiy, Eric Miao, linux-kernel, Haojian Zhuang,
linux-mtd, Lei Wen
(cc Lei Wen)
On Fri, Jun 3, 2011 at 9:34 AM, Axel Lin <axel.lin@gmail.com> wrote:
> Remove unused variable 'mtd' to eliminate below warning.
>
> CC drivers/mtd/nand/pxa3xx_nand.o
> drivers/mtd/nand/pxa3xx_nand.c: In function 'pxa3xx_nand_suspend':
> drivers/mtd/nand/pxa3xx_nand.c:1167: warning: unused variable 'mtd'
> drivers/mtd/nand/pxa3xx_nand.c: In function 'pxa3xx_nand_resume':
> drivers/mtd/nand/pxa3xx_nand.c:1180: warning: unused variable 'mtd'
>
> Signed-off-by: Axel Lin <axel.lin@gmail.com>
Lei, I had the same patch queued up on my side and dropped it now. Can
you apply this one?
Thanks,
Daniel
> ---
> drivers/mtd/nand/pxa3xx_nand.c | 2 --
> 1 files changed, 0 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
> index 1fb3b3a..5cae8ae 100644
> --- a/drivers/mtd/nand/pxa3xx_nand.c
> +++ b/drivers/mtd/nand/pxa3xx_nand.c
> @@ -1164,7 +1164,6 @@ static int pxa3xx_nand_probe(struct platform_device *pdev)
> static int pxa3xx_nand_suspend(struct platform_device *pdev, pm_message_t state)
> {
> struct pxa3xx_nand_info *info = platform_get_drvdata(pdev);
> - struct mtd_info *mtd = info->mtd;
>
> if (info->state) {
> dev_err(&pdev->dev, "driver busy, state = %d\n", info->state);
> @@ -1177,7 +1176,6 @@ static int pxa3xx_nand_suspend(struct platform_device *pdev, pm_message_t state)
> static int pxa3xx_nand_resume(struct platform_device *pdev)
> {
> struct pxa3xx_nand_info *info = platform_get_drvdata(pdev);
> - struct mtd_info *mtd = info->mtd;
>
> nand_writel(info, NDTR0CS0, info->ndtr0cs0);
> nand_writel(info, NDTR1CS0, info->ndtr1cs0);
> --
> 1.7.4.1
>
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [PATCH] mtd: pxa3xx_nand: remove unused variable 'mtd'
@ 2011-06-03 13:00 ` Daniel Mack
0 siblings, 0 replies; 10+ messages in thread
From: Daniel Mack @ 2011-06-03 13:00 UTC (permalink / raw)
To: Axel Lin
Cc: linux-kernel, Haojian Zhuang, Artem Bityutskiy, Eric Miao,
linux-mtd, Lei Wen
(cc Lei Wen)
On Fri, Jun 3, 2011 at 9:34 AM, Axel Lin <axel.lin@gmail.com> wrote:
> Remove unused variable 'mtd' to eliminate below warning.
>
> CC drivers/mtd/nand/pxa3xx_nand.o
> drivers/mtd/nand/pxa3xx_nand.c: In function 'pxa3xx_nand_suspend':
> drivers/mtd/nand/pxa3xx_nand.c:1167: warning: unused variable 'mtd'
> drivers/mtd/nand/pxa3xx_nand.c: In function 'pxa3xx_nand_resume':
> drivers/mtd/nand/pxa3xx_nand.c:1180: warning: unused variable 'mtd'
>
> Signed-off-by: Axel Lin <axel.lin@gmail.com>
Lei, I had the same patch queued up on my side and dropped it now. Can
you apply this one?
Thanks,
Daniel
> ---
> drivers/mtd/nand/pxa3xx_nand.c | 2 --
> 1 files changed, 0 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mtd/nand/pxa3xx_nand.c b/drivers/mtd/nand/pxa3xx_nand.c
> index 1fb3b3a..5cae8ae 100644
> --- a/drivers/mtd/nand/pxa3xx_nand.c
> +++ b/drivers/mtd/nand/pxa3xx_nand.c
> @@ -1164,7 +1164,6 @@ static int pxa3xx_nand_probe(struct platform_device *pdev)
> static int pxa3xx_nand_suspend(struct platform_device *pdev, pm_message_t state)
> {
> struct pxa3xx_nand_info *info = platform_get_drvdata(pdev);
> - struct mtd_info *mtd = info->mtd;
>
> if (info->state) {
> dev_err(&pdev->dev, "driver busy, state = %d\n", info->state);
> @@ -1177,7 +1176,6 @@ static int pxa3xx_nand_suspend(struct platform_device *pdev, pm_message_t state)
> static int pxa3xx_nand_resume(struct platform_device *pdev)
> {
> struct pxa3xx_nand_info *info = platform_get_drvdata(pdev);
> - struct mtd_info *mtd = info->mtd;
>
> nand_writel(info, NDTR0CS0, info->ndtr0cs0);
> nand_writel(info, NDTR1CS0, info->ndtr1cs0);
> --
> 1.7.4.1
>
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] mtd: pxa3xx_nand: remove unused variable 'mtd'
2011-06-03 7:34 ` Axel Lin
@ 2011-06-06 10:20 ` Artem Bityutskiy
-1 siblings, 0 replies; 10+ messages in thread
From: Artem Bityutskiy @ 2011-06-06 10:20 UTC (permalink / raw)
To: Axel Lin; +Cc: linux-mtd, Eric Miao, linux-kernel, Haojian Zhuang
On Fri, 2011-06-03 at 15:34 +0800, Axel Lin wrote:
> Remove unused variable 'mtd' to eliminate below warning.
>
> CC drivers/mtd/nand/pxa3xx_nand.o
> drivers/mtd/nand/pxa3xx_nand.c: In function 'pxa3xx_nand_suspend':
> drivers/mtd/nand/pxa3xx_nand.c:1167: warning: unused variable 'mtd'
> drivers/mtd/nand/pxa3xx_nand.c: In function 'pxa3xx_nand_resume':
> drivers/mtd/nand/pxa3xx_nand.c:1180: warning: unused variable 'mtd'
>
> Signed-off-by: Axel Lin <axel.lin@gmail.com>
Pushed to l2-mtd-2.6.git, thanks!
--
Best Regards,
Artem Bityutskiy (Артём Битюцкий)
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] mtd: pxa3xx_nand: remove unused variable 'mtd'
@ 2011-06-06 10:20 ` Artem Bityutskiy
0 siblings, 0 replies; 10+ messages in thread
From: Artem Bityutskiy @ 2011-06-06 10:20 UTC (permalink / raw)
To: Axel Lin; +Cc: linux-kernel, Eric Miao, Haojian Zhuang, linux-mtd
On Fri, 2011-06-03 at 15:34 +0800, Axel Lin wrote:
> Remove unused variable 'mtd' to eliminate below warning.
>
> CC drivers/mtd/nand/pxa3xx_nand.o
> drivers/mtd/nand/pxa3xx_nand.c: In function 'pxa3xx_nand_suspend':
> drivers/mtd/nand/pxa3xx_nand.c:1167: warning: unused variable 'mtd'
> drivers/mtd/nand/pxa3xx_nand.c: In function 'pxa3xx_nand_resume':
> drivers/mtd/nand/pxa3xx_nand.c:1180: warning: unused variable 'mtd'
>
> Signed-off-by: Axel Lin <axel.lin@gmail.com>
Pushed to l2-mtd-2.6.git, thanks!
--
Best Regards,
Artem Bityutskiy (Артём Битюцкий)
^ permalink raw reply [flat|nested] 10+ messages in thread