Dan Carpenter <dan.carpenter@xxxxxxxxxx> writes: > On Sat, Feb 28, 2015 at 02:02:24AM -0800, Brian Norris wrote: >> diff --git a/drivers/mtd/devices/docg3.c b/drivers/mtd/devices/docg3.c >> index 448ce42f951e..01eddea1f0d9 100644 >> --- a/drivers/mtd/devices/docg3.c >> +++ b/drivers/mtd/devices/docg3.c >> @@ -2079,8 +2079,6 @@ static int __init docg3_probe(struct platform_device *pdev) >> ret = doc_register_sysfs(pdev, cascade); >> if (ret) >> goto err_probe; >> - if (!found) >> - goto notfound; > > Just delete the "found" variable because there are no other uses. Yes, and yes. For both changes (condition killing, found++ eradication and found variable declaration) : Acked-by: Robert Jarzmik <robert.jarzmik@xxxxxxx> Cheers. -- Robert -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
