diff options
author | Alexey Khoroshilov <khoroshilov@ispras.ru> | 2013-06-05 17:28:12 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-06-06 16:07:05 -0400 |
commit | 9f6d20ff848c23506c2ec55431479cb350179886 (patch) | |
tree | 1670aae9b3bf74a9ad6afdfc02c4e57d4940bcee /drivers/tty/serial | |
parent | 7f25301d83432efb0e7bb3fba288001a7bb03fdb (diff) |
tty/serial/sirf: fix error propagation in sirfsoc_uart_probe()
If pinctrl_get_select_default() fails, sirfsoc_uart_probe()
returns IS_ERR(result) instead of PTR_ERR(result).
Found by Linux Driver Verification project (linuxtesting.org).
Signed-off-by: Alexey Khoroshilov <khoroshilov@ispras.ru>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/tty/serial')
-rw-r--r-- | drivers/tty/serial/sirfsoc_uart.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/tty/serial/sirfsoc_uart.c b/drivers/tty/serial/sirfsoc_uart.c index 03465b673945..1fd564b8194b 100644 --- a/drivers/tty/serial/sirfsoc_uart.c +++ b/drivers/tty/serial/sirfsoc_uart.c | |||
@@ -687,9 +687,10 @@ int sirfsoc_uart_probe(struct platform_device *pdev) | |||
687 | 687 | ||
688 | if (sirfport->hw_flow_ctrl) { | 688 | if (sirfport->hw_flow_ctrl) { |
689 | sirfport->p = pinctrl_get_select_default(&pdev->dev); | 689 | sirfport->p = pinctrl_get_select_default(&pdev->dev); |
690 | ret = IS_ERR(sirfport->p); | 690 | if (IS_ERR(sirfport->p)) { |
691 | if (ret) | 691 | ret = PTR_ERR(sirfport->p); |
692 | goto err; | 692 | goto err; |
693 | } | ||
693 | } | 694 | } |
694 | 695 | ||
695 | sirfport->clk = clk_get(&pdev->dev, NULL); | 696 | sirfport->clk = clk_get(&pdev->dev, NULL); |