On Fri, Dec 01, 2023 at 05:31:52PM +0000, David Laight wrote: > From: Mathias Nyman > > Sent: 01 December 2023 15:07 ... > > - memset(dbc->eps, 0, sizeof(struct dbc_ep) * ARRAY_SIZE(dbc->eps)); > > + memset(dbc->eps, 0, sizeof_field(struct xhci_dbc, eps)); > > Isn't that just: > memset(dpc->eps, 0, sizeof (dpc->eps)); > perhaps better written as: > memset(&dpc->epc, 0, sizeof (dpc->eps); Maybe... You can send a patch, so it gets tested for regressions! > Otherwise the existing code wouldn't make sense at all. -- With Best Regards, Andy Shevchenko