From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933222AbcEQBmF (ORCPT ); Mon, 16 May 2016 21:42:05 -0400 Received: from host.buserror.net ([209.198.135.123]:33852 "EHLO host.buserror.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933190AbcEQBmB (ORCPT ); Mon, 16 May 2016 21:42:01 -0400 Message-ID: <1463449293.16584.127.camel@buserror.net> From: Scott Wood To: Qiang Zhao Cc: "robh+dt@kernel.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Xiaobo Xie , Yang-Leo Li , "linuxppc-dev@lists.ozlabs.org" Date: Mon, 16 May 2016 20:41:33 -0500 In-Reply-To: References: <1457486494-11377-1-git-send-email-qiang.zhao@nxp.com> <20160516232210.GA21087@home.buserror.net> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.16.5-1ubuntu3.1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-SA-Exim-Connect-IP: 75.72.173.242 X-SA-Exim-Mail-From: oss@buserror.net X-Spam-Report: * -1.0 ALL_TRUSTED Passed through trusted hosts only via SMTP * -15 BAYES_00 BODY: Bayes spam probability is 0 to 1% * [score: 0.0000] Subject: Re: [v5,1/7] QE: Add IC, SI and SIRAM document to device tree bindings. X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:57:07 +0000) X-SA-Exim-Scanned: Yes (on host.buserror.net) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 2016-05-17 at 01:18 +0000, Qiang Zhao wrote: > On Tue, May 17, 2016 at 07:22AM, Scott Wood wrote: > > -----Original Message----- > > From: Scott Wood [mailto:oss@buserror.net] > > Sent: Tuesday, May 17, 2016 7:22 AM > > To: Qiang Zhao > > Cc: robh+dt@kernel.org; devicetree@vger.kernel.org; linux- > > kernel@vger.kernel.org; Xiaobo Xie ; Yang-Leo Li > > ; linuxppc-dev@lists.ozlabs.org > > Subject: Re: [v5,1/7] QE: Add IC, SI and SIRAM document to device tree > > bindings. > > > > On Wed, Mar 09, 2016 at 09:21:28AM +0800, Zhao Qiang wrote: > > > Add IC, SI and SIRAM document of QE to > > > Documentation/devicetree/bindings/powerpc/fsl/cpm_qe/qe.txt > > > > > > Signed-off-by: Zhao Qiang > > > Acked-by: Rob Herring > > > --- > > > changes for v2 > > > - Add interrupt-controller in Required properties > > > - delete address-cells and size-cells for qe-si and qe-siram Changes > > > for v3 > > > - Add SoC specific caompatible strings to qe-si and qe-siram Changes > > > for v4 > > > - NA > > > Changes for v5 > > > - NA > > > > > > .../devicetree/bindings/powerpc/fsl/cpm_qe/qe.txt | 50 > > > ++++++++++++++++++++++ > > > 1 file changed, 50 insertions(+) > > > +* Serial Interface Block (SI) > > > + > > > +The SI manages the routing of eight TDM lines to the QE block serial > > > +drivers , the MCC and the UCCs, for receive and transmit. > > > + > > > +Required properties: > > > +- compatible : should be "fsl,t1040-qe-si". > > > +- reg : Address range of SI register set. > > > > Is t1040 the only chip that has or will ever have this? > > There also be t1024 and ls1043 supporting si. > I thought to add them when adding their device node. > If you think it is better to add them now, I will modify. The binding is saying that the compatible "should" be "fsl,t1040-qe-si" regardless of whether it's actually a t1040. Instead say that compatible must include "fsl,-qe-si" and give t1040 as an example. If you intend "fsl,t1040-qe-si" to be something that other compatible chips list, in addition to their own compatibles, then also say that explicitly. -Scott