-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(linux): Initial documentation support for AM62L platform. #161
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: sadik <s-sadik@ti.com>
New warnings found with rstcheck:
|
'__SDK_SHORT_NAME__' : 'PSDK' | ||
'__SDK_INSTALL_FILE__' : 'ti-processor-sdk-linux-am62lxx-evm-<version>-Linux-x86-Install.bin' | ||
'__SDK_INSTALL_DIR__' : 'ti-processor-sdk-linux-am62lxx-evm-<version>' | ||
'__SDK_DOWNLOAD_URL__' : '`AM62L-SDK-Download-page <https://www.ti.com/tool/PROCESSOR-SDK-AM62L>`__' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This won't be PROCESSOR-SDK-AM62L
. It would be AM62L-LINUX-SDK
as per new convention
@VeeruPrudhvi Can you confirm the above
'CONFIG_image_type' : 'default' | ||
'CONFIG_icss_support' : 'yes' | ||
'CONFIG_rt_linux_support' : 'yes' | ||
'CONFIG_gpu_ip' : 'Rogue_BXS' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CONFIG_gpu_ip
can be dropped as AM62L doesn't have a GPU
# Project name and HTML title | ||
sdk_product = 'null' #todo: remove after the new structure is used for all device families | ||
|
||
project = u'Processor SDK Linux for AM62L' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this would be Linux SDK for AM62L
as per the new convention
@praneethbajjuri @VeeruPrudhvi to confirm
linux/Foundational_Components/Virtualization/Docker | ||
linux/Foundational_Components_Hypervisor | ||
linux/Foundational_Components/Hypervisor/Jailhouse | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Graphics is not applicable for AM62L.
So, Line 117 to 128 can be dropped
linux/How_to_Guides/FAQ/How_to_Integrate_Open_Source_Software | ||
linux/How_to_Guides_Hardware_Setup_with_CCS | ||
linux/How_to_Guides/Hardware_Setup_with_CCS/AM62L_EVM_Hardware_Setup | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- system/Demo_User_Guides/TI_Apps_Launcher_User_Guide
- system/Demo_User_Guides/Seva_Store
- system/Demo_User_Guides/Display_Cluster_User_Guide
- system/Demo_User_Guides/Chromium_Browser
All above can be dropped
.. code-block:: text | ||
|
||
In which directory do you want to install the target filesystem?(if this directory does not exist it will be created) | ||
[ /home/user/ti-processor-sdk-linux-am62pxx-evm-<version>/targetNFS ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[ /home/user/ti-processor-sdk-linux-am62pxx-evm-<version>/targetNFS ] | |
[ /home/user/ti-processor-sdk-linux-am62lxx-evm-<version>/targetNFS ] |
- Linux Manifest: :file:`<PSDK_PATH>/docs/software_manifest.html` | ||
|
||
|
||
Release 11.00 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Drop everything from Line 39 until end.
This will be added at the time of release
|
||
This page covers migration information for applications built on top | ||
of the Processor SDK Linux AM62L. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Drop everything from Line 12 until end.
This will be added at the time of release
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Drop this file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Drop this file
@jmenti , @sskartheekadivi , @bryanbrattlof |
|
|
||
.. note:: | ||
Processor SDK Linux AM62Px contains only the Linux specific source and application intended | ||
to runs on A53/Linux core. For R5F and RTOS/NO-RTOS side source and applications, refer **MCU+ SDK** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's no R5 core in AM62L.
This can be reframed as,
to runs on A53/Linux core. For R5F and RTOS/NO-RTOS side source and applications, refer **MCU+ SDK** | |
to run on A53/Linux core. For A53 based FreeRTOS side source and applications, refer **MCU+ SDK** |
@@ -0,0 +1,6 @@ | |||
devices/AM62X/buildroot/index | |||
devices/AM62X/buildroot/Getting_Started_Guide | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It should be devices/AM62L
here.
devices/AM62X/buildroot/index | ||
devices/AM62X/buildroot/Getting_Started_Guide | ||
devices/AM62L/buildroot/index | ||
devices/AM62L/buildroot/Getting_Started_Guide | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No changes required for this file
/system/Demo_User_Guides/TI_Apps_Launcher_User_Guide | ||
/system/Demo_User_Guides/Seva_Store | ||
/system/Demo_User_Guides/Display_Cluster_User_Guide | ||
/system/Demo_User_Guides/Chromium_Browser |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
||
**For MCU+ SDK RTOS/NO-RTOS documentation, refer below links** | ||
|
||
- `MCU+ SDK Documentation <https://software-dl.ti.com/mcu-plus-sdk/esd/AM62PX/10_01_00_33/exports/docs/api_guide_am62px/index.html>`__ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dont think this file is needed.
+----------------+---------------------------------------------------------------------------------------------------+ | ||
| Name | Description | | ||
+================+===================================================================================================+ | ||
| AM62L EVM | AM62L Starter Kit rev E1 with ARM running at 1200MHz, LPDDR4 data rate 3200 MT/S | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/Starter Kit/Evaluation Module/
3200MT/s is a little fast. You may want to double check this line
|
||
"Minimum (usec)","5","5","5","5" | ||
"Average (usec)","6","6","5","6" | ||
"Maximum (usec)","30","33","31","35" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah ok I think I get what happened. Feel free to drop the RT performance stuff until we get some numbers tested. This looks like it came from 62P or 62A
This PR adds the necessary source files in configs/ and sources/
and the README for AM62L platform. Taking reference
from existing AM62Px platform structure and updating it
accordingly for the AM62L.