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. Also note that the emails were not "threaded", was that on purpose? thanks, greg k-h