| Message ID | 1307086473.7474.1.camel@phoenix |
|---|---|
| State | Accepted |
| Commit | 1676bc18c6648c378029dad365756e12be7da025 |
| Headers | show |
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
> 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
(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/ >
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!
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);
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(-)