Skip to contentSkip to conversation

Powerpc/DIU: Fixed the 800x600 and 1024x768 resolution bug

PATCH New Chang-Ming.Huang at freescale.com 3 participants 3 messages Oct 31, 2011 Framebuffer layer

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Powerpc/DIU: Fixed the 800x600 and 1024x768 resolution bug

From: Jerry Huang <[email protected]>

When the resolution is set to 800x600 and 1024x768,
but, the driver will use 1280x1024 resolution to set the DIU register

---
 drivers/video/fsl_diu_fb.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
drivers/video/fsl_diu_fb.c +2 -0
index e1d6c26..7cb8cb6 100644
@@ -296,8 +296,10 @@ int fsl_diu_init(u16 xres, u16 yres, u32 pixel_format, int gamma_fix)
 		break;
 	case RESOLUTION(800, 600):
 		fsl_diu_mode_db = &fsl_diu_mode_800_600;
+		break;
 	case RESOLUTION(1024, 768):
 		fsl_diu_mode_db = &fsl_diu_mode_1024_768;
+		break;
 	case RESOLUTION(1280, 1024):
 		fsl_diu_mode_db = &fsl_diu_mode_1280_1024;
 		break;
Signature · 1 line
1.7.4.1
#1 lore raw

PATCH 1/1 Powerpc/DIU: Fixed the 800x600 and 1024x768 resolution bug

Quoted 6 lines · On Mon, Oct 31, 2011 at 1:19 AM, <[email protected]> wrote:

From: Jerry Huang <[email protected]>

When the resolution is set to 800x600 and 1024x768,
but, the driver will use 1280x1024 resolution to set the DIU register

Thanks, Jerry. I'm amazed that I didn't notice this mistake earlier.

Signature · 2 lines
Timur Tabi
Linux kernel developer at Freescale
#2 lore raw

PATCH 1/1 Powerpc/DIU: Fixed the 800x600 and 1024x768 resolution bug

In reply to Chang-Ming.Huang at freescale.com
Quoted 9 lines · On Mon, 31 Oct 2011 14:19:54 +0800 <[email protected]> wrote:

From: Jerry Huang <[email protected]>

When the resolution is set to 800x600 and 1024x768,
but, the driver will use 1280x1024 resolution to set the DIU register

---
 drivers/video/fsl_diu_fb.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

Applied to u-boot-video/master, thanks!

Anatolij