On Fri, Mar 16, 2012 at 11:48 AM, Jingoo Han <jg1.han@xxxxxxxxxxx> wrote: >> > It's just bug. >> > >> > struct s3c_fb_pd_win should be used for windows of FIMD IP. >> > It should not be used for selecting multi-LCDs. >> >> why not? s3c_fb_pd_win contains timing values which depend directly on >> LCD size and resolution. >> Please look into the code again - mini6410 doesn't use platform data >> so select different LCD sizes. >> It has only single window in platform data - exactly what you want. >> Now tell my, why you are arguing, that this platform data cannot be >> rewritten dynamically by board setup routines at kernel startup time? >> Many ARM architectures are doing the same - takes kernel command line >> argument to select LCD and passes necessary data for its fb drivers >> via platform data structures. It not a bug, it's correct approach to >> support different sizes of LCD display for the same board. > > > Hey, it is not my point. > To take kernel command line is not problem to support multiple LCDs. > > My point is that s3c_fb_pd_win should not include variable LCD option. > So, if you want to support multiple LCDs, you should use another variable, > instead of using s3c_fb_pd_win[0] and s3c_fb_pd_win[1] as 4.3" LCD, 7.0" LCD. But I use s3c_fb_pd_win only as container to hold timing values in board setup code. s3c-fb platform data has only single window and does not perform any LCD selection. What is wrong here? You suggest to declare some custom structure to hold these timing values? What's the point? > > You use s3c_fb_pd_win[0], s3c_fb_pd_win[1] as 4.3" LCD, 7.0" LCD. > However, s3c_fb_pd_win[0], s3c_fb_pd_win[1] means the windows of FIMD IP. There is no multiple windows in platform data! driver sees only single window. And this wont change in any way if you drop LCD selection from board setup code. Driver will see the same single window. So - your are arguing, that board setup code sets platform data in bad way. I don't accept your opinion, because tens of boards are doing the same and nobody call it bug. > -- 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
 |