Skip to content

Conversation

@jeevantelukula
Copy link
Collaborator

This adds the support for device specific build sheet information for the devices AM62X, AM62PX, AM64X and AM62DX in github documentation.

@github-actions
Copy link

New issues found with check_toc_txt.py:

WARNING:root:Invalid slug: configs/AM62DX/AM62DX_linux_toc.txt:14 devices/AM62DX/linux/Release_Specific_Workarounds
WARNING:root:Invalid slug: configs/AM62DX/AM62DX_linux_toc.txt:77 linux/How_to_Guides/Hardware_Setup_with_CCS/AM62DX_EVM_Hardware_Setup

StaticRocket
StaticRocket previously approved these changes Jul 12, 2025
@StaticRocket
Copy link
Member

Also the AM62X Release_Specific_Build_Sheet.rst doesn't end with a newline (?). Are you manually copying these around? I added some jinja templating for the index file in #358 . If we need to extend that more for massaging CSVs into this format we can. It's not that bad.

This adds the support for device specific build sheet
information for the devices AM62X, AM62PX, AM64X and AM62DX
in github documentation.

Signed-off-by: Telukula Jeevan Kumar Sahu <[email protected]>
@github-actions
Copy link

New issues found with check_toc_txt.py:

WARNING:root:Invalid slug: configs/AM62DX/AM62DX_linux_toc.txt:14 devices/AM62DX/linux/Release_Specific_Workarounds
WARNING:root:Invalid slug: configs/AM62DX/AM62DX_linux_toc.txt:77 linux/How_to_Guides/Hardware_Setup_with_CCS/AM62DX_EVM_Hardware_Setup

@jeevantelukula
Copy link
Collaborator Author

jeevantelukula commented Jul 14, 2025

Also the AM62X Release_Specific_Build_Sheet.rst doesn't end with a newline (?). Are you manually copying these around? I added some jinja templating for the index file in #358 . If we need to extend that more for massaging CSVs into this format we can. It's not that bad.

Thanks for the catch. I actually copied the entire csv file and added under the csv-table directive.
Updated the PR with new line character at the end of each file.

Copy link
Contributor

@VeeruPrudhvi VeeruPrudhvi left a comment

Choose a reason for hiding this comment

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

LGTM

@sadik-smd
Copy link
Contributor

@cshilwant and @StaticRocket to review.

@jeevantelukula jeevantelukula merged commit 8001858 into TexasInstruments:master Jul 14, 2025
3 of 6 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.

6 participants