Browse Source

serial: imx: simplify tracking of dma being initialized

The .dma_is_inited member is only set to a value != 0 when the port's
startup function calls imx_uart_dma_init(). On shutdown of the port
imx_uart_dma_exit is called which sets the value back to 0. So
.dma_is_inited is always 0 when imx_startup() is called (assuming
.startup() and .shutdown() are correctly balanced) and the check for
!sport->dma_is_inited can go away.

This allows to replace .dma_is_inited by a variable local to
imx_startup.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Reviewed-by: Shawn Guo <shawnguo@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Uwe Kleine-König 7 years ago
parent
commit
4238c00bb1
1 changed files with 4 additions and 8 deletions
  1. 4 8
      drivers/tty/serial/imx.c

+ 4 - 8
drivers/tty/serial/imx.c

@@ -205,7 +205,6 @@ struct imx_port {
 	struct mctrl_gpios *gpios;
 	struct mctrl_gpios *gpios;
 
 
 	/* DMA fields */
 	/* DMA fields */
-	unsigned int		dma_is_inited:1;
 	unsigned int		dma_is_enabled:1;
 	unsigned int		dma_is_enabled:1;
 	unsigned int		dma_is_rxing:1;
 	unsigned int		dma_is_rxing:1;
 	unsigned int		dma_is_txing:1;
 	unsigned int		dma_is_txing:1;
@@ -1114,8 +1113,6 @@ static void imx_uart_dma_exit(struct imx_port *sport)
 		dma_release_channel(sport->dma_chan_tx);
 		dma_release_channel(sport->dma_chan_tx);
 		sport->dma_chan_tx = NULL;
 		sport->dma_chan_tx = NULL;
 	}
 	}
-
-	sport->dma_is_inited = 0;
 }
 }
 
 
 static int imx_uart_dma_init(struct imx_port *sport)
 static int imx_uart_dma_init(struct imx_port *sport)
@@ -1168,8 +1165,6 @@ static int imx_uart_dma_init(struct imx_port *sport)
 		goto err;
 		goto err;
 	}
 	}
 
 
-	sport->dma_is_inited = 1;
-
 	return 0;
 	return 0;
 err:
 err:
 	imx_uart_dma_exit(sport);
 	imx_uart_dma_exit(sport);
@@ -1217,6 +1212,7 @@ static int imx_startup(struct uart_port *port)
 	struct imx_port *sport = (struct imx_port *)port;
 	struct imx_port *sport = (struct imx_port *)port;
 	int retval, i;
 	int retval, i;
 	unsigned long flags, temp;
 	unsigned long flags, temp;
+	int dma_is_inited = 0;
 
 
 	retval = clk_prepare_enable(sport->clk_per);
 	retval = clk_prepare_enable(sport->clk_per);
 	if (retval)
 	if (retval)
@@ -1241,8 +1237,8 @@ static int imx_startup(struct uart_port *port)
 	writel(temp & ~UCR4_DREN, sport->port.membase + UCR4);
 	writel(temp & ~UCR4_DREN, sport->port.membase + UCR4);
 
 
 	/* Can we enable the DMA support? */
 	/* Can we enable the DMA support? */
-	if (!uart_console(port) && !sport->dma_is_inited)
-		imx_uart_dma_init(sport);
+	if (!uart_console(port) && imx_uart_dma_init(sport) == 0)
+		dma_is_inited = 1;
 
 
 	spin_lock_irqsave(&sport->port.lock, flags);
 	spin_lock_irqsave(&sport->port.lock, flags);
 	/* Reset fifo's and state machines */
 	/* Reset fifo's and state machines */
@@ -1261,7 +1257,7 @@ static int imx_startup(struct uart_port *port)
 	writel(USR1_RTSD | USR1_DTRD, sport->port.membase + USR1);
 	writel(USR1_RTSD | USR1_DTRD, sport->port.membase + USR1);
 	writel(USR2_ORE, sport->port.membase + USR2);
 	writel(USR2_ORE, sport->port.membase + USR2);
 
 
-	if (sport->dma_is_inited && !sport->dma_is_enabled)
+	if (dma_is_inited && !sport->dma_is_enabled)
 		imx_enable_dma(sport);
 		imx_enable_dma(sport);
 
 
 	temp = readl(sport->port.membase + UCR1) & ~UCR1_RRDYEN;
 	temp = readl(sport->port.membase + UCR1) & ~UCR1_RRDYEN;