* [meta-ti][PATCH 1/3] ti-k3-secdev: move from meta-ti-extras to meta-ti-bsp @ 2023-01-16 12:16 Jose Quaresma 2023-01-16 12:16 ` [meta-ti][PATCH 2/3] ti-k3-secdev: sets TI_K3_SECDEV_INSTALL_DIR_RECIPE default value Jose Quaresma ` (2 more replies) 0 siblings, 3 replies; 8+ messages in thread From: Jose Quaresma @ 2023-01-16 12:16 UTC (permalink / raw) To: meta-ti; +Cc: ricardo, Jose Quaresma The meta-ti-bsp layer is still broken for K3 HS platforms. The patch [1] makes the check-layer happy again as it does not fail any more during bitbake parsing but it will be difficult for the end user to discover this dependency on meta-ti-extras. On the other side, meta-ti-extras depends on meta-ti-bsp so this is a circular dependency. Currently the layer is broken for am62xx-evm/am64xx-evm machines: | ERROR: Nothing PROVIDES 'ti-k3-secdev-native' (but mc:k3r5:/srv/oe/build/conf/../../layers/meta-ti/meta-ti-bsp/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb DEPENDS on or otherwise requires it). Close matches: | libtspi-dev-native | makedevs-native [1] - 8e43835c ti-sci-fw: make dependency on meta-ti-extras soft Signed-off-by: Jose Quaresma <jose.quaresma@foundries.io> --- .../recipes-ti/secdev/ti-k3-secdev_git.bb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) rename {meta-ti-extras => meta-ti-bsp}/recipes-ti/secdev/ti-k3-secdev_git.bb (96%) diff --git a/meta-ti-extras/recipes-ti/secdev/ti-k3-secdev_git.bb b/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb similarity index 96% rename from meta-ti-extras/recipes-ti/secdev/ti-k3-secdev_git.bb rename to meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb index 3763c339..fd3bd120 100644 --- a/meta-ti-extras/recipes-ti/secdev/ti-k3-secdev_git.bb +++ b/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb @@ -4,7 +4,7 @@ SECTION = "devel" LICENSE = "BSD-3-Clause" LIC_FILES_CHKSUM = "file://manifest/k3-secdev-0.2-manifest.html;md5=f632a78870cc64550078d7b3cbac0892" -require recipes-ti/includes/ti-paths.inc +include recipes-ti/includes/ti-paths.inc # Native host tool only COMPATIBLE_MACHINE = "null" -- 2.34.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [meta-ti][PATCH 2/3] ti-k3-secdev: sets TI_K3_SECDEV_INSTALL_DIR_RECIPE default value 2023-01-16 12:16 [meta-ti][PATCH 1/3] ti-k3-secdev: move from meta-ti-extras to meta-ti-bsp Jose Quaresma @ 2023-01-16 12:16 ` Jose Quaresma 2023-01-16 17:08 ` Andrew Davis 2023-01-16 12:16 ` [meta-ti][PATCH 3/3] ti-sci-fw: sets TI_K3_SECDEV_INSTALL_DIR " Jose Quaresma 2023-01-16 16:48 ` [meta-ti][PATCH 1/3] ti-k3-secdev: move from meta-ti-extras to meta-ti-bsp Andrew Davis 2 siblings, 1 reply; 8+ messages in thread From: Jose Quaresma @ 2023-01-16 12:16 UTC (permalink / raw) To: meta-ti; +Cc: ricardo, Jose Quaresma The TI_K3_SECDEV_INSTALL_DIR_RECIPE is defined on the meta-ti-extras layer so when the layer is not used the variable is undefined. For such cases we can use default value that is the same defined on the recipes-ti/includes/ti-paths.inc Signed-off-by: Jose Quaresma <jose.quaresma@foundries.io> --- meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb | 2 ++ 1 file changed, 2 insertions(+) diff --git a/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb b/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb index fd3bd120..6a28add7 100644 --- a/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb +++ b/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb @@ -4,6 +4,8 @@ SECTION = "devel" LICENSE = "BSD-3-Clause" LIC_FILES_CHKSUM = "file://manifest/k3-secdev-0.2-manifest.html;md5=f632a78870cc64550078d7b3cbac0892" +# set a default value for TI_K3_SECDEV_INSTALL_DIR_RECIPE +export TI_K3_SECDEV_INSTALL_DIR_RECIPE = "${datadir}/ti/ti-k3-secdev" include recipes-ti/includes/ti-paths.inc # Native host tool only -- 2.34.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [meta-ti][PATCH 2/3] ti-k3-secdev: sets TI_K3_SECDEV_INSTALL_DIR_RECIPE default value 2023-01-16 12:16 ` [meta-ti][PATCH 2/3] ti-k3-secdev: sets TI_K3_SECDEV_INSTALL_DIR_RECIPE default value Jose Quaresma @ 2023-01-16 17:08 ` Andrew Davis 2023-01-16 17:41 ` Jose Quaresma 0 siblings, 1 reply; 8+ messages in thread From: Andrew Davis @ 2023-01-16 17:08 UTC (permalink / raw) To: Jose Quaresma, meta-ti; +Cc: ricardo, Jose Quaresma On 1/16/23 6:16 AM, Jose Quaresma wrote: > The TI_K3_SECDEV_INSTALL_DIR_RECIPE is defined on the meta-ti-extras layer > so when the layer is not used the variable is undefined. > For such cases we can use default value that is the same defined > on the recipes-ti/includes/ti-paths.inc > > Signed-off-by: Jose Quaresma <jose.quaresma@foundries.io> > --- > meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb b/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb > index fd3bd120..6a28add7 100644 > --- a/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb > +++ b/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb > @@ -4,6 +4,8 @@ SECTION = "devel" > LICENSE = "BSD-3-Clause" > LIC_FILES_CHKSUM = "file://manifest/k3-secdev-0.2-manifest.html;md5=f632a78870cc64550078d7b3cbac0892" > > +# set a default value for TI_K3_SECDEV_INSTALL_DIR_RECIPE > +export TI_K3_SECDEV_INSTALL_DIR_RECIPE = "${datadir}/ti/ti-k3-secdev" > include recipes-ti/includes/ti-paths.inc > Using ti-paths.inc was not really right to begin with, I only did that as I was in a time crunch, should be just removed (all of the file actually, but we can fix that later). Like other -native tool recipes, the output should end up in a standard place that can be accessed by recipes that depend on it. If -native packages could set env vars that would be nice, but without that, lets just assume location and drop the include on ti-paths.inc. For ti-k3-secdev_git.bb I'm thinking something like: --- a/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb +++ b/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb @@ -4,8 +4,6 @@ SECTION = "devel" LICENSE = "BSD-3-Clause" LIC_FILES_CHKSUM = "file://manifest/k3-secdev-0.2-manifest.html;md5=f632a78870cc64550078d7b3cbac0892" -require recipes-ti/includes/ti-paths.inc - # Native host tool only COMPATIBLE_MACHINE = "null" COMPATIBLE_MACHINE:class-native = "(.*)" @@ -23,11 +21,11 @@ S = "${WORKDIR}/git" do_install() { CP_ARGS="-Prf --preserve=mode,links,timestamps --no-preserve=ownership" - install -d ${D}${TI_K3_SECDEV_INSTALL_DIR_RECIPE} - cp ${CP_ARGS} ${S}/* ${D}${TI_K3_SECDEV_INSTALL_DIR_RECIPE} + install -d ${D}${datadir}/ti/ti-k3-secdev + cp ${CP_ARGS} ${S}/* ${D}${datadir}/ti/ti-k3-secdev } -FILES:${PN} += "${TI_K3_SECDEV_INSTALL_DIR_RECIPE}" +FILES:${PN} += "${D}${datadir}/ti/ti-k3-secdev" INSANE_SKIP:${PN} = "arch ldflags file-rdeps" ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [meta-ti][PATCH 2/3] ti-k3-secdev: sets TI_K3_SECDEV_INSTALL_DIR_RECIPE default value 2023-01-16 17:08 ` Andrew Davis @ 2023-01-16 17:41 ` Jose Quaresma 2023-01-16 18:48 ` Andrew Davis 0 siblings, 1 reply; 8+ messages in thread From: Jose Quaresma @ 2023-01-16 17:41 UTC (permalink / raw) To: Andrew Davis; +Cc: meta-ti, ricardo, Jose Quaresma [-- Attachment #1: Type: text/plain, Size: 3220 bytes --] Hi Andrew, Thanks for your comment. Andrew Davis <afd@ti.com> escreveu no dia segunda, 16/01/2023 à(s) 17:08: > On 1/16/23 6:16 AM, Jose Quaresma wrote: > > The TI_K3_SECDEV_INSTALL_DIR_RECIPE is defined on the meta-ti-extras > layer > > so when the layer is not used the variable is undefined. > > For such cases we can use default value that is the same defined > > on the recipes-ti/includes/ti-paths.inc > > > > Signed-off-by: Jose Quaresma <jose.quaresma@foundries.io> > > --- > > meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb > b/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb > > index fd3bd120..6a28add7 100644 > > --- a/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb > > +++ b/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb > > @@ -4,6 +4,8 @@ SECTION = "devel" > > LICENSE = "BSD-3-Clause" > > LIC_FILES_CHKSUM = > "file://manifest/k3-secdev-0.2-manifest.html;md5=f632a78870cc64550078d7b3cbac0892" > > > > +# set a default value for TI_K3_SECDEV_INSTALL_DIR_RECIPE > > +export TI_K3_SECDEV_INSTALL_DIR_RECIPE = "${datadir}/ti/ti-k3-secdev" > > include recipes-ti/includes/ti-paths.inc > > > > Using ti-paths.inc was not really right to begin with, I only did that as > I was in a time > crunch, should be just removed (all of the file actually, but we can fix > that later). Like > other -native tool recipes, the output should end up in a standard place > that can be accessed > by recipes that depend on it. If -native packages could set env vars that > would be nice, but > without that, lets just assume location and drop the include on > ti-paths.inc. > I think this can be done in a follow up patch, in this series I am trying to make it simple changing only the strictly required to remove the dependencies of meta-ti-extras. > > For ti-k3-secdev_git.bb I'm thinking something like: > > > --- a/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb > +++ b/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb > @@ -4,8 +4,6 @@ SECTION = "devel" > LICENSE = "BSD-3-Clause" > LIC_FILES_CHKSUM = > "file://manifest/k3-secdev-0.2-manifest.html;md5=f632a78870cc64550078d7b3cbac0892" > > -require recipes-ti/includes/ti-paths.inc > - > # Native host tool only > COMPATIBLE_MACHINE = "null" > COMPATIBLE_MACHINE:class-native = "(.*)" > @@ -23,11 +21,11 @@ S = "${WORKDIR}/git" > > do_install() { > CP_ARGS="-Prf --preserve=mode,links,timestamps > --no-preserve=ownership" > - install -d ${D}${TI_K3_SECDEV_INSTALL_DIR_RECIPE} > - cp ${CP_ARGS} ${S}/* ${D}${TI_K3_SECDEV_INSTALL_DIR_RECIPE} > + install -d ${D}${datadir}/ti/ti-k3-secdev > + cp ${CP_ARGS} ${S}/* ${D}${datadir}/ti/ti-k3-secdev > } > > -FILES:${PN} += "${TI_K3_SECDEV_INSTALL_DIR_RECIPE}" > +FILES:${PN} += "${D}${datadir}/ti/ti-k3-secdev" > I agree with this approach and if any additional change is needed it can be done on the distro layer with a bbappend or something else. Jose > > INSANE_SKIP:${PN} = "arch ldflags file-rdeps" > > -- Best regards, José Quaresma [-- Attachment #2: Type: text/html, Size: 4940 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [meta-ti][PATCH 2/3] ti-k3-secdev: sets TI_K3_SECDEV_INSTALL_DIR_RECIPE default value 2023-01-16 17:41 ` Jose Quaresma @ 2023-01-16 18:48 ` Andrew Davis 0 siblings, 0 replies; 8+ messages in thread From: Andrew Davis @ 2023-01-16 18:48 UTC (permalink / raw) To: Jose Quaresma; +Cc: meta-ti, ricardo, Jose Quaresma On 1/16/23 11:41 AM, Jose Quaresma wrote: > Hi Andrew, > > Thanks for your comment. > > Andrew Davis <afd@ti.com <mailto:afd@ti.com>> escreveu no dia segunda, 16/01/2023 à(s) 17:08: > > On 1/16/23 6:16 AM, Jose Quaresma wrote: > > The TI_K3_SECDEV_INSTALL_DIR_RECIPE is defined on the meta-ti-extras layer > > so when the layer is not used the variable is undefined. > > For such cases we can use default value that is the same defined > > on the recipes-ti/includes/ti-paths.inc > > > > Signed-off-by: Jose Quaresma <jose.quaresma@foundries.io <mailto:jose.quaresma@foundries.io>> > > --- > > meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb <http://ti-k3-secdev_git.bb> | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb <http://ti-k3-secdev_git.bb> b/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb <http://ti-k3-secdev_git.bb> > > index fd3bd120..6a28add7 100644 > > --- a/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb <http://ti-k3-secdev_git.bb> > > +++ b/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb <http://ti-k3-secdev_git.bb> > > @@ -4,6 +4,8 @@ SECTION = "devel" > > LICENSE = "BSD-3-Clause" > > LIC_FILES_CHKSUM = "file://manifest/k3-secdev-0.2-manifest.html;md5=f632a78870cc64550078d7b3cbac0892" > > > > +# set a default value for TI_K3_SECDEV_INSTALL_DIR_RECIPE > > +export TI_K3_SECDEV_INSTALL_DIR_RECIPE = "${datadir}/ti/ti-k3-secdev" > > include recipes-ti/includes/ti-paths.inc > > > > Using ti-paths.inc was not really right to begin with, I only did that as I was in a time > crunch, should be just removed (all of the file actually, but we can fix that later). Like > other -native tool recipes, the output should end up in a standard place that can be accessed > by recipes that depend on it. If -native packages could set env vars that would be nice, but > without that, lets just assume location and drop the include on ti-paths.inc. > > > I think this can be done in a follow up patch, in this series I am trying to make it simple > changing only the strictly required to remove the dependencies of meta-ti-extras. > That's fine, just giving future ideas/plans, for this current series, Acked-by: Andrew Davis <afd@ti.com> > > For ti-k3-secdev_git.bb <http://ti-k3-secdev_git.bb> I'm thinking something like: > > > --- a/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb <http://ti-k3-secdev_git.bb> > +++ b/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb <http://ti-k3-secdev_git.bb> > @@ -4,8 +4,6 @@ SECTION = "devel" > LICENSE = "BSD-3-Clause" > LIC_FILES_CHKSUM = "file://manifest/k3-secdev-0.2-manifest.html;md5=f632a78870cc64550078d7b3cbac0892" > > -require recipes-ti/includes/ti-paths.inc > - > # Native host tool only > COMPATIBLE_MACHINE = "null" > COMPATIBLE_MACHINE:class-native = "(.*)" > @@ -23,11 +21,11 @@ S = "${WORKDIR}/git" > > do_install() { > CP_ARGS="-Prf --preserve=mode,links,timestamps --no-preserve=ownership" > - install -d ${D}${TI_K3_SECDEV_INSTALL_DIR_RECIPE} > - cp ${CP_ARGS} ${S}/* ${D}${TI_K3_SECDEV_INSTALL_DIR_RECIPE} > + install -d ${D}${datadir}/ti/ti-k3-secdev > + cp ${CP_ARGS} ${S}/* ${D}${datadir}/ti/ti-k3-secdev > } > > -FILES:${PN} += "${TI_K3_SECDEV_INSTALL_DIR_RECIPE}" > +FILES:${PN} += "${D}${datadir}/ti/ti-k3-secdev" > > > I agree with this approach and if any additional change is needed it can be done on the distro layer with a bbappend > or something else. > > Jose > > > INSANE_SKIP:${PN} = "arch ldflags file-rdeps" > > > > -- > Best regards, > > José Quaresma ^ permalink raw reply [flat|nested] 8+ messages in thread
* [meta-ti][PATCH 3/3] ti-sci-fw: sets TI_K3_SECDEV_INSTALL_DIR default value 2023-01-16 12:16 [meta-ti][PATCH 1/3] ti-k3-secdev: move from meta-ti-extras to meta-ti-bsp Jose Quaresma 2023-01-16 12:16 ` [meta-ti][PATCH 2/3] ti-k3-secdev: sets TI_K3_SECDEV_INSTALL_DIR_RECIPE default value Jose Quaresma @ 2023-01-16 12:16 ` Jose Quaresma 2023-01-16 16:48 ` [meta-ti][PATCH 1/3] ti-k3-secdev: move from meta-ti-extras to meta-ti-bsp Andrew Davis 2 siblings, 0 replies; 8+ messages in thread From: Jose Quaresma @ 2023-01-16 12:16 UTC (permalink / raw) To: meta-ti; +Cc: ricardo, Jose Quaresma The TI_K3_SECDEV_INSTALL_DIR is defined on the meta-ti-extras layer so when the layer is not used the variable is undefined. For such cases we can use default value that is the same defined on the recipes-ti/includes/ti-paths.inc Signed-off-by: Jose Quaresma <jose.quaresma@foundries.io> --- meta-ti-bsp/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb | 2 ++ 1 file changed, 2 insertions(+) diff --git a/meta-ti-bsp/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb b/meta-ti-bsp/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb index c99cd660..8e88b58b 100644 --- a/meta-ti-bsp/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb +++ b/meta-ti-bsp/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb @@ -1,4 +1,6 @@ require recipes-bsp/ti-linux-fw/ti-linux-fw.inc +# set a default value for TI_K3_SECDEV_INSTALL_DIR +export TI_K3_SECDEV_INSTALL_DIR = "${STAGING_DIR_NATIVE}${datadir}/ti/ti-k3-secdev" include recipes-ti/includes/ti-paths.inc DEPENDS = "openssl-native u-boot-mkimage-native dtc-native virtual/bootloader" -- 2.34.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [meta-ti][PATCH 1/3] ti-k3-secdev: move from meta-ti-extras to meta-ti-bsp 2023-01-16 12:16 [meta-ti][PATCH 1/3] ti-k3-secdev: move from meta-ti-extras to meta-ti-bsp Jose Quaresma 2023-01-16 12:16 ` [meta-ti][PATCH 2/3] ti-k3-secdev: sets TI_K3_SECDEV_INSTALL_DIR_RECIPE default value Jose Quaresma 2023-01-16 12:16 ` [meta-ti][PATCH 3/3] ti-sci-fw: sets TI_K3_SECDEV_INSTALL_DIR " Jose Quaresma @ 2023-01-16 16:48 ` Andrew Davis 2023-01-17 22:09 ` Jose Quaresma 2 siblings, 1 reply; 8+ messages in thread From: Andrew Davis @ 2023-01-16 16:48 UTC (permalink / raw) To: Jose Quaresma, meta-ti; +Cc: ricardo, Jose Quaresma On 1/16/23 6:16 AM, Jose Quaresma wrote: > The meta-ti-bsp layer is still broken for K3 HS platforms. > > The patch [1] makes the check-layer happy again as it does not > fail any more during bitbake parsing but it will be difficult > for the end user to discover this dependency on meta-ti-extras. > On the other side, meta-ti-extras depends on meta-ti-bsp so > this is a circular dependency. > > Currently the layer is broken for am62xx-evm/am64xx-evm machines: > > | ERROR: Nothing PROVIDES 'ti-k3-secdev-native' (but mc:k3r5:/srv/oe/build/conf/../../layers/meta-ti/meta-ti-bsp/recipes-bsp/ti-sci-fw/ti-sci-fw_git.bb DEPENDS on or otherwise requires it). Close matches: > | libtspi-dev-native > | makedevs-native > > [1] - 8e43835c ti-sci-fw: make dependency on meta-ti-extras soft > > Signed-off-by: Jose Quaresma <jose.quaresma@foundries.io> > --- Acked-by: Andrew Davis <afd@ti.com> > .../recipes-ti/secdev/ti-k3-secdev_git.bb | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > rename {meta-ti-extras => meta-ti-bsp}/recipes-ti/secdev/ti-k3-secdev_git.bb (96%) > > diff --git a/meta-ti-extras/recipes-ti/secdev/ti-k3-secdev_git.bb b/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb > similarity index 96% > rename from meta-ti-extras/recipes-ti/secdev/ti-k3-secdev_git.bb > rename to meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb > index 3763c339..fd3bd120 100644 > --- a/meta-ti-extras/recipes-ti/secdev/ti-k3-secdev_git.bb > +++ b/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb > @@ -4,7 +4,7 @@ SECTION = "devel" > LICENSE = "BSD-3-Clause" > LIC_FILES_CHKSUM = "file://manifest/k3-secdev-0.2-manifest.html;md5=f632a78870cc64550078d7b3cbac0892" > > -require recipes-ti/includes/ti-paths.inc > +include recipes-ti/includes/ti-paths.inc > > # Native host tool only > COMPATIBLE_MACHINE = "null" > > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#15567): https://lists.yoctoproject.org/g/meta-ti/message/15567 > Mute This Topic: https://lists.yoctoproject.org/mt/96304962/3619733 > Group Owner: meta-ti+owner@lists.yoctoproject.org > Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [afd@ti.com] > -=-=-=-=-=-=-=-=-=-=-=- > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [meta-ti][PATCH 1/3] ti-k3-secdev: move from meta-ti-extras to meta-ti-bsp 2023-01-16 16:48 ` [meta-ti][PATCH 1/3] ti-k3-secdev: move from meta-ti-extras to meta-ti-bsp Andrew Davis @ 2023-01-17 22:09 ` Jose Quaresma 0 siblings, 0 replies; 8+ messages in thread From: Jose Quaresma @ 2023-01-17 22:09 UTC (permalink / raw) To: Andrew Davis; +Cc: meta-ti, Ricardo Salveti, Jose Quaresma [-- Attachment #1: Type: text/plain, Size: 2452 bytes --] Gently ping. A segunda, 16/01/2023, 16:48, Andrew Davis <afd@ti.com> escreveu: > On 1/16/23 6:16 AM, Jose Quaresma wrote: > > The meta-ti-bsp layer is still broken for K3 HS platforms. > > > > The patch [1] makes the check-layer happy again as it does not > > fail any more during bitbake parsing but it will be difficult > > for the end user to discover this dependency on meta-ti-extras. > > On the other side, meta-ti-extras depends on meta-ti-bsp so > > this is a circular dependency. > > > > Currently the layer is broken for am62xx-evm/am64xx-evm machines: > > > > | ERROR: Nothing PROVIDES 'ti-k3-secdev-native' (but > mc:k3r5:/srv/oe/build/conf/../../layers/meta-ti/meta-ti-bsp/recipes-bsp/ti-sci-fw/ > ti-sci-fw_git.bb DEPENDS on or otherwise requires it). Close matches: > > | libtspi-dev-native > > | makedevs-native > > > > [1] - 8e43835c ti-sci-fw: make dependency on meta-ti-extras soft > > > > Signed-off-by: Jose Quaresma <jose.quaresma@foundries.io> > > --- > > Acked-by: Andrew Davis <afd@ti.com> > > > .../recipes-ti/secdev/ti-k3-secdev_git.bb | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > rename {meta-ti-extras => meta-ti-bsp}/recipes-ti/secdev/ > ti-k3-secdev_git.bb (96%) > > > > diff --git a/meta-ti-extras/recipes-ti/secdev/ti-k3-secdev_git.bb > b/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb > > similarity index 96% > > rename from meta-ti-extras/recipes-ti/secdev/ti-k3-secdev_git.bb > > rename to meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb > > index 3763c339..fd3bd120 100644 > > --- a/meta-ti-extras/recipes-ti/secdev/ti-k3-secdev_git.bb > > +++ b/meta-ti-bsp/recipes-ti/secdev/ti-k3-secdev_git.bb > > @@ -4,7 +4,7 @@ SECTION = "devel" > > LICENSE = "BSD-3-Clause" > > LIC_FILES_CHKSUM = > "file://manifest/k3-secdev-0.2-manifest.html;md5=f632a78870cc64550078d7b3cbac0892" > > > > -require recipes-ti/includes/ti-paths.inc > > +include recipes-ti/includes/ti-paths.inc > > > > # Native host tool only > > COMPATIBLE_MACHINE = "null" > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- > > Links: You receive all messages sent to this group. > > View/Reply Online (#15567): > https://lists.yoctoproject.org/g/meta-ti/message/15567 > > Mute This Topic: https://lists.yoctoproject.org/mt/96304962/3619733 > > Group Owner: meta-ti+owner@lists.yoctoproject.org > > Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [afd@ti.com] > > -=-=-=-=-=-=-=-=-=-=-=- > > > [-- Attachment #2: Type: text/html, Size: 4709 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2023-01-17 22:09 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2023-01-16 12:16 [meta-ti][PATCH 1/3] ti-k3-secdev: move from meta-ti-extras to meta-ti-bsp Jose Quaresma 2023-01-16 12:16 ` [meta-ti][PATCH 2/3] ti-k3-secdev: sets TI_K3_SECDEV_INSTALL_DIR_RECIPE default value Jose Quaresma 2023-01-16 17:08 ` Andrew Davis 2023-01-16 17:41 ` Jose Quaresma 2023-01-16 18:48 ` Andrew Davis 2023-01-16 12:16 ` [meta-ti][PATCH 3/3] ti-sci-fw: sets TI_K3_SECDEV_INSTALL_DIR " Jose Quaresma 2023-01-16 16:48 ` [meta-ti][PATCH 1/3] ti-k3-secdev: move from meta-ti-extras to meta-ti-bsp Andrew Davis 2023-01-17 22:09 ` Jose Quaresma
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.