On 13 March 2012 15:17, Jingoo Han <jg1.han@xxxxxxxxxxx> wrote: >> -----Original Message----- >> From: Thomas Abraham [mailto:thomas.abraham@xxxxxxxxxx] >> Sent: Tuesday, March 13, 2012 6:00 AM >> To: linux-fbdev@xxxxxxxxxxxxxxx >> Cc: FlorianSchandinat@xxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-samsung-soc@xxxxxxxxxxxxxxx; >> kgene.kim@xxxxxxxxxxx; jg1.han@xxxxxxxxxxx; ben-linux@xxxxxxxxx; patches@xxxxxxxxxx; Kyungmin Park; >> JeongHyeon Kim; Heiko Stuebner; Kwangwoo Lee; Mark Brown; Peter Korsgaard; Darius Augulis; Maurus >> Cuelenaere >> Subject: [PATCH v2 3/3] ARM: Samsung: Rework platform data of s3c-fb driver >> >> For all the Samsung SoC based boards which have the platform data for >> s3c-fb driver, the 'default_win' element in the platform data is removed >> and the lcd panel video timing values are moved out of individual window >> configuration data. >> >> Cc: Jingoo Han <jg1.han@xxxxxxxxxxx> >> Cc: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> >> Cc: JeongHyeon Kim <jhkim@xxxxxxxxxxxxxx> >> Cc: Kukjin Kim <kgene.kim@xxxxxxxxxxx> >> Cc: Heiko Stuebner <heiko@xxxxxxxxx> >> Cc: Ben Dooks <ben-linux@xxxxxxxxx> >> Cc: Kwangwoo Lee <kwangwoo.lee@xxxxxxxxx> >> Cc: Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx> >> Cc: Peter Korsgaard <jacmet@xxxxxxxxxx> >> Cc: Darius Augulis <augulis.darius@xxxxxxxxx> >> Cc: Maurus Cuelenaere <mcuelenaere@xxxxxxxxx> >> Signed-off-by: Thomas Abraham <thomas.abraham@xxxxxxxxxx> >> --- >> arch/arm/mach-exynos/mach-nuri.c | 26 ++++++++++------- >> arch/arm/mach-exynos/mach-origen.c | 24 ++++++++++------- >> arch/arm/mach-exynos/mach-smdkv310.c | 28 +++++++++++-------- >> arch/arm/mach-exynos/mach-universal_c210.c | 26 ++++++++++------- >> arch/arm/mach-s3c24xx/mach-smdk2416.c | 27 ++++++++++-------- >> arch/arm/mach-s3c64xx/mach-anw6410.c | 25 ++++++++++------- >> arch/arm/mach-s3c64xx/mach-crag6410.c | 25 ++++++++++------- >> arch/arm/mach-s3c64xx/mach-hmt.c | 24 ++++++++++------- >> arch/arm/mach-s3c64xx/mach-mini6410.c | 40 ++++++++++++--------------- >> arch/arm/mach-s3c64xx/mach-real6410.c | 40 ++++++++++++--------------- >> arch/arm/mach-s3c64xx/mach-smartq5.c | 26 ++++++++++------- >> arch/arm/mach-s3c64xx/mach-smartq7.c | 26 ++++++++++------- >> arch/arm/mach-s3c64xx/mach-smdk6410.c | 25 ++++++++++------- >> arch/arm/mach-s5p64x0/mach-smdk6440.c | 24 ++++++++++------- >> arch/arm/mach-s5p64x0/mach-smdk6450.c | 24 ++++++++++------- >> arch/arm/mach-s5pc100/mach-smdkc100.c | 27 ++++++++++-------- >> arch/arm/mach-s5pv210/mach-aquila.c | 36 +++++++++++-------------- >> arch/arm/mach-s5pv210/mach-goni.c | 26 ++++++++++------- >> arch/arm/mach-s5pv210/mach-smdkv210.c | 24 ++++++++++------- >> 19 files changed, 285 insertions(+), 238 deletions(-) >> > > [.....] > >> diff --git a/arch/arm/mach-s3c64xx/mach-mini6410.c b/arch/arm/mach-s3c64xx/mach-mini6410.c >> index c34c2ab..24dcdc9 100644 >> --- a/arch/arm/mach-s3c64xx/mach-mini6410.c >> +++ b/arch/arm/mach-s3c64xx/mach-mini6410.c >> @@ -153,36 +153,32 @@ static struct s3c2410_platform_nand mini6410_nand_info = { >> >> static struct s3c_fb_pd_win mini6410_fb_win[] = { >> { >> - .win_mode = { /* 4.3" 480x272 */ >> - .left_margin = 3, >> - .right_margin = 2, >> - .upper_margin = 1, >> - .lower_margin = 1, >> - .hsync_len = 40, >> - .vsync_len = 1, >> - .xres = 480, >> - .yres = 272, >> - }, >> .max_bpp = 32, >> .default_bpp = 16, >> + .xres = 480, >> + .yres = 272, >> }, { >> - .win_mode = { /* 7.0" 800x480 */ >> - .left_margin = 8, >> - .right_margin = 13, >> - .upper_margin = 7, >> - .lower_margin = 5, >> - .hsync_len = 3, >> - .vsync_len = 1, >> - .xres = 800, >> - .yres = 480, >> - }, >> .max_bpp = 32, >> .default_bpp = 16, >> + .xres = 800, >> + .yres = 480, >> }, >> }; >> >> +static struct fb_videomode mini6410_lcd_timing = { >> + .left_margin = 8, >> + .right_margin = 13, >> + .upper_margin = 7, >> + .lower_margin = 5, >> + .hsync_len = 3, >> + .vsync_len = 1, >> + .xres = 800, >> + .yres = 480, >> +}; >> + >> static struct s3c_fb_platdata mini6410_lcd_pdata __initdata = { >> .setup_gpio = s3c64xx_fb_gpio_setup_24bpp, >> + .vtiming = &mini6410_lcd_timing, >> .win[0] = &mini6410_fb_win[0], >> .vidcon0 = VIDCON0_VIDOUT_RGB | VIDCON0_PNRMODE_RGB, >> .vidcon1 = VIDCON1_INV_HSYNC | VIDCON1_INV_VSYNC, >> @@ -310,8 +306,8 @@ static void __init mini6410_machine_init(void) >> mini6410_lcd_pdata.win[0] = &mini6410_fb_win[features.lcd_index]; >> >> printk(KERN_INFO "MINI6410: selected LCD display is %dx%d\n", >> - mini6410_lcd_pdata.win[0]->win_mode.xres, >> - mini6410_lcd_pdata.win[0]->win_mode.yres); >> + mini6410_lcd_pdata.win[0]->xres, >> + mini6410_lcd_pdata.win[0]->yres); >> >> s3c_nand_set_platdata(&mini6410_nand_info); >> s3c_fb_set_platdata(&mini6410_lcd_pdata); >> diff --git a/arch/arm/mach-s3c64xx/mach-real6410.c b/arch/arm/mach-s3c64xx/mach-real6410.c >> index be2a9a2..41e4f74 100644 >> --- a/arch/arm/mach-s3c64xx/mach-real6410.c >> +++ b/arch/arm/mach-s3c64xx/mach-real6410.c >> @@ -119,36 +119,32 @@ static struct platform_device real6410_device_eth = { >> >> static struct s3c_fb_pd_win real6410_fb_win[] = { >> { >> - .win_mode = { /* 4.3" 480x272 */ >> - .left_margin = 3, >> - .right_margin = 2, >> - .upper_margin = 1, >> - .lower_margin = 1, >> - .hsync_len = 40, >> - .vsync_len = 1, >> - .xres = 480, >> - .yres = 272, >> - }, >> .max_bpp = 32, >> .default_bpp = 16, >> + .xres = 480, >> + .yres = 272, >> }, { >> - .win_mode = { /* 7.0" 800x480 */ >> - .left_margin = 8, >> - .right_margin = 13, >> - .upper_margin = 7, >> - .lower_margin = 5, >> - .hsync_len = 3, >> - .vsync_len = 1, >> - .xres = 800, >> - .yres = 480, >> - }, >> .max_bpp = 32, >> .default_bpp = 16, >> + .xres = 800, >> + .yres = 480, >> }, >> }; >> >> +static struct fb_videomode real6410_lcd_timing = { >> + .left_margin = 3, >> + .right_margin = 2, >> + .upper_margin = 1, >> + .lower_margin = 1, >> + .hsync_len = 40, >> + .vsync_len = 1, >> + .xres = 800, >> + .yres = 480, >> +}; > > > What is the difference between mini6410_lcd_timing and real6410_lcd_timing? > In my opinion, it would be good as follows: > > +static struct fb_videomode real6410_lcd_timing = { > + .left_margin = 8, > + .right_margin = 13, > + .upper_margin = 7, > + .lower_margin = 5, > + .hsync_len = 3, > + .vsync_len = 1, > + .xres = 800, > + .yres = 480, > +}; > > Before this patch series, both real6410 and mini6410 had 'default_win' = 0 in the platform data. And, the s3c-fb driver selected the video timing from the window selected by the default_win parameter in s3c-fb platform data, i.e window 0 for both mini6440 and real6410. So, in this patch, while moving the timing values out of window data, the timing values for window 0 was selected. The timing value for window 1 was never used on mini6410 and real6410. So I would suggest to use timing value of window 0 in this patch. Thanks for your comments. Regards, Thomas. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html