Hi, before I review your patches, one comment changbin.du@xxxxxxxxx writes: > From: "Du, Changbin" <changbin.du@xxxxxxxxx> > > The first patch removed unnecessary checking for debugfs api call; > The second patch fix a memory leak issue; > The third patch add one new entry to debufs. > > Du, Changbin (3): > usb: dwc3: make dwc3_debugfs_init return value be void this is _not_ a fix > usb: dwc3: free dwc->regset on dwc3_debugfs_exit but this is. Why isn't this, at least, the first patch in the list ? In fact, it would be preferred that this patch be sent by itself and the following two patches should be on another branch completely without any dependencies to the memory leak fix. -- balbi
Attachment:
signature.asc
Description: PGP signature
