Received: by 2002:a25:824b:0:0:0:0:0 with SMTP id d11csp3618798ybn; Fri, 27 Sep 2019 08:52:46 -0700 (PDT) X-Google-Smtp-Source: APXvYqzlJVbUbKehpA0BGTjSvZVdwD/uveJhNHfxWHzuK00qJQ5C6he/Q9liHdC9FzcS+kAh9yKD X-Received: by 2002:aa7:cf81:: with SMTP id z1mr5358172edx.224.1569599566269; Fri, 27 Sep 2019 08:52:46 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1569599566; cv=none; d=google.com; s=arc-20160816; b=mBOTOSYVgOrdIixiGr5arJ8YdNzi6OO2icZBSHuc+vaJCGBisfPUR10pSh25ymdSWw RrnSmVWP/y9TQO8l8mAgzIcuXtKzFykBc0RoYQHmzof+MiR6UmR1P6Xq2cBmlo/Mx0V4 sZtifccBI3XfK6vqB3Z93F3/QODmOt3H4L8SyfA6sBNfJJeElgupmd2YEFUX9Dxb5Oiu VuwEk4O//RCl9G4BOi30t7uAwGkKFjV77e4P/8Ysby1wA6OtrR/MZ9nAzIxrHU/tDiy4 BaeAahThdN4BOynvLDkZWomjvAlItjvGLaF/RLk5qPlgHRn66NfbW86InM8Tf3ask9Bd XWgg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-transfer-encoding:content-disposition:mime-version :references:message-id:subject:cc:to:from:date; bh=2JNqIZKp6GqxqA7xiR7ISWAW++PU7T0DhHZ/3EtL4cQ=; b=KNKfj3YkqF67oJpyqbAs+qk2+2lowLjTR0WnI3lqr8rjoEitar9Jjeng++Lx2g4q9z w1s7qH+65Uhebm93kqX0WVGOsy8TjVCoQTx1Pu/RvN2Av6f/ZdXkfUpMkICcItrwo+nh mnWhK+cSHJ3huUr1yRLqQ+a8vC1a9+wrdHb0mnG7Ykn1ZDZITIVP166uXgMGD8zfblJb zKTrOD3ghjwDROJlzJSiqvc6ft+KjZ19lJSorJzZTU0i6ApZSbYzTA32+uKrBX9X7A3F y8mMVNHDjaQ5gAAmtOjWO+CzygiraUVHZrsGf9IP/gm2ZixKMPb6pJu/APTZtFU60jx3 srJA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id cw1si3314125ejb.117.2019.09.27.08.52.20; Fri, 27 Sep 2019 08:52:46 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727800AbfI0PwN (ORCPT + 99 others); Fri, 27 Sep 2019 11:52:13 -0400 Received: from mail-oi1-f196.google.com ([209.85.167.196]:40873 "EHLO mail-oi1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727349AbfI0PwN (ORCPT ); Fri, 27 Sep 2019 11:52:13 -0400 Received: by mail-oi1-f196.google.com with SMTP id k9so5594746oib.7; Fri, 27 Sep 2019 08:52:12 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to:user-agent; bh=2JNqIZKp6GqxqA7xiR7ISWAW++PU7T0DhHZ/3EtL4cQ=; b=s9VnHW/sZvyPiYUEub1njioKpkZ+qK5A0l6+04mXxmiLY6turXUd/ay9FPSTM4kOPt tEBbbNQSO6VoifGsiKWni1gOjBFIjpD2ETEhgO/9eSbA3Rr3yjsfxXKgibps+F2rL02s Lfc423tOcWdRJmBHBmyYRV49F3i1T2eNNOaTxphIzzcTdkNKpr5raz4jtZ4yDevmtN8v jGEFPhnXYqiQwzoOfVKAKWsRPw+WFFZaKqiGujSVRtVVJFqvqsOa8rj1GSUyO+3RWZ2Z uXgNijGc6f2qhkTgXlUd0utJfytI/deOJEdwwok51na5r7D4GgKJ5lfo1JfAb8taw06R sSRQ== X-Gm-Message-State: APjAAAWZA2IkegYg0hlWbpobzxY4VfUh3rvtqlDNVB8/7EALNf78pUmB BGEbNidAOMWhLd5BHRQ1Jg== X-Received: by 2002:aca:6057:: with SMTP id u84mr7898632oib.29.1569599532015; Fri, 27 Sep 2019 08:52:12 -0700 (PDT) Received: from localhost (24-155-109-49.dyn.grandenetworks.net. [24.155.109.49]) by smtp.gmail.com with ESMTPSA id r26sm1692866oij.46.2019.09.27.08.52.10 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 27 Sep 2019 08:52:11 -0700 (PDT) Date: Fri, 27 Sep 2019 10:52:09 -0500 From: Rob Herring To: Lars Poeschel Cc: "David S. Miller" , Mark Rutland , "open list:NFC SUBSYSTEM" , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , open list , Johan Hovold , Simon Horman Subject: Re: [PATCH v8 2/7] nfc: pn532: Add uart phy docs and rename it Message-ID: <20190927155209.GA6261@bogus> References: <20190919091645.16439-1-poeschel@lemonage.de> <20190919091645.16439-2-poeschel@lemonage.de> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20190919091645.16439-2-poeschel@lemonage.de> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Sep 19, 2019 at 11:16:39AM +0200, Lars Poeschel wrote: > This adds documentation about the uart phy to the pn532 binding doc. As > the filename "pn533-i2c.txt" is not appropriate any more, rename it to > the more general "pn532.txt". > This also documents the deprecation of the compatible strings ending > with "...-i2c". > > Cc: Johan Hovold > Cc: Simon Horman > Signed-off-by: Lars Poeschel > --- > Changes in v8: > - Update existing binding doc instead of adding a new one: > - Add uart phy example > - Add general "pn532" compatible string > - Deprecate "...-i2c" compatible strings > - Rename file to a more general filename > - Intentionally drop Rob's Reviewed-By as I guess this rather big change > requires a new review > > Changes in v7: > - Accidentally lost Rob's Reviewed-By > > Changes in v6: > - Rebased the patch series on v5.3-rc5 > - Picked up Rob's Reviewed-By > > Changes in v4: > - Add documentation about reg property in case of i2c > > Changes in v3: > - seperate binding doc instead of entry in trivial-devices.txt > > .../devicetree/bindings/net/nfc/pn532.txt | 46 +++++++++++++++++++ > .../devicetree/bindings/net/nfc/pn533-i2c.txt | 29 ------------ > 2 files changed, 46 insertions(+), 29 deletions(-) > create mode 100644 Documentation/devicetree/bindings/net/nfc/pn532.txt > delete mode 100644 Documentation/devicetree/bindings/net/nfc/pn533-i2c.txt In the future, use '-M' option (I recommend making this the default). > > diff --git a/Documentation/devicetree/bindings/net/nfc/pn532.txt b/Documentation/devicetree/bindings/net/nfc/pn532.txt > new file mode 100644 > index 000000000000..f0591f160bee > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/nfc/pn532.txt > @@ -0,0 +1,46 @@ > +* NXP Semiconductors PN532 NFC Controller > + > +Required properties: > +- compatible: Should be > + - "nxp,pn532" Place a node with this inside the devicetree node of the bus > + where the NFC chip is connected to. > + Currently the kernel has phy bindings for uart and i2c. > + - "nxp,pn532-i2c" (DEPRECATED) only works for the i2c binding. > + - "nxp,pn533-i2c" (DEPRECATED) only works for the i2c binding. No more pm533 support? > + > +Required properties if connected on i2c: > +- clock-frequency: I?C work frequency. > +- reg: for the I?C bus address. This is fixed at 0x24 for the PN532. > +- interrupts: GPIO interrupt to which the chip is connected UART attached case has no irq? I guess it could just start sending data... > + > +Optional SoC Specific Properties: > +- pinctrl-names: Contains only one value - "default". > +- pintctrl-0: Specifies the pin control groups used for this controller. > + > +Example (for ARM-based BeagleBone with PN532 on I2C2): > + > +&i2c2 { > + > + > + pn532: pn532@24 { nfc@24 > + > + compatible = "nxp,pn532"; > + > + reg = <0x24>; > + clock-frequency = <400000>; > + > + interrupt-parent = <&gpio1>; > + interrupts = <17 IRQ_TYPE_EDGE_FALLING>; > + > + }; > +}; > + > +Example (for PN532 connected via uart): > + > +uart4: serial@49042000 { > + compatible = "ti,omap3-uart"; > + > + pn532: nfc { > + compatible = "nxp,pn532"; > + }; > +};