On 10:27 Wed 14 Mar , Jean-Christophe PLAGNIOL-VILLARD wrote:
> Hi,
>
> except the boolean that I need to update is it ok?
Grant is it ok?
Best Regards,
J.
> On 10:15 Wed 07 Mar , Jean-Christophe PLAGNIOL-VILLARD wrote:
> > Allow to compile the driver all the time if AT91 enabled.
> >
> > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@xxxxxxxxxxxx>
> > Cc: Nicolas Ferre <nicolas.ferre@xxxxxxxxx>
> > Cc: linux-usb@xxxxxxxxxxxxxxx
> > ---
> > Hi Greg
> >
> > if it's ok with you I apply with the rest of the USB patch series via
> > at91
> >
> > Best Regards,
> > J. .../devicetree/bindings/usb/atmel-usb.txt | 86 ++++++++
> > drivers/usb/gadget/Kconfig | 2 +-
> > drivers/usb/gadget/atmel_usba_udc.c | 209 +++++++++++++++-----
> > drivers/usb/gadget/atmel_usba_udc.h | 1 +
> > 4 files changed, 248 insertions(+), 50 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/usb/atmel-usb.txt b/Documentation/devicetree/bindings/usb/atmel-usb.txt
> > index 60bd215..b44f87e 100644
> > --- a/Documentation/devicetree/bindings/usb/atmel-usb.txt
> > +++ b/Documentation/devicetree/bindings/usb/atmel-usb.txt
> > @@ -47,3 +47,89 @@ usb1: gadget@fffa4000 {
> > interrupts = <10 4>;
> > atmel,vbus-gpio = <&pioC 5 0>;
> > };
> > +
> > +Atmel High-Speed USB device controller
> > +
> > +Required properties:
> > + - compatible: Should be "atmel,at91sam9rl-udc"
> > + - reg: Address and length of the register set for the device
> > + - interrupts: Should contain macb interrupt
> > + - ep childnode: To specifiy the number of endpoints and their properties.
> > +
> > +Optional properties:
> > + - atmel,vbus-gpio: If present, specifies a gpio that needs to be
> > + activated for the bus to be powered.
> > +
> > +Required child node properties:
> > + - name: Name of the endpoint.
> > + - reg: Num of the endpoint.
> > + - atmel,fifo-size: Size of the fifo.
> > + - atmel,nb-banks: Number of banks.
> > + - atmel,can-dma: Boolean to specify if the endpoint support DMA.
> > + - atmel,can-isoc: Boolean to specify if the endpoint support ISOC.
> > +
> > +usb2: gadget@fff78000 {
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > + compatible = "atmel,at91sam9rl-udc";
> > + reg = <0x00600000 0x80000
> > + 0xfff78000 0x400>;
> > + interrupts = <27 4>;
> > + atmel,vbus-gpio = <&pioB 19 0>;
> > +
> > + ep0 {
> > + reg = <0>;
> > + atmel,fifo-size = <64>;
> > + atmel,nb-banks = <1>;
> > + atmel,can-dma = <0>;
> > + atmel,can-isoc = <0>;
> > + };
> > +
> > + ep1 {
> > + reg = <1>;
> > + atmel,fifo-size = <1024>;
> > + atmel,nb-banks = <2>;
> > + atmel,can-dma = <1>;
> > + atmel,can-isoc = <1>;
> > + };
> > +
> > + ep2 {
> > + reg = <2>;
> > + atmel,fifo-size = <1024>;
> > + atmel,nb-banks = <2>;
> > + atmel,can-dma = <1>;
> > + atmel,can-isoc = <1>;
> > + };
> > +
> > + ep3 {
> > + reg = <3>;
> > + atmel,fifo-size = <1024>;
> > + atmel,nb-banks = <3>;
> > + atmel,can-dma = <1>;
> > + atmel,can-isoc = <0>;
> > + };
> > +
> > + ep4 {
> > + reg = <4>;
> > + atmel,fifo-size = <1024>;
> > + atmel,nb-banks = <3>;
> > + atmel,can-dma = <1>;
> > + atmel,can-isoc = <0>;
> > + };
> > +
> > + ep5 {
> > + reg = <5>;
> > + atmel,fifo-size = <1024>;
> > + atmel,nb-banks = <3>;
> > + atmel,can-dma = <1>;
> > + atmel,can-isoc = <1>;
> > + };
> > +
> > + ep6 {
> > + reg = <6>;
> > + atmel,fifo-size = <1024>;
> > + atmel,nb-banks = <3>;
> > + atmel,can-dma = <1>;
> > + atmel,can-isoc = <1>;
> > + };
> > +};
> > diff --git a/drivers/usb/gadget/Kconfig b/drivers/usb/gadget/Kconfig
> > index edf1144..1dbba6c 100644
> > --- a/drivers/usb/gadget/Kconfig
> > +++ b/drivers/usb/gadget/Kconfig
> > @@ -150,7 +150,7 @@ config USB_AT91
> > config USB_ATMEL_USBA
> > tristate "Atmel USBA"
> > select USB_GADGET_DUALSPEED
> > - depends on AVR32 || ARCH_AT91SAM9RL || ARCH_AT91SAM9G45
> > + depends on AVR32 || ARCH_AT91
> > help
> > USBA is the integrated high-speed USB Device controller on
> > the AT32AP700x, some AT91SAM9 and AT91CAP9 processors from Atmel.
> > diff --git a/drivers/usb/gadget/atmel_usba_udc.c b/drivers/usb/gadget/atmel_usba_udc.c
> > index ce9dffb..234eabb 100644
> > --- a/drivers/usb/gadget/atmel_usba_udc.c
> > +++ b/drivers/usb/gadget/atmel_usba_udc.c
> > @@ -21,6 +21,8 @@
> > #include <linux/usb/gadget.h>
> > #include <linux/usb/atmel_usba_udc.h>
> > #include <linux/delay.h>
> > +#include <linux/of.h>
> > +#include <linux/of_gpio.h>
> >
> > #include <asm/gpio.h>
> > #include <mach/board.h>
> > @@ -1885,9 +1887,143 @@ static int atmel_usba_stop(struct usb_gadget_driver *driver)
> > return 0;
> > }
> >
> > -static int __init usba_udc_probe(struct platform_device *pdev)
> > +#ifdef CONFIG_OF
> > +static struct usba_ep * __devinit atmel_udc_of_init(struct platform_device *pdev,
> > + struct usba_udc *udc)
> > +{
> > + u32 val;
> > + const char *name;
> > + enum of_gpio_flags flags;
> > + struct device_node *np = pdev->dev.of_node;
> > + struct device_node *pp;
> > + int i, ret;
> > + struct usba_ep *eps, *ep;
> > +
> > + udc->num_ep = 0;
> > +
> > + udc->vbus_pin = of_get_named_gpio_flags(np, "atmel,vbus-gpio", 0,
> > + &flags);
> > + udc->vbus_pin_inverted = (flags & OF_GPIO_ACTIVE_LOW) ? 1 : 0;
> > +
> > + pp = NULL;
> > + while ((pp = of_get_next_child(np, pp)))
> > + udc->num_ep++;
> > +
> > + eps = devm_kzalloc(&pdev->dev, sizeof(struct usba_ep) * udc->num_ep,
> > + GFP_KERNEL);
> > + if (!eps)
> > + return ERR_PTR(-ENOMEM);
> > +
> > + udc->gadget.ep0 = &eps[0].ep;
> > +
> > + INIT_LIST_HEAD(&eps[0].ep.ep_list);
> > +
> > + pp = NULL;
> > + i = 0;
> > + while ((pp = of_get_next_child(np, pp))) {
> > + ep = &eps[i];
> > +
> > + ret = of_property_read_u32(pp, "reg", &val);
> > + if (ret) {
> > + dev_err(&pdev->dev, "of_probe: reg error(%d)\n", ret);
> > + goto err;
> > + }
> > + ep->index = val;
> > +
> > + ret = of_property_read_u32(pp, "atmel,fifo-size", &val);
> > + if (ret) {
> > + dev_err(&pdev->dev, "of_probe: fifo-size error(%d)\n", ret);
> > + goto err;
> > + }
> > + ep->fifo_size = val;
> > +
> > + ret = of_property_read_u32(pp, "atmel,nb-banks", &val);
> > + if (ret) {
> > + dev_err(&pdev->dev, "of_probe: nb-banks error(%d)\n", ret);
> > + goto err;
> > + }
> > + ep->nr_banks = val;
> > +
> > + ep->can_dma = of_property_read_bool(pp, "atmel,can-dma");
> > + ep->can_isoc = of_property_read_bool(pp, "atmel,can-isoc");
> > +
> > + ret = of_property_read_string(pp, "name", &name);
> > + ep->ep.name = name;
> > +
> > + ep->ep_regs = udc->regs + USBA_EPT_BASE(i);
> > + ep->dma_regs = udc->regs + USBA_DMA_BASE(i);
> > + ep->fifo = udc->fifo + USBA_FIFO_BASE(i);
> > + ep->ep.ops = &usba_ep_ops;
> > + ep->ep.maxpacket = ep->fifo_size;
> > + ep->udc = udc;
> > + INIT_LIST_HEAD(&ep->queue);
> > +
> > + if (i)
> > + list_add_tail(&ep->ep.ep_list, &udc->gadget.ep_list);
> > +
> > + i++;
> > + }
> > +
> > + return eps;
> > +err:
> > + return ERR_PTR(ret);
> > +}
> > +#else
> > +static struct usba_ep * __devinit atmel_udc_of_init(struct platform_device *pdev,
> > + struct usba_udc *udc)
> > +{
> > + return ERR_PTR(-ENOSYS);
> > +}
> > +#endif
> > +
> > +static struct usba_ep * __devinit usba_udc_pdata(struct platform_device *pdev,
> > + struct usba_udc *udc)
> > {
> > struct usba_platform_data *pdata = pdev->dev.platform_data;
> > + struct usba_ep *eps;
> > + int i;
> > +
> > + if (!pdata)
> > + return ERR_PTR(-ENXIO);
> > +
> > + eps = devm_kzalloc(&pdev->dev, sizeof(struct usba_ep) * pdata->num_ep,
> > + GFP_KERNEL);
> > + if (!eps)
> > + return ERR_PTR(-ENOMEM);
> > +
> > + udc->gadget.ep0 = &eps[0].ep;
> > +
> > + udc->vbus_pin = pdata->vbus_pin;
> > + udc->vbus_pin_inverted = pdata->vbus_pin_inverted;
> > + udc->num_ep = pdata->num_ep;
> > +
> > + INIT_LIST_HEAD(&eps[0].ep.ep_list);
> > +
> > + for (i = 0; i < pdata->num_ep; i++) {
> > + struct usba_ep *ep = &eps[i];
> > +
> > + ep->ep_regs = udc->regs + USBA_EPT_BASE(i);
> > + ep->dma_regs = udc->regs + USBA_DMA_BASE(i);
> > + ep->fifo = udc->fifo + USBA_FIFO_BASE(i);
> > + ep->ep.ops = &usba_ep_ops;
> > + ep->ep.name = pdata->ep[i].name;
> > + ep->fifo_size = ep->ep.maxpacket = pdata->ep[i].fifo_size;
> > + ep->udc = udc;
> > + INIT_LIST_HEAD(&ep->queue);
> > + ep->nr_banks = pdata->ep[i].nr_banks;
> > + ep->index = pdata->ep[i].index;
> > + ep->can_dma = pdata->ep[i].can_dma;
> > + ep->can_isoc = pdata->ep[i].can_isoc;
> > +
> > + if (i)
> > + list_add_tail(&ep->ep.ep_list, &udc->gadget.ep_list);
> > + }
> > +
> > + return eps;
> > +}
> > +
> > +static int __init usba_udc_probe(struct platform_device *pdev)
> > +{
> > struct resource *regs, *fifo;
> > struct clk *pclk, *hclk;
> > struct usba_udc *udc = &the_udc;
> > @@ -1895,7 +2031,7 @@ static int __init usba_udc_probe(struct platform_device *pdev)
> >
> > regs = platform_get_resource(pdev, IORESOURCE_MEM, CTRL_IOMEM_ID);
> > fifo = platform_get_resource(pdev, IORESOURCE_MEM, FIFO_IOMEM_ID);
> > - if (!regs || !fifo || !pdata)
> > + if (!regs || !fifo)
> > return -ENXIO;
> >
> > irq = platform_get_irq(pdev, 0);
> > @@ -1945,46 +2081,14 @@ static int __init usba_udc_probe(struct platform_device *pdev)
> > usba_writel(udc, CTRL, USBA_DISABLE_MASK);
> > clk_disable(pclk);
> >
> > - usba_ep = kzalloc(sizeof(struct usba_ep) * pdata->num_ep,
> > - GFP_KERNEL);
> > - if (!usba_ep)
> > - goto err_alloc_ep;
> > -
> > - the_udc.gadget.ep0 = &usba_ep[0].ep;
> > -
> > - INIT_LIST_HEAD(&usba_ep[0].ep.ep_list);
> > - usba_ep[0].ep_regs = udc->regs + USBA_EPT_BASE(0);
> > - usba_ep[0].dma_regs = udc->regs + USBA_DMA_BASE(0);
> > - usba_ep[0].fifo = udc->fifo + USBA_FIFO_BASE(0);
> > - usba_ep[0].ep.ops = &usba_ep_ops;
> > - usba_ep[0].ep.name = pdata->ep[0].name;
> > - usba_ep[0].ep.maxpacket = pdata->ep[0].fifo_size;
> > - usba_ep[0].udc = &the_udc;
> > - INIT_LIST_HEAD(&usba_ep[0].queue);
> > - usba_ep[0].fifo_size = pdata->ep[0].fifo_size;
> > - usba_ep[0].nr_banks = pdata->ep[0].nr_banks;
> > - usba_ep[0].index = pdata->ep[0].index;
> > - usba_ep[0].can_dma = pdata->ep[0].can_dma;
> > - usba_ep[0].can_isoc = pdata->ep[0].can_isoc;
> > -
> > - for (i = 1; i < pdata->num_ep; i++) {
> > - struct usba_ep *ep = &usba_ep[i];
> > -
> > - ep->ep_regs = udc->regs + USBA_EPT_BASE(i);
> > - ep->dma_regs = udc->regs + USBA_DMA_BASE(i);
> > - ep->fifo = udc->fifo + USBA_FIFO_BASE(i);
> > - ep->ep.ops = &usba_ep_ops;
> > - ep->ep.name = pdata->ep[i].name;
> > - ep->ep.maxpacket = pdata->ep[i].fifo_size;
> > - ep->udc = &the_udc;
> > - INIT_LIST_HEAD(&ep->queue);
> > - ep->fifo_size = pdata->ep[i].fifo_size;
> > - ep->nr_banks = pdata->ep[i].nr_banks;
> > - ep->index = pdata->ep[i].index;
> > - ep->can_dma = pdata->ep[i].can_dma;
> > - ep->can_isoc = pdata->ep[i].can_isoc;
> > + if (pdev->dev.of_node)
> > + usba_ep = atmel_udc_of_init(pdev, udc);
> > + else
> > + usba_ep = usba_udc_pdata(pdev, udc);
> >
> > - list_add_tail(&ep->ep.ep_list, &udc->gadget.ep_list);
> > + if (IS_ERR(usba_ep)) {
> > + ret = PTR_ERR(usba_ep);
> > + goto err_alloc_ep;
> > }
> >
> > ret = request_irq(irq, usba_udc_irq, 0, "atmel_usba_udc", udc);
> > @@ -2001,10 +2105,8 @@ static int __init usba_udc_probe(struct platform_device *pdev)
> > goto err_device_add;
> > }
> >
> > - if (gpio_is_valid(pdata->vbus_pin)) {
> > - if (!gpio_request(pdata->vbus_pin, "atmel_usba_udc")) {
> > - udc->vbus_pin = pdata->vbus_pin;
> > - udc->vbus_pin_inverted = pdata->vbus_pin_inverted;
> > + if (gpio_is_valid(udc->vbus_pin)) {
> > + if (!gpio_request(udc->vbus_pin, "atmel_usba_udc")) {
> >
> > ret = request_irq(gpio_to_irq(udc->vbus_pin),
> > usba_vbus_irq, 0,
> > @@ -2029,13 +2131,13 @@ static int __init usba_udc_probe(struct platform_device *pdev)
> > goto err_add_udc;
> >
> > usba_init_debugfs(udc);
> > - for (i = 1; i < pdata->num_ep; i++)
> > + for (i = 1; i < udc->num_ep; i++)
> > usba_ep_init_debugfs(udc, &usba_ep[i]);
> >
> > return 0;
> >
> > err_add_udc:
> > - if (gpio_is_valid(pdata->vbus_pin)) {
> > + if (gpio_is_valid(udc->vbus_pin)) {
> > free_irq(gpio_to_irq(udc->vbus_pin), udc);
> > gpio_free(udc->vbus_pin);
> > }
> > @@ -2064,13 +2166,12 @@ static int __exit usba_udc_remove(struct platform_device *pdev)
> > {
> > struct usba_udc *udc;
> > int i;
> > - struct usba_platform_data *pdata = pdev->dev.platform_data;
> >
> > udc = platform_get_drvdata(pdev);
> >
> > usb_del_gadget_udc(&udc->gadget);
> >
> > - for (i = 1; i < pdata->num_ep; i++)
> > + for (i = 1; i < udc->num_ep; i++)
> > usba_ep_cleanup_debugfs(&usba_ep[i]);
> > usba_cleanup_debugfs(udc);
> >
> > @@ -2091,11 +2192,21 @@ static int __exit usba_udc_remove(struct platform_device *pdev)
> > return 0;
> > }
> >
> > +#if defined(CONFIG_OF)
> > +static const struct of_device_id atmel_udc_dt_ids[] = {
> > + { .compatible = "atmel,at91sam9rl-udc" },
> > + { /* sentinel */ }
> > +};
> > +
> > +MODULE_DEVICE_TABLE(of, atmel_udc_dt_ids);
> > +#endif
> > +
> > static struct platform_driver udc_driver = {
> > .remove = __exit_p(usba_udc_remove),
> > .driver = {
> > .name = "atmel_usba_udc",
> > .owner = THIS_MODULE,
> > + .of_match_table = of_match_ptr(atmel_udc_dt_ids),
> > },
> > };
> >
> > diff --git a/drivers/usb/gadget/atmel_usba_udc.h b/drivers/usb/gadget/atmel_usba_udc.h
> > index 88a2e07..2c068e7 100644
> > --- a/drivers/usb/gadget/atmel_usba_udc.h
> > +++ b/drivers/usb/gadget/atmel_usba_udc.h
> > @@ -324,6 +324,7 @@ struct usba_udc {
> > int irq;
> > int vbus_pin;
> > int vbus_pin_inverted;
> > + int num_ep;
> > struct clk *pclk;
> > struct clk *hclk;
> >
> > --
> > 1.7.7
> >
> _______________________________________________
> devicetree-discuss mailing list
> devicetree-discuss@xxxxxxxxxxxxxxxx
> https://lists.ozlabs.org/listinfo/devicetree-discuss
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
