Skip to content

Smw enclave upgrade to BSP LF6.12.20-2.0.0#2371

Open
zelan-nxp wants to merge 2 commits intoFreescale:masterfrom
nxp-upstream:smw-enclave-upgrade
Open

Smw enclave upgrade to BSP LF6.12.20-2.0.0#2371
zelan-nxp wants to merge 2 commits intoFreescale:masterfrom
nxp-upstream:smw-enclave-upgrade

Conversation

@zelan-nxp
Copy link
Contributor

swm: Upgrade to LF6.12.20-2.0.0
imx-secure-enclave: Upgrade to LF6.12.20-2.0.0
imx-secure-enclave-seco: Extend imx-secure-enclave for seco

@zelan-nxp zelan-nxp changed the title Smw enclave upgrade Smw enclave upgrade to BSP LF6.12.20-2.0.0 Sep 15, 2025
LIC_FILES_CHKSUM = "file://LICENSE;md5=8636bd68fc00cc6a3809b7b58b45f982"

DEPENDS = " mbedtls openssl"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The dependency mbedtls need hook in meta-openembedded/meta-networking layer.

Copy link
Member

Choose a reason for hiding this comment

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

In this case, this recipe should be moved over the dynamic recipes directory so we still allow the use of the layer without this dependency.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@thochstein what's your opinion move it to dynamic layer? i.MX release set hook meta-openembedded/meta-networking by default https://github.com/nxp-imx/meta-imx/blob/walnascar-6.12.34-2.1.0/tools/imx-setup-release.sh#L180

Copy link
Contributor

Choose a reason for hiding this comment

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

It is fine. It will not cause any problem for our setup.

Copy link
Member

Choose a reason for hiding this comment

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

I saw that you created a file in the dynamic layer and I'm wondering why you are not moving the recipe completely to there.

Comment on lines +26 to +24
SRCBRANCH_smw = "release/version_5.x"
SRCBRANCH_psa = "main"
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we should overcomplicate the recipe here. I think that would be better if we put those parameters in the source URL above.

LIC_FILES_CHKSUM = "file://LICENSE;md5=8636bd68fc00cc6a3809b7b58b45f982"

DEPENDS = " mbedtls openssl"

Copy link
Member

Choose a reason for hiding this comment

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

In this case, this recipe should be moved over the dynamic recipes directory so we still allow the use of the layer without this dependency.

Copy link
Contributor

@thochstein thochstein Sep 15, 2025

Choose a reason for hiding this comment

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

I did a local diff and found this file has no newline at end of file. This is also visible on the Files Changed page with a red icon.

Please add a final newline.

@angolini
Copy link
Member

#2325

@zelan-nxp zelan-nxp force-pushed the smw-enclave-upgrade branch 2 times, most recently from 4fe3947 to 85594a9 Compare September 16, 2025 06:57
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

Comments