usb: replace ehci_*_remove() with usb_deregister()
The remove callbacks of EHCI drivers are often just a wrapper of ehci_deregister. Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> Acked-by: Stephen Warren <swarren@nvidia.com>
This commit is contained in:
		
							parent
							
								
									720873bf42
								
							
						
					
					
						commit
						4052734273
					
				|  | @ -128,17 +128,6 @@ static int ehci_atmel_probe(struct udevice *dev) | ||||||
| 	return ehci_register(dev, hccr, hcor, NULL, 0, USB_INIT_HOST); | 	return ehci_register(dev, hccr, hcor, NULL, 0, USB_INIT_HOST); | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| static int ehci_atmel_remove(struct udevice *dev) |  | ||||||
| { |  | ||||||
| 	int ret; |  | ||||||
| 
 |  | ||||||
| 	ret = ehci_deregister(dev); |  | ||||||
| 	if (ret) |  | ||||||
| 		return ret; |  | ||||||
| 
 |  | ||||||
| 	return 0; |  | ||||||
| } |  | ||||||
| 
 |  | ||||||
| static const struct udevice_id ehci_usb_ids[] = { | static const struct udevice_id ehci_usb_ids[] = { | ||||||
| 	{ .compatible = "atmel,at91sam9g45-ehci", }, | 	{ .compatible = "atmel,at91sam9g45-ehci", }, | ||||||
| 	{ } | 	{ } | ||||||
|  | @ -149,7 +138,7 @@ U_BOOT_DRIVER(ehci_atmel) = { | ||||||
| 	.id		= UCLASS_USB, | 	.id		= UCLASS_USB, | ||||||
| 	.of_match	= ehci_usb_ids, | 	.of_match	= ehci_usb_ids, | ||||||
| 	.probe		= ehci_atmel_probe, | 	.probe		= ehci_atmel_probe, | ||||||
| 	.remove		= ehci_atmel_remove, | 	.remove		= ehci_deregister, | ||||||
| 	.ops		= &ehci_usb_ops, | 	.ops		= &ehci_usb_ops, | ||||||
| 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | ||||||
| 	.priv_auto_alloc_size = sizeof(struct ehci_atmel_priv), | 	.priv_auto_alloc_size = sizeof(struct ehci_atmel_priv), | ||||||
|  |  | ||||||
|  | @ -118,17 +118,6 @@ static int ehci_fsl_probe(struct udevice *dev) | ||||||
| 	return ehci_register(dev, hccr, hcor, &fsl_ehci_ops, 0, USB_INIT_HOST); | 	return ehci_register(dev, hccr, hcor, &fsl_ehci_ops, 0, USB_INIT_HOST); | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| static int ehci_fsl_remove(struct udevice *dev) |  | ||||||
| { |  | ||||||
| 	int ret; |  | ||||||
| 
 |  | ||||||
| 	ret = ehci_deregister(dev); |  | ||||||
| 	if (ret) |  | ||||||
| 		return ret; |  | ||||||
| 
 |  | ||||||
| 	return 0; |  | ||||||
| } |  | ||||||
| 
 |  | ||||||
| static const struct udevice_id ehci_usb_ids[] = { | static const struct udevice_id ehci_usb_ids[] = { | ||||||
| 	{ .compatible = "fsl-usb2-mph", }, | 	{ .compatible = "fsl-usb2-mph", }, | ||||||
| 	{ .compatible = "fsl-usb2-dr", }, | 	{ .compatible = "fsl-usb2-dr", }, | ||||||
|  | @ -141,7 +130,7 @@ U_BOOT_DRIVER(ehci_fsl) = { | ||||||
| 	.of_match = ehci_usb_ids, | 	.of_match = ehci_usb_ids, | ||||||
| 	.ofdata_to_platdata = ehci_fsl_ofdata_to_platdata, | 	.ofdata_to_platdata = ehci_fsl_ofdata_to_platdata, | ||||||
| 	.probe = ehci_fsl_probe, | 	.probe = ehci_fsl_probe, | ||||||
| 	.remove = ehci_fsl_remove, | 	.remove = ehci_deregister, | ||||||
| 	.ops	= &ehci_usb_ops, | 	.ops	= &ehci_usb_ops, | ||||||
| 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | ||||||
| 	.priv_auto_alloc_size = sizeof(struct ehci_fsl_priv), | 	.priv_auto_alloc_size = sizeof(struct ehci_fsl_priv), | ||||||
|  |  | ||||||
|  | @ -44,11 +44,6 @@ static int ehci_usb_probe(struct udevice *dev) | ||||||
| 	return ehci_register(dev, hccr, hcor, NULL, 0, USB_INIT_HOST); | 	return ehci_register(dev, hccr, hcor, NULL, 0, USB_INIT_HOST); | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| static int ehci_usb_remove(struct udevice *dev) |  | ||||||
| { |  | ||||||
| 	return ehci_deregister(dev); |  | ||||||
| } |  | ||||||
| 
 |  | ||||||
| static const struct udevice_id ehci_usb_ids[] = { | static const struct udevice_id ehci_usb_ids[] = { | ||||||
| 	{ .compatible = "generic-ehci" }, | 	{ .compatible = "generic-ehci" }, | ||||||
| 	{ } | 	{ } | ||||||
|  | @ -59,7 +54,7 @@ U_BOOT_DRIVER(ehci_generic) = { | ||||||
| 	.id	= UCLASS_USB, | 	.id	= UCLASS_USB, | ||||||
| 	.of_match = ehci_usb_ids, | 	.of_match = ehci_usb_ids, | ||||||
| 	.probe = ehci_usb_probe, | 	.probe = ehci_usb_probe, | ||||||
| 	.remove = ehci_usb_remove, | 	.remove = ehci_deregister, | ||||||
| 	.ops	= &ehci_usb_ops, | 	.ops	= &ehci_usb_ops, | ||||||
| 	.priv_auto_alloc_size = sizeof(struct generic_ehci), | 	.priv_auto_alloc_size = sizeof(struct generic_ehci), | ||||||
| 	.flags	= DM_FLAG_ALLOC_PRIV_DMA, | 	.flags	= DM_FLAG_ALLOC_PRIV_DMA, | ||||||
|  |  | ||||||
|  | @ -94,17 +94,6 @@ static int ehci_mvebu_probe(struct udevice *dev) | ||||||
| 	return ehci_register(dev, hccr, hcor, NULL, 0, USB_INIT_HOST); | 	return ehci_register(dev, hccr, hcor, NULL, 0, USB_INIT_HOST); | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| static int ehci_mvebu_remove(struct udevice *dev) |  | ||||||
| { |  | ||||||
| 	int ret; |  | ||||||
| 
 |  | ||||||
| 	ret = ehci_deregister(dev); |  | ||||||
| 	if (ret) |  | ||||||
| 		return ret; |  | ||||||
| 
 |  | ||||||
| 	return 0; |  | ||||||
| } |  | ||||||
| 
 |  | ||||||
| static const struct udevice_id ehci_usb_ids[] = { | static const struct udevice_id ehci_usb_ids[] = { | ||||||
| 	{ .compatible = "marvell,orion-ehci", }, | 	{ .compatible = "marvell,orion-ehci", }, | ||||||
| 	{ } | 	{ } | ||||||
|  | @ -115,7 +104,7 @@ U_BOOT_DRIVER(ehci_mvebu) = { | ||||||
| 	.id	= UCLASS_USB, | 	.id	= UCLASS_USB, | ||||||
| 	.of_match = ehci_usb_ids, | 	.of_match = ehci_usb_ids, | ||||||
| 	.probe = ehci_mvebu_probe, | 	.probe = ehci_mvebu_probe, | ||||||
| 	.remove = ehci_mvebu_remove, | 	.remove = ehci_deregister, | ||||||
| 	.ops	= &ehci_usb_ops, | 	.ops	= &ehci_usb_ops, | ||||||
| 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | ||||||
| 	.priv_auto_alloc_size = sizeof(struct ehci_mvebu_priv), | 	.priv_auto_alloc_size = sizeof(struct ehci_mvebu_priv), | ||||||
|  |  | ||||||
|  | @ -451,17 +451,6 @@ static int ehci_usb_probe(struct udevice *dev) | ||||||
| 	return ehci_register(dev, hccr, hcor, &mx6_ehci_ops, 0, priv->init_type); | 	return ehci_register(dev, hccr, hcor, &mx6_ehci_ops, 0, priv->init_type); | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| static int ehci_usb_remove(struct udevice *dev) |  | ||||||
| { |  | ||||||
| 	int ret; |  | ||||||
| 
 |  | ||||||
| 	ret = ehci_deregister(dev); |  | ||||||
| 	if (ret) |  | ||||||
| 		return ret; |  | ||||||
| 
 |  | ||||||
| 	return 0; |  | ||||||
| } |  | ||||||
| 
 |  | ||||||
| static const struct udevice_id mx6_usb_ids[] = { | static const struct udevice_id mx6_usb_ids[] = { | ||||||
| 	{ .compatible = "fsl,imx27-usb" }, | 	{ .compatible = "fsl,imx27-usb" }, | ||||||
| 	{ } | 	{ } | ||||||
|  | @ -472,7 +461,7 @@ U_BOOT_DRIVER(usb_mx6) = { | ||||||
| 	.id	= UCLASS_USB, | 	.id	= UCLASS_USB, | ||||||
| 	.of_match = mx6_usb_ids, | 	.of_match = mx6_usb_ids, | ||||||
| 	.probe	= ehci_usb_probe, | 	.probe	= ehci_usb_probe, | ||||||
| 	.remove = ehci_usb_remove, | 	.remove = ehci_deregister, | ||||||
| 	.ops	= &ehci_usb_ops, | 	.ops	= &ehci_usb_ops, | ||||||
| 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | ||||||
| 	.priv_auto_alloc_size = sizeof(struct ehci_mx6_priv_data), | 	.priv_auto_alloc_size = sizeof(struct ehci_mx6_priv_data), | ||||||
|  |  | ||||||
|  | @ -126,17 +126,6 @@ static int ehci_pci_probe(struct udevice *dev) | ||||||
| 	return ehci_register(dev, hccr, hcor, NULL, 0, USB_INIT_HOST); | 	return ehci_register(dev, hccr, hcor, NULL, 0, USB_INIT_HOST); | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| static int ehci_pci_remove(struct udevice *dev) |  | ||||||
| { |  | ||||||
| 	int ret; |  | ||||||
| 
 |  | ||||||
| 	ret = ehci_deregister(dev); |  | ||||||
| 	if (ret) |  | ||||||
| 		return ret; |  | ||||||
| 
 |  | ||||||
| 	return 0; |  | ||||||
| } |  | ||||||
| 
 |  | ||||||
| static const struct udevice_id ehci_pci_ids[] = { | static const struct udevice_id ehci_pci_ids[] = { | ||||||
| 	{ .compatible = "ehci-pci" }, | 	{ .compatible = "ehci-pci" }, | ||||||
| 	{ } | 	{ } | ||||||
|  | @ -146,7 +135,7 @@ U_BOOT_DRIVER(ehci_pci) = { | ||||||
| 	.name	= "ehci_pci", | 	.name	= "ehci_pci", | ||||||
| 	.id	= UCLASS_USB, | 	.id	= UCLASS_USB, | ||||||
| 	.probe = ehci_pci_probe, | 	.probe = ehci_pci_probe, | ||||||
| 	.remove = ehci_pci_remove, | 	.remove = ehci_deregister, | ||||||
| 	.of_match = ehci_pci_ids, | 	.of_match = ehci_pci_ids, | ||||||
| 	.ops	= &ehci_usb_ops, | 	.ops	= &ehci_usb_ops, | ||||||
| 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | ||||||
|  |  | ||||||
|  | @ -846,17 +846,6 @@ static int ehci_usb_probe(struct udevice *dev) | ||||||
| 			     plat->init_type); | 			     plat->init_type); | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| static int ehci_usb_remove(struct udevice *dev) |  | ||||||
| { |  | ||||||
| 	int ret; |  | ||||||
| 
 |  | ||||||
| 	ret = ehci_deregister(dev); |  | ||||||
| 	if (ret) |  | ||||||
| 		return ret; |  | ||||||
| 
 |  | ||||||
| 	return 0; |  | ||||||
| } |  | ||||||
| 
 |  | ||||||
| static const struct udevice_id ehci_usb_ids[] = { | static const struct udevice_id ehci_usb_ids[] = { | ||||||
| 	{ .compatible = "nvidia,tegra20-ehci", .data = USB_CTLR_T20 }, | 	{ .compatible = "nvidia,tegra20-ehci", .data = USB_CTLR_T20 }, | ||||||
| 	{ .compatible = "nvidia,tegra30-ehci", .data = USB_CTLR_T30 }, | 	{ .compatible = "nvidia,tegra30-ehci", .data = USB_CTLR_T30 }, | ||||||
|  | @ -871,7 +860,7 @@ U_BOOT_DRIVER(usb_ehci) = { | ||||||
| 	.of_match = ehci_usb_ids, | 	.of_match = ehci_usb_ids, | ||||||
| 	.ofdata_to_platdata = ehci_usb_ofdata_to_platdata, | 	.ofdata_to_platdata = ehci_usb_ofdata_to_platdata, | ||||||
| 	.probe = ehci_usb_probe, | 	.probe = ehci_usb_probe, | ||||||
| 	.remove = ehci_usb_remove, | 	.remove = ehci_deregister, | ||||||
| 	.ops	= &ehci_usb_ops, | 	.ops	= &ehci_usb_ops, | ||||||
| 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | ||||||
| 	.priv_auto_alloc_size = sizeof(struct fdt_usb), | 	.priv_auto_alloc_size = sizeof(struct fdt_usb), | ||||||
|  |  | ||||||
|  | @ -73,17 +73,6 @@ static int ehci_zynq_probe(struct udevice *dev) | ||||||
| 	return ehci_register(dev, hccr, hcor, NULL, 0, plat->init_type); | 	return ehci_register(dev, hccr, hcor, NULL, 0, plat->init_type); | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| static int ehci_zynq_remove(struct udevice *dev) |  | ||||||
| { |  | ||||||
| 	int ret; |  | ||||||
| 
 |  | ||||||
| 	ret = ehci_deregister(dev); |  | ||||||
| 	if (ret) |  | ||||||
| 		return ret; |  | ||||||
| 
 |  | ||||||
| 	return 0; |  | ||||||
| } |  | ||||||
| 
 |  | ||||||
| static const struct udevice_id ehci_zynq_ids[] = { | static const struct udevice_id ehci_zynq_ids[] = { | ||||||
| 	{ .compatible = "xlnx,zynq-usb-2.20a" }, | 	{ .compatible = "xlnx,zynq-usb-2.20a" }, | ||||||
| 	{ } | 	{ } | ||||||
|  | @ -95,7 +84,7 @@ U_BOOT_DRIVER(ehci_zynq) = { | ||||||
| 	.of_match = ehci_zynq_ids, | 	.of_match = ehci_zynq_ids, | ||||||
| 	.ofdata_to_platdata = ehci_zynq_ofdata_to_platdata, | 	.ofdata_to_platdata = ehci_zynq_ofdata_to_platdata, | ||||||
| 	.probe = ehci_zynq_probe, | 	.probe = ehci_zynq_probe, | ||||||
| 	.remove = ehci_zynq_remove, | 	.remove = ehci_deregister, | ||||||
| 	.ops	= &ehci_usb_ops, | 	.ops	= &ehci_usb_ops, | ||||||
| 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | ||||||
| 	.priv_auto_alloc_size = sizeof(struct zynq_ehci_priv), | 	.priv_auto_alloc_size = sizeof(struct zynq_ehci_priv), | ||||||
|  |  | ||||||
		Loading…
	
		Reference in New Issue