* [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
* [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 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
* 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.