All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denys@ti.com>
To: "Stiffler, Jacob" <j-stiffler@ti.com>
Cc: "meta-arago@arago-project.org" <meta-arago@arago-project.org>
Subject: Re: [tisdk-setup-scripts][PATCH 2/2] setup-uboot-env-k2g-evm.sh: update for latest default env
Date: Tue, 6 Dec 2016 14:30:11 -0500	[thread overview]
Message-ID: <20161206193011.GJ28380@edge> (raw)
In-Reply-To: <4B26FB5A64EBA14C87266869E037F9582536DBC0@DFLE09.ent.ti.com>

On Tue, Dec 06, 2016 at 07:18:21AM -0500, Stiffler, Jacob wrote:
> 
> 
> > -----Original Message-----
> > From: Dmytriyenko, Denys
> > Sent: Monday, December 05, 2016 10:56 PM
> > To: Stiffler, Jacob
> > Cc: meta-arago@arago-project.org
> > Subject: Re: [meta-arago] [tisdk-setup-scripts][PATCH 2/2] setup-uboot-
> > env-k2g-evm.sh: update for latest default env
> > 
> > On Thu, Dec 01, 2016 at 07:29:52AM -0500, Jacob Stiffler wrote:
> > > * We now need to call "findfdt" so that "name_fdt" is defined.
> > >
> > > Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> > 
> > Acked-by: Denys Dmytriyenko <denys@ti.com>
> > 
> > 
> > Please push both of these in.
> > 
> 
> Should I add the "Acked-by" in the commit message?

If you use Patchwork, you can just grab "mbox" formatted patch from there and 
it will have all the collected Acks or SoBs for you ready to apply.

-- 
Denys


> > > ---
> > >  setup-uboot-env-k2g-evm.sh | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/setup-uboot-env-k2g-evm.sh b/setup-uboot-env-k2g-evm.sh
> > > index f41d278..c80e506 100755
> > > --- a/setup-uboot-env-k2g-evm.sh
> > > +++ b/setup-uboot-env-k2g-evm.sh
> > > @@ -187,7 +187,7 @@ do_expect "\"$prompt\"" "send \"setenv name_ubi
> > $ubifsimage\"" $cwd/setupBoard.m
> > >  do_expect "\"$prompt\"" "send \"setenv name_kern $kernelimage\""
> > $cwd/setupBoard.minicom
> > >  do_expect "\"$prompt\"" "send \"setenv name_uboot $ubootimage\""
> > $cwd/setupBoard.minicom
> > >
> > > -do_expect "\"$prompt\"" "send \"setenv bootcmd 'run envboot; run
> > init_\${fs_mode} get_pmmc_\${kern_mode} run_pmmc
> > get_fdt_\${kern_mode} get_mon_\${kern_mode} get_kern_\${kern_mode}
> > run_mon run_kern'\"" $cwd/setupBoard.minicom
> > > +do_expect "\"$prompt\"" "send \"setenv bootcmd 'run envboot; run
> > init_\${fs_mode} get_pmmc_\${kern_mode} run_pmmc
> > get_mon_\${kern_mode} run_mon findfdt get_fdt_\${kern_mode}
> > get_kern_\${kern_mode} run_kern'\"" $cwd/setupBoard.minicom
> > >
> > >  do_expect "\"$prompt\"" "send \"saveenv\"" $cwd/setupBoard.minicom
> > >
> > > --
> > > 1.9.1
> > >
> > > _______________________________________________
> > > meta-arago mailing list
> > > meta-arago@arago-project.org
> > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


  reply	other threads:[~2016-12-06 19:30 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-12-01 12:29 [tisdk-setup-scripts][PATCH 1/2] setup-uboot-env: am335x, am437x, am57xx: update for latest default env Jacob Stiffler
2016-12-01 12:29 ` [tisdk-setup-scripts][PATCH 2/2] setup-uboot-env-k2g-evm.sh: " Jacob Stiffler
2016-12-06  3:55   ` Denys Dmytriyenko
2016-12-06 12:18     ` Stiffler, Jacob
2016-12-06 19:30       ` Denys Dmytriyenko [this message]
2016-12-06  3:54 ` [tisdk-setup-scripts][PATCH 1/2] setup-uboot-env: am335x, am437x, am57xx: " Denys Dmytriyenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20161206193011.GJ28380@edge \
    --to=denys@ti.com \
    --cc=j-stiffler@ti.com \
    --cc=meta-arago@arago-project.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.