[lkml]   [2018]   [Dec]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH v2 2/5] devfreq: add support for suspend/resume of a devfreq device
On Tue 2018-12-04 14:36:11, Chanwoo Choi wrote:
> Hi Lukasz,
> Looks good to me. But, I add the some comments.
> If you will fix it, feel free to add my tag:
> Reviewed-by: Chanwoo choi <>
> On 2018년 12월 03일 23:31, Lukasz Luba wrote:
> > The patch prepares devfreq device for handling suspend/resume
> > functionality. The new fields will store needed information during this
> nitpick. Remove unneeded space. There are two spaces between '.' and 'The new'.

Yes. And that's because two spaces are okay at that place.

Please don't cause unneeded noise on the list with trivial
comments. If the patch is okay, apply the patch, no need to find
trivial nit everywhere.
(cesky, pictures)
[unhandled content-type:application/pgp-signature]
 \ /
  Last update: 2018-12-09 10:01    [W:0.087 / U:8.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site