3 files changed, 44 insertions, 15 deletions

diff --git a/tools/power/cpupower/utils/idle_monitor/cpuidle_sysfs.c b/tools/power/cpupower/utils/idle_monitor/cpuidle_sysfs.c
index 5b3205f16217..5b8c4956ff9a 100644
--- a/tools/power/cpupower/utils/idle_monitor/cpuidle_sysfs.c
+++ b/tools/power/cpupower/utils/idle_monitor/cpuidle_sysfs.c
@@ -126,6 +126,20 @@ void fix_up_intel_idle_driver_name(char *tmp, int num)
         }
 }
 
+#ifdef __powerpc__
+void map_power_idle_state_name(char *tmp)
+{
+        if (!strncmp(tmp, "stop0_lite", CSTATE_NAME_LEN))
+                strcpy(tmp, "stop0L");
+        else if (!strncmp(tmp, "stop1_lite", CSTATE_NAME_LEN))
+                strcpy(tmp, "stop1L");
+        else if (!strncmp(tmp, "stop2_lite", CSTATE_NAME_LEN))
+                strcpy(tmp, "stop2L");
+}
+#else
+void map_power_idle_state_name(char *tmp) { }
+#endif
+
 static struct cpuidle_monitor *cpuidle_register(void)
 {
         int num;
@@ -145,6 +159,7 @@ static struct cpuidle_monitor *cpuidle_register(void)
                 if (tmp == NULL)
                         continue;
 
+                map_power_idle_state_name(tmp);
                 fix_up_intel_idle_driver_name(tmp, num);
                 strncpy(cpuidle_cstates[num].name, tmp, CSTATE_NAME_LEN - 1);
                 free(tmp);
diff --git a/tools/power/cpupower/utils/idle_monitor/cpupower-monitor.c b/tools/power/cpupower/utils/idle_monitor/cpupower-monitor.c
index 05f953f0f0a0..051da0a7c454 100644
--- a/tools/power/cpupower/utils/idle_monitor/cpupower-monitor.c
+++ b/tools/power/cpupower/utils/idle_monitor/cpupower-monitor.c
@@ -70,36 +70,43 @@ void print_n_spaces(int n)
                 printf(" ");
 }
 
-/* size of s must be at least n + 1 */
+/*s is filled with left and right spaces
+ *to make its length atleast n+1
+ */
 int fill_string_with_spaces(char *s, int n)
 {
+        char *temp;
         int len = strlen(s);
-        if (len > n)
+
+        if (len >= n)
                 return -1;
+
+        temp = malloc(sizeof(char) * (n+1));
         for (; len < n; len++)
                 s[len] = ' ';
         s[len] = '\0';
+        snprintf(temp, n+1, " %s", s);
+        strcpy(s, temp);
+        free(temp);
         return 0;
 }
 
+#define MAX_COL_WIDTH 6
 void print_header(int topology_depth)
 {
         int unsigned mon;
         int state, need_len;
         cstate_t s;
         char buf[128] = "";
-        int percent_width = 4;
 
         fill_string_with_spaces(buf, topology_depth * 5 - 1);
         printf("%s|", buf);
 
         for (mon = 0; mon < avail_monitors; mon++) {
-                need_len = monitors[mon]->hw_states_num * (percent_width + 3)
+                need_len = monitors[mon]->hw_states_num * (MAX_COL_WIDTH + 1)
                         - 1;
-                if (mon != 0) {
-                        printf("|| ");
-                        need_len--;
-                }
+                if (mon != 0)
+                        printf("||");
                 sprintf(buf, "%s", monitors[mon]->name);
                 fill_string_with_spaces(buf, need_len);
                 printf("%s", buf);
@@ -107,23 +114,21 @@ void print_header(int topology_depth)
         printf("\n");
 
         if (topology_depth > 2)
-                printf("PKG |");
+                printf(" PKG|");
         if (topology_depth > 1)
                 printf("CORE|");
         if (topology_depth > 0)
-                printf("CPU |");
+                printf(" CPU|");
 
         for (mon = 0; mon < avail_monitors; mon++) {
                 if (mon != 0)
-                        printf("|| ");
-                else
-                        printf(" ");
+                        printf("||");
                 for (state = 0; state < monitors[mon]->hw_states_num; state++) {
                         if (state != 0)
-                                printf(" | ");
+                                printf("|");
                         s = monitors[mon]->hw_states[state];
                         sprintf(buf, "%s", s.name);
-                        fill_string_with_spaces(buf, percent_width);
+                        fill_string_with_spaces(buf, MAX_COL_WIDTH);
                         printf("%s", buf);
                 }
                 printf(" ");
diff --git a/tools/power/cpupower/utils/idle_monitor/cpupower-monitor.h b/tools/power/cpupower/utils/idle_monitor/cpupower-monitor.h
index 9e43f3371fbc..2ae50b499e0a 100644
--- a/tools/power/cpupower/utils/idle_monitor/cpupower-monitor.h
+++ b/tools/power/cpupower/utils/idle_monitor/cpupower-monitor.h
@@ -15,7 +15,16 @@
 
 #define MONITORS_MAX 20
 #define MONITOR_NAME_LEN 20
+
+/* CSTATE_NAME_LEN is limited by header field width defined
+ * in cpupower-monitor.c. Header field width is defined to be
+ * sum of percent width and two spaces for padding.
+ */
+#ifdef __powerpc__
+#define CSTATE_NAME_LEN 7
+#else
 #define CSTATE_NAME_LEN 5
+#endif
 #define CSTATE_DESC_LEN 60
 
 int cpu_count;