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

Docs Update: Update the Top Level Makefile and cross-compile instructions #47

Merged
merged 2 commits into from
Nov 30, 2024

Conversation

jeevantelukula
Copy link
Collaborator

Updates the cross compile instructions for AM335x and AM437x to avoid the build errors while compiling the binaries for the target.
Update the docs for packages required by host while building with Top Level Makefile

cshilwant
cshilwant previously approved these changes Nov 21, 2024
source/linux/Overview/GCC_ToolChain.rst Outdated Show resolved Hide resolved
Copy link

New warnings found with rstcheck:

source/linux/Overview/GCC_ToolChain.rst:51: (INFO/1) Duplicate implicit target name: "linux-devkit".

Copy link

New warnings found with rstcheck:

source/linux/Overview/GCC_ToolChain.rst:51: (INFO/1) Duplicate implicit target name: "linux-devkit".

@jeevantelukula
Copy link
Collaborator Author

For AM335x/AM437x devices, CROSS_COMPILE_64, SYSROOT_64 and ENV_SETUP_64 paths are to be modified with appropritate locations in the installer.

@cshilwant cshilwant dismissed stale reviews from uditkumarti and StaticRocket November 22, 2024 15:32

Comments addressed by the author

source/linux/Overview/Top_Level_Makefile.rst Outdated Show resolved Hide resolved
source/linux/Overview/GCC_ToolChain.rst Outdated Show resolved Hide resolved
…Makefile

Update the host packages required for AM335x, AM437x, AM65x, AM57x devices for building the SDK using top level makefile

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

New warnings found with rstcheck:

source/linux/Overview/GCC_ToolChain.rst:51: (INFO/1) Duplicate implicit target name: "linux-devkit".

@cshilwant cshilwant dismissed StaticRocket’s stale review November 25, 2024 13:10

Changes addressed by the author

source/linux/Overview/GCC_ToolChain.rst Outdated Show resolved Hide resolved
Copy link

New warnings found with rstcheck:

source/linux/Overview/GCC_ToolChain.rst:51: (INFO/1) Duplicate implicit target name: "linux-devkit".

@cshilwant cshilwant dismissed StaticRocket’s stale review November 26, 2024 08:56

Changes addressed by the Author

@StaticRocket
Copy link
Member

AM57 is not mentioned in any of these common document changes. Is that intentional?

Copy link
Contributor

@nsaulnier-ti nsaulnier-ti left a comment

Choose a reason for hiding this comment

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

Without actually testing the steps, changes look good

Copy link

New warnings found with rstcheck:

source/linux/Overview/GCC_ToolChain.rst:51: (INFO/1) Duplicate implicit target name: "linux-devkit".

@jeevantelukula
Copy link
Collaborator Author

AM57 is not mentioned in any of these common document changes. Is that intentional?

These changes were applicable for AM57x too. Updated the files.

Copy link

New warnings found with rstcheck:

source/linux/Overview/GCC_ToolChain.rst:51: (INFO/1) Duplicate implicit target name: "linux-devkit".

configs/AM57X/AM57X_linux_config.txt Outdated Show resolved Hide resolved
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.

Update optee machine for AM57x

…M335x/AM437x/AM57x

Documentation update for the cross compile and sysroot paths specific to AM335x/AM437x/AM57x different from other sitara platforms

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

New warnings found with rstcheck:

source/linux/Overview/GCC_ToolChain.rst:51: (INFO/1) Duplicate implicit target name: "linux-devkit".

@cshilwant cshilwant dismissed StaticRocket’s stale review November 28, 2024 05:35

Changes Addressed by Author

@cshilwant cshilwant merged commit bd02d22 into TexasInstruments:master Nov 30, 2024
1 of 2 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.

5 participants