From b96f09b47b8163dfd645d7d8531c1f4286b9db67 Mon Sep 17 00:00:00 2001
From: Youngwan Kim <y103.kim@samsung.com>
Date: Tue, 17 Jul 2018 14:23:20 +0900
Subject: [PATCH] [COMMON] lib: libdss: move time check if to the front of
 iteration

Change-Id: Icc1d83dc6f1296dfcefada67633a3310b721cfde
Signed-off-by: Youngwan Kim <y103.kim@samsung.com>
---
 lib/libdss.c | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/lib/libdss.c b/lib/libdss.c
index 9991c0a798a2..6ed852efc2fa 100644
--- a/lib/libdss.c
+++ b/lib/libdss.c
@@ -27,30 +27,32 @@ int main(int argc, char *argv[])
 	printf("log = []\n");
 	for (i = 0; i < DSS_NR_CPUS; i++) {
 		for (j = 0; j < DSS_LOG_MAX_NUM; j++) {
+			if (p->task[i][j].time == 0)
+				break;
 			printf("log.append({'time':%.9f, 'type' : 'sched', 'cpu' : %d, 'comm' : '%s', 'pid' : %d})\n",
 					p->task[i][j].time/1.0e9,
 					i,
 					p->task[i][j].task_comm,
 					p->task[i][j].pid);
-			if (p->task[i][j].time == 0)
-				break;
 		}
 	}
 
 #ifdef CONFIG_DEBUG_SNAPSHOT_FREQ
 	for (i = 0; i < DSS_LOG_MAX_NUM; i++) {
+		if (p->freq[i].time == 0)
+			break;
 		printf("log.append({'time':%.9f, 'type' : 'freq', 'cpu' : %d, 'cluster' : %d, 'freq' : %lu })\n",
 				p->freq[i].time/1.0e9,
 				p->freq[i].cpu,
 				p->freq[i].type,
 				p->freq[i].target_freq);
-		if (p->freq[i].time == 0)
-			break;
 	}
 #endif
 
 	for (i = 0; i < DSS_NR_CPUS; i++) {
 		for (j = 0; j < DSS_LOG_MAX_NUM; j++) {
+			if (p->irq[i][j].time == 0)
+				break;
 			printf("log.append({'time':%.9f, 'type' : 'irq', 'cpu' : %d, 'num' : %d,"
 					"'en' : %d, 'func' : '%p'})\n",
 					p->irq[i][j].time/1.0e9,
@@ -58,23 +60,23 @@ int main(int argc, char *argv[])
 					p->irq[i][j].irq,
 					p->irq[i][j].en,
 					p->irq[i][j].fn);
-			if (p->irq[i][j].time == 0)
-				break;
 		}
 	}
 
 	for (i = 0; i < DSS_NR_CPUS; i++) {
 		for (j = 0; j < DSS_LOG_MAX_NUM; j++) {
+			if (p->cpuidle[i][j].time == 0)
+				break;
 			printf("log.append({'time':%.9f, 'type' : 'cpuidle', 'cpu' : %d, 'state' : %d})\n",
 					p->cpuidle[i][j].time/1.0e9,
 					i,
 					p->cpuidle[i][j].state);
-			if (p->cpuidle[i][j].time == 0)
-				break;
 		}
 	}
 #ifdef CONFIG_DEBUG_SNAPSHOT_BINDER
 	for (i = 0; i < DSS_API_MAX_NUM << 2; i++) {
+		if (p->binder[i].time == 0)
+			break;
 		printf("log.append({'time':%.9f, 'type' : 'binder', 'cpu' : %d, 'trace_type' : %d, 'transaction_id' : %d,"
 				" 'from_pid' : %d, 'from_tid' : %d, 'to_pid' : %d, 'to_tid' : %d,"
 				" 'from_pid_comm' : '%s', 'from_tid_comm' : '%s',"
@@ -91,12 +93,12 @@ int main(int argc, char *argv[])
 				p->binder[i].transaction.flags, p->binder[i].transaction.code,
 				p->binder[i].error.return_error, p->binder[i].error.return_error_param,
 				p->binder[i].error.return_error_line);
-		if (p->binder[i].time == 0)
-			break;
 	}
 #endif
 #ifdef CONFIG_DEBUG_SNAPSHOT_ACPM
 	for (i = 0; i < DSS_LOG_MAX_NUM; i++) {
+		if (p->acpm[i].time == 0)
+			break;
 		for (j = 0; j < 8; j++) {
 			if (!((p->acpm[i].log[j] >= 'a' && p->acpm[i].log[j] <= 'z') || p->acpm[i].log[j] == '_'))
 				p->acpm[i].log[j] = 0;
@@ -106,8 +108,6 @@ int main(int argc, char *argv[])
 				p->acpm[i].acpm_time/1.0e9,
 				p->acpm[i].log,
 				p->acpm[i].data);
-		if (p->acpm[i].time == 0)
-			break;
 	}
 #endif
 	return 0;
-- 
2.20.1