Thanks, Vaibhav Hiremath
> -----Original Message----- > From: Tomi Valkeinen [mailto:[EMAIL PROTECTED] > Sent: Friday, November 14, 2008 9:24 PM > To: Hiremath, Vaibhav > Cc: [EMAIL PROTECTED]; linux- > [EMAIL PROTECTED] > Subject: RE: [REVIEW PATCH] Added OMAP3EVM support on Tomis > FBDEV/DSS Patches > > Hi, > On Fri, 2008-11-14 at 18:05 +0530, ext Hiremath, Vaibhav wrote: > > > > > > > I think the LCD panel on SDP and EVM boards are the same, so we > > > should > > > only have one driver. Do you have specifications about the LCD? > The > > > manufacturer, model etc?. I didn't find any proper information > about > > > the > > > LCD. > > > > > [Hiremath, Vaibhav] The LCD on OMAP3EVM is Sharp LCD (Model No. - > LS037V7DW01). > > Koen has already conformed that panel3430sdp.c is working with > OMAP3EVM, let me try at my end. If it works properly then we can > have same file to build for both SDP and EVM. But in that case it > make sense to rename file to panel-lcd.c. > > > > I took the liberty to add you EVM changes to my tree, with a comment > about the origin. I also changed the SDP panel name to sharp panel, > and > made both EVM and SDP use that. But I can't test the EVM, as I don't > have one. > [Hiremath, Vaibhav] It's ok; you can incorporate required changes from my patches, no issues at all. Please share the code base which you merged from my patch so that I can validate it on EVM, since you don't have EVM to test. > > > The #ifdef mode selection in DVI panel is quit hack, I agree. > The > > > mode > > > selection should be possible to be done runtime (or with kernel -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
