Re: [PATCH v2 3/3] ARM: Samsung: Rework platform data of s3c-fb driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux