]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
MIPS: OCTEON: add missing put_device() call in dwc3_octeon_device_init()
authorYu Kuai <yukuai3@huawei.com>
Tue, 21 Jul 2020 13:47:18 +0000 (21:47 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 19 Aug 2020 06:23:59 +0000 (08:23 +0200)
[ Upstream commit e8b9fc10f2615b9a525fce56981e40b489528355 ]

if of_find_device_by_node() succeed, dwc3_octeon_device_init() doesn't have
a corresponding put_device(). Thus add put_device() to fix the exception
handling for this function implementation.

Fixes: 93e502b3c2d4 ("MIPS: OCTEON: Platform support for OCTEON III USB controller")
Signed-off-by: Yu Kuai <yukuai3@huawei.com>
Signed-off-by: Thomas Bogendoerfer <tsbogend@alpha.franken.de>
Signed-off-by: Sasha Levin <sashal@kernel.org>
arch/mips/cavium-octeon/octeon-usb.c

index cc88a08bc1f736422a561a093f3cd24cf757ea2b..4017398519cf9411b76ba378955f29a5bd6a9009 100644 (file)
@@ -518,6 +518,7 @@ static int __init dwc3_octeon_device_init(void)
 
                        res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
                        if (res == NULL) {
+                               put_device(&pdev->dev);
                                dev_err(&pdev->dev, "No memory resources\n");
                                return -ENXIO;
                        }
@@ -529,8 +530,10 @@ static int __init dwc3_octeon_device_init(void)
                         * know the difference.
                         */
                        base = devm_ioremap_resource(&pdev->dev, res);
-                       if (IS_ERR(base))
+                       if (IS_ERR(base)) {
+                               put_device(&pdev->dev);
                                return PTR_ERR(base);
+                       }
 
                        mutex_lock(&dwc3_octeon_clocks_mutex);
                        dwc3_octeon_clocks_start(&pdev->dev, (u64)base);