usb: host: xhci: cleanup hcd private size (32479d4b) · Commits · 戴 / test

This patch cleanups the hcd private size to suitable size. The previous code has "sizeof(struct xhci_hcd *)" in xhci_hc_driver as hcd_priv_size and sizeof(struct xhci_hcd) in xhci_plat_overrides or xhci_pci_overrides as extra_priv_size. However, the xhci driver uses a "sizeof(struct xhcd_hcd)" memory space in each hcd (main_hcd and shared_hcd) actually. Signed-off-by:

Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com> Signed-off-by:

Mathias Nyman <mathias.nyman@linux.intel.com> Signed-off-by:

Greg Kroah-Hartman <gregkh@linuxfoundation.org>

drivers/usb/host/xhci-pci.c

+0 −1

Original line number	Diff line number	Diff line
		@@ -53,7 +53,6 @@ static struct hc_driver __read_mostly xhci_pci_hc_driver;
		static int xhci_pci_setup(struct usb_hcd *hcd);

		static const struct xhci_driver_overrides xhci_pci_overrides __initconst = {
		.extra_priv_size = sizeof(struct xhci_hcd),
		.reset = xhci_pci_setup,
		};

drivers/usb/host/xhci-plat.c

+0 −1

Original line number	Diff line number	Diff line
		@@ -31,7 +31,6 @@ static int xhci_plat_setup(struct usb_hcd *hcd);
		static int xhci_plat_start(struct usb_hcd *hcd);

		static const struct xhci_driver_overrides xhci_plat_overrides __initconst = {
		.extra_priv_size = sizeof(struct xhci_hcd),
		.reset = xhci_plat_setup,
		.start = xhci_plat_start,
		};

drivers/usb/host/xhci.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -4952,7 +4952,7 @@ EXPORT_SYMBOL_GPL(xhci_gen_setup);
		static const struct hc_driver xhci_hc_driver = {
		.description = "xhci-hcd",
		.product_desc = "xHCI Host Controller",
		.hcd_priv_size = sizeof(struct xhci_hcd *),
		.hcd_priv_size = sizeof(struct xhci_hcd),

		/*
		* generic hardware linkage

Admin message