Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760509AbaGSJzJ (ORCPT ); Sat, 19 Jul 2014 05:55:09 -0400 Received: from mo4-p00-ob.smtp.rzone.de ([81.169.146.160]:13013 "EHLO mo4-p00-ob.smtp.rzone.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756654AbaGSJzG convert rfc822-to-8bit (ORCPT ); Sat, 19 Jul 2014 05:55:06 -0400 X-RZG-AUTH: :JGIXVUS7cutRB/49FwqZ7WcKdUCnXG6JabOfSXKWrat5gtPsyuCM X-RZG-CLASS-ID: mo00 Subject: Re: [PATCH 1/5] arm: dts: omap3-gta04: Add missing nodes to fully describe gta04 board Mime-Version: 1.0 (Apple Message framework v1085) Content-Type: text/plain; charset=us-ascii From: "Dr. H. Nikolaus Schaller" In-Reply-To: Date: Sat, 19 Jul 2014 11:54:31 +0200 Cc: Joachim Eastwood , Marek Belisko , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King - ARM Linux , Benoit Cousson , Tony Lindgren , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , linux-omap Content-Transfer-Encoding: 8BIT Message-Id: References: <1405369228-27197-1-git-send-email-marek@goldelico.com> <1405369228-27197-2-git-send-email-marek@goldelico.com> <0C2AB07A-C515-4D5E-A823-DDF7C871515A@goldelico.com> To: Javier Martinez Canillas X-Mailer: Apple Mail (2.1085) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi all, Am 18.07.2014 um 11:21 schrieb Javier Martinez Canillas: > Hello Marek and Dr. H. Nikolaus, > > On Fri, Jul 18, 2014 at 8:55 AM, Joachim Eastwood wrote: >> On 16 July 2014 09:17, Dr. H. Nikolaus Schaller wrote: >>> Am 15.07.2014 um 14:45 schrieb Joachim Eastwood: >>> >>>> Hi Marek, >>>> >>>> You seem to add some DT nodes for hw that doesn't have drivers in >>>> mainline. I think you should leave those out until the driver itself >>>> is upstream and the bindings for it is documented. >>> is there some policy for only having nodes for existing drivers in DT files? >> > > I strongly agree with Joachim on this regard. > >>> If I understand the device tree concept correctly, it should not describe drivers >>> (and hence nothing about the state of them being mainlined), but it should statically >>> describe the given hardware in a way that kernel and drivers can read it (or ignore). >>> And even other kernels can use it (because they run on the same hardware). >>> > > Yes, it should describe hardware but that description should be > previously agreed and properly documented as was mentioned before. Ok, it is a little a hen and egg problem - who should start with the documentation. The device tree file, more or less describing what the hardware requirements are. Or the drivers which describe what they support. And you are right - these two ends must match and that can only be resolved by discussions and negotiations between both ends. > >>> So unless there is an important reason not to have "currently unused" nodes >>> in the DT source/binary (loading time is IMHO neglectable), I would ask that we >>> can keep with the complete DT instead of splitting it up artificially and getting back >>> to the same status (because the hardware does not change over time). >> > > I understand your motivation since it will allow you to not have to > maintain a patch-set for your downstream DTS changes but I would like > to ask you the opposite question. What's the benefit for the kernel > community to keep in a mainline DTS a bunch of device nodes with DT > bindings that have not been neither discussed nor documented? Less patches to review individually. Just a single one instead. But this might be a weak benefit. > > Developers doing a board bring-up usually use the DTS in mainline as a > reference for their boards and having non-documented/agreed DT binding > on the upstream DTS will only bring confusion and frustration to them > IMHO. Well, my experience (at least for the current status) is that in most times new boards need new drivers and DT nodes anyways. But this might change. > > We already have some issues with Device Trees (bindings that broke > backward compatibility, drivers implementing custom DT binding instead > of using standard ones, bindings that don't really reflect the actual > H/W, etc), please don't make this even more complicated to developers. Ok, then I think we will do the way as suggested and only submit DT nodes for already existing drivers. Or submit new ones bundled with driver and documentation patches. Marek will update the patches anyways, so that they apply without errors. > > Thanks a lot and best regards, > Javier BR, Nikolaus -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/