Hi, > From: Felipe Balbi [mailto:balbi@xxxxxx] > Sent: Monday, October 26, 2015 11:27 PM > > Hi, > > Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx> writes: > > This patch fixes the hcd private size to sutabile size. > ^^^^^^^^ > suitable Oops, Thank you for the point. > > 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. > > Nothing against the patch itself, but you're describing what exactly > you're fixing. Did you find a bug because of this or is this basically a > cleanup ? Your subject says you're "fixing" hcd private size, but I > don't how this could be fixing anything. Care to explain ? My description is wrong. This is basically a cleanup. I don't find any bugs. So, I will fix this subject and description in v4. Best regards, Yoshihiro Shimoda > -- > balbi -- 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