On Thu, Aug 28, 2014 at 10:34:32AM +0100, Ajay kumar wrote: > On Thu, Aug 28, 2014 at 2:45 PM, Mark Rutland <mark.rutland@xxxxxxx> wrote: > > On Thu, Aug 28, 2014 at 06:34:33AM +0100, Ajay kumar wrote: > >> On Wed, Aug 27, 2014 at 8:31 PM, Mark Rutland <mark.rutland@xxxxxxx> wrote: > >> > On Wed, Aug 27, 2014 at 03:48:27PM +0100, Ajay Kumar wrote: > >> >> Add DT nodes for ptn3460 bridge chip and panel. > >> >> Add backlight enable pin and backlight power supply for pwm-backlight. > >> >> Also add bridge phandle needed by dp to enable display on snow. > >> >> > >> >> Note that, snow doesn't support "chunghwa,claa101wb01" panel, > >> >> but still we choose to reuse the binding since "chunghwa,claa101wb01" > >> >> has similar LCD timings. > >> > > >> > What does it actually have? > >> > > >> > It's fine to have "chunghwa,claa101wb01" as a fallback but we should > >> > have an identifier for the actual display, too. > >> Actual display used is "AUO,B116XTN0", for which no data currently exists > >> in panel file. Instead of adding a new panel_desc for B116XTN0, > >> why not we reuse existing ones? what is the "identifier" you are talking about? > > > > The compatible string. > > > > So in the dts we'd have compatible = "au0,b116xtn0", "chunghwa,claa101wb01"; > Sorry, hardware engineer says actual LCD name is "auo,b116xw03". Ok. > > That way if we need specific data we can add it later. > I think we can add a new panel_desc for "auo,b116xw03". > Because its actual LCD dimension is 256x144, but "chungwa, claa101wb01" has > LCD dimension 223x125. > What is Thierry's opinion on adding this new LCD? > > > What vendor is AU0? Can we add a vendor-prefix in parallel? > its "auo" (already present: AU Optronics Corporation) Ah, whoops. Mistook 'O' for '0'. Thanks, Mark. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html