cpupower: Fix no-rounding MHz frequency output
authorPrarit Bhargava <prarit@redhat.com>
Wed, 25 Oct 2017 13:51:32 +0000 (09:51 -0400)
committerShuah Khan <shuahkh@osg.samsung.com>
Fri, 3 Nov 2017 15:28:02 +0000 (09:28 -0600)
'cpupower frequency-info -ln' returns kHz values on systems with MHz range
minimum CPU frequency range.  For example, on a 800MHz to 4.20GHz system
the command returns

hardware limits: 800000 MHz - 4.200000 GHz

The code that causes this error can be removed.  The next else if clause
will handle the output correctly such that

hardware limits: 800.000 MHz - 4.200000 GHz

is displayed correctly.

[v2]: Remove two lines instead of fixing broken code.

Signed-off-by: Prarit Bhargava <prarit@redhat.com>
Cc: Thomas Renninger <trenn@suse.com>
Cc: Stafford Horne <shorne@gmail.com>
Cc: Shuah Khan <shuah@kernel.org>
Reviewed-by: Stafford Horne <shorne@gmail.com>
Signed-off-by: Shuah Khan <shuahkh@osg.samsung.com>
tools/power/cpupower/utils/cpufreq-info.c

index 3e701f0..df43cd4 100644 (file)
@@ -93,8 +93,6 @@ static void print_speed(unsigned long speed)
                if (speed > 1000000)
                        printf("%u.%06u GHz", ((unsigned int) speed/1000000),
                                ((unsigned int) speed%1000000));
-               else if (speed > 100000)
-                       printf("%u MHz", (unsigned int) speed);
                else if (speed > 1000)
                        printf("%u.%03u MHz", ((unsigned int) speed/1000),
                                (unsigned int) (speed%1000));