Skip to content
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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sadik-smd
Copy link

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.

Copy link

New warnings found with rstcheck:

source/devices/AM62L/index.rst:9: (SEVERE/4) Problems with "include" directive path:
source/devices/AM62L/index.rst:26: (SEVERE/4) Problems with "include" directive path:
source/devices/AM62L/linux/Release_Specific_Release_Notes.rst:5: (INFO/1) Duplicate implicit target name: "u-boot".
source/devices/AM62L/linux/Release_Specific_Workarounds.rst:4: (INFO/1) Duplicate implicit target name: "workarounds".
source/devices/AM62L/linux/Release_Specific_Migration_Guide.rst:27: (INFO/1) Duplicate implicit target name: "processor sdk 11.00".
source/devices/AM62L/linux/Overview/Download_and_Install_the_SDK.rst:11: (ERROR/3) Malformed table.

'__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>`__'
Copy link
Member

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'
Copy link
Member

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'
Copy link
Member

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

Copy link
Member

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

Copy link
Member

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 ]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[ /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
Copy link
Member

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.

Copy link
Member

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

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Drop this file

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Drop this file

@praneethbajjuri
Copy link
Collaborator

@jmenti , @sskartheekadivi , @bryanbrattlof
To review

@cshilwant
Copy link
Member


.. 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**
Copy link
Member

@cshilwant cshilwant Feb 11, 2025

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,

Suggested change
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

Copy link
Collaborator

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

Copy link
Collaborator

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
Copy link
Collaborator

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>`__
Copy link
Collaborator

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 |
Copy link
Collaborator

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"
Copy link
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants