Hi Sekhar, On Mon, Oct 08, 2012 at 18:10:12, Nori, Sekhar wrote: > Hi Prakash, > > On 10/5/2012 7:21 PM, Manjunathappa, Prakash wrote: > > Moving panel information from driver to platform file, patch also made > > compliant to fb_videomode data. > > > > Signed-off-by: Manjunathappa, Prakash <prakash.pm@xxxxxx> > > Why do you have to do this? Just moving panel data from driver to > platform code doesn't seem to buy anything. > > If you are passed DT data, then use it else continue the existing > platform data method? Once all the boards using this driver are > converted to DT, then all the panel information can be removed from > driver. That will save code. > Because of following reasons I moved it out of driver 1)This patch also converts panel information compliant to fb_videomode. Patch "of: add display helper"[1] under review expects panel data in fb_videomode format. 2)I felt difficult and unclean to have driver supporting both panel data from driver and panel data from DT. 3)This effort will also ease adding DT support of this driver. [1]http://marc.info/?l=linux-fbdev&m=134937359209227&w=2 Thanks, Prakash > This patch looks like needless churn. > > Thanks, > Sekhar > -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
