On Fri, Sep 23, 2016 at 01:48:39PM +0200, Mark Kettenis wrote: > Essentially fixes for clang warnings. Clang doesn't like the > > sc = sc; > > statements in dwc2_shutdown(), dwc2_childdet() and dwc2_activate(). > Those functions aren't actually used. This diff moves them inside the > #if 0 that's already there for dwc2_suspend() and dwc2_resume(). But > perhaps I should just remove the code altogether?
I think these can be removed. They seem to be parts of netbsd's driver interface that we don't have, and keeping them around doesn't really make updating dwc2 any easier. Either this diff or one that removes them is ok by me.
