All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] tisdk-readme: Specify that uEnv.txt is optional.
@ 2015-04-08 19:31 Jacob Stiffler
  2015-04-08 19:38 ` Cooper Jr., Franklin
  0 siblings, 1 reply; 6+ messages in thread
From: Jacob Stiffler @ 2015-04-08 19:31 UTC (permalink / raw)
  To: meta-arago

Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
---
 .../recipes-tisdk/tisdk-readme/tisdk-readme.bb     |    2 +-
 .../recipes-tisdk/tisdk-readme/tisdk-readme/README |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
index 6fdcaff..4ee3ec4 100644
--- a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
+++ b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
@@ -6,7 +6,7 @@ SRC_URI = "\
     file://README \
 "
 
-PR = "r0"
+PR = "r1"
 PV = "1.0"
 
 PACKAGE_ARCH = "${MACHINE_ARCH}"
diff --git a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme/README b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme/README
index 873d1e8..b9aeefe 100644
--- a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme/README
+++ b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme/README
@@ -11,7 +11,7 @@ By default these locations are:
 ================================================================================
 |    MLO           | boot partition
 |    u-boot.img    | boot partition
-|    uEnv.txt      | boot partition
+|    uEnv.txt      | boot partition (optional)
 |    uImage        | /boot directory of the rootfs partition
 |    *.dtb         | /boot directory of the rootfs partition
 ================================================================================
-- 
1.7.9.5



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

* Re: [PATCH] tisdk-readme: Specify that uEnv.txt is optional.
  2015-04-08 19:31 [PATCH] tisdk-readme: Specify that uEnv.txt is optional Jacob Stiffler
@ 2015-04-08 19:38 ` Cooper Jr., Franklin
  2015-04-08 19:41   ` Stiffler, Jacob
  0 siblings, 1 reply; 6+ messages in thread
From: Cooper Jr., Franklin @ 2015-04-08 19:38 UTC (permalink / raw)
  To: Stiffler, Jacob, meta-arago@arago-project.org

I still don't understand the need to include this especially since it was never included before and all this information is documented.

But anyway the wording needs to be updated. Remove all the parts that says "usually called" since that's what it is called. ZImage should be used and not uImage.

> -----Original Message-----
> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> bounces@arago-project.org] On Behalf Of Stiffler, Jacob
> Sent: Wednesday, April 08, 2015 2:32 PM
> To: meta-arago@arago-project.org
> Subject: [meta-arago] [PATCH] tisdk-readme: Specify that uEnv.txt is
> optional.
> 
> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> ---
>  .../recipes-tisdk/tisdk-readme/tisdk-readme.bb     |    2 +-
>  .../recipes-tisdk/tisdk-readme/tisdk-readme/README |    2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
> b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
> index 6fdcaff..4ee3ec4 100644
> --- a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
> +++ b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
> @@ -6,7 +6,7 @@ SRC_URI = "\
>      file://README \
>  "
> 
> -PR = "r0"
> +PR = "r1"
>  PV = "1.0"
> 
>  PACKAGE_ARCH = "${MACHINE_ARCH}"
> diff --git a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-
> readme/README b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-
> readme/README
> index 873d1e8..b9aeefe 100644
> --- a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme/README
> +++ b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-
> readme/README
> @@ -11,7 +11,7 @@ By default these locations are:
> 
> ==========================================================
> ======================
>  |    MLO           | boot partition
>  |    u-boot.img    | boot partition
> -|    uEnv.txt      | boot partition
> +|    uEnv.txt      | boot partition (optional)
>  |    uImage        | /boot directory of the rootfs partition
>  |    *.dtb         | /boot directory of the rootfs partition
> 
> ==========================================================
> ======================
> --
> 1.7.9.5
> 
> _______________________________________________
> 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: [PATCH] tisdk-readme: Specify that uEnv.txt is optional.
  2015-04-08 19:38 ` Cooper Jr., Franklin
@ 2015-04-08 19:41   ` Stiffler, Jacob
  2015-04-08 19:45     ` Cooper Jr., Franklin
  0 siblings, 1 reply; 6+ messages in thread
From: Stiffler, Jacob @ 2015-04-08 19:41 UTC (permalink / raw)
  To: Cooper Jr., Franklin, meta-arago@arago-project.org

This is included now because for ti33xx and ti43xx, tisdk-readme is included if the brand is not amsdk.

Should this package be removed completely for all brands?


-----Original Message-----
From: Cooper Jr., Franklin 
Sent: Wednesday, April 08, 2015 3:38 PM
To: Stiffler, Jacob; meta-arago@arago-project.org
Subject: RE: [meta-arago] [PATCH] tisdk-readme: Specify that uEnv.txt is optional.

I still don't understand the need to include this especially since it was never included before and all this information is documented.

But anyway the wording needs to be updated. Remove all the parts that says "usually called" since that's what it is called. ZImage should be used and not uImage.

> -----Original Message-----
> From: meta-arago-bounces@arago-project.org [mailto:meta-arago- 
> bounces@arago-project.org] On Behalf Of Stiffler, Jacob
> Sent: Wednesday, April 08, 2015 2:32 PM
> To: meta-arago@arago-project.org
> Subject: [meta-arago] [PATCH] tisdk-readme: Specify that uEnv.txt is 
> optional.
> 
> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> ---
>  .../recipes-tisdk/tisdk-readme/tisdk-readme.bb     |    2 +-
>  .../recipes-tisdk/tisdk-readme/tisdk-readme/README |    2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git 
> a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
> b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
> index 6fdcaff..4ee3ec4 100644
> --- a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
> +++ b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
> @@ -6,7 +6,7 @@ SRC_URI = "\
>      file://README \
>  "
> 
> -PR = "r0"
> +PR = "r1"
>  PV = "1.0"
> 
>  PACKAGE_ARCH = "${MACHINE_ARCH}"
> diff --git a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-
> readme/README b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-
> readme/README
> index 873d1e8..b9aeefe 100644
> --- a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme/README
> +++ b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-
> readme/README
> @@ -11,7 +11,7 @@ By default these locations are:
> 
> ==========================================================
> ======================
>  |    MLO           | boot partition
>  |    u-boot.img    | boot partition
> -|    uEnv.txt      | boot partition
> +|    uEnv.txt      | boot partition (optional)
>  |    uImage        | /boot directory of the rootfs partition
>  |    *.dtb         | /boot directory of the rootfs partition
> 
> ==========================================================
> ======================
> --
> 1.7.9.5
> 
> _______________________________________________
> 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: [PATCH] tisdk-readme: Specify that uEnv.txt is optional.
  2015-04-08 19:41   ` Stiffler, Jacob
@ 2015-04-08 19:45     ` Cooper Jr., Franklin
  2015-04-22 19:18       ` Denys Dmytriyenko
  0 siblings, 1 reply; 6+ messages in thread
From: Cooper Jr., Franklin @ 2015-04-08 19:45 UTC (permalink / raw)
  To: Stiffler, Jacob, meta-arago@arago-project.org

I would just include it for omap5-evm which is the only user of the customized uEnv.txt.
I don't see anyone else using it.

> -----Original Message-----
> From: Stiffler, Jacob
> Sent: Wednesday, April 08, 2015 2:42 PM
> To: Cooper Jr., Franklin; meta-arago@arago-project.org
> Subject: RE: [meta-arago] [PATCH] tisdk-readme: Specify that uEnv.txt is
> optional.
> 
> This is included now because for ti33xx and ti43xx, tisdk-readme is included if
> the brand is not amsdk.
> 
> Should this package be removed completely for all brands?
> 
> 
> -----Original Message-----
> From: Cooper Jr., Franklin
> Sent: Wednesday, April 08, 2015 3:38 PM
> To: Stiffler, Jacob; meta-arago@arago-project.org
> Subject: RE: [meta-arago] [PATCH] tisdk-readme: Specify that uEnv.txt is
> optional.
> 
> I still don't understand the need to include this especially since it was never
> included before and all this information is documented.
> 
> But anyway the wording needs to be updated. Remove all the parts that says
> "usually called" since that's what it is called. ZImage should be used and not
> uImage.
> 
> > -----Original Message-----
> > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > bounces@arago-project.org] On Behalf Of Stiffler, Jacob
> > Sent: Wednesday, April 08, 2015 2:32 PM
> > To: meta-arago@arago-project.org
> > Subject: [meta-arago] [PATCH] tisdk-readme: Specify that uEnv.txt is
> > optional.
> >
> > Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> > ---
> >  .../recipes-tisdk/tisdk-readme/tisdk-readme.bb     |    2 +-
> >  .../recipes-tisdk/tisdk-readme/tisdk-readme/README |    2 +-
> >  2 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git
> > a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
> > b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
> > index 6fdcaff..4ee3ec4 100644
> > --- a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
> > +++ b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
> > @@ -6,7 +6,7 @@ SRC_URI = "\
> >      file://README \
> >  "
> >
> > -PR = "r0"
> > +PR = "r1"
> >  PV = "1.0"
> >
> >  PACKAGE_ARCH = "${MACHINE_ARCH}"
> > diff --git a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-
> > readme/README b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-
> > readme/README
> > index 873d1e8..b9aeefe 100644
> > --- a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-
> readme/README
> > +++ b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-
> > readme/README
> > @@ -11,7 +11,7 @@ By default these locations are:
> >
> >
> ==========================================================
> > ======================
> >  |    MLO           | boot partition
> >  |    u-boot.img    | boot partition
> > -|    uEnv.txt      | boot partition
> > +|    uEnv.txt      | boot partition (optional)
> >  |    uImage        | /boot directory of the rootfs partition
> >  |    *.dtb         | /boot directory of the rootfs partition
> >
> >
> ==========================================================
> > ======================
> > --
> > 1.7.9.5
> >
> > _______________________________________________
> > 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: [PATCH] tisdk-readme: Specify that uEnv.txt is optional.
  2015-04-08 19:45     ` Cooper Jr., Franklin
@ 2015-04-22 19:18       ` Denys Dmytriyenko
  2015-04-22 19:21         ` Jacob Stiffler
  0 siblings, 1 reply; 6+ messages in thread
From: Denys Dmytriyenko @ 2015-04-22 19:18 UTC (permalink / raw)
  To: Cooper Jr., Franklin; +Cc: meta-arago@arago-project.org

Was this patch superseeded by packagegroup-*-amsdk-sdk-host removal of 
tisdk-readme?

-- 
Denys


On Wed, Apr 08, 2015 at 07:45:48PM +0000, Cooper Jr., Franklin wrote:
> I would just include it for omap5-evm which is the only user of the customized uEnv.txt.
> I don't see anyone else using it.
> 
> > -----Original Message-----
> > From: Stiffler, Jacob
> > Sent: Wednesday, April 08, 2015 2:42 PM
> > To: Cooper Jr., Franklin; meta-arago@arago-project.org
> > Subject: RE: [meta-arago] [PATCH] tisdk-readme: Specify that uEnv.txt is
> > optional.
> > 
> > This is included now because for ti33xx and ti43xx, tisdk-readme is included if
> > the brand is not amsdk.
> > 
> > Should this package be removed completely for all brands?
> > 
> > 
> > -----Original Message-----
> > From: Cooper Jr., Franklin
> > Sent: Wednesday, April 08, 2015 3:38 PM
> > To: Stiffler, Jacob; meta-arago@arago-project.org
> > Subject: RE: [meta-arago] [PATCH] tisdk-readme: Specify that uEnv.txt is
> > optional.
> > 
> > I still don't understand the need to include this especially since it was never
> > included before and all this information is documented.
> > 
> > But anyway the wording needs to be updated. Remove all the parts that says
> > "usually called" since that's what it is called. ZImage should be used and not
> > uImage.
> > 
> > > -----Original Message-----
> > > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > > bounces@arago-project.org] On Behalf Of Stiffler, Jacob
> > > Sent: Wednesday, April 08, 2015 2:32 PM
> > > To: meta-arago@arago-project.org
> > > Subject: [meta-arago] [PATCH] tisdk-readme: Specify that uEnv.txt is
> > > optional.
> > >
> > > Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> > > ---
> > >  .../recipes-tisdk/tisdk-readme/tisdk-readme.bb     |    2 +-
> > >  .../recipes-tisdk/tisdk-readme/tisdk-readme/README |    2 +-
> > >  2 files changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git
> > > a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
> > > b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
> > > index 6fdcaff..4ee3ec4 100644
> > > --- a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
> > > +++ b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-readme.bb
> > > @@ -6,7 +6,7 @@ SRC_URI = "\
> > >      file://README \
> > >  "
> > >
> > > -PR = "r0"
> > > +PR = "r1"
> > >  PV = "1.0"
> > >
> > >  PACKAGE_ARCH = "${MACHINE_ARCH}"
> > > diff --git a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-
> > > readme/README b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-
> > > readme/README
> > > index 873d1e8..b9aeefe 100644
> > > --- a/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-
> > readme/README
> > > +++ b/meta-arago-distro/recipes-tisdk/tisdk-readme/tisdk-
> > > readme/README
> > > @@ -11,7 +11,7 @@ By default these locations are:
> > >
> > >
> > ==========================================================
> > > ======================
> > >  |    MLO           | boot partition
> > >  |    u-boot.img    | boot partition
> > > -|    uEnv.txt      | boot partition
> > > +|    uEnv.txt      | boot partition (optional)
> > >  |    uImage        | /boot directory of the rootfs partition
> > >  |    *.dtb         | /boot directory of the rootfs partition
> > >
> > >
> > ==========================================================
> > > ======================
> > > --
> > > 1.7.9.5
> > >
> > > _______________________________________________
> > > meta-arago mailing list
> > > meta-arago@arago-project.org
> > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> 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: [PATCH] tisdk-readme: Specify that uEnv.txt is optional.
  2015-04-22 19:18       ` Denys Dmytriyenko
@ 2015-04-22 19:21         ` Jacob Stiffler
  0 siblings, 0 replies; 6+ messages in thread
From: Jacob Stiffler @ 2015-04-22 19:21 UTC (permalink / raw)
  To: Denys Dmytriyenko, Cooper Jr., Franklin; +Cc: meta-arago@arago-project.org

Yes.

On 4/22/2015 3:18 PM, Denys Dmytriyenko wrote:
> Was this patch superseeded by packagegroup-*-amsdk-sdk-host removal of
> tisdk-readme?
>



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

end of thread, other threads:[~2015-04-22 19:21 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-04-08 19:31 [PATCH] tisdk-readme: Specify that uEnv.txt is optional Jacob Stiffler
2015-04-08 19:38 ` Cooper Jr., Franklin
2015-04-08 19:41   ` Stiffler, Jacob
2015-04-08 19:45     ` Cooper Jr., Franklin
2015-04-22 19:18       ` Denys Dmytriyenko
2015-04-22 19:21         ` Jacob Stiffler

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.