From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: [PATCH] intel_idle: stop exposing platform acronyms in sysfs Date: Wed, 1 Mar 2017 00:55:40 +0100 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Len Brown Cc: Linux PM , Linux Kernel Mailing List , Len Brown , Thomas Renninger List-Id: linux-pm@vger.kernel.org On Tue, Feb 28, 2017 at 10:32 PM, Len Brown wrote: > From: Len Brown > > Cosmetic only -- no functional change in this patch. > > sysfs before: > > state4/desc:MWAIT 0x20 > state4/name:C6-HSW > > sysfs after: > > state4/desc:MWAIT 0x20 > state4/name:C6 > > We remove the platform acronyms from the end of the state name > (-HSW in this case) for three reasonse. > > 1. more consistency with acpi_idle, which prints C1, C2, C3 etc. > > 2. users know what platform they are on already > an acronym for the processor code name here > seems to cause more confusion than clarity. > > 3. less clutter in "cpupower monitor" output, > which truncates the names to 4 columns. > > The precise definition of the state continues to be available in "desc". > > Cc: Thomas Renninger > Reported-by: Artem Bityutskiy > Signed-off-by: Len Brown I think I can apply this one right away if you want me to do that. Thanks, Rafael