Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753250AbbBKPb3 (ORCPT ); Wed, 11 Feb 2015 10:31:29 -0500 Received: from mail-we0-f172.google.com ([74.125.82.172]:43696 "EHLO mail-we0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752448AbbBKPb1 (ORCPT ); Wed, 11 Feb 2015 10:31:27 -0500 From: Aurelien BOUIN To: gregkh@linuxfoundation.org, jslaby@suse.cz, sascha@saschahauer.de Cc: festevam@gmail.com, linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org, Aurelien BOUIN Subject: [PATCH 2/2] serial: imx: checkpatch cleanup Date: Wed, 11 Feb 2015 16:30:35 +0100 Message-Id: <1423668635-30838-2-git-send-email-a.bouin@gmail.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1423668635-30838-1-git-send-email-a.bouin@gmail.com> References: <1423668635-30838-1-git-send-email-a.bouin@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3974 Lines: 113 This patch solve various issue detected by checkpatch.pl Signed-off-by: Aurelien BOUIN diff --git a/drivers/tty/serial/imx.c b/drivers/tty/serial/imx.c index a086eef..63eb457 100644 --- a/drivers/tty/serial/imx.c +++ b/drivers/tty/serial/imx.c @@ -275,9 +275,12 @@ static struct platform_device_id imx_uart_devtype[] = { MODULE_DEVICE_TABLE(platform, imx_uart_devtype); static struct of_device_id imx_uart_dt_ids[] = { - { .compatible = "fsl,imx6q-uart", .data = &imx_uart_devdata[IMX6Q_UART], }, - { .compatible = "fsl,imx1-uart", .data = &imx_uart_devdata[IMX1_UART], }, - { .compatible = "fsl,imx21-uart", .data = &imx_uart_devdata[IMX21_UART], }, + { .compatible = "fsl,imx6q-uart", + .data = &imx_uart_devdata[IMX6Q_UART], }, + { .compatible = "fsl,imx1-uart", + .data = &imx_uart_devdata[IMX1_UART], }, + { .compatible = "fsl,imx21-uart", + .data = &imx_uart_devdata[IMX21_UART], }, { /* sentinel */ } }; MODULE_DEVICE_TABLE(of, imx_uart_dt_ids); @@ -430,6 +433,7 @@ static void imx_stop_tx(struct uart_port *port) if (USE_IRDA(sport)) { /* half duplex - wait for end of transmission */ int n = 256; + while ((--n > 0) && !(readl(sport->port.membase + USR2) & USR2_TXDC)) { udelay(5); @@ -621,7 +625,6 @@ static void imx_dma_tx(struct imx_port *sport) sport->dma_is_txing = 1; dmaengine_submit(desc); dma_async_issue_pending(chan); - return; } /* @@ -870,7 +873,8 @@ static unsigned int imx_tx_empty(struct uart_port *port) struct imx_port *sport = (struct imx_port *)port; unsigned int ret; - ret = (readl(sport->port.membase + USR2) & USR2_TXDC) ? TIOCSER_TEMT : 0; + ret = (readl(sport->port.membase + USR2) & USR2_TXDC) ? + TIOCSER_TEMT : 0; /* If the TX DMA is working, return 0. */ if (sport->dma_is_enabled && sport->dma_is_txing) @@ -1256,6 +1260,7 @@ static int imx_startup(struct uart_port *port) if (USE_IRDA(sport)) { struct imxuart_platform_data *pdata; + pdata = dev_get_platdata(sport->port.dev); sport->irda_inv_rx = pdata->irda_inv_rx; sport->irda_inv_tx = pdata->irda_inv_tx; @@ -1299,6 +1304,7 @@ static void imx_shutdown(struct uart_port *port) if (USE_IRDA(sport)) { struct imxuart_platform_data *pdata; + pdata = dev_get_platdata(sport->port.dev); if (pdata->irda_enable) pdata->irda_enable(0); @@ -1754,7 +1760,8 @@ imx_console_get_options(struct imx_port *sport, int *baud, ubir = readl(sport->port.membase + UBIR) & 0xffff; ubmr = readl(sport->port.membase + UBMR) & 0xffff; - ucfr_rfdiv = (readl(sport->port.membase + UFCR) & UFCR_RFDIV) >> 7; + ucfr_rfdiv = (readl(sport->port.membase + UFCR) & UFCR_RFDIV) + >> 7; if (ucfr_rfdiv == 6) ucfr_rfdiv = 7; else @@ -1763,12 +1770,13 @@ imx_console_get_options(struct imx_port *sport, int *baud, uartclk = clk_get_rate(sport->clk_per); uartclk /= ucfr_rfdiv; - { /* - * The next code provides exact computation of - * baud_raw = round(((uartclk/16) * (ubir + 1)) / (ubmr + 1)) - * without need of float support or long long division, - * which would be required to prevent 32bit arithmetic overflow - */ + { + /* + * The next code provides exact computation of + * baud_raw = round(((uartclk/16) * (ubir + 1)) / (ubmr + 1)) + * without need of float support or long long division, + * which would be required to prevent 32bit arithmetic overflow + */ unsigned int mul = ubir + 1; unsigned int div = 16 * (ubmr + 1); unsigned int rem = uartclk % div; @@ -1844,7 +1852,7 @@ static struct console imx_console = { .data = &imx_reg, }; -#define IMX_CONSOLE &imx_console +#define IMX_CONSOLE (&imx_console) #else #define IMX_CONSOLE NULL #endif -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/