# \[PATCH v2\] video: vt8500lcdfb: remove unneeded continue

- Subsystems: FRAMEBUFFER LAYER, ARM/VT8500 ARM ARCHITECTURE, THE REST
- Maintainers: Helge Deller, Alexey Charkov, Krzysztof Kozlowski, Linus Torvalds
- List: linux-fbdev
- 2 messages, 2 authors
- Page: https://ratatoskr.run/linux-fbdev/2015/01/4404384/t

## \[PATCH v2\] video: vt8500lcdfb: remove unneeded continue

- From: Sudip Mukherjee \<sudipm.mukherjee@gmail.com\>
- Date: 2015-01-07 05:36:04 
- Message: https://ratatoskr.run/linux-fbdev/2015/01/4404384.md

```text
continue is not needed at the end of a for loop, also removed the
braces which were no longer required.

Signed-off-by: Sudip Mukherjee <redacted>
---

v2: removed braces

 drivers/video/fbdev/vt8500lcdfb.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/video/fbdev/vt8500lcdfb.c b/drivers/video/fbdev/vt8500lcdfb.c
index ffaf29e..1a1176b 100644
--- a/drivers/video/fbdev/vt8500lcdfb.c
+++ b/drivers/video/fbdev/vt8500lcdfb.c
@@ -113,10 +113,8 @@ static int vt8500lcd_set_par(struct fb_info *info)
 	}
 
 	for (i = 0; i < 8; i++) {
-		if (bpp_values[i] = info->var.bits_per_pixel) {
+		if (bpp_values[i] = info->var.bits_per_pixel)
 			reg_bpp = i;
-			continue;
-		}
 	}
 
 	control0 = readl(fbi->regbase) & ~0xf;
-- 
1.8.1.2

```

## Re: \[PATCH v2\] video: vt8500lcdfb: remove unneeded continue

- From: Tomi Valkeinen \<hidden\>
- Date: 2015-01-13 11:36:20 
- Message: https://ratatoskr.run/linux-fbdev/2015/01/4408660.md

```text
On 07/01/15 07:35, Sudip Mukherjee wrote:
> continue is not needed at the end of a for loop, also removed the
> braces which were no longer required.
> 

Thanks, queued for 3.20.

 Tomi



```
