From: Simon Glass <sjg@chromium.org>
Date: Sat, 30 Jan 2016 22:45:16 +0000 (-0700)
Subject: video: Use 'int' for loop variables instead of short
X-Git-Tag: v2016.03-rc2~92^2~7
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=2b80b4e246e4f20aec84c15e4da229d07a4c6fad;p=oweals%2Fu-boot.git

video: Use 'int' for loop variables instead of short

Using short doesn't save anything and is confusing when the width and height
variables are ulong.

This may fix Coverity CID134902 but I doubt it.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Tom Rini <trini@konsulko.com>
---

diff --git a/drivers/video/video_bmp.c b/drivers/video/video_bmp.c
index c9075d62dd..fb7943e06d 100644
--- a/drivers/video/video_bmp.c
+++ b/drivers/video/video_bmp.c
@@ -194,7 +194,7 @@ int video_bmp_display(struct udevice *dev, ulong bmp_image, int x, int y,
 {
 	struct video_priv *priv = dev_get_uclass_priv(dev);
 	ushort *cmap_base = NULL;
-	ushort i, j;
+	int i, j;
 	uchar *fb;
 	struct bmp_image *bmp = map_sysmem(bmp_image, 0);
 	uchar *bmap;