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

imx-boot: Fix 8M non multi-config build problem #2009

Merged
merged 1 commit into from
Oct 29, 2024

Conversation

esben
Copy link
Contributor

@esben esben commented Oct 29, 2024

The fix made in commit 2db7047
("imx-boot: Fix 8M multi-config build problems")
broke builds not using U-Boot multi-config, as the link created ends up being a simple recursive link when UBOOT_DTB_NAME_EXTRA is the same as UBOOT_DTB_NAME.

It fails with something like this:

| ./../scripts/dtb_check.sh imx8mq-evk.dtb evk.dtb imx8mq-var-dart-dt8mcustomboard.dtb
|  Can't find u-boot DTB file, please copy from u-boot

caused by a symlink like this:

lrwxrwxrwx 1 1000 1000 35 Oct 29 21:32 imx8mq-var-dart-dt8mcustomboard.dtb -> imx8mq-var-dart-dt8mcustomboard.dtb

Fixes: 2db7047 ("imx-boot: Fix 8M multi-config build problems")

The fix made in commit 2db7047
("imx-boot: Fix 8M multi-config build problems")
broke builds not using U-Boot multi-config, as the link created ends up being a
simple recursive link when UBOOT_DTB_NAME_EXTRA is the same as UBOOT_DTB_NAME.

It fails with something like this:
```
| ./../scripts/dtb_check.sh imx8mq-evk.dtb evk.dtb imx8mq-var-dart-dt8mcustomboard.dtb
|  Can't find u-boot DTB file, please copy from u-boot
```

caused by a symlink like this:
```
lrwxrwxrwx 1 esben 1000001 35 Oct 29 21:32 imx8mq-var-dart-dt8mcustomboard.dtb -> imx8mq-var-dart-dt8mcustomboard.dtb
```

Fixes: 2db7047 ("imx-boot: Fix 8M multi-config build problems")
Signed-off-by: Esben Haabendal <[email protected]>
@esben
Copy link
Contributor Author

esben commented Oct 29, 2024

Backport to scarthgap branch is needed.

Copy link

Successfully created backport PR #2010 for scarthgap.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants