From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomi Valkeinen Subject: Re: [pull] tilcdc-next for 3.9 Date: Mon, 18 Feb 2013 11:02:41 +0200 Message-ID: <5121EE31.7020807@iki.fi> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-lb0-f172.google.com (mail-lb0-f172.google.com [209.85.217.172]) by gabe.freedesktop.org (Postfix) with ESMTP id 446A7E5CBC for ; Mon, 18 Feb 2013 01:02:46 -0800 (PST) Received: by mail-lb0-f172.google.com with SMTP id n8so4113342lbj.3 for ; Mon, 18 Feb 2013 01:02:45 -0800 (PST) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: Rob Clark Cc: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org On 2013-02-18 01:02, Rob Clark wrote: > Hi Dave, > > Here is pull request for tilcdc drm driver.. it also includes a > handful of dependent patches from me, plus a couple fixes from Daniel > Vetter which haven't showed up yet in drm-next. Why is the TFP410 driver integrated into the tilcdc, but the TDA998x is a generic driver? I think the DT bindings do not match the DT guidelines. For example, compatible = "tilcdc,slave" should be "ti,tilcdc,slave". And TI specific properties should also be prepended with "ti,". Who is going to maintain this now that you're no longer in TI? I presume you may do some small stuff, but I think this driver needs quite a lot of further development in the future. But my main concern for this series is still that it creates custom panel stuff, and adds DT bindings for them. Which means we need to support those custom DT bindings in the future, even though it's quite sure that CDF should be used also for this driver, changing the bindings. Tomi