Messages in this thread | | | From | Robert Jarzmik <> | Subject | Re: [PATCH 1/7] mtd: docg3: drop dead code | Date | Mon, 02 Mar 2015 22:39:42 +0100 |
| |
Dan Carpenter <dan.carpenter@oracle.com> 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@free.fr>
Cheers.
-- Robert
| |