From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris BREZILLON Subject: Re: [RFC PATCH 0/2] drm/panel: add simple-panel description using DT Date: Fri, 16 May 2014 09:29:43 +0200 Message-ID: <5375BE67.8000103@free-electrons.com> References: <1399645002-18000-1-git-send-email-boris.brezillon@free-electrons.com> <20140513075148.GF6754@ulmo> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20140513075148.GF6754@ulmo> Sender: linux-doc-owner@vger.kernel.org To: Thierry Reding Cc: Randy Dunlap , David Airlie , Jean-Jacques Hiblot , Nicolas Ferre , dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: devicetree@vger.kernel.org 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