Hi Greg, On 3/23/2021 15:24, Greg Kroah-Hartman wrote: > On Thu, Mar 18, 2021 at 10:12:46AM +0400, Artur Petrosyan wrote: >> - Added a default param "power_saving" to enable or >> disable hibernation or partial power down features. >> >> - Printed hibernation param in hw_params_show and >> power_saving param in params_show. > > This says what you are doing, but not _why_ you are doing this. Why is > this needed at all? > > Please read the documentation for how to write a good changelog text for > when you resubmit this series. Thank you for your reply. I have corrected changelog of the patch according the documentation. Will submit in series. > > Also note that the emails were not "threaded", was that on purpose? Yes, the patches that were included in the "usb: dwc2: Fix and improve power saving modes" which was submitted earlier were too large and needed to be split up into smaller patches. We had a discussion about that with Felipe Balbi earlier. Here is the link to the discussion thread (https://marc.info/?l=linux-usb&m=160379651904064&w=2) So I did the split and obviously the cover letter and the subject were changed therefor I have submitted the updates with this thread and mentioned in the cover letter that these are continuation of the previous patch set that was too large. > > thanks, > > greg k-h > Regards, Artur