Add the missing uart_unregister_driver() before return
from sci_probe_single() in the error handling case.

Signed-off-by: Mao Wenan <maowe...@huawei.com>
---
 v1->v2: add uart_unregister_driver() if mctrl_gpio_init is failed.
 drivers/tty/serial/sh-sci.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/drivers/tty/serial/sh-sci.c b/drivers/tty/serial/sh-sci.c
index 64bbeb7d7e0c..fb5034390795 100644
--- a/drivers/tty/serial/sh-sci.c
+++ b/drivers/tty/serial/sh-sci.c
@@ -3254,12 +3254,16 @@ static int sci_probe_single(struct platform_device *dev,
        mutex_unlock(&sci_uart_registration_lock);
 
        ret = sci_init_single(dev, sciport, index, p, false);
-       if (ret)
+       if (ret) {
+               uart_unregister_driver(&sci_uart_driver);
                return ret;
+       }
 
        sciport->gpios = mctrl_gpio_init(&sciport->port, 0);
-       if (IS_ERR(sciport->gpios) && PTR_ERR(sciport->gpios) != -ENOSYS)
+       if (IS_ERR(sciport->gpios) && PTR_ERR(sciport->gpios) != -ENOSYS) {
+               uart_unregister_driver(&sci_uart_driver);
                return PTR_ERR(sciport->gpios);
+       }
 
        if (sciport->has_rtscts) {
                if (!IS_ERR_OR_NULL(mctrl_gpio_to_gpiod(sciport->gpios,
-- 
2.20.1

Reply via email to