On 07/15/2017 04:38 PM, Rafael J. Wysocki wrote:

On Sunday, July 16, 2017 01:34:53 AM Mason wrote:

quoted

On 16/07/2017 01:24, Rafael J. Wysocki wrote:

quoted

On Saturday, July 15, 2017 10:20:27 AM Florian Fainelli wrote:

quoted

The enum offers the advantage of centralizing how many different states
exist for all the platforms we know about in the kernel, it's easy to
define common values for platforms that have the same semantics, just
like it's simple to add new values for platform specific details.

Well, you seem to be liking this, so why don't you just implement it?

At the end of his message, Florian wrote:

quoted

In any case, just agree and I will be happy to follow-up with patches.

But it may be hard to convince everybody without posting code changes
and often enough showing a patch makes a good argument.

I had the patches ready last night, saw the emails this morning and
decided to go mountain bike for a bit to think about it some more. You
will find my follow-up patches that hopefully implement your recommendation.

Thanks
-- 
Florian