From mboxrd@z Thu Jan 1 00:00:00 1970 From: Samuel Ortiz Subject: Re: [linux-nfc] [PATCH v3 9/9] NFC: nfcmrvl: update device tree bindings for Marvell NFC Date: Sat, 24 Oct 2015 01:00:34 +0200 Message-ID: <20151023230034.GA23609@zurbaran.home> References: <1445602986-3254-1-git-send-email-cuissard@marvell.com> <1445602986-3254-10-git-send-email-cuissard@marvell.com> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Vincent Cuissard Cc: Rob Herring , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-nfc-hn68Rpc1hR1g9hUCZPvPmw@public.gmane.org" List-Id: devicetree@vger.kernel.org Hi Vincent, On Fri, Oct 23, 2015 at 01:47:56PM +0000, Vincent Cuissard wrote: > Hi Rob, >=20 > > Le 23 oct. 2015 =E0 15:18, Rob Herring a =E9crit = : > >=20 > >> On Fri, Oct 23, 2015 at 7:23 AM, Vincent Cuissard wrote: > >> Align NFC bindgins to use marvell instead of mrvl. > >>=20 > >> Signed-off-by: Vincent Cuissard > >> --- > >> Documentation/devicetree/bindings/net/nfc/nfcmrvl.txt | 12 ++++++-= ----- > >> drivers/nfc/nfcmrvl/i2c.c | 2 +- > >> drivers/nfc/nfcmrvl/spi.c | 2 +- > >> drivers/nfc/nfcmrvl/uart.c | 2 +- > >> 4 files changed, 9 insertions(+), 9 deletions(-) > >>=20 > >> diff --git a/Documentation/devicetree/bindings/net/nfc/nfcmrvl.txt= b/Documentation/devicetree/bindings/net/nfc/nfcmrvl.txt > >> index 41058fc..0f95924 100644 > >> --- a/Documentation/devicetree/bindings/net/nfc/nfcmrvl.txt > >> +++ b/Documentation/devicetree/bindings/net/nfc/nfcmrvl.txt > >> @@ -2,9 +2,9 @@ > >>=20 > >> Required properties: > >> - compatible: Should be: > >> - - "mrvl,nfc-uart" for UART devices > >> - - "mrvl,nfc-i2c" for I2C devices > >> - - "mrvl,nfc-spi" for SPI devices > >> + - "marvell,nfc-uart" for UART devices > >> + - "marvell,nfc-i2c" for I2C devices > >> + - "marvell,nfc-spi" for SPI devices > >=20 > > There is no point in changing these and doing so breaks backwards c= ompatibility. >=20 > There is no break since these 2 drivers are new. In the same patchset= I first set mrvl and replace it to marvell since I figured out that th= is is the name for all other Marvell drivers. > I agree the i2c and spi drivers are new, but not the uart one. That one should be supported in a backward compatible way. Cheers, Samuel. -- To unsubscribe from this list: send the line "unsubscribe devicetree" i= n the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html