hal: add startup code for TMS570LC43 #63
Conversation
|
@vaishnavachath hello, can I request you to take a look at this PR as it block another PR in zephyr to add a new board. No hurry, but the original PR has been going on for quite a while now! |
| * | ||
| * These functions are adapted from TI Halcogen generated code. | ||
| */ | ||
|
|
There was a problem hiding this comment.
@ntavish can you please combine the both copyright statements together?
ntavish
force-pushed
the
feature/add-tms570lc43-halcogen-code
branch
from
9d1ece5 to
30d1509
Compare
|
@ntavish could you give more details why soc related zephyr code in HAL, e.g dts information its zephyr macro? |
Hello, the only reason to move this code to ti_hal is licensing, this is adapted from HALcoGen output, and I cannot relicense it. The only possible way to inlclude this is in zephyr seems to be to move it here. TMS570/hercules does not provide a HAL per se, only generated output from HALCoGen. Please check the related PR as well zephyrproject-rtos/zephyr#88741 Some files can be cleaned up to remove unnecessary includes from zephyr, and the dts macro could be moved to zephyr side - let me update the PR to improve that. |
Adds soc init code including MPU configuration and PLL errata handling for the TI Hercules family SoC TMS570LC43xx. Portions of this code were taken or adapter from Halcogen code generator. This HAL is not really complete, as the currently implemented drivers do not depend on Halcogen and so were not needed. Signed-off-by: Tavish Naruka <[email protected]>
ntavish
force-pushed
the
feature/add-tms570lc43-halcogen-code
branch
from
30d1509 to
43c7050
Compare
|
Moved the Apache 2.0 licensed headers out of here and added some description in a README as requested. (updated the zephyr side PR for TMS570 as well) |
Adds soc init code including MPU configuration and PLL errata handling for the TI Hercules family SoC TMS570LC43xx. Portions of this code were taken or adapter from Halcogen code generator.
This HAL is not really complete, as the currently implemented drivers do not depend on Halcogen and so were not needed. Main reason for this PR is the code license (BSD 3 clause).
This PR is related to this one on zephyr which adds support for TMS570: zephyrproject-rtos/zephyr#88741