On 13/05/2014 09:51, Thierry Reding wrote: > On Fri, May 09, 2014 at 04:16:40PM +0200, Boris BREZILLON wrote: >> Hello Thierry, >> >> I noticed you're describing each new panel with a new entry in the >> of_platform_match table and a new compatible string. >> I guess you have a good reason to do it this way, because retrieving >> panel description from DT would be pretty easy (see this series ;-)). >> >> Could tell me why you chose this approach ? > The reason is that devicetree mandates that a device be identified using > a compatible value and that compatible value should be as specific as > possible. That compatible value should give the device driver enough > information to know everything it needs (resolution, timings, physical > dimension). > > Having all of that data in the device tree is redundant. Okay, thanks for your answer. As a result, you'll see a patch adding support for the FL500WVR00-A0T (foxlink) panel soon ;-). Best Regards, Boris -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
