AM64 U-boot Remote proc support and conditional compile MCUR5 boot variations for J7 devices.#257
AM64 U-boot Remote proc support and conditional compile MCUR5 boot variations for J7 devices.#257h-nagalla wants to merge 4 commits intoTexasInstruments:masterfrom
Conversation
h-nagalla
requested review from
StaticRocket,
VeeruPrudhvi,
aniket-l,
cshilwant,
praneethbajjuri and
uditkumarti
as code owners
github-actions
bot
assigned
bryanbrattlof,
devarsht,
jmenti,
praneethbajjuri,
r-vignesh and
uditkumarti
github-actions
bot
requested review from
bryanbrattlof,
devarsht,
jmenti and
r-vignesh
praneethbajjuri
left a comment
praneethbajjuri
left a comment
There was a problem hiding this comment.
Minor typo in commitmsg on "jacinto" name, otherwise lgtm.
@3V3RYONE for review as well.
|
|
||
| Cortex-R5F can be used in both lockstep (both cores run same code) and split | ||
| mode (each core runs independently). | ||
| Cortex-R5F can be used in both lockstep (both cores run same code) and split |
There was a problem hiding this comment.
| Cortex-R5F can be used in both lockstep (both cores run same code) and split | |
| Cortex-R5F can be used in both lockstep (both cores run same code) and split |
Do not indent section contents.
There was a problem hiding this comment.
Thanks. Will fix the indents.
| Lockstep and Split mode | ||
| ^^^^^^^^^^^^^^^^^^^^^^^ | ||
| .. ifconfig:: CONFIG_part_variant in ('J722S') | ||
| .. ifconfig:: (CONFIG_sdk not in ('SITARA')) and (CONFIG_part_variant not in ('J722S')) |
There was a problem hiding this comment.
Invalid use of ifconfig:
https://www.sphinx-doc.org/en/master/usage/extensions/ifconfig.html
Sections count as structural changes.
There was a problem hiding this comment.
Humm. you mean, we can't use logical boolean expressions to select content? Chrome bowser AI extension suggested this conditional inclusion using python boolean expression.
There was a problem hiding this comment.
No, I mean the ifconfig directive only masks content. It's not like ifdef. That content is still parsed. The ref engine is a giant state machine that controls all documents references and sections. Because of this the ifconfigs can mask sections that still have a ref, resulting broken links and toctrees. Move the section header outside of the ifconfig.
Update documentation to reflect support for both LPDDR4 and DDR4 on AM62L devices. Signed-off-by: Hari Nagalla <hnagalla@ti.com>
This reverts commit c3fcc6e.
Documentation for u-boot remoteproc loading and starting on AM64x devices. Signed-off-by: Hari Nagalla <hnagalla@ti.com>
Sitara devices do not use MCU-R5 for boot purposes. So, keep MCU-R5 specifics to Jancinto devices. Signed-off-by: Hari Nagalla <hnagalla@ti.com>
|
Fixed the comments and sending a new PR |
8 participants
a) Add support for AM64 u-boot remote proc boot.
b) Conditional compile of MCU-R5 boot variations for J7 devices.