Skip to content

Conversation

@sadik-smd
Copy link
Contributor

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.

@github-actions
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.

@praneethbajjuri
Copy link
Collaborator

@jmenti , @sskartheekadivi , @bryanbrattlof
To review

@cshilwant
Copy link
Member

@cshilwant
Copy link
Member

@jmenti , @sskartheekadivi , @bryanbrattlof To review

@sskartheekadivi @jmenti Any comments / feedback from your end?

@github-actions
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/RT_Linux_Performance_Guide.rst:61: (WARNING/2) The "csv-table" directive requires content; none supplied.
source/devices/AM62L/linux/Linux_Performance_Guide.rst:12: (ERROR/3) Malformed table.

@github-actions
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:25: (SEVERE/4) Problems with "include" directive path:

Copy link
Collaborator

@bryanbrattlof bryanbrattlof left a comment

Choose a reason for hiding this comment

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

those CPU and DDR frequencies look weird but this is a good start 👍

Copy link
Member

@StaticRocket StaticRocket left a comment

Choose a reason for hiding this comment

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

May want to use the directories AM62LX to keep in line with what was done for AM62PX. Otherwise this is fine.

@sadik-smd
Copy link
Contributor Author

May want to use the directories AM62LX to keep in line with what was done for AM62PX. Otherwise this is fine.

@praneethbajjuri to confirm the directory as 'AM62LX'

@cshilwant cshilwant dismissed their stale review February 20, 2025 12:24

Author has updated the PR

@cshilwant
Copy link
Member

@jeevantelukula @VeeruPrudhvi @jsuhaas22 for final reviews

@praneethbajjuri
Copy link
Collaborator

May want to use the directories AM62LX to keep in line with what was done for AM62PX. Otherwise this is fine.

@praneethbajjuri to confirm the directory as 'AM62LX'

Ack to keep it consistent "AM62LX"

@github-actions
Copy link

New warnings found with rstcheck:

source/devices/AM62LX/index.rst:9: (SEVERE/4) Problems with "include" directive path:
source/devices/AM62LX/index.rst:25: (SEVERE/4) Problems with "include" directive path:

Copy link
Member

@cshilwant cshilwant left a comment

Choose a reason for hiding this comment

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

@sadik-smd Provided more comments

@github-actions
Copy link

New warnings found with rstcheck:

source/devices/AM62LX/index.rst:9: (SEVERE/4) Problems with "include" directive path:
source/devices/AM62LX/index.rst:25: (SEVERE/4) Problems with "include" directive path:

@cshilwant cshilwant dismissed stale reviews from StaticRocket and themself February 21, 2025 17:15

Author has updated the PR

@praneethbajjuri praneethbajjuri merged commit 98f5e6f into TexasInstruments:master Feb 23, 2025
1 of 3 checks passed
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.

8 participants