summaryrefslogtreecommitdiffstats
path: root/drivers/serial
diff options
context:
space:
mode:
authorDaniel Glöckner <dg@emlix.com>2009-06-11 14:39:21 +0100
committerLinus Torvalds <torvalds@linux-foundation.org>2009-06-11 08:51:08 -0700
commit9f322ad064f9210e7d472dfe77e702274d5c9dba (patch)
treee7799ca401c1334e9c4bea40b02e7a5b6214a392 /drivers/serial
parent2e1463922a35584c863f71d4021e1e71f76eaed0 (diff)
downloadop-kernel-dev-9f322ad064f9210e7d472dfe77e702274d5c9dba.zip
op-kernel-dev-9f322ad064f9210e7d472dfe77e702274d5c9dba.tar.gz
imx: serial: handle initialisation failure correctly
correctly de-initialise device when setting up failed, call to pdata->exit() was missing. Signed-off-by: Daniel Glöckner <dg@emlix.com> Signed-off-by: Oskar Schirmer <os@emlix.com> Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/serial')
-rw-r--r--drivers/serial/imx.c7
1 files changed, 6 insertions, 1 deletions
diff --git a/drivers/serial/imx.c b/drivers/serial/imx.c
index e6c2ba2..cbd4f32 100644
--- a/drivers/serial/imx.c
+++ b/drivers/serial/imx.c
@@ -1157,10 +1157,15 @@ static int serial_imx_probe(struct platform_device *pdev)
goto clkput;
}
- uart_add_one_port(&imx_reg, &sport->port);
+ ret = uart_add_one_port(&imx_reg, &sport->port);
+ if (ret)
+ goto deinit;
platform_set_drvdata(pdev, &sport->port);
return 0;
+deinit:
+ if (pdata->exit)
+ pdata->exit(pdev);
clkput:
clk_put(sport->clk);
clk_disable(sport->clk);
OpenPOWER on IntegriCloud