uz3eg-iocc: add support for lmp-base#328
Merged
ricardosalveti merged 3 commits intofoundriesio:masterfrom Apr 30, 2021
Merged
Conversation
Support includes applying optee-os overlay and loading fpga bitstream before loading and jumping to the kernel. Signed-off-by: Ricardo Salveti <[email protected]>
Similar to lmp.cfg, but without a fixed bootcommand configuration. Signed-off-by: Ricardo Salveti <[email protected]>
Set correct WKS_FILE and boot files for lmp-base support. Signed-off-by: Ricardo Salveti <[email protected]>
ricardosalveti
requested review from
Tim-Anderson,
angolini,
ldts,
mike-scott and
vanmaegima
Tim-Anderson
approved these changes
Apr 29, 2021
Contributor
Tim-Anderson
left a comment
Tim-Anderson
left a comment
There was a problem hiding this comment.
One question but other than that LGTM
| bootcmd_load_k=fatload ${devtype} ${devnum}:1 ${kernel_addr_r} Image | ||
| bootcmd_args=setenv bootargs console=ttyPS0,115200 root=/dev/mmcblk1p2 rootwait rw | ||
| bootcmd_run=booti ${kernel_addr_r} - ${fdt_addr_r} | ||
| bootcmd=run bootcmd_dtb; run bootcmd_tee_ovy; run bootcmd_fpga; run bootcmd_args; run bootcmd_load_k; run bootcmd_run |
Contributor
There was a problem hiding this comment.
why is this a .in file since there is nothing in here to be substituted
Member
Author
There was a problem hiding this comment.
This is just because the recipe expects a file named like that, in case there is something to be dynamically replaced (which is not required for this file in particular).
.in would be used in cases such as rpi, which would have a different boot command for 32 and 64 bits.
vanmaegima
approved these changes
Apr 30, 2021
angolini
approved these changes
Apr 30, 2021
quaresmajose
pushed a commit
that referenced
this pull request
Nov 9, 2022
Relevant changes: - e785ded Merge pull request #338 from MaxKellermann/underscore - 78f05c8 classes/sdcard_image-sunxi.bbclass: fix misplaced ":" - 02df911 cover up build error with kernel headers - 897f9c0 move ext4 dep - 1f4dba3 fix sd-card creation - use new syntax - 080eb40 conf: Set LAYERSERIES_COMPAT to honister only - dcd5790 conf: machine: Fix paths for tunes - 6a1103b misc: More fixes for new syntax - 668bc68 conf: Added honister layer to LAYERSERIAS_COMPAT - fd330bb Merge pull request #328 from dev-0x7C6/master - e66258a layer: convert to new override syntax 💥 Signed-off-by: Daiane Angolini <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.