From: Ulf Samuelsson <ulf@atmel.com>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] Using 'ver' env var
Date: Tue, 14 Aug 2007 07:02:21 +0200 [thread overview]
Message-ID: <007401c7de59$7124a0f0$dcc4af0a@atmel.com> (raw)
In-Reply-To: 20070814001111.014262405E@gemini.denx.de
> In message <5CBE65F7D9232C47861CB09B0954861C66A961@MAIL.infinitychannel.local> you wrote:
>>
>> I was thinking this already.. I just thought that I was missing
>> something which is why I asked the questions. I'll send a patch to
>> the group for review later.
>
> Don't bother to do that. I will not accept patches that automatically
> save the environment. U-Boot shall only act on user's request and
> never to any magical things behind my back.
>
> If you want this function, it can be implemented using U-Boot's
> scripting capabilities, so there is no need for a patch.
>
You assume that there is no users request, but there is.
If you, from Linux, are upgrading the U-Boot, you could
*if you wanted to*, interpret this as an indirect
request to update the "ver" variable.
This should only happen right after boot and before the user has any chance to
update anything else, so that only the "ver" variable is updated by this save.
The drawback of this is that a user will not be able to have a ver variable
which differs from the version. As soon as a user does
setenv ver <whatever>
U-Boot will revert that at the next change.
----------------
If U-Boot supported a read only environment area (generated at compile time)
then there would be no need for doing things behind your back.
If that is not acceptable, then the only way forward I see
is that Linux writes a new version of the bootcmd to the environment
so that the request is made explicit.
This requires that the user can write the current version to "ver"
from a command.
> Thanks.
>
> Best regards,
>
> Wolfgang Denk
>
> --
Best Regards
Ulf Samuelsson
next prev parent reply other threads:[~2007-08-14 5:02 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-13 19:55 [U-Boot-Users] Using 'ver' env var Zach Sadecki
2007-08-13 20:05 ` Wolfgang Denk
2007-08-13 20:26 ` Zach Sadecki
2007-08-13 20:45 ` Wolfgang Denk
2007-08-13 21:41 ` Ulf Samuelsson
2007-08-13 21:25 ` Zach Sadecki
2007-08-13 22:07 ` [U-Boot-Users] PATCH: save 'ver' env var on update Zach Sadecki
2007-08-14 0:10 ` Zach Sadecki
2007-08-14 0:18 ` Wolfgang Denk
2007-08-14 0:13 ` Wolfgang Denk
2007-08-14 0:11 ` [U-Boot-Users] Using 'ver' env var Wolfgang Denk
2007-08-14 5:02 ` Ulf Samuelsson [this message]
2007-08-14 13:39 ` Wolfgang Denk
2007-08-14 14:40 ` Ulf Samuelsson
2007-08-14 15:23 ` Wolfgang Denk
2007-08-14 18:18 ` Ulf Samuelsson
2007-08-14 18:55 ` Rune Torgersen
2007-08-14 19:03 ` Wolfgang Denk
2007-08-14 19:12 ` Zach Sadecki
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='007401c7de59$7124a0f0$dcc4af0a@atmel.com' \
--to=ulf@atmel.com \
--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