On Wed, 2018-01-10 at 23:25 +0100, Rafael J. Wysocki wrote: > On Wed, Jan 10, 2018 at 2:24 PM, Andy Shevchenko > <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote: > > On Wed, 2018-01-10 at 13:26 +0100, Rafael J. Wysocki wrote: > > Just a nitpick: Can we leave S2IDLE instead of S2I? > > Would it make sense for potential code readers? > > I wanted it to be shorter, but if that is a problem, I'd rather call > it PLATFORM than S2IDLE (as technically they are related to the > low-power mode of the platform). It's not problem per se, though without a context it would take time to get into S2I acronym from the code for not familiar reader. > I'll send an update shortly. Thanks! -- Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Intel Finland Oy
