am64x/am243: tamagawa: resolve reviewer comments

- resolve reviewer comments

Fixes: PINDSW-5472

Signed-off-by: Manoj Koppolu <manoj_koppolu@ti.com>
This commit is contained in:
Manoj Koppolu 2023-12-20 19:54:23 +05:30
parent 70a05f6dae
commit 10ae899ab6
16 changed files with 16 additions and 22 deletions

View File

@ -5,7 +5,6 @@ let device = "am243x";
const files = {
common: [
"tamagawa_diagnostic.c",
"tamagawa_periodic_trigger.h",
"tamagawa_periodic_trigger.c",
"main.c",
],
@ -36,6 +35,7 @@ const includes_freertos_r5f = {
"${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/FreeRTOS-Kernel/include",
"${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/portable/TI_ARM_CLANG/ARM_CR5F",
"${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/config/am243x/r5f",
"${MOTOR_CONTROL_SDK_PATH}/examples/position_sense/tamagawa_diagnostic",
],
};

View File

@ -5,7 +5,6 @@ let device = "am64x";
const files = {
common: [
"tamagawa_diagnostic.c",
"tamagawa_periodic_trigger.h",
"tamagawa_periodic_trigger.c",
"main.c",
],
@ -36,6 +35,7 @@ const includes_freertos_r5f = {
"${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/FreeRTOS-Kernel/include",
"${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/portable/TI_ARM_CLANG/ARM_CR5F",
"${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/config/am64x/r5f",
"${MOTOR_CONTROL_SDK_PATH}/examples/position_sense/tamagawa_diagnostic",
],
};

View File

@ -35,6 +35,7 @@
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/FreeRTOS-Kernel/include
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/portable/TI_ARM_CLANG/ARM_CR5F
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/config/am243x/r5f
-I${MOTOR_CONTROL_SDK_PATH}/examples/position_sense/tamagawa_diagnostic
-mcpu=cortex-r5
-mfloat-abi=hard
-mfpu=vfpv3-d16
@ -97,8 +98,6 @@
<pathVariable name="MOTOR_CONTROL_SDK_PATH" path="${COM_TI_MOTOR_CONTROL_SDK_AMXXX_INSTALL_DIR}" scope="project" />
<file path="../../../../tamagawa_diagnostic.c" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../../../../tamagawa_periodic_trigger.h" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../../../../tamagawa_periodic_trigger.c" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../main.c" openOnCreation="false" excludeFromBuild="false" action="copy">

View File

@ -39,7 +39,6 @@ endif
FILES_common := \
tamagawa_diagnostic.c \
tamagawa_periodic_trigger.h \
tamagawa_periodic_trigger.c \
main.c \
ti_drivers_config.c \
@ -63,6 +62,7 @@ INCLUDES_common := \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/FreeRTOS-Kernel/include \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/portable/TI_ARM_CLANG/ARM_CR5F \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/config/am243x/r5f \
-I${MOTOR_CONTROL_SDK_PATH}/examples/position_sense/tamagawa_diagnostic \
-Igenerated \
DEFINES_common := \

View File

@ -35,6 +35,7 @@
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/FreeRTOS-Kernel/include
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/portable/TI_ARM_CLANG/ARM_CR5F
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/config/am243x/r5f
-I${MOTOR_CONTROL_SDK_PATH}/examples/position_sense/tamagawa_diagnostic
-mcpu=cortex-r5
-mfloat-abi=hard
-mfpu=vfpv3-d16
@ -97,8 +98,6 @@
<pathVariable name="MOTOR_CONTROL_SDK_PATH" path="${COM_TI_MOTOR_CONTROL_SDK_AMXXX_INSTALL_DIR}" scope="project" />
<file path="../../../../tamagawa_diagnostic.c" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../../../../tamagawa_periodic_trigger.h" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../../../../tamagawa_periodic_trigger.c" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../main.c" openOnCreation="false" excludeFromBuild="false" action="copy">

View File

@ -39,7 +39,6 @@ endif
FILES_common := \
tamagawa_diagnostic.c \
tamagawa_periodic_trigger.h \
tamagawa_periodic_trigger.c \
main.c \
ti_drivers_config.c \
@ -63,6 +62,7 @@ INCLUDES_common := \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/FreeRTOS-Kernel/include \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/portable/TI_ARM_CLANG/ARM_CR5F \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/config/am243x/r5f \
-I${MOTOR_CONTROL_SDK_PATH}/examples/position_sense/tamagawa_diagnostic \
-Igenerated \
DEFINES_common := \

View File

@ -35,6 +35,7 @@
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/FreeRTOS-Kernel/include
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/portable/TI_ARM_CLANG/ARM_CR5F
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/config/am64x/r5f
-I${MOTOR_CONTROL_SDK_PATH}/examples/position_sense/tamagawa_diagnostic
-mcpu=cortex-r5
-mfloat-abi=hard
-mfpu=vfpv3-d16
@ -97,8 +98,6 @@
<pathVariable name="MOTOR_CONTROL_SDK_PATH" path="${COM_TI_MOTOR_CONTROL_SDK_AMXXX_INSTALL_DIR}" scope="project" />
<file path="../../../../tamagawa_diagnostic.c" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../../../../tamagawa_periodic_trigger.h" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../../../../tamagawa_periodic_trigger.c" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../main.c" openOnCreation="false" excludeFromBuild="false" action="copy">

View File

@ -39,7 +39,6 @@ endif
FILES_common := \
tamagawa_diagnostic.c \
tamagawa_periodic_trigger.h \
tamagawa_periodic_trigger.c \
main.c \
ti_drivers_config.c \
@ -63,6 +62,7 @@ INCLUDES_common := \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/FreeRTOS-Kernel/include \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/portable/TI_ARM_CLANG/ARM_CR5F \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/config/am64x/r5f \
-I${MOTOR_CONTROL_SDK_PATH}/examples/position_sense/tamagawa_diagnostic \
-Igenerated \
DEFINES_common := \

View File

@ -5,7 +5,6 @@ let device = "am243x";
const files = {
common: [
"tamagawa_diagnostic.c",
"tamagawa_periodic_trigger.h",
"tamagawa_periodic_trigger.c",
"main.c",
],
@ -36,6 +35,7 @@ const includes_freertos_r5f = {
"${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/FreeRTOS-Kernel/include",
"${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/portable/TI_ARM_CLANG/ARM_CR5F",
"${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/config/am243x/r5f",
"${MOTOR_CONTROL_SDK_PATH}/examples/position_sense/tamagawa_diagnostic",
],
};

View File

@ -5,7 +5,6 @@ let device = "am64x";
const files = {
common: [
"tamagawa_diagnostic.c",
"tamagawa_periodic_trigger.h",
"tamagawa_periodic_trigger.c",
"main.c",
],
@ -36,6 +35,7 @@ const includes_freertos_r5f = {
"${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/FreeRTOS-Kernel/include",
"${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/portable/TI_ARM_CLANG/ARM_CR5F",
"${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/config/am64x/r5f",
"${MOTOR_CONTROL_SDK_PATH}/examples/position_sense/tamagawa_diagnostic",
],
};

View File

@ -35,6 +35,7 @@
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/FreeRTOS-Kernel/include
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/portable/TI_ARM_CLANG/ARM_CR5F
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/config/am243x/r5f
-I${MOTOR_CONTROL_SDK_PATH}/examples/position_sense/tamagawa_diagnostic
-mcpu=cortex-r5
-mfloat-abi=hard
-mfpu=vfpv3-d16
@ -97,8 +98,6 @@
<pathVariable name="MOTOR_CONTROL_SDK_PATH" path="${COM_TI_MOTOR_CONTROL_SDK_AMXXX_INSTALL_DIR}" scope="project" />
<file path="../../../../tamagawa_diagnostic.c" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../../../../tamagawa_periodic_trigger.h" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../../../../tamagawa_periodic_trigger.c" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../main.c" openOnCreation="false" excludeFromBuild="false" action="copy">

View File

@ -39,7 +39,6 @@ endif
FILES_common := \
tamagawa_diagnostic.c \
tamagawa_periodic_trigger.h \
tamagawa_periodic_trigger.c \
main.c \
ti_drivers_config.c \
@ -63,6 +62,7 @@ INCLUDES_common := \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/FreeRTOS-Kernel/include \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/portable/TI_ARM_CLANG/ARM_CR5F \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/config/am243x/r5f \
-I${MOTOR_CONTROL_SDK_PATH}/examples/position_sense/tamagawa_diagnostic \
-Igenerated \
DEFINES_common := \

View File

@ -35,6 +35,7 @@
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/FreeRTOS-Kernel/include
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/portable/TI_ARM_CLANG/ARM_CR5F
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/config/am243x/r5f
-I${MOTOR_CONTROL_SDK_PATH}/examples/position_sense/tamagawa_diagnostic
-mcpu=cortex-r5
-mfloat-abi=hard
-mfpu=vfpv3-d16
@ -97,8 +98,6 @@
<pathVariable name="MOTOR_CONTROL_SDK_PATH" path="${COM_TI_MOTOR_CONTROL_SDK_AMXXX_INSTALL_DIR}" scope="project" />
<file path="../../../../tamagawa_diagnostic.c" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../../../../tamagawa_periodic_trigger.h" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../../../../tamagawa_periodic_trigger.c" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../main.c" openOnCreation="false" excludeFromBuild="false" action="copy">

View File

@ -39,7 +39,6 @@ endif
FILES_common := \
tamagawa_diagnostic.c \
tamagawa_periodic_trigger.h \
tamagawa_periodic_trigger.c \
main.c \
ti_drivers_config.c \
@ -63,6 +62,7 @@ INCLUDES_common := \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/FreeRTOS-Kernel/include \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/portable/TI_ARM_CLANG/ARM_CR5F \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/config/am243x/r5f \
-I${MOTOR_CONTROL_SDK_PATH}/examples/position_sense/tamagawa_diagnostic \
-Igenerated \
DEFINES_common := \

View File

@ -35,6 +35,7 @@
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/FreeRTOS-Kernel/include
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/portable/TI_ARM_CLANG/ARM_CR5F
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/config/am64x/r5f
-I${MOTOR_CONTROL_SDK_PATH}/examples/position_sense/tamagawa_diagnostic
-mcpu=cortex-r5
-mfloat-abi=hard
-mfpu=vfpv3-d16
@ -97,8 +98,6 @@
<pathVariable name="MOTOR_CONTROL_SDK_PATH" path="${COM_TI_MOTOR_CONTROL_SDK_AMXXX_INSTALL_DIR}" scope="project" />
<file path="../../../../tamagawa_diagnostic.c" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../../../../tamagawa_periodic_trigger.h" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../../../../tamagawa_periodic_trigger.c" openOnCreation="false" excludeFromBuild="false" action="copy">
</file>
<file path="../main.c" openOnCreation="false" excludeFromBuild="false" action="copy">

View File

@ -39,7 +39,6 @@ endif
FILES_common := \
tamagawa_diagnostic.c \
tamagawa_periodic_trigger.h \
tamagawa_periodic_trigger.c \
main.c \
ti_drivers_config.c \
@ -63,6 +62,7 @@ INCLUDES_common := \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/FreeRTOS-Kernel/include \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/portable/TI_ARM_CLANG/ARM_CR5F \
-I${MOTOR_CONTROL_SDK_PATH}/mcu_plus_sdk/source/kernel/freertos/config/am64x/r5f \
-I${MOTOR_CONTROL_SDK_PATH}/examples/position_sense/tamagawa_diagnostic \
-Igenerated \
DEFINES_common := \