All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-arago PATCH] tisdk-core-bundle: Install sysfw.itb in the prebuilt-images directory
@ 2021-06-03  3:20 Aparna M
  2021-06-03  8:56 ` Vignesh Raghavendra
  0 siblings, 1 reply; 6+ messages in thread
From: Aparna M @ 2021-06-03  3:20 UTC (permalink / raw)
  To: meta-arago

Install the sysfw.itb file in the prebuilt images directory of the
tisdk-core-bundle package.

Signed-off-by: Aparna M <a-m1@ti.com>
---
 meta-arago-distro/recipes-core/images/tisdk-core-bundle.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.bb b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.bb
index 253bfbb6..d1bfdc55 100644
--- a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.bb
+++ b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.bb
@@ -14,6 +14,7 @@ DEPLOY_IMAGES_NAME_append_j7-evm = " sysfw.itb"
 DEPLOY_IMAGES_NAME_append_j7-hs-evm = " sysfw.itb"
 DEPLOY_IMAGES_NAME_append_am65xx-evm = " ti-sci-firmware-am65x-gp.bin sysfw-am65x-evm.itb sysfw-am65x_sr2-evm.itb"
 DEPLOY_IMAGES_NAME_append_j7-evm = " ti-fs-firmware-j721e-gp.bin"
+DEPLOY_IMAGES_NAME_append_am64xx-evm = " sysfw.itb"
 
 ARAGO_TISDK_IMAGE ?= "tisdk-core-bundle"
 export IMAGE_BASENAME = "${ARAGO_TISDK_IMAGE}"
-- 
2.17.1



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [meta-arago PATCH] tisdk-core-bundle: Install sysfw.itb in the prebuilt-images directory
  2021-06-03  3:20 [meta-arago PATCH] tisdk-core-bundle: Install sysfw.itb in the prebuilt-images directory Aparna M
@ 2021-06-03  8:56 ` Vignesh Raghavendra
  2021-06-07 21:23   ` Nishanth Menon
  0 siblings, 1 reply; 6+ messages in thread
From: Vignesh Raghavendra @ 2021-06-03  8:56 UTC (permalink / raw)
  To: Aparna M, meta-arago

+praneeth

On 6/3/21 8:50 AM, Aparna M wrote:
> Install the sysfw.itb file in the prebuilt images directory of the
> tisdk-core-bundle package.
> 

Why do we need to package sysfw.itb given that its already part of
tiboot3.bin in case of am64x?



> Signed-off-by: Aparna M <a-m1@ti.com>
> ---
>  meta-arago-distro/recipes-core/images/tisdk-core-bundle.bb | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.bb b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.bb
> index 253bfbb6..d1bfdc55 100644
> --- a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.bb
> +++ b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.bb
> @@ -14,6 +14,7 @@ DEPLOY_IMAGES_NAME_append_j7-evm = " sysfw.itb"
>  DEPLOY_IMAGES_NAME_append_j7-hs-evm = " sysfw.itb"
>  DEPLOY_IMAGES_NAME_append_am65xx-evm = " ti-sci-firmware-am65x-gp.bin sysfw-am65x-evm.itb sysfw-am65x_sr2-evm.itb"
>  DEPLOY_IMAGES_NAME_append_j7-evm = " ti-fs-firmware-j721e-gp.bin"
> +DEPLOY_IMAGES_NAME_append_am64xx-evm = " sysfw.itb"
>  
>  ARAGO_TISDK_IMAGE ?= "tisdk-core-bundle"
>  export IMAGE_BASENAME = "${ARAGO_TISDK_IMAGE}"
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-arago PATCH] tisdk-core-bundle: Install sysfw.itb in the prebuilt-images directory
  2021-06-03  8:56 ` Vignesh Raghavendra
@ 2021-06-07 21:23   ` Nishanth Menon
  2021-06-08 13:44     ` Nikhil Devshatwar
  0 siblings, 1 reply; 6+ messages in thread
From: Nishanth Menon @ 2021-06-07 21:23 UTC (permalink / raw)
  To: Vignesh Raghavendra; +Cc: meta-arago

On 14:26-20210603, Vignesh Raghavendra wrote:
> +praneeth
> 
> On 6/3/21 8:50 AM, Aparna M wrote:
> > Install the sysfw.itb file in the prebuilt images directory of the
> > tisdk-core-bundle package.
> > 
> 
> Why do we need to package sysfw.itb given that its already part of
> tiboot3.bin in case of am64x?


NAK. We are using multi-certificate for am64x.

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-arago PATCH] tisdk-core-bundle: Install sysfw.itb in the prebuilt-images directory
  2021-06-07 21:23   ` Nishanth Menon
@ 2021-06-08 13:44     ` Nikhil Devshatwar
  2021-06-08 13:52       ` Nishanth Menon
  0 siblings, 1 reply; 6+ messages in thread
From: Nikhil Devshatwar @ 2021-06-08 13:44 UTC (permalink / raw)
  To: Nishanth Menon; +Cc: meta-arago

On 16:23-20210607, Nishanth Menon wrote:
> On 14:26-20210603, Vignesh Raghavendra wrote:
> > +praneeth
> > 
> > On 6/3/21 8:50 AM, Aparna M wrote:
> > > Install the sysfw.itb file in the prebuilt images directory of the
> > > tisdk-core-bundle package.
> > > 
> > 
> > Why do we need to package sysfw.itb given that its already part of
> > tiboot3.bin in case of am64x?
Vignesh,

This was done for consistency. It's no harm to include this.
It was a feedback on the SDK from the apps team.

> 
> 
> NAK. We are using multi-certificate for am64x.

Pardon me for not knowing, What is a multi-certificate?

Regards,
Nikhil D

> 
> -- 
> Regards,
> Nishanth Menon
> Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-arago PATCH] tisdk-core-bundle: Install sysfw.itb in the prebuilt-images directory
  2021-06-08 13:44     ` Nikhil Devshatwar
@ 2021-06-08 13:52       ` Nishanth Menon
  2021-06-09 12:58         ` Nikhil Devshatwar
  0 siblings, 1 reply; 6+ messages in thread
From: Nishanth Menon @ 2021-06-08 13:52 UTC (permalink / raw)
  To: Nikhil Devshatwar; +Cc: meta-arago

On 19:14-20210608, Nikhil Devshatwar wrote:
> On 16:23-20210607, Nishanth Menon wrote:
> > On 14:26-20210603, Vignesh Raghavendra wrote:
> > > +praneeth
> > > 
> > > On 6/3/21 8:50 AM, Aparna M wrote:
> > > > Install the sysfw.itb file in the prebuilt images directory of the
> > > > tisdk-core-bundle package.
> > > > 
> > > 
> > > Why do we need to package sysfw.itb given that its already part of
> > > tiboot3.bin in case of am64x?
> Vignesh,
> 
> This was done for consistency. It's no harm to include this.
> It was a feedback on the SDK from the apps team.

The feedback is wrong. putting a random binary in the boot folder which
does no function is broken understanding of the boot process - not to
mention deployment and upgrade burden that we pitch on customers.

> > NAK. We are using multi-certificate for am64x.
> 
> Pardon me for not knowing, What is a multi-certificate?


multi-certificate x509 boot - where SPL, sysfw etc are packaged as part
of tiboot3.bin.


-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [meta-arago PATCH] tisdk-core-bundle: Install sysfw.itb in the prebuilt-images directory
  2021-06-08 13:52       ` Nishanth Menon
@ 2021-06-09 12:58         ` Nikhil Devshatwar
  0 siblings, 0 replies; 6+ messages in thread
From: Nikhil Devshatwar @ 2021-06-09 12:58 UTC (permalink / raw)
  To: Nishanth Menon; +Cc: meta-arago

On 08:52-20210608, Nishanth Menon wrote:
> On 19:14-20210608, Nikhil Devshatwar wrote:
> > On 16:23-20210607, Nishanth Menon wrote:
> > > On 14:26-20210603, Vignesh Raghavendra wrote:
> > > > +praneeth
> > > > 
> > > > On 6/3/21 8:50 AM, Aparna M wrote:
> > > > > Install the sysfw.itb file in the prebuilt images directory of the
> > > > > tisdk-core-bundle package.
> > > > > 
> > > > 
> > > > Why do we need to package sysfw.itb given that its already part of
> > > > tiboot3.bin in case of am64x?
> > Vignesh,
> > 
> > This was done for consistency. It's no harm to include this.
> > It was a feedback on the SDK from the apps team.
> 
> The feedback is wrong. putting a random binary in the boot folder which
> does no function is broken understanding of the boot process - not to
> mention deployment and upgrade burden that we pitch on customers.

I was not aware if there is any case where the ITB would be required.
From your comment it seems it's not required at all.

Aparna,
Let us not add this in the installer. and also update the documentation to
specifically call out the contents of tiboot3.bin for AM64

Regards,
Nikhil D

> 
> > > NAK. We are using multi-certificate for am64x.
> > 
> > Pardon me for not knowing, What is a multi-certificate?
> 
> 
> multi-certificate x509 boot - where SPL, sysfw etc are packaged as part
> of tiboot3.bin.
> 
> 
> -- 
> Regards,
> Nishanth Menon
> Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2021-06-09 12:57 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-03  3:20 [meta-arago PATCH] tisdk-core-bundle: Install sysfw.itb in the prebuilt-images directory Aparna M
2021-06-03  8:56 ` Vignesh Raghavendra
2021-06-07 21:23   ` Nishanth Menon
2021-06-08 13:44     ` Nikhil Devshatwar
2021-06-08 13:52       ` Nishanth Menon
2021-06-09 12:58         ` Nikhil Devshatwar

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.