Skip to content

Remove conditional imx-boot-container configuration from machines and move to extender#1465

Merged
otavio merged 2 commits intomasterfrom
topic/simplify-imx-boot-container
Mar 31, 2023
Merged

Remove conditional imx-boot-container configuration from machines and move to extender#1465
otavio merged 2 commits intomasterfrom
topic/simplify-imx-boot-container

Conversation

@otavio
Copy link
Copy Markdown
Member

@otavio otavio commented Mar 29, 2023

This pull request includes two commits that make changes related to the imx-boot-container configuration.

The first commit removes the conditional imx-boot-container MACHINEOVERRIDES from machines. The conditional belongs to the bootloader recipe and can be removed from machines to simplify the configuration.

The second commit moves the imx-boot-container MACHINEOVERRIDES to extender as it is SoC specific and should not be set inside the machine. The commit moves it to the imx-base.inc file to ensure proper configuration.

Together, these changes improve the configuration of imx-boot-container in our software project by removing unnecessary conditional statements from machines and moving the configuration to a more appropriate location.

otavio added 2 commits March 29, 2023 13:57
The imx-boot-container MACHINEOVERRIDES allows the bootloader recipe to
enable the binman support for the SoCs which uses it. The conditional
belongs to the bootloader recipe and can be removed from machines.

Signed-off-by: Otavio Salvador <[email protected]>
The imx-boot-container is SoC specific so it should not be set inside
the machine but in the BSP basic settings. This commit moves it to the
imx-base.inc file as the bootloader recipe can still opt to not inherit
the class specifically.

Signed-off-by: Otavio Salvador <[email protected]>
@otavio otavio requested a review from thochstein March 29, 2023 18:44
@otavio otavio marked this pull request as ready for review March 29, 2023 18:44
@thochstein
Copy link
Copy Markdown
Contributor

I think it makes sense.

Note that the imx-boot recipe is used by NXP for all i.MX 8 families. Do you know why we limit imx-boot-container to 8M?

@thochstein
Copy link
Copy Markdown
Contributor

Actually, I think you should set UBOOT_PROVIDES_BOOT_CONTAINER with a weak default.

@otavio
Copy link
Copy Markdown
Member Author

otavio commented Mar 29, 2023

Actually, I think you should set UBOOT_PROVIDES_BOOT_CONTAINER with a weak default.

This variable can be removed as well. The class would need more rework, but I prefer to do that as another PR so we don't mix changes.

@otavio otavio requested review from junzhuimx and zandrey March 29, 2023 20:34
@otavio
Copy link
Copy Markdown
Member Author

otavio commented Mar 29, 2023

About limiting it to i.MX8M SoCs, @zandrey did most of this work. But first, we must verify which platforms already use binman inside U-Boot.

Either way, we should first clean up this, and later, we can extend the functionality.

@otavio otavio merged commit 9bbadd4 into master Mar 31, 2023
@otavio otavio deleted the topic/simplify-imx-boot-container branch March 31, 2023 13:34
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Apr 4, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Apr 10, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Apr 10, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Apr 14, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Apr 24, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Apr 26, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Apr 27, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Apr 28, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request May 3, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request May 3, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request May 5, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request May 8, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request May 8, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request May 9, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request May 10, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request May 10, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Jun 29, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Jun 29, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Jun 29, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Jun 30, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Jul 3, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Jul 4, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Jul 4, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Jul 7, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Jul 13, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Jul 31, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Aug 3, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Aug 4, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to quaresmajose/meta-lmp that referenced this pull request Aug 8, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Aug 8, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Aug 11, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Aug 28, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Aug 30, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Sep 4, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Sep 5, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Sep 6, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Sep 8, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Sep 18, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Sep 22, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Oct 2, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Oct 9, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Oct 12, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Oct 13, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Oct 19, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Oct 19, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

Signed-off-by: Jose Quaresma <[email protected]>
quaresmajose added a commit to foundriesio/meta-lmp that referenced this pull request Oct 19, 2023
The imx-boot-container MACHINEOVERRIDES is now SoC specific [1]
with higher prioririty than mx8-generic-bsp.
The meta-freescale layer also check for 1/0 so set it to 0.

[1] Freescale/meta-freescale#1465

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.

2 participants