diff options
author | Felipe Balbi <balbi@ti.com> | 2012-09-06 08:45:28 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-09-06 12:17:01 -0400 |
commit | 1b42c8b29b1822436e690fd0a7906aaa62099f91 (patch) | |
tree | 68415b46c24da4d99fc45ee759bdb31fdffee88e /drivers/tty | |
parent | 93220dcc3052182e7156c09655ad1316055564b9 (diff) |
serial: omap: drop unnecessary check from remove
if platform_get_drvdata() returns NULL, that's
quite a nasty bug on the driver which we want to
catch ASAP. Otherwise, that check is hugely
unneeded.
Tested-by: Shubhrajyoti D <shubhrajyoti@ti.com>
Acked-by: Santosh Shilimkar <santosh.shilimkar@ti.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/tty')
-rw-r--r-- | drivers/tty/serial/omap-serial.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/tty/serial/omap-serial.c b/drivers/tty/serial/omap-serial.c index 8a696a4efbee..c19d340f6ba8 100644 --- a/drivers/tty/serial/omap-serial.c +++ b/drivers/tty/serial/omap-serial.c | |||
@@ -1369,13 +1369,10 @@ static int serial_omap_remove(struct platform_device *dev) | |||
1369 | { | 1369 | { |
1370 | struct uart_omap_port *up = platform_get_drvdata(dev); | 1370 | struct uart_omap_port *up = platform_get_drvdata(dev); |
1371 | 1371 | ||
1372 | if (up) { | 1372 | pm_runtime_disable(up->dev); |
1373 | pm_runtime_disable(up->dev); | 1373 | uart_remove_one_port(&serial_omap_reg, &up->port); |
1374 | uart_remove_one_port(&serial_omap_reg, &up->port); | 1374 | pm_qos_remove_request(&up->pm_qos_request); |
1375 | pm_qos_remove_request(&up->pm_qos_request); | ||
1376 | } | ||
1377 | 1375 | ||
1378 | platform_set_drvdata(dev, NULL); | ||
1379 | return 0; | 1376 | return 0; |
1380 | } | 1377 | } |
1381 | 1378 | ||