From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756531AbaJHNg0 (ORCPT ); Wed, 8 Oct 2014 09:36:26 -0400 Received: from mout.kundenserver.de ([212.227.17.13]:63890 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756235AbaJHNgY (ORCPT ); Wed, 8 Oct 2014 09:36:24 -0400 From: Arnd Bergmann To: Scott Branden Cc: linux-arm-kernel@lists.infradead.org, Christian Daudt , Matt Porter , Russell King , bcm-kernel-feedback-list@broadcom.com, Mike Turquette , Alex Elder , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Andrew Morton , "David S. Miller" , Greg Kroah-Hartman , Joe Perches , Mauro Carvalho Chehab , Antti Palosaari , devicetree@vger.kernel.org, Ray Jui , linux-kernel@vger.kernel.org, Jonathan Richardson , JD Zheng Subject: Re: [PATCH 3/6] dt-bindings: Document Broadcom Cygnus SoC and clock driver Date: Wed, 08 Oct 2014 15:35:27 +0200 Message-ID: <2797894.oaYXOOJrDl@wuerfel> User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; ) In-Reply-To: <543518C7.9070809@broadcom.com> References: <1412746025-11998-4-git-send-email-sbranden@broadcom.com> <2480056.Cs3dr8EM2e@wuerfel> <543518C7.9070809@broadcom.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V02:K0:8tadGmAJvUjgZUddziU0VHVfWBpvMQKmSX0vdc7qv7f 2jDZGkdHtLRGpG0gbRXTisHCog6A+nOIlNkdZMcYPCc4v0Bd9S oixJJInuOsv+KwZ67NxpwmJu85QIOA+/NPStH4eHMoHL1m9eXf q/e/HcjadLl/wsILQ0uOTFOkjDJkijmgim3h1EcsPfuB95AxZx zeInE2jY8xkbxawOsnfu/QTjVABuy0WKmRVq9O9Mii/Xw6tDjT vfok2r7KRM9LtJW+beL2OI1qNblvKg7Qg4DbhrVcMDckXt2t2R EdXCxOQjy9sdJLNeMT8icLFUPpaGgSRIKzJvgxfHNiS0Fc0NZE XWvl1Jy6DIATZnFLXvw8= X-UI-Out-Filterresults: notjunk:1; Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wednesday 08 October 2014 03:58:15 Scott Branden wrote: > On 14-10-08 12:57 AM, Arnd Bergmann wrote: > > On Tuesday 07 October 2014 22:27:02 Scott Branden wrote: > >> diff --git a/Documentation/devicetree/bindings/arm/cygnus.txt b/Documentation/devicetree/bindings/arm/cygnus.txt > >> new file mode 100644 > >> index 0000000..a210377 > >> --- /dev/null > >> +++ b/Documentation/devicetree/bindings/arm/cygnus.txt > >> @@ -0,0 +1,12 @@ > >> +Broadcom Cygnus device tree bindings > >> +------------------------------------ > >> + > >> +All Cygnus boards shall have the following properties: > >> + > >> +Required root node property: > >> + - compatible = "brcm,cygnus"; > > > > Can you add a more specific model number for the soc as well? > > I assume that cygnus refers to more than one model, so you > > should probably list both the family and the individual > > SoC type here. > Add specific model numbers does not seem necessary because they are not > needed to be specified in the device tree for compatible purposes for > the kernel to work. Plus, the list of model numbers will continue to > grow over time and compatible strings would need to be added to the > kernel for no apparent reason? It's always helpful to have them in the DT though, in case you ever need to add a workaround for one of the SoCs. You can change the description to just state the format of the more specific model number so you don't need to add the particular type to the binding, but I'd definitely recommend listing both a generic and a more specific name here, such as compatible = "brcm,bcm123456", "brcm,cygnus"; > >> +Boards variants shall have the following additional properties: > >> + > >> +Required root node property for the BCM911360_ENTPHN board: > >> + - compatible = "brcm,bcm911360_entphn"; > >> > > > > We normally don't list specific boards here, since it's hard to > > keep up. > Thanks - removed specific boards from file and moved the cygnus.txt file > to devicetree/bindings/arm/bcm/cygnus.txt in next version. This seems > to be like the appropriate place to put this document. We will need to > add back in board specific bindings when ALSA support is added. But > that will be a later commit to deal with. I believe most new alsa drivers can now use the binding from Documentation/devicetree/bindings/sound/simple-card.txt so you should no longer need a board specific driver or binding for those. Arnd