Skip to content

fix(build): update the build commands for AM62L TFA and Uboot#710

Open
ti-scaria wants to merge 1 commit into
TexasInstruments:masterfrom
ti-scaria:badge-build
Open

fix(build): update the build commands for AM62L TFA and Uboot#710
ti-scaria wants to merge 1 commit into
TexasInstruments:masterfrom
ti-scaria:badge-build

Conversation

@ti-scaria
Copy link
Copy Markdown
Contributor

The target boards mentioned for building TFA on AM62L-badge is incorrect. Updating it from 'am62l3_badge' to 'am62l3-badge'.

Build command for bl1 is incorrect as there is no make target for 'am62l_bl1' on ti-tfa-2.14.y. Updating it with correct build commands for both ti-master and ti-tfa.2.14.y.

Also enhancing the build instuctions by including the correct PLAT and TFA_BOARD configuration for building on both ti-master and ti-tfa-2.14.y.


$ make CROSS_COMPILE="$CROSS_COMPILE_64" ARCH=aarch64 PLAT=k3low TARGET_BOARD=am62lx am62l_bl1
$ make CROSS_COMPILE="$CROSS_COMPILE_64" ARCH=aarch64 PLAT=k3low TARGET_BOARD=am62lx bl1 # on ti-tfa-2.14.y
$ make CROSS_COMPILE="$CROSS_COMPILE_64" ARCH=aarch64 PLAT=k3 TARGET_BOARD=am62l am62l_bl1 # on ti-master
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This ti-master(intended for 11.x) is not applicable form docs master branch(intended for 12.x).

Please strip that part and send pr for '11' branch seperately.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

okay, ill only include the changes for ti-tfa-2.14.y branch here

The target boards mentioned for building TFA on AM62L-badge is incorrect.
Updating it from 'am62l3_badge' to 'am62l3-badge'.

Build command for bl1 is incorrect as there is no make target for
'am62l_bl1' on ti-tfa-2.14.y. Updating it with correct build commands
for ti-tfa.2.14.y.

Signed-off-by: Scaria Kochidanadu <s-kochidanadu@ti.com>
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