You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There are at least 3 different targets that we need to build edbg for:
hostOS
petitboot
ebmc
Each of these environments have different build requirements and necessary supported function.
For example, ebmc will need istep support. But neither hostOS or petitboot need that function. That particular use case can be controlled by existing defines like ECMD_REMOVE_INIT_FUNCTIONS.
However, we will have cases internal to the code as well. One such case is ebmc needs to define a different help text path than hostOS/petitboot.
To solve this, I purpose each build target get its own compile define. That compile define can then be used to control the enabling of things like ECMD_REMOVE defines as well as unique paths in common edbg code.
I would purpose:
hostOS = EDBG_BUILD_HOSTOS
petitboot = EDBG_BUILD_PETITBOOT
ebmc = EDBG_BUILD_EBMC
When existing defines already do the job, we need to make sure to use those. We don't want the code littered with build target specific defines. Defines would be controlled by src/common/ecmdDefines.H (which isn't in master yet, but I have locally) and largely removed from being passed in on the command line of the compile.
The text was updated successfully, but these errors were encountered:
There are at least 3 different targets that we need to build edbg for:
Each of these environments have different build requirements and necessary supported function.
For example, ebmc will need istep support. But neither hostOS or petitboot need that function. That particular use case can be controlled by existing defines like ECMD_REMOVE_INIT_FUNCTIONS.
However, we will have cases internal to the code as well. One such case is ebmc needs to define a different help text path than hostOS/petitboot.
To solve this, I purpose each build target get its own compile define. That compile define can then be used to control the enabling of things like ECMD_REMOVE defines as well as unique paths in common edbg code.
I would purpose:
hostOS = EDBG_BUILD_HOSTOS
petitboot = EDBG_BUILD_PETITBOOT
ebmc = EDBG_BUILD_EBMC
When existing defines already do the job, we need to make sure to use those. We don't want the code littered with build target specific defines. Defines would be controlled by
src/common/ecmdDefines.H
(which isn't in master yet, but I have locally) and largely removed from being passed in on the command line of the compile.The text was updated successfully, but these errors were encountered: