Dear Tushar Behera, In message <1316076867-2138-1-git-send-email-tushar.behera@xxxxxxxxxx> you wrote: > ORIGEN board is fitted with 7" LCD panel HV070WSA. The pixel > resolution of the LCD panel is 1024x600. ... > +static struct s3c_fb_pd_win origen_fb_win0 = { > + .win_mode = { > + .left_margin = 64, > + .right_margin = 16, > + .upper_margin = 64, > + .lower_margin = 16, > + .hsync_len = 48, > + .vsync_len = 3, > + .xres = 1024, > + .yres = 600, > + }, > + .max_bpp = 32, > + .default_bpp = 24, > +}; Does it still make sense to hard-code such parameters? In PowerPC-land we pass display mode information in the device tree using a verbatim EDID block. Would it be not better (and way more flexible) to do the same here, now that ARM has device tree support? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@xxxxxxx The only way to get rid of a temptation is to yield to it. - Oscar Wilde -- 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