Re: [PATCH] usb: chipidea: fix device tree binding for zevio/nspire usb driver
On Thu, Dec 05, 2013 at 05:37:01PM +1100, dt.ta...@gmail.com wrote: > From: Daniel Tang > > The device tree binding chosen for the nspire-usb driver was inappropriate and > should be renamed to lsi,zevio-usb. > > References to nspire have been replaced with zevio (the SoC name) > > Signed-off-by: Daniel Tang > --- > .../devicetree/bindings/usb/ci-hdrc-nspire.txt | 17 - > .../devicetree/bindings/usb/ci-hdrc-zevio.txt | 17 + > drivers/usb/chipidea/Makefile |2 +- > drivers/usb/chipidea/ci_hdrc_nspire.c | 72 > > drivers/usb/chipidea/ci_hdrc_zevio.c | 72 > > 5 files changed, 90 insertions(+), 90 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/usb/ci-hdrc-nspire.txt > create mode 100644 Documentation/devicetree/bindings/usb/ci-hdrc-zevio.txt > delete mode 100644 drivers/usb/chipidea/ci_hdrc_nspire.c > create mode 100644 drivers/usb/chipidea/ci_hdrc_zevio.c > > diff --git a/Documentation/devicetree/bindings/usb/ci-hdrc-nspire.txt > b/Documentation/devicetree/bindings/usb/ci-hdrc-nspire.txt > deleted file mode 100644 > index ef1fcbf..000 > --- a/Documentation/devicetree/bindings/usb/ci-hdrc-nspire.txt > +++ /dev/null > @@ -1,17 +0,0 @@ > -* TI-Nspire USB OTG Controller > - > -Required properties: > -- compatible: Should be "lsi,nspire-usb" > -- reg: Should contain registers location and length > -- interrupts: Should contain controller interrupt > - > -Recommended properies: > -- vbus-supply: regulator for vbus > - > -Examples: > - usb0: usb@B000 { > - reg = <0xB000 0x1000>; > - compatible = "lsi,nspire-usb"; > - interrupts = <8>; > - vbus-supply = <&vbus_reg>; > - }; > diff --git a/Documentation/devicetree/bindings/usb/ci-hdrc-zevio.txt > b/Documentation/devicetree/bindings/usb/ci-hdrc-zevio.txt > new file mode 100644 > index 000..b6c3b5c > --- /dev/null > +++ b/Documentation/devicetree/bindings/usb/ci-hdrc-zevio.txt > @@ -0,0 +1,17 @@ > +* TI-Nspire USB OTG Controller > + TI-Nspire is a platform which uses zevio SoC, correct? So, please do not use TI-Nspire above. Peter > +Required properties: > +- compatible: Should be "lsi,zevio-usb" > +- reg: Should contain registers location and length > +- interrupts: Should contain controller interrupt > + > +Recommended properies: > +- vbus-supply: regulator for vbus > + > +Examples: > + usb0: usb@B000 { > + reg = <0xB000 0x1000>; > + compatible = "lsi,zevio-usb"; > + interrupts = <8>; > + vbus-supply = <&vbus_reg>; > + }; > diff --git a/drivers/usb/chipidea/Makefile b/drivers/usb/chipidea/Makefile > index 245ea4d..7635407 100644 > --- a/drivers/usb/chipidea/Makefile > +++ b/drivers/usb/chipidea/Makefile > @@ -10,7 +10,7 @@ ci_hdrc-$(CONFIG_USB_CHIPIDEA_DEBUG)+= debug.o > # Glue/Bridge layers go here > > obj-$(CONFIG_USB_CHIPIDEA) += ci_hdrc_msm.o > -obj-$(CONFIG_USB_CHIPIDEA) += ci_hdrc_nspire.o > +obj-$(CONFIG_USB_CHIPIDEA) += ci_hdrc_zevio.o > > # PCI doesn't provide stubs, need to check > ifneq ($(CONFIG_PCI),) > diff --git a/drivers/usb/chipidea/ci_hdrc_nspire.c > b/drivers/usb/chipidea/ci_hdrc_nspire.c > deleted file mode 100644 > index c5c2dde..000 > --- a/drivers/usb/chipidea/ci_hdrc_nspire.c > +++ /dev/null > @@ -1,72 +0,0 @@ > -/* > - * Copyright (C) 2013 Daniel Tang > - * > - * This program is free software; you can redistribute it and/or modify > - * it under the terms of the GNU General Public License version 2, as > - * published by the Free Software Foundation. > - * > - * Based off drivers/usb/chipidea/ci_hdrc_msm.c > - * > - */ > - > -#include > -#include > -#include > -#include > - > -#include "ci.h" > - > -static struct ci_hdrc_platform_data ci_hdrc_nspire_platdata = { > - .name = "ci_hdrc_nspire", > - .flags = CI_HDRC_REGS_SHARED, > - .capoffset = DEF_CAPOFFSET, > -}; > - > -static int ci_hdrc_nspire_probe(struct platform_device *pdev) > -{ > - struct platform_device *ci_pdev; > - > - dev_dbg(&pdev->dev, "ci_hdrc_nspire_probe\n"); > - > - ci_pdev = ci_hdrc_add_device(&pdev->dev, > - pdev->resource, pdev->num_resources, > - &ci_hdrc_nspire_platdata); > - > - if (IS_ERR(ci_pdev)) { > - dev_err(&pdev->dev, "ci_hdrc_add_device failed!\n"); > - return PTR_ERR(ci_pdev); > - } > - > - platform_set_drvdata(pdev, ci_pdev); > - > - return 0; > -} > - > -static int ci_hdrc_nspire_remove(struct platform_device *pdev) > -{ > - struct platform_device *ci_pdev = platform_get_drvdata(pdev); > - > - ci_hdrc_remove_device(ci_pdev); > - > - return 0; > -} > - > -static const
[PATCH] usb: chipidea: fix device tree binding for zevio/nspire usb driver
From: Daniel Tang The device tree binding chosen for the nspire-usb driver was inappropriate and should be renamed to lsi,zevio-usb. References to nspire have been replaced with zevio (the SoC name) Signed-off-by: Daniel Tang --- .../devicetree/bindings/usb/ci-hdrc-nspire.txt | 17 - .../devicetree/bindings/usb/ci-hdrc-zevio.txt | 17 + drivers/usb/chipidea/Makefile |2 +- drivers/usb/chipidea/ci_hdrc_nspire.c | 72 drivers/usb/chipidea/ci_hdrc_zevio.c | 72 5 files changed, 90 insertions(+), 90 deletions(-) delete mode 100644 Documentation/devicetree/bindings/usb/ci-hdrc-nspire.txt create mode 100644 Documentation/devicetree/bindings/usb/ci-hdrc-zevio.txt delete mode 100644 drivers/usb/chipidea/ci_hdrc_nspire.c create mode 100644 drivers/usb/chipidea/ci_hdrc_zevio.c diff --git a/Documentation/devicetree/bindings/usb/ci-hdrc-nspire.txt b/Documentation/devicetree/bindings/usb/ci-hdrc-nspire.txt deleted file mode 100644 index ef1fcbf..000 --- a/Documentation/devicetree/bindings/usb/ci-hdrc-nspire.txt +++ /dev/null @@ -1,17 +0,0 @@ -* TI-Nspire USB OTG Controller - -Required properties: -- compatible: Should be "lsi,nspire-usb" -- reg: Should contain registers location and length -- interrupts: Should contain controller interrupt - -Recommended properies: -- vbus-supply: regulator for vbus - -Examples: - usb0: usb@B000 { - reg = <0xB000 0x1000>; - compatible = "lsi,nspire-usb"; - interrupts = <8>; - vbus-supply = <&vbus_reg>; - }; diff --git a/Documentation/devicetree/bindings/usb/ci-hdrc-zevio.txt b/Documentation/devicetree/bindings/usb/ci-hdrc-zevio.txt new file mode 100644 index 000..b6c3b5c --- /dev/null +++ b/Documentation/devicetree/bindings/usb/ci-hdrc-zevio.txt @@ -0,0 +1,17 @@ +* TI-Nspire USB OTG Controller + +Required properties: +- compatible: Should be "lsi,zevio-usb" +- reg: Should contain registers location and length +- interrupts: Should contain controller interrupt + +Recommended properies: +- vbus-supply: regulator for vbus + +Examples: + usb0: usb@B000 { + reg = <0xB000 0x1000>; + compatible = "lsi,zevio-usb"; + interrupts = <8>; + vbus-supply = <&vbus_reg>; + }; diff --git a/drivers/usb/chipidea/Makefile b/drivers/usb/chipidea/Makefile index 245ea4d..7635407 100644 --- a/drivers/usb/chipidea/Makefile +++ b/drivers/usb/chipidea/Makefile @@ -10,7 +10,7 @@ ci_hdrc-$(CONFIG_USB_CHIPIDEA_DEBUG) += debug.o # Glue/Bridge layers go here obj-$(CONFIG_USB_CHIPIDEA) += ci_hdrc_msm.o -obj-$(CONFIG_USB_CHIPIDEA) += ci_hdrc_nspire.o +obj-$(CONFIG_USB_CHIPIDEA) += ci_hdrc_zevio.o # PCI doesn't provide stubs, need to check ifneq ($(CONFIG_PCI),) diff --git a/drivers/usb/chipidea/ci_hdrc_nspire.c b/drivers/usb/chipidea/ci_hdrc_nspire.c deleted file mode 100644 index c5c2dde..000 --- a/drivers/usb/chipidea/ci_hdrc_nspire.c +++ /dev/null @@ -1,72 +0,0 @@ -/* - * Copyright (C) 2013 Daniel Tang - * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License version 2, as - * published by the Free Software Foundation. - * - * Based off drivers/usb/chipidea/ci_hdrc_msm.c - * - */ - -#include -#include -#include -#include - -#include "ci.h" - -static struct ci_hdrc_platform_data ci_hdrc_nspire_platdata = { - .name = "ci_hdrc_nspire", - .flags = CI_HDRC_REGS_SHARED, - .capoffset = DEF_CAPOFFSET, -}; - -static int ci_hdrc_nspire_probe(struct platform_device *pdev) -{ - struct platform_device *ci_pdev; - - dev_dbg(&pdev->dev, "ci_hdrc_nspire_probe\n"); - - ci_pdev = ci_hdrc_add_device(&pdev->dev, - pdev->resource, pdev->num_resources, - &ci_hdrc_nspire_platdata); - - if (IS_ERR(ci_pdev)) { - dev_err(&pdev->dev, "ci_hdrc_add_device failed!\n"); - return PTR_ERR(ci_pdev); - } - - platform_set_drvdata(pdev, ci_pdev); - - return 0; -} - -static int ci_hdrc_nspire_remove(struct platform_device *pdev) -{ - struct platform_device *ci_pdev = platform_get_drvdata(pdev); - - ci_hdrc_remove_device(ci_pdev); - - return 0; -} - -static const struct of_device_id ci_hdrc_nspire_dt_ids[] = { - { .compatible = "lsi,nspire-usb", }, - { /* sentinel */ } -}; - -static struct platform_driver ci_hdrc_nspire_driver = { - .probe = ci_hdrc_nspire_probe, - .remove = ci_hdrc_nspire_remove, - .driver = { - .name = "nspire_usb", - .owner = THIS_MODULE, - .of_match_table = ci_hdrc