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

meta-bsp: layer.conf: Provide UBOOT defaults instead overrides #17

Open
wants to merge 1 commit into
base: langdale-6.1.1-1.0.0
Choose a base branch
from

Conversation

AtanasBunchev
Copy link

Commit 2655f05
turns the definition of default values for
UBOOT_MAKE_TARGET and UBOOT_SUFFIX to overrides.

Overrides are not good because they can't be redefined easily. (Values need to be changed if using customized u-boot-imx.)

Defining default values is sufficient here.

Commit 2655f05
turns the definition of default values for
UBOOT_MAKE_TARGET and UBOOT_SUFFIX to overrides.

Overrides are not good because they can't be redefined easily.
(Values need to be changed if using customized u-boot-imx.)

Defining default values is sufficient here.

Signed-off-by: Atanas Bunchev <atanas.bunchev@konsulko.com>
@AtanasBunchev
Copy link
Author

I ran a bitbake -e test on my patch with Poky and both of these variables get their values from the meta-bsp/conf/layer.conf defaults, so the patch should not affect current builds.

@AtanasBunchev
Copy link
Author

Hi.
I'd like to ask if there are any issues with my pull request.
Is there anything i can help with to get the patch merged?

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.

1 participant