On Fri, Oct 23, 2015 at 7:23 AM, Vincent Cuissard <cuissard@xxxxxxxxxxx> wrote: > Align NFC bindgins to use marvell instead of mrvl. > > Signed-off-by: Vincent Cuissard <cuissard@xxxxxxxxxxx> > --- > 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(-) > > 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 @@ > > 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 There is no point in changing these and doing so breaks backwards compatibility. These bindings have a bigger issue in that it doesn't describe the hardware, but rather just something to bind the Linux driver. The compatible strings should be tied to what NFC chip you have (i.e. the part number). So either leave it alone or fix the bigger problem. Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
