lkml.org 
[lkml]   [2006]   [Jan]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] [MMC] Lindent wbsd driver
On Sun, Jan 08, 2006 at 12:17:48AM +0100, Pierre Ossman wrote:
> Fix the coding style in the wbsd driver once and for all.

> --- a/drivers/mmc/wbsd.c
> +++ b/drivers/mmc/wbsd.c
> @@ -61,21 +61,21 @@
> #ifdef CONFIG_PNP
>
> static const struct pnp_device_id pnp_dev_table[] = {
> - { "WEC0517", 0 },
> - { "WEC0518", 0 },
> - { "", 0 },
> + {"WEC0517", 0},
> + {"WEC0518", 0},
> + {"", 0},

Was OK.

> -#endif /* CONFIG_PNP */
> +#endif /* CONFIG_PNP */

Was OK.

> +static inline char *wbsd_kmap_sg(struct wbsd_host *host)
> {
> host->mapped_sg = kmap_atomic(host->cur_sg->page, KM_BIO_SRC_IRQ) +
> - host->cur_sg->offset;
> + host->cur_sg->offset;

Doesn't make sense.

> + for (i = 0; i < 4; i++) {
> cmd->resp[i] =
> - wbsd_read_index(host, WBSD_IDX_RESP1 + i * 4) << 24;
> + wbsd_read_index(host, WBSD_IDX_RESP1 + i * 4) << 24;
> cmd->resp[i] |=
> - wbsd_read_index(host, WBSD_IDX_RESP2 + i * 4) << 16;
> + wbsd_read_index(host, WBSD_IDX_RESP2 + i * 4) << 16;
> cmd->resp[i] |=
> - wbsd_read_index(host, WBSD_IDX_RESP3 + i * 4) << 8;
> + wbsd_read_index(host, WBSD_IDX_RESP3 + i * 4) << 8;
> cmd->resp[i] |=
> - wbsd_read_index(host, WBSD_IDX_RESP4 + i * 4) << 0;
> + wbsd_read_index(host, WBSD_IDX_RESP4 + i * 4) << 0;

Doesn't make sense.

> @@ -967,15 +924,15 @@ static void wbsd_request(struct mmc_host
> return;
> }
>
> -done:
> + done:

Was OK. GNU indent is broken wrt labels.

"Was OK" for all labels below.


> dma_unmap_single(host->mmc->dev, host->dma_addr, WBSD_DMA_SIZE,
> - DMA_BIDIRECTIONAL);
> - host->dma_addr = (dma_addr_t)NULL;
> + DMA_BIDIRECTIONAL);
> + host->dma_addr = (dma_addr_t) NULL;

Leave space after cast or not?

> @@ -2112,9 +2049,9 @@ static struct platform_driver wbsd_drive
>
> .suspend = wbsd_platform_suspend,
> .resume = wbsd_platform_resume,
> - .driver = {
> - .name = DRIVER_NAME,
> - },
> + .driver = {
> + .name = DRIVER_NAME,
> + },

Was OK.

-
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/

\
 
 \ /
  Last update: 2006-01-08 02:14    [W:0.306 / U:0.508 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site