Message ID | 20221207214312.21735-1-afd@ti.com |
---|---|
State | Accepted |
Delegated to: | Ryan Eatmon |
Headers | show |
Series | [meta-ti,master/kirkstone,1/7] conf: machine: Remove multi-certificate image comment | expand |
Finally was able to spend some time reviewing this patchset - see comments to individual patches. On Wed, Dec 07, 2022 at 03:43:06PM -0600, Andrew Davis via lists.yoctoproject.org wrote: > Multi-cert is already now the default, no need to add a comment on that. > > Signed-off-by: Andrew Davis <afd@ti.com> > --- > meta-ti-bsp/conf/machine/include/j7.inc | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/meta-ti-bsp/conf/machine/include/j7.inc b/meta-ti-bsp/conf/machine/include/j7.inc > index 4ea3a5d8..cf4cc440 100644 > --- a/meta-ti-bsp/conf/machine/include/j7.inc > +++ b/meta-ti-bsp/conf/machine/include/j7.inc > @@ -3,5 +3,3 @@ SOC_FAMILY:append = ":j7" > > OPTEEMACHINE = "k3-j721e" > OPTEEOUTPUTMACHINE = "k3" > - > -# Use default IMAGE_BOOT_FILES_LEGACY files > -- > 2.38.1
Ryan, I believe we are fine merging this series as is and addressing the remaining cleanup items afterwards. Thanks.
Ping. On Mon, Dec 19, 2022 at 04:23:04PM -0500, Denys Dmytriyenko wrote: > Ryan, > > I believe we are fine merging this series as is and addressing the remaining > cleanup items afterwards. Thanks. > > -- > Denys > > > On Wed, Dec 07, 2022 at 03:43:06PM -0600, Andrew Davis via lists.yoctoproject.org wrote: > > Multi-cert is already now the default, no need to add a comment on that. > > > > Signed-off-by: Andrew Davis <afd@ti.com> > > --- > > meta-ti-bsp/conf/machine/include/j7.inc | 2 -- > > 1 file changed, 2 deletions(-) > > > > diff --git a/meta-ti-bsp/conf/machine/include/j7.inc b/meta-ti-bsp/conf/machine/include/j7.inc > > index 4ea3a5d8..cf4cc440 100644 > > --- a/meta-ti-bsp/conf/machine/include/j7.inc > > +++ b/meta-ti-bsp/conf/machine/include/j7.inc > > @@ -3,5 +3,3 @@ SOC_FAMILY:append = ":j7" > > > > OPTEEMACHINE = "k3-j721e" > > OPTEEOUTPUTMACHINE = "k3" > > - > > -# Use default IMAGE_BOOT_FILES_LEGACY files > > -- > > 2.38.1
diff --git a/meta-ti-bsp/conf/machine/include/j7.inc b/meta-ti-bsp/conf/machine/include/j7.inc index 4ea3a5d8..cf4cc440 100644 --- a/meta-ti-bsp/conf/machine/include/j7.inc +++ b/meta-ti-bsp/conf/machine/include/j7.inc @@ -3,5 +3,3 @@ SOC_FAMILY:append = ":j7" OPTEEMACHINE = "k3-j721e" OPTEEOUTPUTMACHINE = "k3" - -# Use default IMAGE_BOOT_FILES_LEGACY files
Multi-cert is already now the default, no need to add a comment on that. Signed-off-by: Andrew Davis <afd@ti.com> --- meta-ti-bsp/conf/machine/include/j7.inc | 2 -- 1 file changed, 2 deletions(-)