Patchwork [v3,10/10] mtd: nand: omap2: use gpmc provided irqs

login
register
mail settings
Submitter Tony Lindgren
Date Sept. 11, 2012, 12:17 a.m.
Message ID <20120911001710.GD1303@atomide.com>
Download mbox | patch
Permalink /patch/182978/
State New
Headers show

Comments

Tony Lindgren - Sept. 11, 2012, 12:17 a.m.
* Tony Lindgren <tony@atomide.com> [120830 12:54]:
> * Artem Bityutskiy <dedekind1@gmail.com> [120825 04:49]:
> > On Tue, 2012-08-21 at 15:14 +0530, Afzal Mohammed wrote:
> > > GPMC platform initialization provides it's clients
> > > with interrupts that can be used through struct
> > > resource. Make use of it for irq mode functionality.
> > > 
> > > Also now write protect disable is done by GPMC,
> > > hence remove it.
> > > 
> > > Signed-off-by: Afzal Mohammed <afzal@ti.com>
> > 
> > Acked-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
> 
> Thanks I'll apply these into omap devel-gpmc branch.

FYI, looks like a new warning got introduced, so I've
committed the following trivial patch on top of this series.

Regards,

Tony

From: Tony Lindgren <tony@atomide.com>
Date: Mon, 10 Sep 2012 17:14:13 -0700
Subject: [PATCH] mtd: nand omap2: Fix uninitialized err warning

Commit bd4156fd (mtd: nand: omap2: use gpmc provided irqs)
introduced a new warning:

drivers/mtd/nand/omap2.c: In function ‘omap_nand_probe’:
drivers/mtd/nand/omap2.c:1267: warning: ‘err’ may be used uninitialized in this function

Signed-off-by: Tony Lindgren <tony@atomide.com>

Patch

--- a/drivers/mtd/nand/omap2.c
+++ b/drivers/mtd/nand/omap2.c
@@ -1389,6 +1389,7 @@  static int __devinit omap_nand_probe(struct platform_device *pdev)
 		info->gpmc_irq_fifo = platform_get_irq(pdev, 0);
 		if (info->gpmc_irq_fifo <= 0) {
 			dev_err(&pdev->dev, "error getting fifo irq\n");
+			err = -ENODEV;
 			goto out_release_mem_region;
 		}
 		err = request_irq(info->gpmc_irq_fifo,	omap_nand_irq,