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

Make initrd-flash support more configurable #1140

Merged
merged 3 commits into from
Jan 17, 2023

Conversation

madisongh
Copy link
Member

Clean up some global settings so that distros/users that want to use an alternative initrd-flash image, or exclude that support altogether to cut down on build times, can do so.

TEGRAFLASH_INITRD_FLASH_IMAGE can be set to the null string to exclude support, or to the name of another recipe.

Also refactor some more common settings to tegra-common.inc, and avoid direct appends to INITRAMFS_IMAGE_FSTYPES, allowing users to specify alternative types for their initramfs builds by setting TEGRA_INITRAMFS_FSTYPES (and TEGRA_INITRD_FLASH_INITRAMFS_FSTYPES for the initrd flasher).

* Move common settings from SoC-specific includes to tegra-common.inc
* Avoid direct :append to INITRAMFS_FSTYPES
* Use ?= instead of hard = for more settings
* Use separate variable for initrd-flash initrafms fstype

Signed-off-by: Matt Madison <[email protected]>
The initrd-flash kernel/initrd image has to be the Android
(cboot-style) image type, and now that we have a separate
variable in the machine configs to specify that, use that
variable to set INITRAMFS_FSTYPES in the recipe.

Signed-off-by: Matt Madison <[email protected]>
Make the image dependencies and inclusion of the initrd-flash
image conditional on whether TEGRAFLASH_INITRD_FLASH_IMAGE is
non-null.  This allows users to exclude initrd-flash support
from their builds, if desired.

Signed-off-by: Matt Madison <[email protected]>
@quaresmajose
Copy link
Contributor

This should solve a couple of issues and with it in place we can chose the initrd types for other images that may exist and use the one specified in this layer for flashing

@ichergui
Copy link
Member

ichergui commented Jan 17, 2023

Thanks @madisongh.
LGTM
I will definitely try the master branch later and will let you know

@madisongh madisongh merged commit f3f9936 into master Jan 17, 2023
@madisongh madisongh deleted the make-initrd-flash-optional branch January 17, 2023 15:58
quaresmajose added a commit to quaresmajose/meta-lmp that referenced this pull request Jan 20, 2023
TEGRA_INITRAMFS_FSTYPES it been introduced to avoid changing
the global INITRAMFS_FSTYPES that affects any recipe.

OE4T/meta-tegra#1140

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to quaresmajose/meta-lmp that referenced this pull request Jan 20, 2023
TEGRA_INITRAMFS_FSTYPES it been introduced to avoid changing
the global INITRAMFS_FSTYPES that affects every recipe.

OE4T/meta-tegra#1140

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to quaresmajose/meta-lmp that referenced this pull request Jan 20, 2023
TEGRA_INITRAMFS_FSTYPES it's been introduced to avoid changing
the global INITRAMFS_FSTYPES that affects every recipe.

OE4T/meta-tegra#1140

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Jan 20, 2023
TEGRA_INITRAMFS_FSTYPES it's been introduced to avoid changing
the global INITRAMFS_FSTYPES that affects every recipe.

OE4T/meta-tegra#1140

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to quaresmajose/meta-lmp that referenced this pull request Jan 23, 2023
TEGRA_INITRAMFS_FSTYPES it's been introduced to avoid changing
the global INITRAMFS_FSTYPES that affects every recipe.

OE4T/meta-tegra#1140

Signed-off-by: Jose Quaresma <[email protected]>
ricardosalveti pushed a commit to foundriesio/meta-lmp that referenced this pull request Jan 23, 2023
TEGRA_INITRAMFS_FSTYPES it's been introduced to avoid changing
the global INITRAMFS_FSTYPES that affects every recipe.

OE4T/meta-tegra#1140

Signed-off-by: Jose Quaresma <[email protected]>
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.

3 participants