Commit 2c58e560 authored by Erwan Le Ray's avatar Erwan Le Ray Committed by Greg Kroah-Hartman

serial: stm32: fix the get_irq error case

Checks the returned values of platform_get_irq() for both required
"event" and optional "wakeup" IRQs during probe. This allows the driver
probe to be deferred if needed.
Removes redundant checks for 'cfg.has_wakeup'.
Signed-off-by: default avatarFabien Dessenne <fabien.dessenne@st.com>
Signed-off-by: default avatarErwan Le Ray <erwan.leray@st.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 5297f274
...@@ -852,12 +852,31 @@ static int stm32_init_port(struct stm32_port *stm32port, ...@@ -852,12 +852,31 @@ static int stm32_init_port(struct stm32_port *stm32port,
port->flags = UPF_BOOT_AUTOCONF; port->flags = UPF_BOOT_AUTOCONF;
port->ops = &stm32_uart_ops; port->ops = &stm32_uart_ops;
port->dev = &pdev->dev; port->dev = &pdev->dev;
port->irq = platform_get_irq(pdev, 0);
ret = platform_get_irq(pdev, 0);
if (ret <= 0) {
if (ret != -EPROBE_DEFER)
dev_err(&pdev->dev, "Can't get event IRQ: %d\n", ret);
return ret ? ret : -ENODEV;
}
port->irq = ret;
port->rs485_config = stm32_config_rs485; port->rs485_config = stm32_config_rs485;
stm32_init_rs485(port, pdev); stm32_init_rs485(port, pdev);
stm32port->wakeirq = platform_get_irq(pdev, 1); if (stm32port->info->cfg.has_wakeup) {
stm32port->wakeirq = platform_get_irq(pdev, 1);
if (stm32port->wakeirq <= 0 && stm32port->wakeirq != -ENXIO) {
if (stm32port->wakeirq != -EPROBE_DEFER)
dev_err(&pdev->dev,
"Can't get event wake IRQ: %d\n",
stm32port->wakeirq);
return stm32port->wakeirq ? stm32port->wakeirq :
-ENODEV;
}
}
stm32port->fifoen = stm32port->info->cfg.has_fifo; stm32port->fifoen = stm32port->info->cfg.has_fifo;
res = platform_get_resource(pdev, IORESOURCE_MEM, 0); res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
...@@ -1064,7 +1083,7 @@ static int stm32_serial_probe(struct platform_device *pdev) ...@@ -1064,7 +1083,7 @@ static int stm32_serial_probe(struct platform_device *pdev)
if (ret) if (ret)
return ret; return ret;
if (stm32port->info->cfg.has_wakeup && stm32port->wakeirq >= 0) { if (stm32port->wakeirq > 0) {
ret = device_init_wakeup(&pdev->dev, true); ret = device_init_wakeup(&pdev->dev, true);
if (ret) if (ret)
goto err_uninit; goto err_uninit;
...@@ -1094,11 +1113,11 @@ static int stm32_serial_probe(struct platform_device *pdev) ...@@ -1094,11 +1113,11 @@ static int stm32_serial_probe(struct platform_device *pdev)
return 0; return 0;
err_wirq: err_wirq:
if (stm32port->info->cfg.has_wakeup && stm32port->wakeirq >= 0) if (stm32port->wakeirq > 0)
dev_pm_clear_wake_irq(&pdev->dev); dev_pm_clear_wake_irq(&pdev->dev);
err_nowup: err_nowup:
if (stm32port->info->cfg.has_wakeup && stm32port->wakeirq >= 0) if (stm32port->wakeirq > 0)
device_init_wakeup(&pdev->dev, false); device_init_wakeup(&pdev->dev, false);
err_uninit: err_uninit:
...@@ -1112,7 +1131,6 @@ static int stm32_serial_remove(struct platform_device *pdev) ...@@ -1112,7 +1131,6 @@ static int stm32_serial_remove(struct platform_device *pdev)
struct uart_port *port = platform_get_drvdata(pdev); struct uart_port *port = platform_get_drvdata(pdev);
struct stm32_port *stm32_port = to_stm32_port(port); struct stm32_port *stm32_port = to_stm32_port(port);
struct stm32_usart_offsets *ofs = &stm32_port->info->ofs; struct stm32_usart_offsets *ofs = &stm32_port->info->ofs;
struct stm32_usart_config *cfg = &stm32_port->info->cfg;
stm32_clr_bits(port, ofs->cr3, USART_CR3_DMAR); stm32_clr_bits(port, ofs->cr3, USART_CR3_DMAR);
...@@ -1134,7 +1152,7 @@ static int stm32_serial_remove(struct platform_device *pdev) ...@@ -1134,7 +1152,7 @@ static int stm32_serial_remove(struct platform_device *pdev)
TX_BUF_L, stm32_port->tx_buf, TX_BUF_L, stm32_port->tx_buf,
stm32_port->tx_dma_buf); stm32_port->tx_dma_buf);
if (cfg->has_wakeup && stm32_port->wakeirq >= 0) { if (stm32_port->wakeirq > 0) {
dev_pm_clear_wake_irq(&pdev->dev); dev_pm_clear_wake_irq(&pdev->dev);
device_init_wakeup(&pdev->dev, false); device_init_wakeup(&pdev->dev, false);
} }
...@@ -1252,7 +1270,7 @@ static void stm32_serial_enable_wakeup(struct uart_port *port, bool enable) ...@@ -1252,7 +1270,7 @@ static void stm32_serial_enable_wakeup(struct uart_port *port, bool enable)
struct stm32_usart_config *cfg = &stm32_port->info->cfg; struct stm32_usart_config *cfg = &stm32_port->info->cfg;
u32 val; u32 val;
if (!cfg->has_wakeup || stm32_port->wakeirq < 0) if (stm32_port->wakeirq <= 0)
return; return;
if (enable) { if (enable) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment