1_sanity_check does not provide a meaningful name of what this
script does. Rename 1_sanity_check.sh to cpuhotplug_00.sh to be
the first subtest to check if all cpus are online before
starting the remaining cpuhotplug tests.

Also, rename sanity_check() function to check_cpus_online() to
make it clear of what the function does.

Signed-off-by: Lisa Nguyen <lisa.ngu...@linaro.org>
---
 cpuhotplug/1_sanity_check.sh |   39 ---------------------------------------
 cpuhotplug/cpuhotplug_00.sh  |   39 +++++++++++++++++++++++++++++++++++++++
 2 files changed, 39 insertions(+), 39 deletions(-)
 delete mode 100755 cpuhotplug/1_sanity_check.sh
 create mode 100755 cpuhotplug/cpuhotplug_00.sh

diff --git a/cpuhotplug/1_sanity_check.sh b/cpuhotplug/1_sanity_check.sh
deleted file mode 100755
index d4bc02a..0000000
--- a/cpuhotplug/1_sanity_check.sh
+++ /dev/null
@@ -1,39 +0,0 @@
-#!/bin/sh
-#
-# PM-QA validation test suite for the power management on Linux
-#
-# Copyright (C) 2011, Linaro Limited.
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License
-# as published by the Free Software Foundation; either version 2
-# of the License, or (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, 
USA.
-#
-# Contributors:
-#     Daniel Lezcano <daniel.lezc...@linaro.org> (IBM Corporation)
-#       - initial API and implementation
-#
-# URL :
-
-. ../include/functions.sh
-
-sanity_check() {
-       ret=$(cat $CPU_PATH/offline)
-       if [ -z "$ret" ]; then
-               return 0
-       else
-               return 1
-       fi
-}
-
-check "all cpu are online" "sanity_check"
-test_status_show
diff --git a/cpuhotplug/cpuhotplug_00.sh b/cpuhotplug/cpuhotplug_00.sh
new file mode 100755
index 0000000..9b2f1ee
--- /dev/null
+++ b/cpuhotplug/cpuhotplug_00.sh
@@ -0,0 +1,39 @@
+#!/bin/sh
+#
+# PM-QA validation test suite for the power management on Linux
+#
+# Copyright (C) 2011, Linaro Limited.
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License
+# as published by the Free Software Foundation; either version 2
+# of the License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, 
USA.
+#
+# Contributors:
+#     Daniel Lezcano <daniel.lezc...@linaro.org> (IBM Corporation)
+#       - initial API and implementation
+#
+# URL :
+
+. ../include/functions.sh
+
+check_cpus_online() {
+       ret=$(cat $CPU_PATH/offline)
+       if [ -z "$ret" ]; then
+               return 0
+       else
+               return 1
+       fi
+}
+
+check "all cpus are online" "check_cpus_online"
+test_status_show
-- 
1.7.9.5


_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev

Reply via email to