Added spaces on the left side of parenthesis and on both sides of binary operators. Also realigned else and else if so it matches the parenthesis line. This refactoring makes the code more readable. Signed-off-by: Fabian Krueger <fabian.krueger@xxxxxx> Signed-off-by: Michael Scheiderer <michael.scheiderer@xxxxxx> Cc: <linux-kernel@xxxxxxxxxxxx> --- drivers/staging/kpc2000/kpc2000_spi.c | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/drivers/staging/kpc2000/kpc2000_spi.c b/drivers/staging/kpc2000/kpc2000_spi.c index b4dba0e42c72..d5b4bd7b2ea7 100644 --- a/drivers/staging/kpc2000/kpc2000_spi.c +++ b/drivers/staging/kpc2000/kpc2000_spi.c @@ -226,10 +226,9 @@ kp_spi_txrx_pio(struct spi_device *spidev, struct spi_transfer *transfer) kp_spi_write_reg(cs, KP_SPI_REG_TXDATA, val); processed++; } - } - else if(rx) { + } else if (rx) { for (i = 0 ; i < c ; i++) { - char test=0; + char test = 0; kp_spi_write_reg(cs, KP_SPI_REG_TXDATA, 0x00); res = kp_spi_wait_for_reg_bit(cs, KP_SPI_REG_STATUS, @@ -267,9 +266,8 @@ kp_spi_setup(struct spi_device *spidev) cs = spidev->controller_state; if (!cs) { cs = kzalloc(sizeof(*cs), GFP_KERNEL); - if(!cs) { + if (!cs) return -ENOMEM; - } cs->base = kpspi->base; cs->conf_cache = -1; spidev->controller_state = cs; @@ -429,7 +427,7 @@ kp_spi_probe(struct platform_device *pldev) int i; drvdata = pldev->dev.platform_data; - if (!drvdata){ + if (!drvdata) { dev_err(&pldev->dev, "kp_spi_probe: platform_data is NULL!\n"); return -ENODEV; } @@ -479,7 +477,7 @@ kp_spi_probe(struct platform_device *pldev) spi_new_device(master, &(table[i])); \ } - switch ((drvdata->card_id & 0xFFFF0000) >> 16){ + switch ((drvdata->card_id & 0xFFFF0000) >> 16) { case PCI_DEVICE_ID_DAKTRONICS_KADOKA_P2KR0: NEW_SPI_DEVICE_FROM_BOARD_INFO_TABLE(p2kr0_board_info); break; -- 2.17.1 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel