On Sun, May 17, 2015 at 05:41:01PM +0200, Hauke Mehrtens wrote: > Remove static in front of ctrl. This variable should not be shared > between different instances of brcmnand_probe(), it should be local to > this function and stored on the stack. > > Signed-off-by: Hauke Mehrtens <hauke@xxxxxxxxxx> > --- > drivers/mtd/nand/brcmnand/brcmnand.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/mtd/nand/brcmnand/brcmnand.c b/drivers/mtd/nand/brcmnand/brcmnand.c > index bae30ab..a48ad49 100644 > --- a/drivers/mtd/nand/brcmnand/brcmnand.c > +++ b/drivers/mtd/nand/brcmnand/brcmnand.c > @@ -2069,7 +2069,7 @@ int brcmnand_probe(struct platform_device *pdev, struct brcmnand_soc *soc) > { > struct device *dev = &pdev->dev; > struct device_node *dn = dev->of_node, *child; > - static struct brcmnand_controller *ctrl; > + struct brcmnand_controller *ctrl; > struct resource *res; > int ret; > Good catch. Applied this to l2-mtd.git. I'll come back to the others. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html