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); | ||||
| } | ||||
| 
 | ||||
| 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[] = { | ||||
| 	{ .compatible = "atmel,at91sam9g45-ehci", }, | ||||
| 	{ } | ||||
|  | @ -149,7 +138,7 @@ U_BOOT_DRIVER(ehci_atmel) = { | |||
| 	.id		= UCLASS_USB, | ||||
| 	.of_match	= ehci_usb_ids, | ||||
| 	.probe		= ehci_atmel_probe, | ||||
| 	.remove		= ehci_atmel_remove, | ||||
| 	.remove		= ehci_deregister, | ||||
| 	.ops		= &ehci_usb_ops, | ||||
| 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | ||||
| 	.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); | ||||
| } | ||||
| 
 | ||||
| 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[] = { | ||||
| 	{ .compatible = "fsl-usb2-mph", }, | ||||
| 	{ .compatible = "fsl-usb2-dr", }, | ||||
|  | @ -141,7 +130,7 @@ U_BOOT_DRIVER(ehci_fsl) = { | |||
| 	.of_match = ehci_usb_ids, | ||||
| 	.ofdata_to_platdata = ehci_fsl_ofdata_to_platdata, | ||||
| 	.probe = ehci_fsl_probe, | ||||
| 	.remove = ehci_fsl_remove, | ||||
| 	.remove = ehci_deregister, | ||||
| 	.ops	= &ehci_usb_ops, | ||||
| 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | ||||
| 	.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); | ||||
| } | ||||
| 
 | ||||
| static int ehci_usb_remove(struct udevice *dev) | ||||
| { | ||||
| 	return ehci_deregister(dev); | ||||
| } | ||||
| 
 | ||||
| static const struct udevice_id ehci_usb_ids[] = { | ||||
| 	{ .compatible = "generic-ehci" }, | ||||
| 	{ } | ||||
|  | @ -59,7 +54,7 @@ U_BOOT_DRIVER(ehci_generic) = { | |||
| 	.id	= UCLASS_USB, | ||||
| 	.of_match = ehci_usb_ids, | ||||
| 	.probe = ehci_usb_probe, | ||||
| 	.remove = ehci_usb_remove, | ||||
| 	.remove = ehci_deregister, | ||||
| 	.ops	= &ehci_usb_ops, | ||||
| 	.priv_auto_alloc_size = sizeof(struct generic_ehci), | ||||
| 	.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); | ||||
| } | ||||
| 
 | ||||
| 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[] = { | ||||
| 	{ .compatible = "marvell,orion-ehci", }, | ||||
| 	{ } | ||||
|  | @ -115,7 +104,7 @@ U_BOOT_DRIVER(ehci_mvebu) = { | |||
| 	.id	= UCLASS_USB, | ||||
| 	.of_match = ehci_usb_ids, | ||||
| 	.probe = ehci_mvebu_probe, | ||||
| 	.remove = ehci_mvebu_remove, | ||||
| 	.remove = ehci_deregister, | ||||
| 	.ops	= &ehci_usb_ops, | ||||
| 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | ||||
| 	.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); | ||||
| } | ||||
| 
 | ||||
| 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[] = { | ||||
| 	{ .compatible = "fsl,imx27-usb" }, | ||||
| 	{ } | ||||
|  | @ -472,7 +461,7 @@ U_BOOT_DRIVER(usb_mx6) = { | |||
| 	.id	= UCLASS_USB, | ||||
| 	.of_match = mx6_usb_ids, | ||||
| 	.probe	= ehci_usb_probe, | ||||
| 	.remove = ehci_usb_remove, | ||||
| 	.remove = ehci_deregister, | ||||
| 	.ops	= &ehci_usb_ops, | ||||
| 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | ||||
| 	.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); | ||||
| } | ||||
| 
 | ||||
| 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[] = { | ||||
| 	{ .compatible = "ehci-pci" }, | ||||
| 	{ } | ||||
|  | @ -146,7 +135,7 @@ U_BOOT_DRIVER(ehci_pci) = { | |||
| 	.name	= "ehci_pci", | ||||
| 	.id	= UCLASS_USB, | ||||
| 	.probe = ehci_pci_probe, | ||||
| 	.remove = ehci_pci_remove, | ||||
| 	.remove = ehci_deregister, | ||||
| 	.of_match = ehci_pci_ids, | ||||
| 	.ops	= &ehci_usb_ops, | ||||
| 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | ||||
|  |  | |||
|  | @ -846,17 +846,6 @@ static int ehci_usb_probe(struct udevice *dev) | |||
| 			     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[] = { | ||||
| 	{ .compatible = "nvidia,tegra20-ehci", .data = USB_CTLR_T20 }, | ||||
| 	{ .compatible = "nvidia,tegra30-ehci", .data = USB_CTLR_T30 }, | ||||
|  | @ -871,7 +860,7 @@ U_BOOT_DRIVER(usb_ehci) = { | |||
| 	.of_match = ehci_usb_ids, | ||||
| 	.ofdata_to_platdata = ehci_usb_ofdata_to_platdata, | ||||
| 	.probe = ehci_usb_probe, | ||||
| 	.remove = ehci_usb_remove, | ||||
| 	.remove = ehci_deregister, | ||||
| 	.ops	= &ehci_usb_ops, | ||||
| 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | ||||
| 	.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); | ||||
| } | ||||
| 
 | ||||
| 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[] = { | ||||
| 	{ .compatible = "xlnx,zynq-usb-2.20a" }, | ||||
| 	{ } | ||||
|  | @ -95,7 +84,7 @@ U_BOOT_DRIVER(ehci_zynq) = { | |||
| 	.of_match = ehci_zynq_ids, | ||||
| 	.ofdata_to_platdata = ehci_zynq_ofdata_to_platdata, | ||||
| 	.probe = ehci_zynq_probe, | ||||
| 	.remove = ehci_zynq_remove, | ||||
| 	.remove = ehci_deregister, | ||||
| 	.ops	= &ehci_usb_ops, | ||||
| 	.platdata_auto_alloc_size = sizeof(struct usb_platdata), | ||||
| 	.priv_auto_alloc_size = sizeof(struct zynq_ehci_priv), | ||||
|  |  | |||
		Loading…
	
		Reference in New Issue