From: Brijesh Jadav <brijesh.jadav@ti.com>
Date: Tue, 6 Feb 2018 14:19:02 +0000 (+0530)
Subject: Closed Review comments
X-Git-Tag: DEV.FVID2.01.00.00.00~14
X-Git-Url: https://git.ti.com/gitweb?p=keystone-rtos%2Ffvid2.git;a=commitdiff_plain;h=43c54e1e3986ef555006091847e65bb9397717fa

Closed Review comments

Signed-off-by: Brijesh Jadav <brijesh.jadav@ti.com>
---

diff --git a/include/fvid2.h b/fvid2.h
similarity index 99%
rename from include/fvid2.h
rename to fvid2.h
index 5d9bb8e..a9bcd5f 100644
--- a/include/fvid2.h
+++ b/fvid2.h
@@ -31,9 +31,9 @@
  */
 
 /**
- *  \file csl_fvid2.h
+ *  \file fvid2.h
  *
- *  \brief CSL FVID2 API
+ *  \brief FVID2 API
  *
  *  FVID2 API is an interface which is used to create, control and use different
  *  video drivers. FVID2 API is used not only for on-chip
@@ -187,7 +187,7 @@
 /*                             Include Files                                  */
 /* ========================================================================== */
 
-#include <csl_fvid2_dataTypes.h>
+#include <ti/csl/csl_fvid2_dataTypes.h>
 
 #ifdef __cplusplus
 extern "C" {
diff --git a/fvid2_component.mk b/fvid2_component.mk
index e359883..a3fc99f 100644
--- a/fvid2_component.mk
+++ b/fvid2_component.mk
@@ -35,9 +35,8 @@
 ifeq ($(fvid2_component_make_include), )
 
 fvid2_default_SOCLIST       = j7 am6x
-fvid2_default_BOARDLIST     = simJ7 am6x-sim
 fvid2_default_j7_CORELIST   = mcu1_0 mcu1_1
-fvid2_default_am6x_CORELIST = mcu1_0 mcu1_1
+fvid2_default_am6x_CORELIST = mcu1_0 mcu1_1 mpu1_0
 
 ############################
 # fvid2 package
@@ -69,7 +68,7 @@ export fvid2_MAKEFILE
 export fvid2_LIBNAME
 export fvid2_LIBPATH
 fvid2_BOARD_DEPENDENCY = no
-fvid2_CORE_DEPENDENCY = yes
+fvid2_CORE_DEPENDENCY = no
 export fvid2_COMP_LIST
 export fvid2_BOARD_DEPENDENCY
 export fvid2_CORE_DEPENDENCY
diff --git a/include/fvid2_drvMgr.h b/include/fvid2_drvMgr.h
index 90a62ea..b49288c 100644
--- a/include/fvid2_drvMgr.h
+++ b/include/fvid2_drvMgr.h
@@ -48,9 +48,7 @@
 /*                             Include Files                                  */
 /* ========================================================================== */
 
-#include <fvid2.h>
-/* This is needed for memset/memcpy */
-#include <string.h>
+#include <ti/drv/fvid2/fvid2.h>
 
 #ifdef __cplusplus
 extern "C" {
diff --git a/src/fvid2_drvMgr.c b/src/fvid2_drvMgr.c
index 9f85c52..2ef6bdb 100644
--- a/src/fvid2_drvMgr.c
+++ b/src/fvid2_drvMgr.c
@@ -1,5 +1,5 @@
 /* =============================================================================
- *   Copyright (c) Texas Instruments Incorporated 2012-2015
+ *   Copyright (c) Texas Instruments Incorporated 2012-2018
  *
  *  Redistribution and use in source and binary forms, with or without
  *  modification, are permitted provided that the following conditions
@@ -43,9 +43,11 @@
 /* ========================================================================== */
 
 #include <ti/csl/tistdtypes.h>
-#include <csl_fvid2_dataTypes.h>
+#include <ti/csl/csl_fvid2_dataTypes.h>
 #include <fvid2_drvMgr.h>
 #include <ti/osal/osal.h>
+/* This is needed for memset/memcpy */
+#include <string.h>
 
 /* ========================================================================== */
 /*                           Macros & Typedefs                                */
diff --git a/src/makefile b/src/makefile
index a1027fd..4c1f1f2 100644
--- a/src/makefile
+++ b/src/makefile
@@ -6,12 +6,11 @@ include $(PDK_INSTALL_PATH)/ti/build/Rules.make
 MODULE_NAME = fvid2
 
 SRCDIR = src
-INCDIR = . ./include src
+INCDIR = . include src
 
 # List all the external components/interfaces, whose interface header files
 # need to be included for this component
 INCLUDE_EXTERNAL_INTERFACES = pdk
-INCLUDE_INTERNAL_INTERFACES = csl osal
 
 # Common source files and CFLAGS across all platforms and cores
 PACKAGE_SRCS_COMMON = .