Addition of unit test cases for Switching Backend functionality

Adding Unit test cases for newly added function pdbg_release_dt_root()

Signed-off-by: swarnendu.roy.chowdhury@ibm.com

Makefile.am

@@ -11,7 +11,8 @@ GIT_SHA1 ?= `git --work-tree=$(top_srcdir) --git-dir=$(top_srcdir)/.git describe
 libpdbg_tests = libpdbg_target_test \
 		libpdbg_probe_test1 \
 		libpdbg_probe_test2 \
-		libpdbg_probe_test3
+		libpdbg_probe_test3 \
+		libpdbg_release_dt_root_test
 
 bin_PROGRAMS = pdbg
 check_PROGRAMS = $(libpdbg_tests) libpdbg_dtree_test \
@@ -249,6 +250,11 @@ libpdbg_target_test_CFLAGS = $(libpdbg_test_cflags)
 libpdbg_target_test_LDFLAGS = $(libpdbg_test_ldflags)
 libpdbg_target_test_LDADD = $(libpdbg_test_ldadd)
 
+libpdbg_release_dt_root_test_SOURCES = src/tests/libpdbg_release_dt_root_test.c
+libpdbg_release_dt_root_test_CFLAGS = $(libpdbg_test_cflags)
+libpdbg_release_dt_root_test_LDFLAGS = $(libpdbg_test_ldflags)
+libpdbg_release_dt_root_test_LDADD = $(libpdbg_test_ldadd)
+
 libpdbg_probe_test1_SOURCES = src/tests/libpdbg_probe_test.c
 libpdbg_probe_test1_CFLAGS = $(libpdbg_test_cflags) -DTEST_ID=1
 libpdbg_probe_test1_LDFLAGS = $(libpdbg_test_ldflags)

libpdbg/device.c

@@ -708,8 +708,8 @@ static void pdbg_targets_init_virtual(struct pdbg_target *node, struct pdbg_targ
  * 		  function deletes/removes a target form the list of associated
  * 		  pdbg_target_class object. For internal use only.
  * 
- * @param target The pdbg_target object which will be removed from it's associated
- * 				 pdbg_target_class list
+ * @param target The pdbg_target object which will be removed from
+ * 				 it's associated pdbg_target_class list
  * 
  * @see pdbg_release_target for more details
  */
@@ -724,12 +724,12 @@ static void pdbg_del_target_from_target_class_list(struct pdbg_target* target)
 }
 
 /**
- * @brief A function to delete/release an existing pdbg_target object along
- * 		  with its children from the node list/dev tree. Should be only called 
- * 		  internally. Recursive in nature. For internal use only.
+ * @brief A function to delete/release an existing pdbg_target object
+ * 		  along with its children from the node list/dev tree. Should
+ *		  be only called internally. Recursive in nature.
  * 
- * @param target The target node for which the children and then the node itself
- * 				 will be released/freed
+ * @param target The target node for which the children and then the
+ * 				 node itself will be released/freed
  * 
  * @see pdbg_release_dt_root for more details
  */

libpdbg/dtb.c

@@ -418,12 +418,6 @@ bool pdbg_set_backend(enum pdbg_backend backend, const char *backend_option)
 		return false;
 	}
 
-	if (pdbg_backend == backend)
-	{
-		pdbg_log(PDBG_ERROR, "New backend is same as the current backend. Not proceeding further\n");
-		return false;
-	}
-
 	pdbg_backend = backend;
 	pdbg_backend_option = backend_option;
 

libpdbg/libpdbg.h

@@ -1459,8 +1459,8 @@ bool pdbg_context_short(void);
  * switching the backend in the same process. It clears/releases the 
  * existing dev tree (if any) children by children along with the root node.
  * 
- * Call this function very judiciously before ##pdbg_set_backend() if there is a
- * dev tree already is in place and we are switching the backend
+ * Call this function very judiciously before ##pdbg_set_backend() if there
+ * is a dev tree already is in place and we are switching the backend
  */
 void pdbg_release_dt_root();