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

firmware-qcom-boot-common: provide virtual/bootbins #72

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ricardosalveti
Copy link
Contributor

Define a common virtual provider for bootbins, as we will later have multiple recipes providing boot binaries (based on base boards / SoCs).

This is to help creating generic dependencies to the boot firmware recipe.

Define a common virtual provider for bootbins, as we will later have
multiple recipes providing boot binaries (based on base boards / SoCs).

This is to help creating generic dependencies to the boot firmware
recipe.

Signed-off-by: Ricardo Salveti <[email protected]>
@ndechesne
Copy link
Contributor

the name for the virtual provide is too generic, if we define a new provider, it should be at least with a qcom namespace.

how about we wait until we really need this new provider? For now we are setting the dependency in the machine conf file, so we don't need the virtual provider. or am i missing something?

@ricardosalveti
Copy link
Contributor Author

the name for the virtual provide is too generic, if we define a new provider, it should be at least with a qcom namespace.

how about we wait until we really need this new provider? For now we are setting the dependency in the machine conf file, so we don't need the virtual provider. or am i missing something?

Yup, I was looking at some of the hooks we use in meta-partner / meta-qcom-hwe and noticed the virtual was missing, but I agree we can hold this up until there is an actual need and user for it.

@ricardosalveti ricardosalveti marked this pull request as draft November 15, 2024 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants