> 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 Sure, Balbi. This will be better. I will send out patch v3 and another independent patch. Also include changelog as Greg required. Thanks for checking. Regards, Du, Changbin -- 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
