From mboxrd@z Thu Jan 1 00:00:00 1970 From: jon@ringle.org Date: Wed, 05 Mar 2014 08:19:59 +0000 Subject: [PATCH] video: da8xx-fb: Fix casting of info->pseudo_palette Message-Id: <1394007599-31429-1-git-send-email-jon@ringle.org> List-Id: References: <1393836183-26555-1-git-send-email-jringle@gridpoint.com> In-Reply-To: <1393836183-26555-1-git-send-email-jringle@gridpoint.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Jon Ringle From: Jon Ringle (Resending without corporate disclaimer in email footer) The casting to (u16 *) on info->pseudo_palette is wrong and causes the display to show a blue (garbage) vertical line on every other pixel column Signed-off-by: Jon Ringle --- drivers/video/da8xx-fb.c | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/drivers/video/da8xx-fb.c b/drivers/video/da8xx-fb.c index d042624..83c43b2 100644 --- a/drivers/video/da8xx-fb.c +++ b/drivers/video/da8xx-fb.c @@ -678,15 +678,7 @@ static int fb_setcolreg(unsigned regno, unsigned red, unsigned green, (green << info->var.green.offset) | (blue << info->var.blue.offset); - switch (info->var.bits_per_pixel) { - case 16: - ((u16 *) (info->pseudo_palette))[regno] = v; - break; - case 24: - case 32: - ((u32 *) (info->pseudo_palette))[regno] = v; - break; - } + ((u32 *) (info->pseudo_palette))[regno] = v; if (palette[0] != 0x4000) { update_hw = 1; palette[0] = 0x4000; -- 1.8.5.4