Re: [PATCH 7/9] usb: add Atmel USBA UDC DT support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux