We can return 0 directly instead of branching to the out label. Signed-off-by: Boris Brezillon <boris.brezillon@xxxxxxxxxxxxx> Reviewed-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx> Tested-by: Lubomir Rintel <lkundrak@xxxxx> --- Changes in v2: * Add R-b/T-b --- drivers/mtd/nand/raw/cafe_nand.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/mtd/nand/raw/cafe_nand.c b/drivers/mtd/nand/raw/cafe_nand.c index 93e8e124df45..84b8d50c7f08 100644 --- a/drivers/mtd/nand/raw/cafe_nand.c +++ b/drivers/mtd/nand/raw/cafe_nand.c @@ -930,7 +930,7 @@ static int cafe_nand_probe(struct pci_dev *pdev, if (err) goto out_cleanup_nand; - goto out; + return 0; out_cleanup_nand: nand_cleanup(&cafe->nand); @@ -941,7 +941,6 @@ static int cafe_nand_probe(struct pci_dev *pdev, GLOBAL_IRQ_MASK); out_ior: pci_iounmap(pdev, cafe->mmio); - out: return err; } -- 2.25.3 ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/