Skip to content

Port to Yocto scarthgap#1532

Closed
quaresmajose wants to merge 144 commits intofoundriesio:mainfrom
quaresmajose:scarthgap-next
Closed

Port to Yocto scarthgap#1532
quaresmajose wants to merge 144 commits intofoundriesio:mainfrom
quaresmajose:scarthgap-next

Conversation

@quaresmajose
Copy link
Copy Markdown
Member

It's not ready yet, but there's not much left and it's already at a point where it can start to be reviewed.

The commits that start with [FIXME]/ [WIP] still need to be worked on

  • [FIXME] bsp: lmp-machine-custom: drop recovery
  • [FIXME] bsp: lmp-machine-custom: imx6ullevk: drop KERNEL_DEVICETREE
  • [FIXME] base: systemd: drop patch
  • [FIXME] base: optee-os-fio: drop patch
  • [WIP] base: docker-compose: drop patch
  • [WIP] base: docker-ce -> docker-moby

@quaresmajose quaresmajose requested a review from a team November 7, 2024 18:46
@ricardosalveti
Copy link
Copy Markdown
Member

We should probably break this pr in smaller ones, otherwise it makes it impossible to review and merge. We can separate the fixme patches as well.

Not concerned to have main in a broken / partial state at this stage, until we land everything we need.

file://0003-FIO-interal-stm32mp1-support-for-boot-script-in-FIP.patch \
"

SRC_URI:append:stm32mp1common = " file://0001-Binutils-2.39-now-warns-when-a-segment-has-RXW-permi.patch"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why this is only for stm32mp1common?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good question, maybe is not need anymore. I will check locally

@@ -1,9 +1,2 @@
FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}:"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there is a typo on this commit log

pacthes/patches

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, I will fix it

@quaresmajose
Copy link
Copy Markdown
Member Author

We should probably break this pr in smaller ones, otherwise it makes it impossible to review and merge. We can separate the fixme patches as well.

Not concerned to have main in a broken / partial state at this stage, until we land everything we need.

I'm going to try to break it into some pieces but in a way that doesn't break the main branch.

Because TI uses multiconfig we can't have this file names shared between machines in [1]

| ERROR: mc:k3r5:u-boot-ti-staging-1_2023.04+git-r0 do_deploy: Recipe u-boot-ti-staging is trying to install files into a shared area when those files already exist. The files and the manifests listing them are:
|   /build/deploy/images/am62xx-evm/u-boot-ti-staging-config
|     (matched in manifest-am62xx_evm-u-boot-ti-staging.deploy)
| Please adjust the recipes so only one recipe provides a given file.

[1] meta-lmp-base/recipes-bsp/u-boot/u-boot-lmp-common.inc

Signed-off-by: Jose Quaresma <[email protected]>
Signed-off-by: Jose Quaresma <[email protected]>
- Update patches with devtool

| Applying patch 0001-85-nm-unmanaged.rules-do-not-manage-docker-bridges.patch
| patching file data/85-nm-unmanaged.rules
| Hunk #1 succeeded at 30 with fuzz 2 (offset 4 lines).

Signed-off-by: Jose Quaresma <[email protected]>
This reverts commit 35696b6.

These patches are already included in ovmf 2022.11

Signed-off-by: Jose Quaresma <[email protected]>
Use the meta-openembedded version that is updated to b4e0673

Signed-off-by: Jose Quaresma <[email protected]>
We are facing some build erros on uz3eg-iocc-sec machine
because we don't have sufficient space.

| spl/u-boot-spl.bin exceeds file size limit:
|   limit:  0x26000 bytes
|   actual: 0x26009 bytes
|   excess: 0x9 bytes

Signed-off-by: Jose Quaresma <[email protected]>
Signed-off-by: Jose Quaresma <[email protected]>
| devtool finish --force-patch-refresh bluez5 ../layers/meta-lmp/meta-lmp-base

Signed-off-by: Jose Quaresma <[email protected]>
Signed-off-by: Jose Quaresma <[email protected]>
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]>
The imx-boot-container [1] is now only enabled on mx8m-generic-bsp
machines.

[1] Freescale/meta-freescale#1488

Signed-off-by: Jose Quaresma <[email protected]>
…dynamic

Add boot container dynamic selection to imx-base.inc
Freescale/meta-freescale#1492

Signed-off-by: Jose Quaresma <[email protected]>
angolini and others added 24 commits November 11, 2024 18:42
Since [1] the uboot-sign class is being refactored and some files were
reorganized.

Taking concat_dtb_helper as example, the dtb files once installed
previous the deploy were migrated to the deploy. It makes only the
simple named dtb files being available during the fitimage its creation.

Because of that, use the variables with the file names for signed or not
signed u-boot dtb files.

The instalation directory also changed.

[1] openembedded/openembedded-core@5e12dc9

Signed-off-by: Daiane Angolini <[email protected]>
Signed-off-by: Jose Quaresma <[email protected]>
Drop lib-zlib-Fix-a-bug-when-getting-a-gzip-header-extra-field.patch

Signed-off-by: Jose Quaresma <[email protected]>
Relevant changes:
- cc8b1802 ktypes/standard: enable CONFIG_CRC64

Signed-off-by: Jose Quaresma <[email protected]>
When we have lmpdebug in DISTRO_FEATURES it will be good to enable systemd coredump

Signed-off-by: Jose Quaresma <[email protected]>
The IMAGE_NAME already includes the IMAGE_NAME_SUFFIX in the scarthgap

IMAGE_NAME="lmp-base-console-image-intel-corei7-64.rootfs-20241014163026"

Signed-off-by: Jose Quaresma <[email protected]>
This image is not used anywhere

Signed-off-by: Jose Quaresma <[email protected]>
When a default dtb is not used the fit now points to the default kernel
index, which is 1 by default.

Signed-off-by: Ricardo Salveti <[email protected]>
Based on scarthgap.

Signed-off-by: Ricardo Salveti <[email protected]>
Align base version with the same default version used in meta-rpi.

Signed-off-by: Ricardo Salveti <[email protected]>
Relevant changes:
- 16feae8a bsp: add BSP fragments for raspberrypi5
- 9bff1de1 bsp: raspberrypi4: config updates for 6.6
- 94d1435a kver: bumping to v6.6.54
- 3c4731cb netfilter: Enable CONFIG_NETFILTER_XTABLES_COMPAT
- 1dd9e613 features/intel-npu: introduce Intel NPU fragment
- 12e25b9a bsp/intel-corei7-64: enable Intel IOMMU support
- 130af14e features/intel-pinctrl: add pinctrl driver for Intel Meteor Lake
- dd8d897b full_nohz: remove RCU_FAST_NO_HZ gone from upstream
- 8f915afd cgl: drop SELINUX_DISABLE option
- 18f9024a nft_test.cfg: Enable CONFIG_VETH
- f4c820c0 features/nf_tables: Add net_fib_* options for greater ptest coverage
- 754686f5 features/nf_tables: nft_objref is now builtin
- 4bf6d3f7 features/net_sched: Add MULTIQ and NET_EMATCH config
- d3a802b6 features/net: Add xdp feature config
- 97f449ea features/vfio: remove CONFIG_VFIO_VIRQFD

Signed-off-by: Ricardo Salveti <[email protected]>
@quaresmajose
Copy link
Copy Markdown
Member Author

I will propose a new version soon in smaller pieces

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.

4 participants