Browse Source

backlight: ili922x: Remove 'else' after a return

Fixed the following checkpatch warning.

  WARNING: else is not generally useful after a break or return

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Jingoo Han 11 years ago
parent
commit
547f60cee1
1 changed files with 6 additions and 5 deletions
  1. 6 5
      drivers/video/backlight/ili922x.c

+ 6 - 5
drivers/video/backlight/ili922x.c

@@ -495,17 +495,18 @@ static int ili922x_probe(struct spi_device *spi)
 			"no LCD found: Chip ID 0x%x, ret %d\n",
 			"no LCD found: Chip ID 0x%x, ret %d\n",
 			reg, ret);
 			reg, ret);
 		return -ENODEV;
 		return -ENODEV;
-	} else {
-		dev_info(&spi->dev, "ILI%x found, SPI freq %d, mode %d\n",
-			 reg, spi->max_speed_hz, spi->mode);
 	}
 	}
 
 
+	dev_info(&spi->dev, "ILI%x found, SPI freq %d, mode %d\n",
+		 reg, spi->max_speed_hz, spi->mode);
+
 	ret = ili922x_read_status(spi, &reg);
 	ret = ili922x_read_status(spi, &reg);
 	if (ret) {
 	if (ret) {
 		dev_err(&spi->dev, "reading RS failed...\n");
 		dev_err(&spi->dev, "reading RS failed...\n");
 		return ret;
 		return ret;
-	} else
-		dev_dbg(&spi->dev, "status: 0x%x\n", reg);
+	}
+
+	dev_dbg(&spi->dev, "status: 0x%x\n", reg);
 
 
 	ili922x_display_init(spi);
 	ili922x_display_init(spi);