From: Mike Frysinger <vapier@gentoo.org>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [PATCH] Blackfin: implement go/boote wrappers
Date: Mon, 21 Apr 2008 06:59:42 -0400 [thread overview]
Message-ID: <200804210659.43481.vapier@gentoo.org> (raw)
In-Reply-To: <200804210635.08416.vapier@gentoo.org>
On Monday 21 April 2008, Mike Frysinger wrote:
> On Monday 21 April 2008, Wolfgang Denk wrote:
> > Providing both a "go" and a "jump" command which differ just in cahce
> > handling seems broken to me. If you want to add such a feature, then
> > I recommend to do it as part of "go", but make it optional, i. e. in-
> > troduce a new optional argument to the "go" command, something like
> >
> > go [ -cache={off,d-off,i-off,on,d-on,i-on} ] addr [ args ... ]
>
> cache is just an example. other arches may want to do other sort of
> "system breakdown/cleanup" before relinquishing control. option flags to
> commands really doesnt fit the style of u-boot (i expect that sort of
> painful option parsing in redboot, not really u-boot).
>
> so we can do:
> go [-noret] addr [args...]
> or we can add "jump" to cmd_boot.c and merge the differences by just using
> a function pointer to "do_go_exec" or "do_jump_exec".
untested poc attached
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 827 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20080421/edee92fc/attachment.pgp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: u-boot-jump-cmd.patch
Type: text/x-diff
Size: 1779 bytes
Desc: not available
Url : http://lists.denx.de/pipermail/u-boot/attachments/20080421/edee92fc/attachment.patch
next prev parent reply other threads:[~2008-04-21 10:59 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-04-19 5:45 [U-Boot-Users] [PATCH] Blackfin: implement go/boote wrappers Mike Frysinger
2008-04-20 23:38 ` Wolfgang Denk
2008-04-21 0:31 ` Mike Frysinger
2008-04-21 5:00 ` Wolfgang Denk
2008-04-21 5:25 ` Mike Frysinger
2008-04-21 7:26 ` Wolfgang Denk
2008-04-21 7:49 ` Mike Frysinger
2008-04-21 9:03 ` Wolfgang Denk
2008-04-21 9:41 ` Mike Frysinger
2008-04-21 10:07 ` Wolfgang Denk
2008-04-21 10:35 ` Mike Frysinger
2008-04-21 10:59 ` Mike Frysinger [this message]
2008-04-21 12:14 ` Wolfgang Denk
2008-04-21 12:13 ` Wolfgang Denk
2008-04-21 18:44 ` Mike Frysinger
2008-04-21 19:45 ` Wolfgang Denk
2008-04-21 20:58 ` Mike Frysinger
2008-04-21 22:19 ` Wolfgang Denk
2008-04-21 23:08 ` Mike Frysinger
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=200804210659.43481.vapier@gentoo.org \
--to=vapier@gentoo.org \
--cc=u-boot@lists.denx.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox