All of lore.kernel.org
 help / color / mirror / Atom feed
* [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.